Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: upgrade bank-vaults #127

Merged
merged 3 commits into from
Jul 26, 2023
Merged

feat: upgrade bank-vaults #127

merged 3 commits into from
Jul 26, 2023

Conversation

sagikazarmark
Copy link
Member

Overview

Fixes #113

@sagikazarmark sagikazarmark added this to the v1.21.0 milestone Jul 24, 2023
@sagikazarmark sagikazarmark requested a review from ramizpolic July 24, 2023 15:52
@sagikazarmark sagikazarmark force-pushed the bank-vaults-upgrade branch 3 times, most recently from ac3d8ab to c2c1142 Compare July 24, 2023 16:37
@akijakya
Copy link
Member

Hi! I tested why the softhsm test fails locally, the configurer throws this error:

error creating kv store: error creating HSM kv store: failed to open p11 module: /usr/lib/softhsm/libsofthsm2.so: failed to load module \"/usr/lib/softhsm/libsofthsm2.so\

And the reason seems to be that softhsm is no longer built into the bank-vaults image. See also the comment in the CR used by the test case.

@sagikazarmark
Copy link
Member Author

Thanks for checking!

Looks like I forgot to put softhsm back to the image.

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@sagikazarmark
Copy link
Member Author

Finally!

@sagikazarmark sagikazarmark merged commit 45651d9 into main Jul 26, 2023
@sagikazarmark sagikazarmark deleted the bank-vaults-upgrade branch July 26, 2023 22:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update bank vaults version after new release
3 participants