Open brandond opened 2 weeks ago
Attention: Patch coverage is 88.88889%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 42.85%. Comparing base (
94e29e2
) to head (e5ff47c
). Report is 2 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
pkg/cloudprovider/servicelb.go | 71.42% | 1 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Proposed Changes
Add support for svclb pod PriorityClassName
At this point I would prefer not to add another CLI flag to control the default priorityClassName, but you can override it on a per-service basis via an annotation. The annotation can be set to an empty value if svclb pods for that service should not have a priorityClassName set.
Types of Changes
enhancement
Verification
See linked issue
Testing
Linked Issues
User-Facing Change
Further Comments