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 support for shareProcessNamespace #168

Closed
marcusianlevine opened this issue Jan 9, 2020 · 1 comment
Closed

Add support for shareProcessNamespace #168

marcusianlevine opened this issue Jan 9, 2020 · 1 comment

Comments

@marcusianlevine
Copy link

We are using TLS certificates issued via LetsEncrypt, which means we need to periodically reload the Vault servers to pick up the new certificates.

The chart already supports sidecar containers via server.extraContainers, but in order to issue the SIGHUP signal and trigger a restart, the Pod spec needs shareProcessNamespace: true to be available

Ideally this would just be a flag like server.shareProcessNamespace with a default value of false

I'm happy to open a PR for this feature if the maintainers are open to the idea

@jasonodonnell
Copy link
Contributor

This functionality was added in #174. You can expect this to be in the next release!

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

No branches or pull requests

2 participants