Repository navigation
feat(tenants): tenant.mail voor SMTP per tenant - #146
Draft
rubenvdlinde wants to merge 1 commit into
Draft
rubenvdlinde wants to merge 1 commit into
rubenvdlinde wants to merge 1 commit into
Conversation
Een tenant zet SMTP aan met tenant.mail (enabled, fromAddress, domain, optioneel port en secure). De ApplicationSet zet dan MAIL_*/SMTP_* env vars (host en login uit nextcloud-secrets: smtp-host, smtp-username, smtp-password, optional) en zet smtp.config.php van de chart aan voor alleen die tenant. create-tenant-secret.sh schrijft de smtp-* sleutels mee als SMTP_HOST, SMTP_USERNAME en SMTP_PASSWORD gezet zijn. validate-values.sh controleert tenant.mail, met elf fixtures. Alle 89 tenants renderen byte voor byte gelijk (values en manifests). Aanleiding: demotenant zuiddrecht-prod (registratie, Woo, notificaties).
rubenvdlinde
added
the
change/platform
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Wat
Een tenant kan nu mail versturen via SMTP met
tenant.mailin zijntenantbestand (
enabled,fromAddress,domain, optioneelportensecure). Aanleiding: de demotenantzuiddrecht-prodheeft mail nodig voorregistratie en notificaties, en geen tenant op het platform had SMTP.
tenant.mail.enabled: truede env varsMAIL_FROM_ADDRESS,MAIL_DOMAIN,SMTP_SECURE,SMTP_PORTen, uitnextcloud-secrets,SMTP_HOST,SMTP_NAME,SMTP_PASSWORD, en zetnextcloud.defaultConfigs.smtp.config.phpaan voor alleen die tenant.smtp-host,smtp-username,smtp-password) zijnoptional. Bewust niet vianextcloud.mail.enabledvan de chart: daar zijnze verplicht en blijft de pod in
CreateContainerConfigErrorzolang zeontbreken.
create-tenant-secret.shschrijft desmtp-*sleutels mee alsSMTP_HOST,SMTP_USERNAMEenSMTP_PASSWORDgezet zijn (alle drie of geen).env.examplenoemde ze al, het script deed er niets mee.validate-values.shcontroleerttenant.mail; elf nieuwe fixtures intests/cases/tenant-mail-*.Blast radius: geen. Alle 89 tenants renderen byte voor byte gelijk (values en
manifests, vergeleken met
helm templateen de goTemplate-emulatie van deApplicationSet). Alleen een tenant die
tenant.mailzet, krijgt nieuwe env varsen een extra configbestand, en herstart dus.
Nog niet: SMTP-sleutels voor tenants met
tenant.secrets.managed: true(ESO).Gecontroleerd
Lokaal:
scripts/verify.sh,nextcloud-platform/tests/run-tests.shenpre-commit run --all-files --hook-stage pre-pushgeven exit 0. Alle 89 bestaande tenants renderen byte-identiek (values en manifests). Niet gedraaid: niets tegen een cluster.Opt-in per tenant; platformwijziging, dus mergen ma tot do na 17:00 en daarna
nextcloud-platform-bootstrapsyncen. Eerste gebruiker: tenantzuiddrecht-prod(#145).🤖 Generated with Claude Code