-
Notifications
You must be signed in to change notification settings - Fork 155
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
[stable/goldilocks] Add pod priority class for goldilocks #1543
base: master
Are you sure you want to change the base?
[stable/goldilocks] Add pod priority class for goldilocks #1543
Conversation
@sudermanjr can this be reviewed |
@vitorvezani can this be reviewed |
@sudermanjr can you please approve |
Hi @harshbhardwaj5 -- thanks for the contribution, added a couple of suggestions to keep consistent with other configuration |
Co-authored-by: Vitor Rodrigo Vezani <[email protected]>
Co-authored-by: Vitor Rodrigo Vezani <[email protected]>
Hi thanks for suggestions, fixed it |
I think @sudermanjr review is mandatory for it to merge |
@sudermanjr Hi can we get a approval here |
Hi @sudermanjr , Can we get this merged? |
Why This PR?
a short description of why this PR is needed
There is no pod priority class support available for goldilocks start so this change add pod priority class in goldilocks deployment
Fixes #1544
Changes
Changes proposed in this pull request:
Checklist:
[stable/goldilocks]
.Chart.yaml
following Semantic Versioning.helm-docs --sort-values-order=file
has been run for the charts that support it.