Skip to content

DOCS-3097: Fix the StagedNetworkPolicy resource reference - #3082

Closed
ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:DOCS-3097-stagednetworkpolicy
Closed

ctauchen wants to merge 2 commits into
tigera:mainfrom
ctauchen:DOCS-3097-stagednetworkpolicy

Conversation

@ctauchen

@ctauchen ctauchen commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

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

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

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

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

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

Preview:

Copilot AI balanced review requested due to automatic review settings October 12, 2026 14:27
@ctauchen
ctauchen requested a review from a team as a code owner 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 succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 5dd776b
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6accfa8f6882e13f4b8a74ce
😎 Deploy Preview https://deploy-preview-3082--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 5dd776b
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6accfa8f9418f0f6107c90d1
😎 Deploy Preview https://deploy-preview-3082--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.

@ctauchen

Copy link
Copy Markdown
Collaborator Author
  • The stagedAction row said a staged policy previews the enforced policy that has the same name. Calico Open Source doesn't do that. A staged policy is evaluated alongside the enforced policies, and nothing matches names (app-policy/checker/check.go).
  • Set now says the policy is evaluated as if enforced, and the result is reported in the policies.pending field of flow logs. Every version reports it (felix/collector/collector.go).
  • Delete now says the staged policy is ignored. The update processor drops it before it reaches Felix (libcalico-go/lib/backend/syncersv1/updateprocessors/stagednetworkpolicyprocessor.go). The rule that other spec fields must be empty, except tier, stays (validator.go).
  • Learn and Ignore are now described as treated like Set. Nothing in Calico Open Source reads them beyond validation.
  • The Selector heading is now Selectors, so the five #selectors links from the shared EntityRule and ServiceAccountMatch partials resolve.

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