-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update to Kueue 0.9.2 #267
base: main
Are you sure you want to change the base?
Conversation
Letting CI do a sniff test; not ready to merge. Disabling visibilityOnDemand is a hack around the Kueue-manager not starting up correctly on a kind cluster due to a brace problem. Need to look more closely and figure out why the role binding that is part of the install.yaml didn't avoid this. |
17e3b09
to
c8f2575
Compare
#269 fixed the hard error when starting kueue without disabling the visibilityOnDemand feature gate. That was a mistake in the way we kustomized Kueue for the AppWrapper CI. There is still an upstream logging issue that causes the kueue 0.9 manager to generate several error messages a minute when visibilityOnDemand is not enabled. |
c8f2575
to
3afffbb
Compare
3afffbb
to
5f23ada
Compare
5f23ada
to
688b9a6
Compare
No description provided.