feat: add priorityClassName support for provisioner and helper pods #525
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for configuring
priorityClassNamefor both the main provisioner pod and helper pods in the local-path-provisioner Helm chart. Previously, users could not set priority classes for these critical storage infrastructure components, limiting their ability to properly prioritize pods in resource-constrained environments.Changes
priorityClassNamefield tovalues.yamlfor the main provisioner podconfigmap.helperPod.priorityClassNamefield tovalues.yamlfor helper podstemplates/deployment.yamlto conditionally renderpriorityClassNamewhen specifiedtemplates/configmap.yamlto make helper pod priority configurable (was hard-coded tosystem-node-critical)README.mdwith configuration examples and parameter descriptionsUsage Scenarios
priorityClassNameto assign priority to the main provisioner deploymentconfigmap.helperPod.priorityClassNameto assign priority to volume operation podssystem-node-critical)Example
Or via command line:
helm install local-path-storage . \ --set priorityClassName=high-priority \ --set configmap.helperPod.priorityClassName=system-cluster-criticalTesting
priorityClassNamefield is rendered when values are emptysystem-node-criticaldefault when not overriddenBackward Compatibility
This change is fully backward compatible. All existing configurations will continue to work as before, with helper pods maintaining their default
system-node-criticalpriority class when not explicitly configured.