Skip to content

Conversation

@itsgc
Copy link

@itsgc itsgc commented May 6, 2018

This solves Issue #340. As this is my first PR, apologies for anything that might be in bad form.

…os_config, junos_facts, junos_jsnapy and junos_table which were missing.
@vnitinv
Copy link
Contributor

vnitinv commented May 7, 2018

@itsgc Can you confirm which version of Ansible you are using?

@itsgc
Copy link
Author

itsgc commented May 7, 2018

Hello @vnitinv this has been tested against Ansible v2.3.1.0. I will attach a full pip freeze as soon as i am on my work computer.

Best

@itsgc
Copy link
Author

itsgc commented May 8, 2018

Hello @vnitinv

Here is the content of my pip freeze. Hope this helps.
pipfreeze.txt

Best Regards

@itsgc
Copy link
Author

itsgc commented May 11, 2018

Hello @vnitinv, anything else that you need for this PR?

@vnitinv
Copy link
Contributor

vnitinv commented May 16, 2018

@itsgc We made the fix via a different approach which will have the cascading effect. Check #357 for more details.

@vnitinv vnitinv closed this May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants