16:01:50 #startmeeting Ansible Network Working Group 16:01:50 Meeting started Wed May 9 16:01:50 2018 UTC. 16:01:50 This meeting is logged and archived in a public location. 16:01:50 The chair is gundalow. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:01:50 Useful Commands: #action #agreed #halp #info #idea #link #topic. 16:01:50 The meeting name has been set to 'ansible_network_working_group' 16:03:07 #info Agenda as always https://github.com/ansible/community/labels/network 16:03:47 #chair acozine1 jmcgill298 Anil_ 16:03:47 Current chairs: Anil_ acozine1 gundalow jmcgill298 16:05:06 #info Fix for NameError in cnos.py #39665 16:05:16 yeap 16:06:05 Anil_: Did that ever work, thought the tests would have caught that 16:06:40 We were focussing on enos and this was never tested. 16:06:46 ah, OK 16:07:07 Wonder if it would be worth adding a basic unit test to ensure the modules load up OK? 16:07:29 Now I started rewamping the code of cnos_facts, cnos_command and cnos_config in tune with how Ansible does it now. 16:08:04 So this bug came up . And this time I am adding Unit test cases 16:08:25 Ace 16:08:33 Merged, I'll do the backport shortly 16:08:44 All th new changes can be Ok to be in 2.6 but the 39665 has to ve in 2.5 as well 16:08:51 Oh Thanks 16:09:48 By this Friday i will be modifying all the cnos_facts, cnos_config and cnos_command and will raise a PR for that too 16:10:35 Then I would like move away from paramiko and use persistent connection for all cnos modules 16:10:49 Cool 16:11:05 I assume that will be for 2.7 though, given the dates 16:11:15 https://github.com/ansible/ansible/blob/devel/docs/docsite/rst/roadmap/ROADMAP_2_6.rst 16:11:34 yes.. i have time till 17th of this month I guess 16:11:46 Correct 16:11:53 Merged by 17th 16:12:10 oh 16:12:11 yes.. I know.. 16:12:15 2018-05-25 Community Freeze (Community Modules) 16:12:28 bah 16:12:31 2018-05-17 Core Freeze (Core Engine and Non-Community Modules) 16:12:35 You were right 16:13:04 Yeah good I have more breathing space.. 16:14:14 https://github.com/ansible/ansible/pull/39916 I'll merge once green 16:14:19 Anil_: Anything else? 16:15:06 Gundalow: Thats all for me, but you will take care of back porting to 2.5 as well right ? 16:15:44 Anil_: https://github.com/ansible/ansible/pull/39916 is the backport to stable-2.5 which I'll merge once Shippable goes green 16:16:17 If you can make the integration tests so we can backport them to stable-2.5 (ie. no new module functionality included) that would be great 16:20:22 gundalow: I have one more item to discuss with you 16:20:27 Sure 16:21:03 The CLIs of cnos is undergoing changes in line with ISCLI 16:21:51 so our switches which works with 2.5 will still work for some more time 16:22:44 but eventually that CLIs may get deprecated 16:23:05 so should we always backport the changes in CLIs to it ? 16:24:13 or will it be Ok that we put in document that u need to use CNOS 10.7 or lesser with Ansible 2.6 16:25:09 what u suggest ? 16:26:04 Anil_: I think so, let me ask people internally what they think. 16:29:27 Anil_: Could you please drop me an email with that in, so I'll make sure I get back to you 16:29:32 Anything else? 16:30:05 sure.. I will mail you..And nothing else from my side 16:30:12 Thanks 16:30:15 #topic Open Floor 16:30:21 Anyone got anything else? 16:35:34 39916 is green 16:36:11 Merged 16:36:26 ok, will close the meeting shortly 16:38:01 sure.. Have a wonderful day gundalow 16:38:28 Thanks 16:38:32 #endmeeting