-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
fix calico rbac issue #9806
fix calico rbac issue #9806
Conversation
/retest |
- list | ||
- create | ||
- update | ||
- delete |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @JaneLiuL
From the https://github.com/projectcalico/calico/blob/master/charts/calico/templates/calico-node-rbac.yaml#L160
The List, Update, Delete is not needed.
Should it better to be removed ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi, that depends on calico version, when i install kubespray with kubernetes 1.24.10 with calico version 3.24.5. we still need it.
and we should both support lower version.
I still suggest that we should keep it.
And both when i upgrade kubernets from 1.24.0 to 1.25 via kubespray, we still need that.
what do you think so?
and what you provide the link is not release yet, it is in master branch.
for latest calico release tag please check here: https://github.com/projectcalico/calico/blob/v3.25.0/charts/calico/templates/calico-node-rbac.yaml#L160
it still need create permission
@yankay
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JaneLiuL Thank you for the PR 👍
Looks good to me based on explanation
@JaneLiuL Thanks |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: floryut, JaneLiuL, yankay The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/kind bug
when i install kubernetes with calico cni, and specific calico version in
inventory/mycluster/group_vas/k8s-cluster.yml
, and specifiy calico version withcalico_version: v3.24.5
.the calico pods will fail, logs as below
so i just modify the rbac, and grant the permission as this pr, and install again, it works
for latest calico release tag please check here: https://github.com/projectcalico/calico/blob/v3.25.0/charts/calico/templates/calico-node-rbac.yaml#L160
it still need create permission