Skip to content

Inventory plugin: Allow "parent_group" to have host variables in it #84372

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.

Sign up for GitHub

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

Jump to bottom
Open
thehe wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Inventory plugin: Allow "parent_group" to have host variables in it #84372

thehe wants to merge 1 commit into from
+1 −1

Conversation

@thehe
Copy link

@thehe thehe commented Nov 22, 2024

SUMMARY

Allow to build up multi-hierarchy group-structure by arbitary inventory-plugins.

e.g. Provider -> Site1 -> Datacenter1
or Provider -> Zone -> DatacenterName

where Zone and Site are dynamically given.

ISSUE TYPE
  • Feature Pull Request

@thehe
Inventory plugin: Allow "parent_group" to have host variables in it
8e01338 
Allow to build up multi-hierarchy  group-structure by arbitary inventory-plugins.

e.g. Provider -> Site1 -> Datacenter1
or Provider -> Zone -> DatacenterName

where Zone and Site are dynamically given.
@ansibot ansibot added feature This issue/PR relates to a feature request. needs_triage Needs a first human triage before being processed. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Nov 22, 2024
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Nov 22, 2024
@s-hertel s-hertel removed the needs_triage Needs a first human triage before being processed. label Dec 3, 2024
s-hertel
s-hertel reviewed Dec 3, 2024
View reviewed changes
Copy link
Contributor

@s-hertel s-hertel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a changelog and tests?

@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Dec 3, 2024
@ansibot ansibot added the needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html label Apr 15, 2025
@ansibot ansibot added the stale_pr This PR has not been pushed to for more than one year. label Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@s-hertel s-hertel s-hertel left review comments

Assignees

No one assigned

Labels

ci_verified Changes made in this PR are causing tests to fail. feature This issue/PR relates to a feature request. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. stale_pr This PR has not been pushed to for more than one year.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@thehe @s-hertel @webknjaz @ansibot