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

Add helm chart post-hook option to clean up state secrets when kube agent run as deployment #50291

Open
deusxanima opened this issue Dec 16, 2024 · 2 comments
Labels
c-cdl Internal Customer Reference feature-request Used for new features in Teleport, improvements to current should be #enhancements helm

Comments

@deusxanima
Copy link
Contributor

What would you like Teleport to do?

Provide helm chart post-hook option to clean up state secrets of rotated teleport kube agents when kube agents are run as part of a deployment.

What problem does this solve?

Some customers run teleport kube-agents in deployment mode (vs. default sts mode). Currently teleport stores the state of each agent in a secret. When agents are managed manually as a deploy vs. sts, restarting the agents creates a new pod with a new secret, but the previous pod's state secret is left behind.

Teleport currently provides a helm post-hook to clean up stale state artifacts on helm delete, but not for kube pod restarts. This in turn leaves a large number of orphaned secrets behind which require manual cleanup.

If a workaround exists, please include it.

Manually track and remove old state secrets.

@deusxanima deusxanima added c-cdl Internal Customer Reference feature-request Used for new features in Teleport, improvements to current should be #enhancements labels Dec 16, 2024
@tigrato
Copy link
Contributor

tigrato commented Dec 17, 2024

Teleport kube agent chart running in deployment mode shouldn't use secrets. The lack of TELEPORT_REPLICA_NAME, KUBE_NAMESPACE and RELEASE_NAME env variables should prevent the pod from using secret storage.

Have you manually added this env vars?

@deusxanima
Copy link
Contributor Author

@tigrato - getting clarity and deploy configs from customer env. will update with further info and details

@zmb3 zmb3 added the helm label Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-cdl Internal Customer Reference feature-request Used for new features in Teleport, improvements to current should be #enhancements helm
Projects
None yet
Development

No branches or pull requests

3 participants