Skip to content

feat(priorityClassName): Add support for priorityClassName to 'contro… #111

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(priorityClassName): Add support for priorityClassName to 'contro… #111

wants to merge 1 commit into from

Conversation

Mart-Kuc
Copy link

Pull Request Checklist

  • Any new images or tags consumed by charts has been added here
  • Chart version has been incremented (if necessary)
  • That helm lint and pack run successfully on the chart.
  • Deployment of the chart has been tested and verified that it functions as expected.
  • Changes to scripting or CI config have been tested to the best of your ability

Types of Change

Added support for priorityClassName configuration in both node and controller components.
New values introduced:

  • .csiNode.priorityClassName
  • .csiController.priorityClassName

Chart version updated to 3.3.1-rancher11.

@Mart-Kuc Mart-Kuc requested review from a team as code owners May 22, 2025 12:11
jiaqiluo
jiaqiluo approved these changes May 22, 2025
View reviewed changes
Copy link
Member

@jiaqiluo jiaqiluo left a comment

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brandond brandond brandond approved these changes

@jiaqiluo jiaqiluo jiaqiluo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Mart-Kuc @brandond @jiaqiluo