-
Notifications
You must be signed in to change notification settings - Fork 743
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
Adding modifiable priority class to controller-manager and audit deployment #1008
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1008 +/- ##
==========================================
+ Coverage 46.70% 47.05% +0.35%
==========================================
Files 62 62
Lines 3987 3987
==========================================
+ Hits 1862 1876 +14
+ Misses 1879 1871 -8
+ Partials 246 240 -6
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
df1f593
to
231b61c
Compare
@BBzzz can you run |
Hi @sozercan! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
f0c479d
to
7236c6b
Compare
7236c6b
to
4fbb611
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sorry for the delay! It looks like I don't have permission to rebase this PR off of current master. Any chance you can do so, @BBzzz, to unblock merging? |
…oyment Signed-off-by: Botond Borbely <botond.borbely@nokia.com>
4fbb611
to
2459a84
Compare
What this PR does / why we need it:
Introducing the priority class to this deployment.
This value can be set from outside (e.g. when installing this helm chart)