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

Bad values if vault send error #53

Open
gyrter opened this issue Oct 4, 2022 · 2 comments
Open

Bad values if vault send error #53

gyrter opened this issue Oct 4, 2022 · 2 comments

Comments

@gyrter
Copy link

gyrter commented Oct 4, 2022

Describe the bug

Helm-vault plugin can install release with out any error with bad values, when Vault return HTTP error.

To Reproduce

Steps to reproduce the behavior:

  1. Create release with a lot of values in Vault.
  2. Start deploy
  3. Turn off Vault during release
  4. See error. For example in fount this in pull secret deploy-token:%!s(<nil>)

Expected behavior

Plugin supress vault error.

Environment:

  • OS: Linux
  • Python Version 3.9
  • Vault Version current from master
@deniskv
Copy link

deniskv commented Dec 1, 2022

Any updates?

@Just-Insane
Copy link
Owner

Hello,

I haven't worked with this project in quite a while and no longer use Vault.

It seems like we aren't checking or handling errors when making API calls to vault. I'll have to look, but I'm pretty sure we use the vault CLI client in python, so it should be relatively easy to add error handling, I hope.

I'll take a look at this tomorrow if I have time and will update the thread.

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

3 participants