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

Add priorityClassName to CRDs #1513

Merged
merged 3 commits into from
Jun 27, 2022

Conversation

cablespaghetti
Copy link

@cablespaghetti cablespaghetti commented Jun 8, 2022

Fixes #687

Tested in my environment and working fine :)

@cablespaghetti cablespaghetti marked this pull request as ready for review June 8, 2022 13:07
@toast-gear toast-gear added this to the v0.25.0 milestone Jun 9, 2022
@cablespaghetti cablespaghetti force-pushed the feature/priorityClassName branch from 7f0945e to dd4f504 Compare June 13, 2022 20:48
@cablespaghetti
Copy link
Author

I'm very unsure what I could have done to break the tests. I've just rebased this branch. Any help figuring it out would be great, thanks :)

@toast-gear
Copy link
Collaborator

it's most likely a ephemeral error and not related to your change, don't worry about it, the change as it is will get merged once we're ready

Copy link
Collaborator

@mumoshu mumoshu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks a lot for your contribution!

@mumoshu mumoshu merged commit bc7a3ca into actions:master Jun 27, 2022
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.

Adding priorityClassName to crd for runners
4 participants