Change default Interpreter

dwydler commented 3 years ago

Today, i found time to test the wrapper in our development environment. After some cups of coffee and some ice, runs ansible in combination with The Bastion. Thanks again for the plugin!

I noticed zwei little things:

In both python scripts you use pyhton als default interpreter. In the system requierments did you write Python 3. I had to create a symlink for the wrapper to work. Wouldn't it be better to change the interpreter to pyhton3?
In the section "Configuration via ansible.cfg" in the README.md you use relative paths. Perhaps you can add a hint, that this only work if ansible (configuration) user based and not system based.

RomainL972 commented 2 years ago

Hi, I also have this issue, I had to install python-is-python3 on Ubuntu for the scripts to work.

dwydler commented 2 years ago

@speed47 Do you have time to look into this issue?

speed47 commented 2 years ago

Sorry about the delay! Can you try with the current main branch? We've just merged #8 that references python3 explicitly

ovh / the-bastion-ansible-wrapper

Change default Interpreter #2