Skip to content

DOCS-3092: Fix inaccuracies on the NetworkSet reference - #3121

Merged
ctauchen merged 4 commits into
tigera:mainfrom
ctauchen:DOCS-3092-fact-check-networkset
Oct 19, 2026
Merged

ctauchen merged 4 commits into
tigera:mainfrom
ctauchen:DOCS-3092-fact-check-networkset

Conversation

@ctauchen

@ctauchen ctauchen commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes inaccuracies on the NetworkSet reference page, found and verified with the calico-fact-check skill.

  • A network set is not limited to network policies in its own namespace. A rule with a namespaceSelector matches network sets in other namespaces, and global network policy rules match network sets in any namespace.
  • name: underscores are rejected and dots are allowed.
  • labels: the labels belong to the network set, not an endpoint.

Changes next, 3.33, 3.32 and 3.31. Each fix was reproduced on a kind cluster running that tree's release: v3.33.0 for next and 3.33, v3.32.2 for 3.32, v3.31.7 for 3.31.

Not changed, for follow-up:

  • nets also accepts bare IP addresses, and 3.33 limits it to 100000 entries of at most 49 characters.
  • The sample uses the staging namespace, which must exist first.
  • The kube-proxy SNAT note was not settled.

Previous attempt: #3091.

Preview the changed page:

Next and 3.33 were verified in both API server mode and native v3 CRD mode.

Copilot AI lite review requested due to automatic review settings October 16, 2026 11:56
@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 86ef5c1
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6ad21106fb46168f7b623200
😎 Deploy Preview https://deploy-preview-3121--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 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 86ef5c1
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6ad2110685bccd72f23d4481
😎 Deploy Preview https://deploy-preview-3121--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: 65 (🔴 down 8 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.

🔵 Needs a closer look

Shared entity-rule documentation still contradicts the updated NetworkSet guidance.

0 open findings

What changed in this PR

Updates NetworkSet references for current Calico and versions 3.31–3.33.

Changes:

  • Clarifies policy matching scope.
  • Corrects valid name characters.
  • Clarifies label ownership.
File Summary
calico/​reference/​resources/​networkset.mdx Updated current NetworkSet reference.
calico_versioned_docs/​version-3.33/​reference/​resources/​networkset.mdx Updated 3.33 reference.
calico_versioned_docs/​version-3.32/​reference/​resources/​networkset.mdx Updated 3.32 reference.
calico_versioned_docs/​version-3.31/​reference/​resources/​networkset.mdx Updated 3.31 reference.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ctauchen
ctauchen marked this pull request as ready for review October 19, 2026 14:20
@ctauchen
ctauchen requested a review from a team as a code owner October 19, 2026 14:20
@ctauchen

Copy link
Copy Markdown
Collaborator Author

LGTM

@ctauchen
ctauchen merged commit 4e836d1 into tigera:main Oct 19, 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