Skip to content

Eai 1452 seaweed s3 rebased 4#747

Merged
brownzebra merged 51 commits into
mainfrom
EAI-1452-seaweed-s3-rebased-4
Jun 12, 2026
Merged

Eai 1452 seaweed s3 rebased 4#747
brownzebra merged 51 commits into
mainfrom
EAI-1452-seaweed-s3-rebased-4

Conversation

@blankdots

Copy link
Copy Markdown
Contributor

No description provided.

@blankdots blankdots requested a review from a team as a code owner June 12, 2026 14:31
pwistbac and others added 21 commits June 12, 2026 17:33
The secret-manager script was strictly write-once: any existing path in
OpenBao was always skipped. An unrendered template literal previously
stored by the old cat|sed bootstrap (e.g. {{ .Values.minio.consoleAccessKey }})
could therefore never be corrected, and leaked through the seaweedfs
ExternalSecret into the rendered s3.json.

Now static secrets are reconciled: if the stored value differs from the
desired envsubst-rendered value, it is updated so downstream
ExternalSecrets converge. random secrets stay write-once so generated
credentials are never regenerated.
@blankdots blankdots force-pushed the EAI-1452-seaweed-s3-rebased-4 branch from 4f37bd7 to 4aa82c0 Compare June 12, 2026 14:35
@brownzebra brownzebra enabled auto-merge June 12, 2026 14:42

@brownzebra brownzebra left a comment

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.

LGTM

@brownzebra brownzebra merged commit bf68248 into main Jun 12, 2026
8 checks passed
@brownzebra brownzebra deleted the EAI-1452-seaweed-s3-rebased-4 branch June 12, 2026 14:42
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.

4 participants