ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-07-23 05:10:22 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Don't skip an inventory source just because it has a comma (#35002)

Browse source 
* Don't skip an inventory source just because it has a comma, make sure it's also doesn't exist as a path. Fixes #34931

* Add integration test for inventory path with commas
This commit is contained in:
Matt Martz 2018-01-24 09:44:55 -06:00 • committed by GitHub
commit 1ac04ba52e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 18 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

1
test/integration/targets/inventory_path_with_comma/aliases Normal file
View file

@ -0,0 +1 @@
posix/ci/group3

9
test/integration/targets/inventory_path_with_comma/playbook.yml Normal file
View file

@ -0,0 +1,9 @@
---
- hosts: all
gather_facts: false
tasks:
- name: Ensure we can see group_vars from path with comma
assert:
that:
- inventory_var_from_path_with_commas is defined
- inventory_var_from_path_with_commas == 'here'

5
test/integration/targets/inventory_path_with_comma/runme.sh Executable file
View file

@ -0,0 +1,5 @@
#!/usr/bin/env bash
set -ux
ansible-playbook -i this,path,has,commas/hosts playbook.yml -v "$@"

1
test/integration/targets/inventory_path_with_comma/this,path,has,commas/group_vars/all.yml Normal file
View file

@ -0,0 +1 @@
inventory_var_from_path_with_commas: 'here'

1
test/integration/targets/inventory_path_with_comma/this,path,has,commas/hosts Normal file
View file

@ -0,0 +1 @@
localhost ansible_connect=local