Repository navigation
DOC-7120 unit 5: convert kubernetes 7.4.6/ to render hooks - #4134
Merged
Merged
Conversation
Runs build/migrate_shortcode_links.py's all-stages pipeline over the content/operate/kubernetes/7.4.6/ frozen version snapshot (64 files, 62 markdown), converting relref shortcode links and note/warning/tip/info/alert callouts to native Markdown render-hook syntax. Implementation-only per the DOC-7104 rs/ frozen-snapshot precedent -- no rendered appearance or behavior change. Hand-fixed the same list-nested callout indentation-loss bug found in this ticket's earlier units, in the same two files the prior units flagged as a strong prior (deployment/openshift/openshift-cli.md: 6 instances, deployment/quick-start.md: 2 instances), plus 10 more instances the script doesn't catch across old-index.md, connect-to-admin-console.md, create-reaadb.md, manage-rec-credentials.md, and collect-logs.md. Also dedented two non-list-nested stray indents to 0 (connect-prometheus-operator.md, manage-rec-credentials.md) that would otherwise have become CommonMark code blocks, and cleaned up a handful of stray whitespace-only blockquote artifact lines the script's callout-to-blockquote conversion leaves behind when the original shortcode's body had leading/trailing whitespace before its closing tag. Verified with a two-build Hugo href diff (0 changes across all 62 pages) and build/check_uncanonicalized_links.py (0 FIXABLE, 0 MOUNT_ONLY, 3 pre-existing DEAD links, unrelated to this conversion). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at 4810840 |
This was referenced Sep 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Unit 5 of DOC-7120 (continuing the DOC-6909→7047→7055→7059→7074→7079→7080→7086→7089→7104→7118 render-hook migration series). Converts every file under
content/operate/kubernetes/7.4.6/(64 files, 62 markdown) fromrelref/callout shortcodes to native Markdown render-hook syntax, usingbuild/migrate_shortcode_links.py all.This is a frozen version snapshot (per
content/operate/kubernetes/AGENTS.md's "Versioned content" section) — this conversion is implementation-only syntax and does not change rendered appearance or behavior, matching the DOC-7104 (rs/) precedent for its own frozen snapshots. No prose, terminology, or identifier was reworded.Changes
reference/kubernetes-api-reference-frontmatter.patchandreference/kubernetes-api-reference-template.tmpl, had no relref/callout syntax and the script correctly no-opped on them)./content/<path>.md[#anchor]form.note/warning/tip/info/alertshortcode instances converted to> [!TYPE]blockquote form.Gotcha instances found and hand-fixed
The script doesn't preserve indentation on blockquote continuation/closing lines for list-nested callouts. Checked every indented callout in the snapshot by hand. Found and fixed:
deployment/openshift/openshift-cli.md— 6 instances (matches unit 3/4's exact file+count prior for this file).deployment/quick-start.md— 2 instances (unit 3/4 found 3 in their copies of this file; this snapshot's copy only has 2 list-nested instances — verified independently rather than assuming the count carried over, per the task's own caution).deployment/openshift/old-index.md— 3 instances (not previously flagged in this ticket; new find in this snapshot).re-clusters/connect-to-admin-console.md— 1 instance.active-active/create-reaadb.md— 2 instances.security/manage-rec-credentials.md— 1 list-nested instance (plus 1 non-list-nested stray 2-space indent, dedented to 0 — see below).logs/collect-logs.md— 1 instance.Non-list-nested stray indents dedented to 0 (would otherwise become CommonMark indented code blocks, breaking the blockquote):
re-clusters/connect-prometheus-operator.md— 1 instance (4-space stray indent, not inside a list).security/manage-rec-credentials.md— 1 instance (2-space stray indent, below the list-continuation threshold for its neighboring numbered list).Also cleaned up several stray whitespace-only blockquote artifact lines (e.g. a leftover
>or>line) that the callout-to-blockquote conversion leaves behind when the original shortcode body had leading/trailing whitespace directly against its closing tag — cosmetic, but tidied while already in the diff for the indentation fix.No no-slash relref concatenation or missing-close-paren relref instances found in this snapshot.
DEAD links found (not fixed, per house style — flagging rather than deciding)
build/check_uncanonicalized_links.pyfound 3 pre-existing broken relrefs, unrelated to this conversion:deployment/openshift/old-getting-started-openshift-crdb.md:48→/operate/platforms/openshift/_index.mddeployment/openshift/old-index.md:30→/operate/rs/installing-upgrading/hardware-requirements.mddeployment/openshift/old-index.md:333→/operate/rs/administering/creating-databases/_index.mdVerification
origin/main(before) and this branch (after) with Hugo in two separate checkouts, foreground.build/diff_rendered_hrefs.py <before> <after> operate/kubernetes/7.4.6— 0 href diffs across all 62 pages.build/check_uncanonicalized_links.py content/operate/kubernetes/7.4.6/— 0 FIXABLE, 0 MOUNT_ONLY, 3 DEAD (listed above).Test plan
/operate/kubernetes/7.4.6/look identical to the previous shortcode-based rendering (blockquote callouts nested correctly inside numbered/bulleted list steps).deployment/openshift/openshift-cli.mdanddeployment/quick-start.mdfor the hand-fixed callout nesting.🤖 Generated with Claude Code
Note
Low Risk
Documentation-only syntax migration in a versioned snapshot with rendered href parity checks; no runtime or product code changes.
Overview
Migrates the frozen
operate/kubernetes/7.4.6/docs snapshot from Hugorelrefand callout shortcodes to native Markdown for render hooks—same pattern as earlier units in the DOC-7120 series.Across 43 markdown files, internal links move from
{{< relref ... >}}to canonical/content/...paths (with anchors preserved), and note/warning-style callouts become GitHub-flavored> [!NOTE]/> [!WARNING]blockquotes. Prose and behavior are unchanged; verification reported 0 href diffs when comparing Hugo renders before and after.Hand-edits fix blockquote indentation where list-nested callouts would break (e.g.
deployment/openshift/openshift-cli.md,deployment/quick-start.md,deployment/openshift/old-index.md, several active-active and security pages). A few stray-indented callouts were dedented so they are not treated as code blocks.Not in scope: three pre-existing broken targets in legacy OpenShift draft pages remain flagged (not retargeted in this PR).
Reviewed by Cursor Bugbot for commit 4810840. Bugbot is set up for automated code reviews on this repo. Configure here.