-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spidercoordinator: It able to get CIDR from kubeadm-config #3130
Spidercoordinator: It able to get CIDR from kubeadm-config #3130
Conversation
798ded4
to
d904872
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release-v0.8 #3130 +/- ##
=============================================
Coverage 81.01% 81.01%
=============================================
Files 49 49
Lines 5340 5340
=============================================
Hits 4326 4326
Misses 856 856
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
d904872
to
91a823f
Compare
CI 失败的原因: release-0.8 的 e2e 安装的 IPv6 单栈集群的 calico 有 bug,它仍有 IPV4、IPv6 的池。故在 IPv6 的集群中 auto 模式下仍获取到了 ipv4 的 pod CIDR。 将 release-0.8 的 test/scripts/install-default-cni.sh 中 151 行修改一下?保证 ipv6 模式下只有calico 的 ipv6 池? ipv6)
kubectl delete ippools default-ipv4-ippool --force
kubectl patch ippools default-ipv6-ippool --patch '{"spec": {"cidr": "'"${CALICO_IPV6POOL_CIDR}"'"}}' --type=merge
;; |
[FAIL] SpiderCoordinator Switch podCIDRType to https://github.com/spidernet-io/spiderpool/actions/runs/7638721733/job/20818914107?pr=3130 |
91a823f
to
f3ad24b
Compare
why can't merge this when we have two approves? |
|
If the kube-controller-manager Pod is running as systemd precess rather than Pod, In this case, We can't get the CIDR from the KCM Pod. We can get the CIDR from the kubeadm-config configMap. Signed-off-by: cyclinder <qifeng.guo@daocloud.io>
f3ad24b
to
0234532
Compare
If the kube-controller-manager Pod is running as systemd precess rather than Pod, In this case, We can't get the CIDR from the KCM Pod. We can get the CIDR from the kubeadm-config configMap.
Thanks for contributing!
What type of PR is this?
What this PR does / why we need it:
Cherry-pick
Which issue(s) this PR fixes:
Fixes #3126
Special notes for your reviewer: