Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

"Ansible for Networking" Lesson and Image #289

Merged
merged 32 commits into from
Jan 28, 2020
Merged

"Ansible for Networking" Lesson and Image #289

merged 32 commits into from
Jan 28, 2020

Conversation

Mierdin
Copy link
Member

@Mierdin Mierdin commented Jan 17, 2020

Note - this implements part of #97 but there is room for more that this lesson might not cover, like specifics of Junos modules, better suited for a dedicated lesson. Should consider closing that issue when this gets merged but opening up new issues for those specific topics

TODO

  • Change all paths (like images) that are pointing to the fork back to this repo

@Mierdin
Copy link
Member Author

Mierdin commented Jan 17, 2020

Things seem to be working pretty well but I did get this error at some point:

[antidote@ansible stage1]$ ansible-playbook -i devices playbook.yml

PLAY [Ansible test within NRE Labs lesson] **********************************************************

TASK [configure the login banner] *******************************************************************
fatal: [vqfx1]: FAILED! => {"ansible_facts": {"discovered_interpreter_python": "/usr/libexec/platform-python"}, "changed": false, "module_stderr": "", "module_stdout": "\n{\"msg\": \"name 'known_hosts_lookup' is not defined\", \"failed\": true, \"invocation\": {\"module_args\": {\"banner\": \"login\", \"port\": 22, \"text\": \"this is my login banner\\nthat contains a multiline\\nstring\\n\", \"state\": \"present\", \"provider\": {\"host\": null, \"port\": 22, \"username\": null, \"password\": null, \"ssh_keyfile\": null, \"timeout\": null, \"transport\": \"netconf\"}, \"active\": true, \"host\": null, \"username\": null, \"password\": null, \"ssh_keyfile\": null, \"timeout\": null, \"transport\": null}}, \"deprecations\": [{\"msg\": \"Param 'port' is deprecated. See the module docs for more information\", \"version\": 2.9}]}\n\n{\"msg\": \"name 'known_hosts_lookup' is not defined\", \"failed\": true, \"invocation\": {\"module_args\": {\"banner\": \"login\", \"port\": 22, \"text\": \"this is my login banner\\nthat contains a multiline\\nstring\\n\", \"state\": \"present\", \"provider\": {\"host\": null, \"port\": 22, \"username\": null, \"password\": null, \"ssh_keyfile\": null, \"timeout\": null, \"transport\": \"netconf\"}, \"active\": true, \"host\": null, \"username\": null, \"password\": null, \"ssh_keyfile\": null, \"timeout\": null, \"transport\": null}}, \"deprecations\": [{\"msg\": \"Param 'port' is deprecated. See the module docs for more information\", \"version\": 2.9}]}\n", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

PLAY RECAP ******************************************************************************************
vqfx1                      : ok=0    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0   

Unfortunately I can't seem to replicate it. When I re-ran the playbook, it ran normally.

@Mierdin
Copy link
Member Author

Mierdin commented Jan 18, 2020

Nevermind, I wasn't actually using the provided ansible.cfg file in the correct location. Moving this fixed the problem, works great now.

IPvSean and others added 3 commits January 22, 2020 12:56
trying to get a working netconf example online , thanks @Mierdin

Signed-off-by: ipvsean <[email protected]>
Signed-off-by: Matt Oswalt <[email protected]>
@Mierdin
Copy link
Member Author

Mierdin commented Jan 22, 2020

@IPvSean Should have port 830 open as well now.

.gitignore Show resolved Hide resolved
@IPvSean
Copy link
Contributor

IPvSean commented Jan 22, 2020 via email

@Mierdin
Copy link
Member Author

Mierdin commented Jan 22, 2020

Okay. I desperately need to update .gitignore in this repo, it's got a whole bunch of stuff in there that is a remnant of what used to be in this repo. Do what you need to do for now, but I'll likely come in after and clean all that crap up anyways

IPvSean and others added 11 commits January 23, 2020 14:04
thanks @Mierdin for merge
syncing stage4 for NRE @Mierdin
please sync this @Mierdin
vqfx doesn't need addresses configured until stage 4, and cumulus
scripts weren't using the correct configuration files - replaced
absolute paths with lookups relative to script location so each script
always uses its own files

Signed-off-by: Matt Oswalt <[email protected]>
Signed-off-by: Matt Oswalt <[email protected]>
@Mierdin Mierdin marked this pull request as ready for review January 28, 2020 02:05
@Mierdin
Copy link
Member Author

Mierdin commented Jan 28, 2020

@IPvSean I think we're good to merge now. I took the liberty of adding a lesson diagram - mostly useful for the last two stages, but still good to have lesson-wide anyways, in case the learner wants to play around.

I'll make sure this is reflected on the test site immediately after I merge, to ensure everything still works, especially the pictures throughout the lesson, etc. If there's anything you wish to change prior to release, don't hesitate to let me know - we are still a few days out from that.

Thanks so much for your quick work on this - I think this is going to be one of our signature lessons. I know it's been one of our most-requested.

@Mierdin Mierdin merged commit 8fc40b2 into nre-learning:master Jan 28, 2020
@Mierdin Mierdin deleted the ansible-networking branch January 28, 2020 02:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants