Installation guidelines will break all other modules/playbooks about ntc-ansible HOT 5 CLOSED

Ran into this also today. I had to rename the files as mentioned ntc-ansible/setup.py, setup.cfg and ntc-templates/ setup.cfg so that ansible would work. I wasn't even calling any of the ntc-ansible modules in my playbooks at the time.

from ntc-ansible.

Hi @spikedrba , sorry for the delay in coming back to you!
Thanks for having got deep into this, I've personally never met this issue and it's definitely something good to be aware of.

Do you know if this happen with both pip and source installation?

from ntc-ansible.

Ah, I had forgotten about this issue, it stopped me from trying ntc-ansible for a while when I encountered this. I think it is probably more an issue in ansible than ntc, but if you try to use the repo folder directly, the setup.py (in both ntc and ntc-templates) and even the setup.cfg file would get loaded by execute_module. Renaming those files works around this (DISABLE-), but I think the real fix would be in ansible to rename the setup module to something less generic or perhaps have a tighter search path for core modules :(

from ntc-ansible.

hey @GGabriele , was away for a month, thanks for your patience. I don't understand the question about pip Vs clone. The git clone in ansible's module path is a requirement as far as I understand per your installation/setup instructions in the README, which is what I did when I set it up. Pip is only to install dependencies etc, but it won't change that doing the git pull will populate the ntc-ansible dir inside the ansible's module path with a setup.py file.
I don't know what the guidelines are to release ansible's modules, but the approach you're taking seems doomed to fail as far as I can see for as long as ansible will look for a setup.py. I agree with @bdowling that a good solution may be for ansible to use more specific names, but there may be others, not sure.

hope that helps,

Spike

from ntc-ansible.

Updated Readme to note this issue (and work around), and have been attempting to get ansible to change their setup.py to gather_facts.py.

@GGabriele should be able to close this since it is a known issue and workaround is documented.

from ntc-ansible.