Skip to content

Conversation

@ksa-real
Copy link
Contributor

@ksa-real ksa-real commented Feb 28, 2022

This is useful for immutable secrets which are typically named as secret-base-name-<.data hash>. Previously old and new secrets were considere completely independent ConfigMap/Secret. With this annotation user can add "helm.sh/base-name: secret-base-name" annotation and compare old and new objects similar to git rename.

#378

@ksa-real
Copy link
Contributor Author

ksa-real commented Apr 8, 2022

@mumoshu Can you please take a look?

This is useful for immutable secrets which are typically named as
secret-base-name-<.data hash>. Previously old and new secrets were
considere completely independent ConfigMap/Secret. With this annotation
user can add "helm.sh/base-name: secret-base-name" annotation and
compare old and new objects similar to git rename.
Copy link
Collaborator

@mumoshu mumoshu left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks a lot for your contribution @ksa-real!

@ksa-real
Copy link
Contributor Author

Would you merge it?

@mumoshu
Copy link
Collaborator

mumoshu commented Apr 26, 2022

@ksa-real Yeah. I was just waiting for CI to pass.

@mumoshu mumoshu merged commit fec08e5 into databus23:master Apr 26, 2022
@ksa-real
Copy link
Contributor Author

Do you do releases? It seems your release script is somewhat broken.

@mumoshu
Copy link
Collaborator

mumoshu commented Apr 27, 2022

@ksa-real I don't do releases so often so if you need to use it real quick, you'd better build and install locally by running make install

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants