neoave / mrack

Multicloud use-case based multihost async provisioner for CIs and testing during development
Apache License 2.0
11 stars 14 forks source link

feat: Allow domain and global section records for ansible inventory #234

Closed Tiboris closed 1 year ago

Tiboris commented 1 year ago
feat(AnsibleInventory): Allow additional domain level ansible inventory values

Add support for domain level ansible_inventory records which
can be eventually overwritten by host defined values.
Added unit tests to test this feature.   

feat(AnsibleInventory): Allow additional global level values

Add support for global level ansible_inventory records
which can be eventually overwritten by domain and host defined values.
Minor fix of logig in pytest multihost code - adding deepcopy.

test(AnsibleInventory): global level output values override

Add test for the feature to support global level
ansible_inventory records which can be eventually
overwritten by domain and host defined values.

Signed-off-by: Tibor Dudlák <tdudlak@redhat.com>
Tiboris commented 1 year ago

I can easily remove the multihost part if that would help to ship the ansible part required by users :innocent:

Tiboris commented 1 year ago

Removed multihost changes

Tiboris commented 1 year ago

Thanks for review :)