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
calico_pool_blocksize doesnt default to kube_network_node_prefix #9843
Comments
Thanks @Payback159 for the report. If you comment out
If blocksize is 24, why is this task break up?
|
Hi @cyclinder, if I comment out if I comment out In my point of view the check works perfectly fine, it's just the comments and the expectation of the person which is going to configure it's inventory. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
We should either have calico_pool_blocksize: "{{ kube_network_node_prefix }}" In the |
/remove-lifecycle rotten |
Correction: calico_pool_blocksize should not default to kube_network_node_prefix (not the same thing, there can be several calico block per node and they are dynamically assigned |
@VannTen Yes, Calico has two IPAM methods: one is |
It's already supported FWICT. (at least #6580 was merged)
|
Environment:
Cloud provider or hardware configuration:
OS (
printf "$(uname -srm)\n$(cat /etc/os-release)\n"
):ansible --version
):ansible --version ansible [core 2.12.5] config file = /kubespray/ansible.cfg configured module search path = ['/kubespray/library'] ansible python module location = /usr/local/lib/python3.8/dist-packages/ansible ansible collection location = /root/.ansible/collections:/usr/share/ansible/collections executable location = /usr/local/bin/ansible python version = 3.8.10 (default, Nov 14 2022, 12:59:47) [GCC 9.4.0] jinja version = 2.11.3 libyaml = True
python --version
):Kubespray version (commit) (
git rev-parse --short HEAD
):Network plugin used:
Calico
Full inventory with variables (
ansible -i inventory/sample/inventory.ini all -m debug -a "var=hostvars[inventory_hostname]"
):Command used to invoke ansible:
Output of ansible run:
breaks at this step https://github.com/kubernetes-sigs/kubespray/blob/master/roles/network_plugin/calico/tasks/check.yml#L159
Anything else do we need to know:
The sample Inventory shows the bug well. In the k8s-net-calico it says in the comments that it goes by default to
kube_network_node_prefix
. https://github.com/kubernetes-sigs/kubespray/blob/master/inventory/sample/group_vars/k8s_cluster/k8s-net-calico.yml#L21Therefore my assumption that if I comment out
calico_pool_blocksize
that the blocksize is a /24 as configured inkube_network_node_prefix
. https://github.com/kubernetes-sigs/kubespray/blob/master/inventory/sample/group_vars/k8s_cluster/k8s-cluster.yml#L98But the problem seems to exist since Kubespray v2.18 that this is not the case in the check here anyway, because in the main.yml of the Role the value
calico_pool_blocksize
is configured to /26 and therefore the check takes the value ofcalico_pool_blocksize
and not as assumed in the comment and from my point of view also in the check tokube_network_node_prefix
.My suggestion would be to comment out calico_pool_blocksize in the main.yml in the role so it reacts as mentioned in the comment but wanted your opinion on it first before I put the PR.
The text was updated successfully, but these errors were encountered: