search

NSO-developer / nso-ansible-demo

Simple demo of the Ansible modules for NSO
Apache License 2.0
34 stars 24 forks source link

Added Python3 version of json2yaml.py #7

Closed sbarvick-dd closed 5 years ago

sbarvick-dd commented 5 years ago

json2yaml.py does not work with Python3 so I modified it and added as json2yaml3.py. Trying to make the one file do both is something perhaps for another time. This addition plus a small change to nso.py in the ansible module_utils make this work with Python 3.5.

Tested with virtualenv Python2.7 and Python3.5 and use case examples in the README.

sbarvick-dd commented 5 years ago

Oh, PR for module change is https://github.com/ansible/ansible/pull/48619 but I suspect the necessary reviews are on CC:

sbarvick-dd commented 5 years ago

FYI, my fix to make the core ansible module was accepted and merged so getting this working with python3 would be a good thing.