Repository navigation
docs(platform-package): note the /21 the kube-ovn variants keep for cilium - #739
europrinter (yankawai) wants to merge 1 commit into
Conversation
✅ Deploy Preview for cozystack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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. Comment |
Aleksei Sviridkin (lexfrei)
left a comment
There was a problem hiding this comment.
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>
00df171 to
56a6830
Compare
|
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 |
Aleksei Sviridkin (lexfrei)
left a comment
There was a problem hiding this comment.
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.
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.podCIDRand 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. thenetworking.podCIDRrow in thenextplatform package reference now says both. nothing for the released versions, the change is on main only.bash hack/check-i18n.sh checkpasses, 29 keys and 20 translated pages.git diff --checkis clean, one reference file changes.merge it together with cozystack/cozystack#4778.
Screenshots
not applicable, reference documentation only.
Downstream repositories