Skip to content
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

[KOTS]: configure blockNewUsers #9718

Merged
merged 2 commits into from
May 4, 2022
Merged

[KOTS]: configure blockNewUsers #9718

merged 2 commits into from
May 4, 2022

Conversation

mrsimonemms
Copy link
Contributor

@mrsimonemms mrsimonemms commented May 3, 2022

Description

The Installed exposes a blockNewUsers block in the config which limits who can create a new account in the installation. This PR adds this to the KOTS config and injects it in if configured - this is in a new section at the bottom called "User management".

By default, this is not enabled:

image

If enabled, it exposes a list of domains that can be added.

image

Related Issue(s)

Fixes #8906

How to test

Install dev-sje via KOTS

Release Notes

[KOTS]: configure blockNewUsers

Documentation

Simon Emms added 2 commits May 3, 2022 10:48
This is just for ease of development as `main.2968` missing jq in the
installer image. This will still be changed by the werft job when publishing
a new release
@mrsimonemms mrsimonemms marked this pull request as ready for review May 3, 2022 12:17
@mrsimonemms mrsimonemms requested a review from a team May 3, 2022 12:17
@mrsimonemms mrsimonemms changed the title [KOTS]: config blockNewUsers [KOTS]: configure blockNewUsers May 3, 2022
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label May 3, 2022
@roboquat roboquat merged commit 0886b21 into main May 4, 2022
@roboquat roboquat deleted the sje/kots-block-signups branch May 4, 2022 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note size/M team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option to configure blockNewUsers
3 participants