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

chart testing workflow failing due to old vault command? #9

Closed
jessebot opened this issue May 20, 2024 · 2 comments · Fixed by #11
Closed

chart testing workflow failing due to old vault command? #9

jessebot opened this issue May 20, 2024 · 2 comments · Fixed by #11
Labels
bug Something isn't working

Comments

@jessebot
Copy link

jessebot commented May 20, 2024

Describe the bug

We're getting the following in the server-test pod during the test workflow:

------------------------------------------------------------------------------------------------------------------------
==> Logs of container openbao-zjqq3ibo8s-server-test
------------------------------------------------------------------------------------------------------------------------
Checking for sealed info in 'vault status' output
Attempt 0...
/bin/sh: line 6: vault: command not found
Attempt 1...
/bin/sh: line 6: vault: command not found
Attempt 2...
/bin/sh: line 6: vault: command not found
Attempt 3...
/bin/sh: line 6: vault: command not found
Attempt 4...
/bin/sh: line 6: vault: command not found
Attempt 5...
/bin/sh: line 6: vault: command not found
Attempt 6...
/bin/sh: line 6: vault: command not found
Attempt 7...
/bin/sh: line 6: vault: command not found
Attempt 8...
/bin/sh: line 6: vault: command not found
Attempt 9...
/bin/sh: line 6: vault: command not found
timed out looking for sealed info in 'vault status' output

Example job where it failed:
https://github.com/openbao/openbao-helm/actions/runs/9165363410/job/25198629301#step:8:395

Environment

github workflow in this repo

Additional context

Pretty sure this is the smoking gun:

vault status -format yaml | grep -E '^sealed: (true|false)' && break

I assume we want this to be changed to bao status, yes? Correct me if I'm wrong. I haven't tried it yet 😅

@jessebot jessebot added the bug Something isn't working label May 20, 2024
@jessebot
Copy link
Author

I am working on this first, so that when I work on other issues, we have at least a valid chart test. This can be assigned to me.

@naphelps
Copy link
Member

@jessebot Yes, the cli command is bao.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants