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

Fix bug creating serviceAccount for external Vault #681

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

Conversation

Iristyle
Copy link
Contributor

@Iristyle Iristyle commented Jan 14, 2022

Stumbled across this one while adding tests for #680

  • Fixes serviceaccount specs to use the correct file (originally
    introduced in 2b2b0dd)

  • Fixed a bug with serviceAccount getting created even
    when an external Vault is specified

@tvoran tvoran added bug Something isn't working vault-server Area: operation and usage of vault server in k8s labels Jan 14, 2022
@hashicorp-cla
Copy link

hashicorp-cla commented Mar 12, 2022

CLA assistant check
All committers have signed the CLA.

@Iristyle Iristyle force-pushed the fix-serviceaccount-creation-for-external-vault branch from f66a3bd to 7a3028d Compare April 4, 2022 17:43
 - Fixes serviceaccount specs to use the correct file (originally
   introduced in 2b2b0dd)

 - Fixed a bug with serviceAccount getting created even
   when an external Vault is specified
@Iristyle Iristyle force-pushed the fix-serviceaccount-creation-for-external-vault branch from 7a3028d to bdd3fe7 Compare April 4, 2022 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working vault-server Area: operation and usage of vault server in k8s
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants