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

Update to Kueue 0.9.1 #267

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

dgrove-oss
Copy link
Collaborator

No description provided.

@dgrove-oss dgrove-oss marked this pull request as draft November 7, 2024 23:22
@dgrove-oss
Copy link
Collaborator Author

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.

@dgrove-oss dgrove-oss force-pushed the kueue-bump branch 2 times, most recently from 17e3b09 to c8f2575 Compare November 12, 2024 01:35
@dgrove-oss
Copy link
Collaborator Author

#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.

@dgrove-oss dgrove-oss changed the title Update to Kueue 0.9.0 Update to Kueue 0.9.1 Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant