Skip to content

Add allowedSourcePrefixes annotation documentation#2613

Open
tomastigera wants to merge 2 commits into
tigera:mainfrom
tomastigera:worktree-allowed-source-prefixes
Open

Add allowedSourcePrefixes annotation documentation#2613
tomastigera wants to merge 2 commits into
tigera:mainfrom
tomastigera:worktree-allowed-source-prefixes

Conversation

@tomastigera

Copy link
Copy Markdown
Contributor

Summary

  • Add new allowed-source-prefixes.mdx page under networking/configuring/ for all three products (Calico, Calico Enterprise, Calico Cloud)
  • Documents the cni.projectcalico.org/allowedSourcePrefixes pod annotation, which allows pods to send egress traffic with source IPs outside their own address
  • Includes prerequisite workloadSourceSpoofing: Any FelixConfiguration setting and security caution about admission control
  • Add sidebar entries in all three products

Test plan

  • Verify page renders correctly for each product
  • Verify sidebar links work
  • Review content accuracy with engineering

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 27, 2026 19:00
@netlify

netlify Bot commented Mar 27, 2026

Copy link
Copy Markdown

Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 332bd33
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6a60f281e46e4e0008007470
😎 Deploy Preview https://deploy-preview-2613--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: 71 (🟢 up 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

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Mar 27, 2026

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 332bd33
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6a60f27e2716d00008080855
😎 Deploy Preview https://deploy-preview-2613--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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.

Pull request overview

Adds new documentation pages across Calico OSS, Calico Enterprise, and Calico Cloud describing the cni.projectcalico.org/allowedSourcePrefixes pod annotation and links them into each product’s “Configure networking” sidebar.

Changes:

  • Added allowed-source-prefixes.mdx under networking/configuring/ for Calico, Enterprise, and Cloud.
  • Documented the required FelixConfiguration setting (workloadSourceSpoofing: Any) and included a security caution about admission control.
  • Added sidebar entries in all three products so the new page is discoverable.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
sidebars-calico.js Adds the new “allowed-source-prefixes” page to the Calico OSS networking/configuring sidebar.
sidebars-calico-enterprise.js Adds the new page to the Calico Enterprise networking/configuring sidebar.
sidebars-calico-cloud.js Adds the new page to the Calico Cloud networking/configuring sidebar.
calico/networking/configuring/allowed-source-prefixes.mdx New OSS doc page for the allowedSourcePrefixes annotation and Felix prerequisite.
calico-enterprise/networking/configuring/allowed-source-prefixes.mdx New Enterprise doc page for the allowedSourcePrefixes annotation and Felix prerequisite.
calico-cloud/networking/configuring/allowed-source-prefixes.mdx New Cloud doc page for the allowedSourcePrefixes annotation and Felix prerequisite.

Comment thread calico-enterprise/networking/configuring/allowed-source-prefixes.mdx Outdated
Comment thread calico-cloud/networking/configuring/allowed-source-prefixes.mdx Outdated
Comment thread calico-cloud/networking/configuring/allowed-source-prefixes.mdx
Comment thread calico/networking/configuring/allowed-source-prefixes.mdx Outdated
Comment thread calico/networking/configuring/allowed-source-prefixes.mdx
Comment thread calico-enterprise/networking/configuring/allowed-source-prefixes.mdx Outdated
@tomastigera
tomastigera marked this pull request as ready for review July 22, 2026 16:13
@tomastigera
tomastigera requested a review from a team as a code owner July 22, 2026 16:13
tomastigera and others added 2 commits July 22, 2026 09:40
Document the cni.projectcalico.org/allowedSourcePrefixes pod annotation
across all three products (Calico, Calico Enterprise, Calico Cloud).
This annotation allows pods to send egress traffic with source IPs
outside their own address, requiring workloadSourceSpoofing: Any in
FelixConfiguration.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add Calico CNI prerequisite with product-correct link (OSS uses
  configure-cni-plugins.mdx; Enterprise/Cloud use
  component-resources/configuration.mdx). The three pages are no longer
  byte-identical for this reason.
- Fix FelixConfiguration apiVersion from crd.projectcalico.org/v1 to
  projectcalico.org/v3, matching the rest of the docs.
- Note the annotation is enforced in both iptables/nftables and eBPF
  dataplanes.
- Clarify that Felix drops (does not program) a pod requesting source
  prefixes until workloadSourceSpoofing: Any is set, per
  felix/calc/validation_filter.go.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@tomastigera
tomastigera force-pushed the worktree-allowed-source-prefixes branch from 05e76dc to 332bd33 Compare July 22, 2026 16:40
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