Skip to content

Conversation

MoteHue
Copy link
Contributor

@MoteHue MoteHue commented Sep 2, 2025

Without any top-level inventory file, Ansible will fail with:

ERROR! Completely failed to parse inventory source /home/ubuntu/ansible-slurm-appliance/environments/$ENV/inventory

@MoteHue MoteHue requested a review from a team as a code owner September 2, 2025 14:40
Without any top-level inventory file, Ansible will fail with:

```
ERROR! Completely failed to parse inventory source /home/ubuntu/ansible-slurm-appliance/environments/$ENV/inventory
```
@MoteHue MoteHue force-pushed the fix-cookiecutter-inventory branch from f7f504a to 40487de Compare September 2, 2025 15:07
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

LGTM, tested it with cookiecutter:

without this PR:

 ansible all -a hostname
ERROR! Completely failed to parse inventory source /home/rocky/slurm-app-rl9/environments/foo/inventory

with this PR:

ansible all -a hostname
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'

@sjpb sjpb merged commit cbf990a into main Sep 4, 2025
1 of 2 checks passed
@sjpb sjpb deleted the fix-cookiecutter-inventory branch September 4, 2025 09:07
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.

2 participants