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

allow users to control group var merge order #22580

Merged
merged 4 commits into from
Mar 16, 2017
Merged

Conversation

bcoca
Copy link
Member

@bcoca bcoca commented Mar 13, 2017

SUMMARY

With ansible_group_priority var at each group will determine merge order with siblings,
falling back to sorted names when priority is equal as it did before.

Parent/children relationships still work as they did and have higher precedence than priority

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

group_vars

ANSIBLE VERSION
2.4

With ansible_group_priority var at each group will determine merge order with siblings,
falling back to sorted names when priority is equal as it did before.

Parent/children relationships still work as they did and have higher precedence than priority
@bcoca bcoca added this to the 2.4.0 milestone Mar 13, 2017
@ansibot ansibot added affects_2.3 This issue/PR affects Ansible v2.3 feature_pull_request needs_triage Needs a first human triage before being processed. labels Mar 13, 2017
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Mar 13, 2017
@mscherer
Copy link
Contributor

This seems ok, but since 1942cd3 had no doc, I think it is time to document it somewhere.

@bcoca
Copy link
Member Author

bcoca commented Mar 13, 2017

doc ... is that a new form of source code?

@ansibot ansibot added c:inventory/host needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Mar 15, 2017
@ansibot ansibot added the affects_2.4 This issue/PR affects Ansible v2.4 label Mar 16, 2017
@bcoca bcoca merged commit 4319ddd into ansible:devel Mar 16, 2017
@bcoca bcoca deleted the group_priority branch June 20, 2017 00:17
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 4, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.3 This issue/PR affects Ansible v2.3 affects_2.4 This issue/PR affects Ansible v2.4 c:inventory/group c:inventory/host c:inventory/ini feature This issue/PR relates to a feature request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants