Skip to content

Add OpenBao Variable Provider - #3719

Merged
fibonacci1729 merged 2 commits into
spinframework:mainfrom
thorstenhans:feature/openbao-provider
Sep 28, 2026
Merged

fibonacci1729 merged 2 commits into
spinframework:mainfrom
thorstenhans:feature/openbao-provider

Conversation

@thorstenhans

@thorstenhans thorstenhans commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

The underlying vaultrs allows us connecting to Vault and OpenBao. I've added dedicated provider and shared the impl between Vault and OpenBao.

Although this being mainly sugar coating on top of the already existing implementation, it's worth it IMO. OpenBao is a truly community driven alternative under the CNCF umbrella umbrella besides Vault that should be supported.

Signed-off-by: Thorsten Hans <thans@akamai.com>
@lann

lann commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Is there any difference between the two implementations? If not then perhaps this could just be a #[serde(alias = "openbao")] on the VariableProviderConfiguration::Vault variant?

@lann

lann commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

or rename everything Vault -> OpenBao and make it #[alias = "vault"] - I don't really have a strong opinion either way 🙂

@lann lann changed the title Add OpenBao Varbiable Provider Add OpenBao Variable Provider Sep 16, 2026
@thorstenhans

Copy link
Copy Markdown
Contributor Author

@lann 🤦🏻 yeah.. obviously the way smarter approach for the implementation. Haven't considered that as an feasible approach... Let' me test locally before refactoring it in that way.

@github-project-automation github-project-automation Bot moved this to Backlog in Spin 4.2 Sep 17, 2026
@michelleN michelleN moved this from Backlog to In progress in Spin 4.2 Sep 17, 2026
Signed-off-by: Thorsten Hans <thans@akamai.com>
@thorstenhans

Copy link
Copy Markdown
Contributor Author

@lann removed dedicated types and went with the alias. Also verified using the existing sample and docs that have been PR'ed to spin-docs and samples repository

@fibonacci1729
fibonacci1729 merged commit efd26d1 into spinframework:main Sep 28, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants