Skip to content

docs(platform-package): note the /21 the kube-ovn variants keep for cilium - #739

Open
europrinter (yankawai) wants to merge 1 commit into
cozystack:mainfrom
yankawai:docs/cilium-address-block
Open

europrinter (yankawai) wants to merge 1 commit into
cozystack:mainfrom
yankawai:docs/cilium-address-block

Conversation

@yankawai

@yankawai europrinter (yankawai) commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

What this PR does

follow-up for cozystack/cozystack#4778. that pr moves the router and gateway api ingress addresses of cilium on the kube-ovn variants into the last /21 of networking.podCIDR and excludes that block from kube-ovn, so the pod cidr has to be /20 or wider there. each node takes a /29 of that block, which caps these variants at 256 nodes. the networking.podCIDR row in the next platform package reference now says both. nothing for the released versions, the change is on main only.

  • bash hack/check-i18n.sh check passes, 29 keys and 20 translated pages.
  • git diff --check is clean, one reference file changes.
  • hugo was not built locally.

merge it together with cozystack/cozystack#4778.

Screenshots

not applicable, reference documentation only.

Downstream repositories

@netlify

netlify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cozystack ready!

Name Link
🔨 Latest commit 56a6830
🔍 Latest deploy log https://app.netlify.com/projects/cozystack/deploys/6ac5691871dbc800085bd687
😎 Deploy Preview https://deploy-preview-739--cozystack.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.

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c5f3280f-7615-4460-8452-be5db8db1e56
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

europrinter (@yankawai) one thing to add before this goes in. The row matches cozystack/cozystack#4778 as it stands now: the last /21 goes to Cilium's router and Ingress addresses, kube-ovn excludes it, and the render needs /20 or shorter. The 256-node ceiling is missing, though. Since the last push of #4778, the networking.podCIDR comment in values.yaml and its release note both say that a /21 of /29s caps these variants at 256 nodes, and a fresh install hits that too. This row is where an operator looks for the constraints on podCIDR, so please add the same sentence here.

The change only touches next/, which is right while #4778 is unmerged.

…ilium

cozystack/cozystack#4778 moves the router and Gateway API Ingress
addresses of Cilium on the Kube-OVN variants into the last /21 of
networking.podCIDR and excludes that block from Kube-OVN, so the pod
CIDR has to be /20 or wider there. Each node takes a /29 of that block,
which caps these variants at 256 nodes. The networking.podCIDR row now
says both.

Assisted-by: LLM
Signed-off-by: Yan Bondarenko <202671653+yankawai@users.noreply.github.com>
@yankawai

Copy link
Copy Markdown
Contributor Author

Aleksei Sviridkin (@lexfrei) added. the row now says each node takes a /29 of that /21, which caps these variants at 256 nodes, same as the networking.podCIDR comment and the release note in cozystack/cozystack#4778. amended into the one commit, 56a6830, and hack/check-i18n.sh check still passes.

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.

europrinter (@yankawai) the 256-node ceiling is in now and the row matches #4778 at its current head. LGTM; this should land together with or after #4778.

This branch has not been deployed

No deployments
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