-
Notifications
You must be signed in to change notification settings - Fork 38
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
Issue 515: add customized labels support #514
Issue 515: add customized labels support #514
Conversation
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.
It looks like unit tests are failing. Could you please fix that?
Codecov Report
@@ Coverage Diff @@
## master #514 +/- ##
==========================================
- Coverage 71.78% 71.74% -0.04%
==========================================
Files 15 15
Lines 3459 3469 +10
==========================================
+ Hits 2483 2489 +6
- Misses 870 872 +2
- Partials 106 108 +2
Continue to review full report at Codecov.
|
Signed-off-by: Aaron Wu <[email protected]>
Signed-off-by: Aaron Wu <[email protected]>
Signed-off-by: Aaron Wu <[email protected]>
Signed-off-by: Aaron Wu <[email protected]>
Signed-off-by: Aaron Wu <[email protected]>
e7a5d53
to
f79bb56
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
* add annotation support Signed-off-by: Aaron Wu <[email protected]> * add fields to crds Signed-off-by: Aaron Wu <[email protected]> * check nil Signed-off-by: Aaron Wu <[email protected]> * add values, update doc, fix UT Signed-off-by: Aaron Wu <[email protected]> * add deepcopy generated code Signed-off-by: Aaron Wu <[email protected]>
Change log description
add controllerPodLabels and segmentStorePodLabels field in the crd, each one is a map with key as the label key and value as the label value.
Purpose of the change
Fixes #515
What the code does
add labels to the controller deployment and pods, and segmentstore sts and pods
How to verify it
deploy pravega with labels set see if labels are attached to pods