-
Notifications
You must be signed in to change notification settings - Fork 130
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
Add annotation hashes for Manager / KB to roll pods over on kb cert c… #1119
Add annotation hashes for Manager / KB to roll pods over on kb cert c… #1119
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.
LGTM, but maybe @tmjd can take a look too
@@ -109,6 +109,8 @@ const ( | |||
EsManagerRoleBinding = "es-manager" | |||
EsKubeControllerRole = "es-calico-kube-controllers" | |||
EsKubeControllerRoleBinding = "es-calico-kube-controllers" | |||
|
|||
KibanaTLSAnnotationHash = "hash.operator.tigera.io/tls-certificate-secret" |
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.
nit: maybe there could be better names for the variable and value - they don't really line up well
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.
Could we use hash.operator.tigera.io/kibana-secrets
here too?
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.
The changes look good, I'd like to see consistency in the naming though. (I know the operator isn't very consistent with some names but it won't get better if we don't try 😀 )
For expediency I'm going to approve, fixing the names isn't required and if you fix it I don't need to review it again.
…hange
Description
For PR author
make gen-files
make gen-versions
For PR reviewers
A note for code reviewers - all pull requests must have the following:
kind/bug
if this is a bugfix.kind/enhancement
if this is a a new feature.enterprise
if this PR applies to Calico Enterprise only.