Skip to content

DOCS-3077: Fix the NetworkPolicy resource reference - #3072

Closed
ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:np-tier-pass
Closed

ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:np-tier-pass

Conversation

@ctauchen

@ctauchen ctauchen commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes two errors in the Calico Open Source NetworkPolicy reference, in next and in 3.31, 3.32, and 3.33.

  • Adds the tier field to the spec table.
  • Fixes the Pass action description. Pass goes to the next tier, not to the profiles. This partial is also used by the GlobalNetworkPolicy and staged policy pages.

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

Preview:

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

netlify Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit b0ed199
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6accfa6202723597a8c2fe0a
😎 Deploy Preview https://deploy-preview-3072--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.

@netlify

netlify Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

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

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

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

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

The review found one gap, in 3.31 only.

  • In 3.31, a network policy outside the default tier must be named ., or create and update fail (libcalico-go/lib/clientv3/networkpolicy.go@v3.31.7:66, libcalico-go/lib/names/policy.go@v3.31.7:105). The 3.31 tier row now says so.
  • Later versions don't enforce the rule (ValidateTieredPolicyName has no caller from v3.32.2 on), so next, 3.33, and 3.32 are unchanged.
  • This PR and DOCS-3085: Fix the GlobalNetworkPolicy resource reference #3081 must merge together: each fixes one of the two defects in the shared _rule.mdx.

Copilot AI lite review requested due to automatic review settings October 12, 2026 15:18

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 tier, order and Pass changes were reproduced on a cluster for next, 3.33, 3.32 and 3.31, and are now in #3114.

@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