device42/ansible_device42

Dynamic Inventory doesn't seem to work due to missing SPLIT_GROUP_BY_COMMA element.

Closed this issue · 1 comments

Hi folks. I've been having some trouble with the dynamic inventory always returning empty data, or errors. Further investigation seems to indicate a couple of problems with the lib.py.

  • SPLIT_GROUP_BY_COMMA is never added to the conf dict, so in Ansible.get_grouping an exception is raised for the missing element.

Also, there seems to be a redundant parameter to d42_ansible_dynamic_inventory.py called --list that as far as I can tell does nothing.

I've submitted PR #12 to resolve these.