openbao / openbao-helm

Helm chart to install OpenBao and other associated components.
Mozilla Public License 2.0
14 stars 7 forks source link

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

Closed jessebot closed 4 months ago

jessebot commented 4 months ago

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: https://github.com/openbao/openbao-helm/blob/b59b6e55bb124e6486f861feeb15c2428096634b/charts/openbao/templates/tests/server-test.yaml#L36

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

jessebot commented 4 months ago

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 commented 4 months ago

@jessebot Yes, the cli command is bao.