Skip to content

Add Master OpenBao Instance#56

Merged
Kidswiss merged 5 commits into
masterfrom
feat/master-openbao
May 7, 2026
Merged

Add Master OpenBao Instance#56
Kidswiss merged 5 commits into
masterfrom
feat/master-openbao

Conversation

@norbertgruszka
Copy link
Copy Markdown
Collaborator

The OpenBao service supports an autounseal configuration. To enable proper end-to-end testing of this feature, a new target has been added to kindev: the Master OpenBao Instance.

What's included:

  • Master OpenBao Instance — a dedicated OpenBao instance that self-initializes via a Kubernetes Job on startup
  • Credentials storage — the root token is saved to the master-openbao-init-credentials Secret after initialization
  • Transit engine setup — the init Job also configures a transit engine, which is required for autounsealing of other OpenBao clusters
  • Documentation — an included README.md provides step-by-step instructions for generating a new token for autounseal use

@norbertgruszka norbertgruszka requested a review from Kidswiss May 6, 2026 11:40
Copy link
Copy Markdown
Contributor

@Kidswiss Kidswiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's mostly just nits.

Comment thread openbao/README.md
make master-openbao-setup
```

**By default, Master OpenBao is not installed as part of `vshnpostgresql` nor `vshnall`.**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Idea: let's introduce a vshnopenbao and add it to vshnall.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good to me, I renamed master-openbao to vshnopenbao. The service name convention matches other services now.

Comment thread master-openbao/README.md Outdated
bao status

# Generate token required for auto-unseal of another OpenBao instance
bao token create -orphan -policy="autounseal" -period=24h -format=json > auto-unseal.json
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't done bao in a while, but this will create a periodic token, that will keep itself from expiring if it's used within the specified period, right?

I'd bump that up to 100h or something for the dev instance, otherwise we'd have to re-generate tokens again after a weekend.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can make it indefinite with

bao token create -orphan -policy="autounseal" -no-default-policy -format=json > auto-unseal.json

so it never expires. I just added updated command to README.md

Comment thread openbao/values.yml
command:
- "/bin/sh"
- "-cex"
- "chown -R 100:1000 /openbao/data"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to be sure 100:1000 is not a typo?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be correct, I think those are default values for UID and GID in OpenBao Helm Chart.

Comment thread Makefile Outdated
vcluster-clean: ## If you break Crossplane hard enough just remove the whole vcluster
$(vcluster_bin) rm controlplane || true

master-openbao-setup: $(master_openbao_sentinel) ## Install local forgejo instance to host argocd repos
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a target to renew the unseal token manually would be nice.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd keep it in README.md for now. Let's see how often it is used and maybe then we can add it as a target?

@Kidswiss Kidswiss merged commit 6e288ca into master May 7, 2026
1 check passed
@norbertgruszka norbertgruszka deleted the feat/master-openbao branch May 7, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants