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

Support setting priority class of operator in Helm chart #956

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DebakelOrakel
Copy link

@DebakelOrakel DebakelOrakel commented Apr 16, 2024

Summary

Support setting priorityClassName of operator deployment in Helm chart.

Checklist

For Code changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:operator
  • Link this PR to related issues
  • I have not made any changes in the charts/ directory.

For Helm Chart changes

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • PR contains the label area:chart
  • PR contains the chart label, e.g. chart:k8up
  • Variables are documented in the values.yaml using the format required by Helm-Docs.
  • Chart Version bumped if immediate release after merging is planned
  • I have run make chart-docs
  • Link this PR to related code release or other issues.

@DebakelOrakel DebakelOrakel requested a review from a team as a code owner April 16, 2024 07:47
@DebakelOrakel DebakelOrakel requested review from tobru and Kidswiss and removed request for a team April 16, 2024 07:47
@TheBigLee
Copy link
Contributor

@DebakelOrakel Please sign your commit for the DCO check to pass.
Also don't forget to run make chart-docs

Signed-off-by: Stephan Feurer <stephan.feurer@vshn.net>
@DebakelOrakel
Copy link
Author

@DebakelOrakel Please sign your commit for the DCO check to pass. Also don't forget to run make chart-docs

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants