mschuchard / linter-ansible-syntax

ansible syntax check linter for pulsar
MIT License
5 stars 3 forks source link

Fails to parse dynamic-inventory #8

Closed serpro69 closed 8 months ago

serpro69 commented 4 years ago

If I include the dynamic inventory script in the settings I'm getting that it failed to parse the inventory file and only implicit localhost is available. Removing the inventory setting altogether naturally throws a null pointer. Is there a workaround for this to use this package with dynamic inventories or disable parsing the inventory (if the rest of the functionality is not dependent on this)?

serpro69 commented 4 years ago

Setting this to /etc/ansible/hosts helped resolve this issue and a few others (for example incorrect line numbers displayed for lint issues). But still not ideal since that hosts list is empty for me

mschuchard commented 4 years ago

Hi, are you still having this issue? If so, can you provide a MCVE of the dynamic inventory for me to test against?

serpro69 commented 4 years ago

Hi. I haven't tried using the plug-in with the dynamic inventory since. But I will try to reproduce it and provide you with the inventory script.

mschuchard commented 8 months ago

No reply; closing.