Repository navigation
DOC-7120 unit 4: convert kubernetes 7.22/ to render hooks - #4133
Merged
Merged
Conversation
Runs build/migrate_shortcode_links.py all over every file under content/operate/kubernetes/7.22/ (the frozen version snapshot), converting relref shortcodes to plain canonicalized Markdown links and note/warning/ tip/info/alert callout shortcodes to native blockquote render-hook syntax. Hand-fixed list-nested callout indentation loss (the conversion only preserves indentation on the blockquote header line, not continuation lines) in 7 files: re-clusters/connect-to-admin-console.md, active-active/prepare-clusters.md, security/allow-resource-adjustment.md, deployment/openshift/openshift-cli.md (6 instances), deployment/quick-start.md (3 instances), active-active/create-reaadb.md, and re-clusters/connect-prometheus-operator.md. The last one was not list-nested but had a stray 4-space indent unrelated to any enclosing list, which would make CommonMark treat the blockquote as an indented code block; dedented it to 0 instead of matching it. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at dec355b |
Contributor
This was referenced Sep 28, 2026
dwdougherty
approved these changes
Sep 28, 2026
dwdougherty
left a comment
dwdougherty
left a comment
Collaborator
There was a problem hiding this comment.
LGTC. (Looks good to Claude.)
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 4 of DOC-7120 (continuing the DOC-6909→7047→…→7118 shortcode-migration series). Converts every file under
content/operate/kubernetes/7.22/(the frozen 7.22 version snapshot) fromrelref/callout shortcodes to native Markdown render-hook syntax, usingbuild/migrate_shortcode_links.py all.reference/api/*_api.mdfiles and 2 non-content files — no-op as expected).]({{< relref "X" >}})to canonicalized](/content/<path>.md[#anchor])form.note/warning/tip/info/alert, both{{< >}}and{{% %}}forms) converted to> [!TYPE]blockquote form.Gotcha instances found and hand-fixed
List-nested callout indentation loss (the conversion only preserves indentation on the blockquote header line, not continuation/closing lines) — found and fixed in 9 instances across 7 files:
re-clusters/connect-to-admin-console.md(1)active-active/prepare-clusters.md(1)security/allow-resource-adjustment.md(1)active-active/create-reaadb.md(1)deployment/openshift/openshift-cli.md(6 instances)deployment/quick-start.md(3 instances)These four files match unit 3's live-tree findings for the same file basenames (
deployment/openshift/openshift-cli.md,deployment/quick-start.md) — confirming this frozen snapshot carries the same pre-migration byte-for-byte content and thus the same bug instances, needing independent fixing here since fixing the live tree doesn't propagate to the frozen copy.One additional case,
re-clusters/connect-prometheus-operator.md, had a stray 4-space indent on its{{<note>}}that was not inside a list (just leftover formatting). Matching that indent would have made CommonMark treat the blockquote as an indented code block, so it was dedented to 0 instead of indent-matched.No no-slash relref concatenation or missing-close-paren relref instances were found in this scope.
DEAD links
None found.
check_uncanonicalized_links.pyreported 0 FIXABLE, 0 MOUNT_ONLY, 0 DEAD.Verification
origin/main(before) and this branch (after) with Hugo in two separate directories, both foreground/blocking (19,645 pages each).build/diff_rendered_hrefs.py before after operate/kubernetes/7.22: 70 vs 70 pages compared, 0 pages only-in-before, 0 only-in-after, 0 href-set changes.build/check_uncanonicalized_links.pyscoped tocontent/operate/kubernetes/7.22/: 0 FIXABLE, 0 MOUNT_ONLY, 0 DEAD.This is purely a shortcode-syntax conversion — no prose, terminology, or identifier changes.
content/operate/kubernetes/AGENTS.mdwas reviewed; legacy product name and all custom-resource identifiers/field paths were left untouched.Test plan
build/migrate_shortcode_links.py allrun over all 70 files in scoperelrefor callout shortcodes incontent/operate/kubernetes/7.22/diff_rendered_hrefs.py: 0 diffscheck_uncanonicalized_links.py: 0 FIXABLE / 0 MOUNT_ONLY / 0 DEAD🤖 Generated with Claude Code
Note
Low Risk
Documentation-only link and callout syntax migration with no runtime or product behavior changes; main risk is broken links if paths were wrong, which the PR claims was verified with Hugo diff and link checks.
Overview
Converts the frozen
content/operate/kubernetes/7.22/snapshot from Hugorelreflinks andnote/warningshortcodes to native Markdown: canonical/content/.../*.mdhrefs (including anchors) and> [!NOTE]/> [!WARNING]blockquotes.The change is mechanical across the 7.22 tree (index pages, Active-Active, deployment, networking, re-clusters, etc.); prose and API identifiers are unchanged. A follow-up pass re-indents list-nested callouts in a handful of files (e.g.
quick-start.md,openshift-cli.md,prepare-clusters.md) so blockquotes stay inside list items after conversion.Reviewed by Cursor Bugbot for commit dec355b. Bugbot is set up for automated code reviews on this repo. Configure here.