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

Document auto restart of pods on secret rotation #1647

Open
ewan-chalmers opened this issue Sep 27, 2024 · 3 comments · May be fixed by #1648
Open

Document auto restart of pods on secret rotation #1647

ewan-chalmers opened this issue Sep 27, 2024 · 3 comments · May be fixed by #1648
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@ewan-chalmers
Copy link

ewan-chalmers commented Sep 27, 2024

Describe the solution you'd like
When secret auto-rotation is enabled, users of Secrets Store CSI Driver may look for a means to have workload pods automatically restarted when a mounted secret is updated. While restart code is assumed not to be in scope of the driver project, the project could outline a suggested approach.

Anything else you would like to add:
I have implemented a kubernetes controller to optionally restart pods when mounted secrets are updated.

I have a document ready to propose for inclusion in docs/topics.

Note that Reloader does not cover the case where secrets are mounted in pod volumes.

This follow from discussion on the slack community: https://kubernetes.slack.com/archives/C013PUP2WRK/p1727349029443369?thread_ts=1724065955.676549&cid=C013PUP2WRK
Environment:

  • Secrets Store CSI Driver version: (use the image tag): N/A
  • Kubernetes version: (use kubectl version): N/A
@ewan-chalmers ewan-chalmers added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 27, 2024
@ewan-chalmers
Copy link
Author

I would like to claim the issue. I have a doc ready for PR

@ewan-chalmers ewan-chalmers linked a pull request Sep 27, 2024 that will close this issue
3 tasks
@enj
Copy link
Contributor

enj commented Sep 30, 2024

(just for PR review purposes)

/assign aramase nilekhc

@ewan-chalmers
Copy link
Author

@aramase @nilekhc is there something more I could do to move this forward?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants