-
Notifications
You must be signed in to change notification settings - Fork 82
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
Supporting custom labels/annotations on deployment/pod #105
Conversation
d4c09a0
to
443b7b0
Compare
4c11179
to
876a717
Compare
876a717
to
a475807
Compare
@jkroepke this needs rebase as well |
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
e56324c
to
4597e91
Compare
@sagikazarmark rebased. |
@sagikazarmark can you please take a look here again? |
@sagikazarmark A friendly reminder. |
@sagikazarmark I would appreciate a review here, please let me know, if I can assist here. |
charts/dex/values.yaml
Outdated
@@ -107,8 +107,15 @@ rbac: | |||
# If disabled, the operator is responsible for creating the necessary resources (ClusterRole and RoleBinding or CRD's) | |||
createClusterScoped: true | |||
|
|||
# -- Annotations to be added to deployment. | |||
annotations: {} |
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.
How about deploymentAnnotations
and deploymentLabels
?
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.
good call, changed.
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de> Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
e01732f
to
3dd4a1d
Compare
@sagikazarmark I address all your points. |
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.
A few more nits. Other than those, it's good to go.
Co-authored-by: Márk Sági-Kazár <sagikazarmark@users.noreply.github.com> Signed-off-by: Jan-Otto Kröpke <github@jkroepke.de>
@sagikazarmark Applied, thanks for review! |
Thanks @jkroepke ! |
Signed-off-by: Jan-Otto Kröpke joe@cloudeteer.de
Overview
Added the ability to define custom annotations on deployment/pods
Tools like https://github.com/stakater/Reloader depends on that.
What this PR does / why we need it
Special notes for your reviewer
Checklist
Chart.yaml
(see the contributing guide for details)Chart.yaml
(see the contributing guide for details)make docs