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.

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

Accommodate specified inventory files #4393

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Accommodate specified inventory files #4393

wants to merge 1 commit into from
+176 −6

Conversation

cavcrosby
Copy link
Contributor

Fixes #4241 and #4225.

I alluded to this in the former issue, but I believe it was intended that ansible-lint would accommodate the inventory files used. Though, for whatever reason, it seems this functionality has been missing.

@cavcrosby cavcrosby requested a review from a team as a code owner November 9, 2024 23:53
@abhikdps abhikdps added the bug label Nov 10, 2024
@cavcrosby cavcrosby mentioned this pull request Nov 11, 2024
Open
@cavcrosby
Accommodate specified inventory files
12c1678 
The ansible_cfg_inventory conditional in the _get_inventory_files method
is used because otherwise, when not passing in an inventory file via
CLI, inventory_files would set to the /etc/ansible/hosts (the default
value for the DEFAULT_HOST_LIST config). In most cases, I'd presume that
wouldn't be desired.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug
Projects
🧰 devtools project board
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Variable groups: 'dict object' has no attribute '{group_name}' - syntax-check[specific]
3 participants
@cavcrosby @ssbarnea @abhikdps