Skip to content

Fix tip admonition rendering on Calico on-premises install page - #3132

Merged
ctauchen merged 1 commit into
mainfrom
ets-fix-onprem-tip
Oct 19, 2026
Merged

ctauchen merged 1 commit into
mainfrom
ets-fix-onprem-tip

Conversation

@tmjd

@tmjd tmjd commented Oct 19, 2026

Copy link
Copy Markdown
Member

Product Version(s):
Calico next, 3.33 (latest), 3.32, 3.31

Issue:
On https://docs.tigera.io/calico/latest/getting-started/kubernetes/self-managed-onprem/onpremises (and the 3.32/3.31 versions), the "Automatic data plane deployment" tip in step 3 renders as literal text (:::tip Automatic data plane deployment ... :::) instead of a tip box.

Link to docs preview:
/calico/next/getting-started/kubernetes/self-managed-onprem/onpremises and /calico/{latest,3.32,3.31}/getting-started/kubernetes/self-managed-onprem/onpremises on the deploy preview.

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:
The tip used the legacy :::tip Title syntax. docusaurus.config.js sets future.v4.mdx1CompatDisabledByDefault: true, so Docusaurus no longer rewrites that to the MDX v3 directive label form, and remark-directive fails to parse it as an admonition. Switched to :::tip[Automatic data plane deployment]. Verified by compiling the page with @docusaurus/mdx-loader 3.10.1 with mdx1Compat disabled: before → plain paragraph (matches the live site), after → admonition with type: "tip" and the title.

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

🤖 Generated with Claude Code

The "Automatic data plane deployment" tip in step 3 used the legacy
":::tip Title" syntax. The site sets mdx1CompatDisabledByDefault, so the
title is no longer rewritten to a directive label and the whole block
rendered as a literal paragraph. Use ":::tip[Title]" instead in next,
3.33, 3.32 and 3.31.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tmjd
tmjd requested a review from a team as a code owner October 19, 2026 15:10
Copilot AI lite review requested due to automatic review settings October 19, 2026 15:10
@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Name Link
🔨 Latest commit eeb2bcc
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6ad63303ae4cc90f7efaf548
😎 Deploy Preview https://deploy-preview-3132--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 98 (🟢 up 4 from production)
Accessibility: 98 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit eeb2bcc
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6ad63303ae4b3ca6d755aac4
😎 Deploy Preview https://deploy-preview-3132--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The syntax fix is consistently applied across all reviewed documentation versions with no unresolved review comments.

No open issues 🎉

What changed in this PR

Updates Calico on-premises documentation so the “Automatic data plane deployment” tip renders correctly with Docusaurus MDX syntax.

Changes:

  • Replaces legacy tip syntax with bracketed directive labels.
  • Applies the fix to next, 3.33, 3.32, and 3.31 documentation.
File Description
calico/​getting-started/​kubernetes/​self-managed-onprem/​onpremises.mdx Fixes the next-version tip syntax.
calico_versioned_docs/​version-3.33/​getting-started/​kubernetes/​self-managed-onprem/​onpremises.mdx Fixes the 3.33 tip syntax.
calico_versioned_docs/​version-3.32/​getting-started/​kubernetes/​self-managed-onprem/​onpremises.mdx Fixes the 3.32 tip syntax.
calico_versioned_docs/​version-3.31/​getting-started/​kubernetes/​self-managed-onprem/​onpremises.mdx Fixes the 3.31 tip syntax.

Review effort: Lite

@ctauchen ctauchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@ctauchen
ctauchen merged commit 08ce8cb into main Oct 19, 2026
12 checks passed
@ctauchen
ctauchen deleted the ets-fix-onprem-tip branch October 19, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants