Skip to content

CLOUD-589 Pass additional Statefulset labels and any Annotations from values.yaml#78

Open
andrewkro wants to merge 5 commits into
Percona-Lab:mainfrom
viatorinc:CLOUD-589-pass-statefulset-annotations
Open

CLOUD-589 Pass additional Statefulset labels and any Annotations from values.yaml#78
andrewkro wants to merge 5 commits into
Percona-Lab:mainfrom
viatorinc:CLOUD-589-pass-statefulset-annotations

Conversation

@andrewkro
Copy link
Copy Markdown

This addition to the statefulset template and values file allows custom
labels to be passed into the metadata labels block AND annotations.

Our usecase for annotations is to set our vault providers
(vault.security.banzaicloud.io) vault-role for permission to get
any secretes required (in this case, credentials.password)

CLOUD-589

This addition to the statefulset template and values file allows custom
labels to be passed into the metadata labels block AND annotations.

Our usecase for annotations is to set our vault providers
(vault.security.banzaicloud.io) vault-role for permission to get
any secretes required (in this case, credentials.password)

CLOUD-589
@it-percona-cla
Copy link
Copy Markdown

it-percona-cla commented Oct 28, 2020

CLA assistant check
All committers have signed the CLA.

@andrewkro andrewkro changed the title Pass additional Statefulset labels and any Annotations from values.yaml CLOUD-589 Pass additional Statefulset labels and any Annotations from values.yaml Oct 28, 2020
Add options to the readme page
Add new line in values EOF and edit comment to explain labels
Change auto-indenting from editor on annotations with loop

CLOUD-589
@andrewkro andrewkro marked this pull request as ready for review October 28, 2020 23:18
Base automatically changed from master to main February 11, 2021 11:10
{{- if .Values.statefulSet.podTemplate.labels }}
{{ toYaml .Values.statefulSet.podTemplate.labels | indent 8 }}
{{- end }}
{{- with .Values.statefulSet.podTemplate.annotations }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @andrewkro ! Sorry, but this got sidetracked somehow!
Could you please rebase on current main branch and the annotations for the pod will need to be changed a bit since we set annotations in case if platform is openshift so we need to merge them and not that one overrides the other values.
Thanks!

@tplavcic tplavcic removed their assignment Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants