Skip to content

chore(oss-prep): replace dev-default secrets with CHANGE_ME, parametrize MONGO_ADMIN_URI#18

Merged
mastermanas805 merged 1 commit into
masterfrom
redact/oss-prep-2026-05-21
May 21, 2026
Merged

chore(oss-prep): replace dev-default secrets with CHANGE_ME, parametrize MONGO_ADMIN_URI#18
mastermanas805 merged 1 commit into
masterfrom
redact/oss-prep-2026-05-21

Conversation

@mastermanas805
Copy link
Copy Markdown
Member

Per OSS-01-secrets-audit + OSS-05-infra-split. k8s/secrets.yaml: 4 dev-default strings → CHANGE_ME. k8s/data/minio-secret.yaml: same MinIO defaults → CHANGE_ME (note: in-cluster MinIO retired 2026-05-20). k8s/provisioner/deployment.yaml: inline mongodb://root:root@... → secretKeyRef. k8s/data/mongodb.yaml: append mongodb-admin Secret stub for the secretKeyRef target. Operator NOTE: git history still contains the dev-default values — run git filter-repo before publishing if history-clean required. DO NOT auto-apply — infra has no auto-deploy; operator must kubectl apply after reviewing.

…ize MONGO_ADMIN_URI

Per /tmp/oss-plan-2026-05-21/OSS-01-secrets-audit.md + OSS-05-infra-split.md:

- k8s/secrets.yaml: PLATFORM_DB_PASSWORD + CUSTOMER_DB_PASSWORD + MINIO_ROOT_USER + MINIO_ROOT_PASSWORD → CHANGE_ME (were 'instant'/'instant_cust'/'minioadmin'/'minioadmin123' — local-dev convenience defaults that operators MUST replace)
- k8s/data/minio-secret.yaml: same MinIO defaults → CHANGE_ME, comment notes self-hosted MinIO is retired
- k8s/provisioner/deployment.yaml: MONGO_ADMIN_URI inline value 'mongodb://root:root@...' → secretKeyRef pointing at mongodb-admin Secret (matches the pattern every other credential already uses)
- k8s/data/mongodb.yaml: append mongodb-admin Secret stub (with CHANGE_ME placeholder) so the secretKeyRef has a target

This unlocks the infra repo for OSS publication. Note: git history still contains the dev-default values — operator should run `git filter-repo` before publishing if history-clean is required.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mastermanas805 mastermanas805 merged commit 07277f4 into master May 21, 2026
0 of 2 checks passed
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.

1 participant