Skip to content
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

Ignoring changes in-cluster changes to annotation & label #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lucapericlp
Copy link

Not sure if this will be helpful, I haven't looked into this deeply but I'm assuming that after the first terraform apply, annotations & labels are getting updated for the flyte-tls-secret (by cert-manager?). I've had to add this in order to make any future changes without disrupting current state.

@lucapericlp
Copy link
Author

@davidmirror-ops can't assign you as a reviewer so just light ping. This repo was super useful in setting things up, kudos!

@davidmirror-ops
Copy link
Collaborator

@lucapericlp thanks for the nice comments and the PR!

I'm assuming that after the first terraform apply, annotations & labels are getting updated for the flyte-tls-secret (by cert-manager?).

Good question. I don't have a GKE environment handy but could you share the output of kubectl describe secret flyte-tls-secret -n flyte? I'm not familiar with a use case that would change secret's labels or annotations after deployment

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