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

24.12.x fails with "Unable to parse /the/path/localhost as an inventory source" #4448

Closed
sverrehu opened this issue Dec 12, 2024 · 0 comments · Fixed by #4450
Closed

24.12.x fails with "Unable to parse /the/path/localhost as an inventory source" #4448

sverrehu opened this issue Dec 12, 2024 · 0 comments · Fixed by #4450
Labels
bug new Triage required

Comments

@sverrehu
Copy link
Contributor

Summary

Previously, ansible-lint would invoke ansible-playbook with the option -i localhost, but after #4393 the trailing comma is gone. This makes ansible-playbook look for a file named localhost instead of treating localhost as a hostname.

Issue Type
  • Bug Report
@sverrehu sverrehu added bug new Triage required labels Dec 12, 2024
ssbarnea added a commit that referenced this issue Dec 12, 2024
@ssbarnea
Revert "Accommodate specified inventory files (#4393)"
30e0efa 
This reverts commit 6fbb12e.
This reverts commit 4d295e8.

Fixes: #4448
Fixes: #4449
@ssbarnea ssbarnea mentioned this issue Dec 12, 2024
Merged
ssbarnea added a commit that referenced this issue Dec 12, 2024
@ssbarnea
Revert "Accommodate specified inventory files (#4393)"
6036a35 
This reverts commit 6fbb12e.
This reverts commit 4d295e8.

Fixes: #4448
Fixes: #4449
cavcrosby added a commit to cavcrosby/ansible-lint that referenced this issue Dec 12, 2024
@cavcrosby
Treat the default inventory_files as host list (ansible#4448)
404e50d 
The --inventory option to the ansible-playbook executable can accept a
value that is comma-separated. This comma-separated value is presumed to
be a host list and not a list of inventory files. This was the behavior
for the inventory_files variable before 4d295e8. There wasn't a reason
for its removal and it was an oversight made.
cavcrosby added a commit to cavcrosby/ansible-lint that referenced this issue Dec 12, 2024
@cavcrosby
Treat the default inventory_files as host list (ansible#4448)
3477eba 
The --inventory option to the ansible-playbook executable can accept a
value that is comma-separated. This comma-separated value is presumed to
be a host list and not a list of inventory files. This was the behavior
for the inventory_files variable before 6fbb12e. There wasn't a reason
for its removal and it was an oversight made.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug new Triage required
Projects
🧰 devtools project board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Revert "Accommodate specified inventory files (#4393)" ansible/ansible-lint
1 participant
@sverrehu