Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[users] Allow user to configure default group append behavior.#2642

Open
TwiggyWan wants to merge 1 commit into
debops:masterdebops/debops:masterfrom
TwiggyWan:users/make_append_configurableTwiggyWan/debops:users/make_append_configurableCopy head branch name to clipboard
Open

[users] Allow user to configure default group append behavior.#2642
TwiggyWan wants to merge 1 commit into
debops:masterdebops/debops:masterfrom
TwiggyWan:users/make_append_configurableTwiggyWan/debops:users/make_append_configurableCopy head branch name to clipboard

Conversation

@TwiggyWan

Copy link
Copy Markdown

Because users configuration can span many group_vars/ and hosts_vars/, current role forces one to set 'append' for each user if one wants to always set 'append' to 'false'.

This is not practical with users configuration spread in many group_vars/ and hosts/vars/.

The solution is to give the user the ability to set a default value through a global default variable.
It is set to 'True' by default, therefore will not alter existing inventories.

See #2641

@TwiggyWan

Copy link
Copy Markdown
Author

please advice on variable name.

@TwiggyWan
TwiggyWan force-pushed the users/make_append_configurable branch from 23ef1ff to 25dedab Compare February 5, 2026 09:15
@TwiggyWan
TwiggyWan force-pushed the users/make_append_configurable branch from 25dedab to d3fd7fa Compare April 2, 2026 10:47
Because users configuration can span many group_vars/ and hosts_vars/,
current role forces one to set 'append' for each user if one wants
to always set 'append' to 'false'.

This is not practical with users configuration spread in many group_vars/
and hosts_vars/.

The solution is to give the user the ability to set a default value through
a global default variable.
It is set to 'True' by default, therefore will not
alter existing inventories.

See debops#2641
@TwiggyWan
TwiggyWan force-pushed the users/make_append_configurable branch from d3fd7fa to e46a09f Compare April 30, 2026 15:19
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.