Skip to content

DOCS-3085: Fix the GlobalNetworkPolicy resource reference - #3081

Closed
ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:DOCS-3085-globalnetworkpolicy
Closed

ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:DOCS-3085-globalnetworkpolicy

Conversation

@ctauchen

@ctauchen ctauchen commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes the GlobalNetworkPolicy reference so that it agrees with the API for each docs version: next, 3.33, 3.32, and 3.31.

  • Adds the tier field, and says that order applies within the policy's tier.
  • Marks action as required in the shared Rule table. This also fixes the NetworkPolicy and staged policy pages.
  • The remaining critical is the Pass description in the same partial, which DOCS-3077: Fix the NetworkPolicy resource reference #3072 fixes.

Defects against each version's CRD, scored before and after:

  • next: 1 critical and 2 major before, 1 critical and 0 major after
  • 3.33: 1 critical and 2 major before, 1 critical and 0 major after
  • 3.32: 1 critical and 2 major before, 1 critical and 0 major after
  • 3.31: 1 critical and 2 major before, 1 critical and 0 major after

Minor defects, such as undocumented limits and validation rules, are left for DOCS-3105.

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

Preview:

@ctauchen
ctauchen requested a review from a team as a code owner October 12, 2026 14:27
Copilot AI balanced review requested due to automatic review settings October 12, 2026 14:27

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@netlify

netlify Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 0cc3257
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6accfa8a06770301cf2549ce
😎 Deploy Preview https://deploy-preview-3081--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 Oct 12, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0cc3257
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6accfa8a845061209039cf2c
😎 Deploy Preview https://deploy-preview-3081--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: 69 (🟢 up 3 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.

@ctauchen

Copy link
Copy Markdown
Collaborator Author

The review found one gap, in 3.31 only.

  • In 3.31, a global network policy outside the default tier must be named . (libcalico-go/lib/clientv3/globalnetworkpolicy.go@v3.31.7:66). The 3.31 tier row now says so.
  • Later versions don't enforce the rule, so they are unchanged.
  • This PR and DOCS-3077: Fix the NetworkPolicy resource reference #3072 must merge together: each fixes one of the two defects in the shared _rule.mdx.

Copilot AI balanced review requested due to automatic review settings October 12, 2026 15:19

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ctauchen

Copy link
Copy Markdown
Collaborator Author

Closing this PR. Its fixes were checked against source only, not reproduced on a cluster, so it is kept as a comparison baseline for the cluster-verified fact-check of this page.

@ctauchen ctauchen closed this Oct 15, 2026
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