-
Notifications
You must be signed in to change notification settings - Fork 0
feat(contabo): enable mendys service-db provisioning (finish #183) #184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 8 commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
cf68db2
feat(contabo): enable mendys service-db provisioning (finish #183)
github-actions[bot] 5398ab3
Merge branch 'main' into claude/issue-183-20260707-1704
izzywdev 2bde1ab
fix(contabo): disable secret-less svc-db entries so mendys can provision
github-actions[bot] 1de7099
Merge remote-tracking branch 'origin/main' into claude/issue-183-2026…
github-actions[bot] 5f346bb
fix(ci): fix gate-sast pip path + test-infrastructure Docker socket
izzywdev 99c30fe
fix(ci): skip test-infrastructure failure when Docker unavailable on …
izzywdev 1cdd2eb
fix(arc): enable Docker-in-DinD sidecar for staging runner
izzywdev 4bf9c57
Merge branch 'main' into claude/issue-183-20260707-1704
izzywdev 9e97d9c
fix(contabo): re-enable fuzesales/fuzecontact/fuzeservice provisioning
izzywdev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@claude why disable those services? they are already now on fuzeinfra
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Claude finished @izzywdev's task in 1m 28s —— View job
Why
fuzesales/fuzecontact/fuzeserviceare disabledShort version: their credentials secrets aren't on the cluster yet, and the provisioning hook is a single shared pod — so leaving them
enabled: truewedges provisioning for every service, includingmendys. I re-verified against prod just now:1. The three secrets do not exist in
fuzeinfra(kubectl -n fuzeinfra get secrets):No
fuzesales-db-credentials,fuzecontact-db-credentials, orfuzeservice-db-credentials.2. Their Postgres roles don't exist either (
SELECT rolname ... LIKE '%_svc'onfuzeinfra-postgres-0):No
fuzesales_svc/fuzecontact_svc/fuzeservice_svc, and nofuzesales/fuzecontact/fuzeservicedatabases.3. The live provisioning Job is currently wedged for exactly this reason. Because
mainstill has those three enabled without their secrets, the hook pod has been inCreateContainerConfigErrorfor ~4 days:That single pod mounts every enabled service's password via a non-optional
secretKeyRef, so one missing secret fails the whole pod → nothing gets provisioned,mendysincluded. That's the whole reason this PR'smendysenablement wouldn't actually take effect until those three are turned back off.So "they're already on fuzeinfra"?
The apps may well be deployed, but their per-service DB credentials haven't been sealed into the
fuzeinfranamespace, and their roles/dbs were never created. Theenabledflag here doesn't track "is the app running" — it tracks "has this service's password Secret landed infuzeinfraso the shared Job can provision its role." For these three, it hasn't. Flipping them on before the secret exists is what caused the 4-day wedge in the first place, and it's what the file's own GITOPS GATE comment warns against.To enable them (the right way)
Same path we used for
mendys(FuzeKeys #136 precedent): land each service's*-db-credentialsSecret infuzeinfrafirst, then flipenabled: truein that same PR. If you want, I can apply the hand-off Secrets for the three now (given their passwords) and re-enable all three in this PR — that would provisionfuzesales_svc/fuzecontact_svc/fuzeservice_svctoo. Just say the word and provide/point me at the passwords (or confirm you want them generated + sealed).Leaving them disabled here is deliberate and non-destructive: it unblocks
mendyswithout regressing anything (those roles/dbs don't exist to lose).·
claude/issue-183-20260707-1704