Skip to content

DOCS-3083: Fix the CalicoNodeStatus resource reference - #3182

Merged
ctauchen merged 5 commits into
tigera:mainfrom
ctauchen:DOCS-3083-caliconodestatus
Nov 4, 2026
Merged

ctauchen merged 5 commits into
tigera:mainfrom
ctauchen:DOCS-3083-caliconodestatus

Conversation

@ctauchen

@ctauchen ctauchen commented Nov 4, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes the CalicoNodeStatus reference for next, 3.33, 3.32, and 3.31. Each fix was reproduced on a kind cluster at the tree's release: v3.33.0 for next and 3.33, v3.32.3, and v3.31.7.

  • The metadata name rule allowed underscores and uppercase letters. The API rejects both.
  • The example output showed the internal projectcalico.org/metadata annotation, which reads never return, and a crd.projectcalico.org/v1 apiVersion in the last-applied annotation.
  • status.agent, status.bgp, and status.routes were described as lists. Each is a single object.
  • Route destination is a CIDR, not an IP.
  • learnedFrom.peerIP includes the port when a BGPPeer session has one, for example 10.0.0.1.port.1790, so its schema is string.

Not changed, for follow-up:

  • learnedFrom.peerIP carrying the port suffix looks like a product bug. status.bgp peer peerIP does not have it.
  • With the Routes class set, a route learned from a localWorkloadSelector peer stops the node from writing any status.
  • In native v3 CRD mode, a resource missing classes, node, or updatePeriodSeconds is accepted and then ignored.
  • The performance figures date from 2021 and need engineering review.

Previews:

Jira: https://tigera.atlassian.net/browse/DOCS-3083

@ctauchen
ctauchen requested a review from a team as a code owner November 4, 2026 14:57
Copilot AI lite review requested due to automatic review settings November 4, 2026 14:57
@netlify

netlify Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit e823440
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6aeb57028a209babdf466e75
😎 Deploy Preview https://deploy-preview-3182--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.

@netlify

netlify Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit e823440
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6aeb57024f2b00e4f241a3cd
😎 Deploy Preview https://deploy-preview-3182--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: 66 (🔴 down 6 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.

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 metadata.name rule is corrected consistently across the current and 3.31–3.33 Calico documentation, with no active review issues identified.

No open issues 🎉

What changed in this PR

Updates the CalicoNodeStatus resource reference so the documented metadata.name format matches the validator across the current documentation and Calico versions 3.31–3.33.

Changes:

  • Restricts names to lowercase alphanumeric characters with optional . or -.
  • Removes the outdated allowance for uppercase letters and underscores.
  • Applies the correction consistently across all four reviewed Calico documentation pages.

Minor follow-up: corresponding Enterprise and Cloud copies still document the previous name rule and should be updated or confirmed as intentionally different.

File Description
calico/​reference/​resources/​caliconodestatus.mdx Corrects the current CalicoNodeStatus metadata name rule.
calico_versioned_docs/​version-3.33/​reference/​resources/​caliconodestatus.mdx Corrects the Calico 3.33 metadata name rule.
calico_versioned_docs/​version-3.32/​reference/​resources/​caliconodestatus.mdx Corrects the Calico 3.32 metadata name rule.
calico_versioned_docs/​version-3.31/​reference/​resources/​caliconodestatus.mdx Corrects the Calico 3.31 metadata name rule.

Review effort: Lite

@ctauchen ctauchen left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@ctauchen
ctauchen merged commit aefc853 into tigera:main Nov 4, 2026
11 checks passed
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.

2 participants