Skip to content

feat(dgraph): add optional ACL bootstrap reconciler and post-install validation subsystem - #153

Open
mlwelles wants to merge 1 commit into
feat-native-tlsfrom
feat-acl-bootstrap-and-validation
Open

feat(dgraph): add optional ACL bootstrap reconciler and post-install validation subsystem#153
mlwelles wants to merge 1 commit into
feat-native-tlsfrom
feat-acl-bootstrap-and-validation

Conversation

@mlwelles

Copy link
Copy Markdown

Add two optional, default-off subsystems: an ACL bootstrap reconciler (rotates groot off its default password, converges declarative users/groups) and a post-install validation subsystem (a helm test Pod, an optional gating hook Job, and a suspended manual-trigger CronJob). Both are absent from a stock render.

Note for maintainers: these are the most opinionated pieces, deliberately at the top of the stack so they can be dropped without unwinding the rest. The reconciler guesses groot's default password on first run to rotate it; the gating Job fails helm upgrade on a failed check (off by default).

Stacked on feat-native-tls; will retarget to main as lower PRs merge.

Part of splitting #140 into per-area PRs. The merge of all split PRs reproduces #140's tree byte-for-byte. #140 is being closed as superseded.

…validation subsystem

Both are default-off and inert on a stock install: nothing renders unless
explicitly enabled.

The ACL bootstrap reconciler is a post-install/post-upgrade hook Job that
rotates groot off its default password and converges the declared users and
groups. It is idempotent, so it is safe to re-run on every helm upgrade.

The validation subsystem asserts the running cluster matches what the chart
rendered (health, membership, ACL enforcement, admin and per-user logins,
group rules, and optionally backup CronJob schedules). It can gate a release
via a post-install hook Job or run on demand via helm test; a suspended
manual-trigger CronJob is also provided.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant