Skip to content
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

Helm chart 4.2.2 broken because of the switch to Lease API #8958

Closed
lbogdan opened this issue Aug 23, 2022 · 11 comments · Fixed by #8959
Closed

Helm chart 4.2.2 broken because of the switch to Lease API #8958

lbogdan opened this issue Aug 23, 2022 · 11 comments · Fixed by #8959
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@lbogdan
Copy link

lbogdan commented Aug 23, 2022

Helm chart 4.2.2 is currently broken because of #8921, as it's referencing a controller image that still uses the ConfigMap leader election, but the ingress-nginx Role has the ConfigMap permissions removed.

@lbogdan lbogdan added the kind/bug Categorizes issue or PR as related to a bug. label Aug 23, 2022
@k8s-ci-robot
Copy link
Contributor

@lbogdan: This issue is currently awaiting triage.

If Ingress contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot added needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-priority labels Aug 23, 2022
@rouke-broersma
Copy link

@tao12345666333 I think you introduced this change just this morning. Any chance you could take a look at this?

@tao12345666333
Copy link
Member

thanks for your report.

I didn't actually notice that we were releasing a new version of the Helm chart.
I will do a quick fix

@tao12345666333
Copy link
Member

#8959

@auwaerter
Copy link

Hey @tao12345666333 i just found this issue as i was running into the same with the following log appearing after trying the new chart:

E0823 05:16:54.075704 6 leaderelection.go:367] Failed to update lock: configmaps "ingress-controller-leader" is forbidden: User "system:serviceaccount:ingress-nginx:nginx-serviceaccount" cannot update resource "configmaps" in API group "" in the namespace "ingress-nginx"

This is the issue we are talking about here, right?

@tao12345666333
Copy link
Member

yes

@auwaerter
Copy link

Thanks a lot for your confirmation! :)

@tao12345666333
Copy link
Member

once #8959 been merged. we will publish new Helm chart 4.2.3

@tao12345666333
Copy link
Member

Please try the latest versio v4.2.3, thanks!

@auwaerter
Copy link

Working perfectly fine to me, thanks a lot! :)

@lbogdan
Copy link
Author

lbogdan commented Aug 23, 2022

I can also confirm freshly installing 4.2.3 works fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants