Skip to content

Add settings.platform.disableRegularOrgCreation - #1337

Merged
ArnobKumarSaha merged 1 commit into
masterfrom
arnob-org-creation
Sep 17, 2026
Merged

ArnobKumarSaha merged 1 commit into
masterfrom
arnob-org-creation

Conversation

@ArnobKumarSaha

Copy link
Copy Markdown
Member

Companion to appscode-cloud/b3#1656.

b3 reads [admin] DISABLE_REGULAR_ORG_CREATION from the merged platform config and lets a siteadmin flip it at runtime, writing back to the ace-user-config secret. This adds the chart value that seeds it.

Why user.ini and not app.ini

ace-config is re-rendered by helm on every upgrade, so a runtime write there is reverted. ace-user-config is created only when absent (lookup guard + helm.sh/resource-policy: keep) and is the secret b3 patches and republishes over NATS — the same place branding and DISABLE_REGISTRATION live.

Default is null, not a deploymentType check

b3 derives AppsCodeHosted from the domain (info.HostedDomain(Domain)), not from deploymentType, so a chart-side guess could contradict it. Left unset, b3 defaults the flag to !AppsCodeHosted — disabled on selfhosted, enabled on appscode.com.

The template emits the [admin] section only when the value is an actual bool (kindIs "bool"), so an explicit false is not silently dropped the way with would drop it.

Existing installs are untouched: the secret already exists, so the template does not render, and the b3-side default covers them.

Verified

helm template for unset / =true / =false, and go build ./....

Seeds [admin] DISABLE_REGULAR_ORG_CREATION into the ace-user-config
secret so a siteadmin can toggle regular org creation at runtime. Left
null by default, which lets the platform pick: disabled on selfhosted,
enabled on the hosted platform.

Signed-off-by: Arnob kumar saha <arnob@appscode.com>
@ArnobKumarSaha
ArnobKumarSaha merged commit 9068b55 into master Sep 17, 2026
4 checks passed
@ArnobKumarSaha
ArnobKumarSaha deleted the arnob-org-creation branch September 17, 2026 11:53
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