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

feat: add extra annotations to customize path of injected token and permissions of the token file sink on vault #650

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

Conversation

srevinsaju
Copy link

adds:

  • vault.hashicorp.com/agent-inject-token-file to specify a different path other than ${vault_volume_name}/token
  • vault.hashicorp.com/agent-inject-token-perms to specify a different permission for the token file, other than the current default, 0640

@srevinsaju srevinsaju requested a review from a team as a code owner June 1, 2024 22:05
…ermissions of the token file sink on vault

adds:
* `vault.hashicorp.com/agent-inject-token-file` to specify a different
  path other than `${vault_volume_name}/token`
* `vault.hashicorp.com/agent-inject-token-perms` to specify a different
  permission for the token file, other than the current default, 0640
@srevinsaju srevinsaju force-pushed the feature/vault-token-annotations-customization branch from 84762d7 to 9db0d8a Compare June 1, 2024 22:46
@dlitster
Copy link

dlitster commented Aug 2, 2024

This would be great. Ran into some trouble with this issue using a distroless container this week. Is this linked to a feature request or github issue?

@srevinsaju
Copy link
Author

nope, it’s currently not linked to a feature request/issue right now

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