-
Notifications
You must be signed in to change notification settings - Fork 3
ADR for managed OpenBao on AppCat #161
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
Conversation
Kidswiss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds reasonable.
Using the integrated raft storage makes things a lot less complex than using an external database.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting this ADR @ioboi!
Overall I like it and I think using the Helm chart here makes total sense as it fits in well with the AppCat framework. While doing my own research on this I also leaned heavily towards this approach.
Just a few things to help me understand:
- How will auto-unseal be configured effectively?
- Regarding the authentication method stated in the document, is the kubernetes authentication method also required for ServiceAccounts to consume Secrets from the secrets engine?
- At the top of the document it states that it will be used for PKI and secrets management. In that case I assume both engines will be configured by default?
@mdnix thank you for the review
|
About the k8s integration: it probably warrants its own ADR. If we actually want to add it to the service offering in the first place. |
b1021ca to
db78a88
Compare
Replaces TODO deployment architecture with complete VSHNOpenBao CRD specification including service configuration, sizing, backup, monitoring, and maintenance parameters. Documents auto-unseal functionality with support for AWS KMS, Azure Key Vault, GCP KMS, and Transit providers, including AWS KMS example configuration.
Adds documentation for the default VSHN-managed auto-unseal configuration and includes a warning that customer-configured auto-unseal providers only support besteffort service level. Also fixes typo in section heading.
0603e83 to
97a1530
Compare
|
@Kidswiss rebased and ready to be merged 🎉 |
Summary
Checklist
change,decision,requirement/quality,requirement/functional,dependencyas they show up in the changelog