Please leave a comment regarding any agenda item you wish to discuss. If you don't show up for the meeting, your item will be skipped.
If your IRC nick is different from your Github username, leave that as well.
See https://github.com/ansible/community/tree/master/group-network for the schedule
All Core Networking issues (most recently updated at the top):
https://github.com/ansible/ansible/issues?q=is%3Aopen+label%3Anetworking+-label%3Aneeds_revision+sort%3Aupdated-desc
Network Working Group: https://github.com/ansible/community/tree/master/group-network
#ansible-network on Freenode IRC
Ansible 2.4.0 FINAL
https://groups.google.com/forum/#!msg/ansible-announce/IrLR3pjNZnk/J8s-uC5EAgAJ
Lot of network improvements, especially to NXOS
Please do take a look
If you find any bugs please report them at https://github.com/ansible/ansible/issues/new
Thank you all for your help during this release.
From @saichint ansible/ansible#29143 Integration Tests only: nxos_udld, nxos_udld_interface, nxos_vxlan_vtep_vni MERGED
Migrated from https://github.com/ansible/community/issues/110#issuecomment-329234544
Proposals for 2.5 that we'd like your input on:
We welcome all of your input on this, making sure we implement something that's useful and easy to understand.
From @Gelob https://github.com/ansible/ansible/issues/30250 Arista EOS module doesn't properly parse vrfs
Migrated from https://github.com/ansible/community/issues/110#issuecomment-330420196
All,
If you could raise PRs to update the list of Network Group members on https://github.com/ansible/community/tree/master/group-network that would be very helpful given the large and growing community.
Add GitHub, real name, company/affiliation and IRC name (if different to GitHub)
Thanks in advance.
From @caphrim007
https://github.com/ansible/ansible/pull/30577 This module allows users to manage partitions on a BIG-IP MERGED
https://github.com/ansible/ansible/pull/30584 Changes ignored and maintainers for the f5 modules. MERGED
Migrated from https://github.com/ansible/community/issues/110#issuecomment-330675257
https://github.com/ansible/ansible/pull/30767 This is 1 module out of roughly 20 that I am looking to contribute. My IRC name is Gesha24
Response
In alignment with other module submissions of this type, and based on discussions with members of the core team, and what is available as documentation here within the review, we are going to decline accepting this pull request.
The primary driver is that it is an arbitrary API module, that has no definable argument spec.
Each module should typically interact with one set of functionality. Should you decide to break this module up into individual components that deal with one set of functionalities with the API that have an explicitly defined argument spec, we would be happy to re-review.
https://github.com/ansible/ansible/pull/31202
Should we add cdp information to the lldp fact, since it is basically the same. We could add an attribute source in ansible_net_neighbors with value cdp/lldp ?
Please merge ansible/ansible#31401
This moves files from lenovo folder to cnos folder
Resolved all issues . Please merge
ansible/ansible#31508 MERGED
ansible/ansible#31509 MERGED
ansible/ansible#31510 MERGED
ansible/ansible#31512 MERGED
ansible/ansible#31565 MERGED
ansible/ansible#31580 MERGED
ansible/ansible#31696
These are util files, plugin files and doc file for enos modules
ansible/ansible#31401
This PR is to move modules from lenovo folder to cnos folder
Updated my PRs to be merged List shown above
We need to complete the conversation around aggregates https://github.com/ansible/proposals/issues/71 and decide if we are going down the loop route or not.
https://github.com/ansible/ansible/pull/32162 (irc nick dt-arista) MERGED & CP'd into 2.4
Meeting ended Wed Oct 18 17:04:59 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot .
Minutes: https://meetbot.fedoraproject.org/ansible-network/2017-10-18/network_working_group.2017-10-18-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2017-10-18/network_working_group.2017-10-18-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2017-10-18/network_working_group.2017-10-18-16.00.log.html
Review wanted on https://github.com/ansible/ansible/pull/31807 Docs: Network getting started example
I would like to to merge these 2 PRs.
https://github.com/ansible/ansible/pull/29143
https://github.com/ansible/ansible/pull/28933~~ MERGED
I would like to to merge these 2 PRs.
ansible/ansible#31696 MERGED
ansible/ansible#31401
PRs:
ansible/ansible#29030 MERGED
ansible/ansible#28939
Meeting ended Wed Oct 25 16:40:12 2017 UTC.
Minutes: https://meetbot.fedoraproject.org/ansible-network/2017-10-25/network_working_group.2017-10-25-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2017-10-25/network_working_group.2017-10-25-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2017-10-25/network_working_group.2017-10-25-16.00.log.html
Meeting ended Wed Nov 1 16:46:43 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot .
Minutes: https://meetbot.fedoraproject.org/ansible-network/2017-11-01/network_working_group.2017-11-01-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2017-11-01/network_working_group.2017-11-01-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2017-11-01/network_working_group.2017-11-01-16.00.log.html
Actions
Please merge ansible/ansible#31696 MERGED
~Please merge ansible/ansible#31401 MERGED
Please merge ansible/ansible#32782 MERGED
Please merge ansible/ansible#32895 MERGED
ansible/ansible#28939
From Core discussions "Deprecate using jinja tests as filters"
Comments welcome on the proposal
Please merge ansible/ansible#32966. MERGED
This contains fix for cli context issues for ENOS switches associated with network_cli
Please merge ansible/ansible#33081
This is enos_config.py module addition. This passes Unit tests in my lab
nmap dynamic inventory https://github.com/ansible/ansible/pull/32857
Now that https://github.com/ansible/ansible/pull/32825 is merged the Persistent Connection is terminated at the end of each play.
network_cli developer documentation https://github.com/ansible/community/blob/master/group-network/network_dev_network_cli.rst
Please merge ansible/ansible#33081 MERGED
Need to discuss the following PR's
Please merge ansible/ansible#33857 MERGED
Hi,
Please merge all mlnxos modules:
I finished pushing all mlnxos_modules for this ansible version, (except maybe bug fixes)
Thanks,
Samer
The 2.5 freeze dates (merge deadlines) have been pushed out by a week.
Please see updated dates
https://github.com/ansible/ansible/blob/devel/docs/docsite/rst/roadmap/ROADMAP_2_5.rst
To see whats change please see https://github.com/ansible/ansible/commit/1826c27e6841eb313dc3a05dc7f19c96e42c7fbf
Please merge a bug fix in cnos.py module_util file. MERGED
ansible/ansible#34607
Some of Switch ports support break-out, such as there maybe "interface eth 1/87/1" for G8296 etc.
This fix address such changes in switch prompts.
ansible/ansible#34888 MERGED
These are pertaining to update of license information. This is pending on approval from Ganesh
Discuss use of VCR for acceptance test. https://pypi.python.org/pypi/vcrpy
DELETED
~~Please merge the following documentation / license PR ansible/ansible#36840 ~~ MERGED
https://github.com/ansible/ansible/pull/36468
discuss vcr as shown in above pull request
Ansible 2.5.0 Release Candidate 2 is now available! Get it on PyPI: pip install ansible==2.5.0rc2, https://releases.ansible.com/ansible/, or GitHub.
unable to open shell!connection: network_cli and connection: netconf. connection: local will continue to work. Please read this the docs and see that platform specific notes for EOS, Junos and NXOS if you are use them.ansible/ansible#37310
please review and merge if OK.
I have another bug fix that based on it
A proposal for eAPI connection type: https://github.com/ansible/proposals/issues/102
Proposal on Netconf modules: https://github.com/ansible/proposals/issues/104
Deprecation of the use of connection=load in network modules that support connection=network_cli or network=netconf
Review HTTP connection proposal ansible/proposals#103
I have a customer looking for these which are waiting for review:
https://github.com/ansible/ansible/pull/37543
https://github.com/ansible/ansible/pull/37580
https://github.com/ansible/ansible/pull/37582
https://github.com/ansible/ansible/pull/37583
https://github.com/ansible/ansible/pull/37676
https://github.com/ansible/ansible/pull/37678
With respect to https://github.com/ansible/ansible/issues/39386#issuecomment-384773801
It's not very clear that the NIOS lookup plugins exist. Is there some way to indicate that in the documentation section for the NIOS net_tools modules?
Please review and merge
ansible/ansible#39665
This is a bug fix on cnos util module. This has to be backported to 2.5 as well.
Thank You.
-Anil
Clarifying terminology for network-engine role. Background:
We've used the word “parser” to describe the file with the matching rules, the module that does the matching, and the combination of the two. We're working to clarify the usage and make the modules in network-engine easier to understand and use. See https://github.com/ansible-network/network-engine/issues/65.
Please review and merge
ansible/ansible#39955
Here I am modifiying cnos_facts, cnos_command to align with how it is done for other vendors. In addition to this I am adding cnos_config also. Here I am using persistent connection. Therefore all the three plugins are also introduced for this purpose viz. action, cliconf and terminal plugins. The documentation snippet in util is brought into each module documentation as they differ here. module_util/cnos.py is added with all methods that are required for this. Added unit test for each of them. In addition to that i have removed few errors from the ignore.txt as they have been addressed here
Please review and merge
Thank You.
-Anil
ansible/ansible#40218
Starting with our recent CNOS release version 10.7, we have moved certain CLI keywords to match with Industry Standard CLI (ISCLI) keywords. For backward compatibility, old CLI keywords will continue to work for next one or two CNOS releases with a condition that “command acronyms” of old CLI keywords, cannot be used. For eg: Use “display version” instead of “disp ver”. This PR is to expand the acronymns if they are present in the python code
This has to be backported to 2.5 as well.
Merged into devel and cherry-picked for release in 2.5.3
I would like to discuss the comments from this PR
I would also like to discuss the following issues:
Please merge the PR
ansible/ansible#40865
This is to fix the copy paste error in CNOS plugin files. Corrected typo ENOS to CNOS. There was few merge issue in some enos license text. This also fixed.
Merged on May 30
I would like to discuss the following issues:
https://github.com/ansible/ansible/issues/40683
https://github.com/ansible/ansible/issues/40902~~
https://github.com/ansible/ansible/issues/40913~~
No feedback has been solicited yet by core members, so I want to bring this up
Covered in Ansible Core meeting https://meetbot.fedoraproject.org/ansible-meeting/2018-06-05/ansible_irc_core_meeting.2018-06-05-19.04.log.html
Please merge the PR
ansible/ansible#41121
This is to fix the copy paste error in CNOS facts . This issue came revealed when I ran a coverity scan.
Merged on Jun 5
Meeting started Wed Jun 13 16:01:01 2018 UTC.
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-06-13/network_working_group.2018-06-13-16.01.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-06-13/network_working_group.2018-06-13-16.01.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-06-13/network_working_group.2018-06-13-16.01.log.html
Same license terms would be included as seen in Ansible's ismount code found here
https://github.com/ansible/ansible/blob/6787fc70a643fb6e2bdd2c6a6202072d21db72ef/lib/ansible/module_utils/ismount.py#L7
F5 wants to do this for the following reason
The reason for this is the current implementation in the modules places a dependency that must be met in Ansible (in the form of tests) and met by F5's customers (remembering to install netaddr). This second issue leads to more support requests, which are unnecessary, because this dependency can be removed.
The requested code to include, is provided natively in Python 3.x. This code would be used as a shim until such time as,
I think that based off of the pre-existing work in Ansible, and the known GPL-compatibility of the PSF and GPL3, that this should be a non-issue provided that the ipaddress.py code is clearly documented as such (and that that license does not extend beyond the ipaddress.py file.
There is existing work in Ansible's module utils to support rudimentary IP address work. F5 needs the enhanced library to support IP networks in addition to the existing IP addresses.
This module utility would only be included in F5's module_utils directory; not the larger network/common or its sub dirs. Other module authors using this module_util would assume the risk of it being removed per the bullets mentioned above.
Agreed but punted to tomorrow's core meeting due to licensing issue @Qalthos
This was approved by core on 2018-06-21 @caphrim007
Meeting started Wed Jun 20 15:59:50 2018 UTC.
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-06-20/network_working_group.2018-06-20-15.59.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-06-20/network_working_group.2018-06-20-15.59.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-06-20/network_working_group.2018-06-20-15.59.log.html
Please merge
ansible/ansible#41998
This is to add the expansion of CNOS CLI acronymns into Change Logs
Ansible 2.6.0 final is now available
See
From a network point view
Connection Plugins
httpapi - New connection method that simplifies connections to platform-specific APIs such as Arista eAPI and Cisco NXAPI. This method replaces connection: local with connection: httpapi and then specifying the specific platform via the ansible_network_os variable.Modules
net_get - downloads a large config file to target network device with SCP/SFTPnet_put - copies a large config file to target network device with SCP/SFTPnetconf_get - fetches the configuration and state data from remote NETCONF enabled network device/hostnetconf_rpc - executes generic NETCONF RPC commands that are defined by NETCONF standard as well as proprietary RPC-based NETCONF capabilities supported by the remote network device/hostGeneral
2018-07-04
Summary of key points
Team status update
09:17 Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-07-04/ansible_network_working_group.2018-07-04-16.00.html
09:17 Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-07-04/ansible_network_working_group.2018-07-04-16.00.txt
09:17 Log: https://meetbot.fedoraproject.org/ansible-network/2018-07-04/ansible_network_working_group.2018-07-04-16.00.log.html
For the July 11th meeting, I would like to quickly discuss an open PR: https://github.com/ansible/ansible/pull/41985
Please merge PR ansible/ansible#42500
In this PR, I am moving the legacy code for ssh connectivity from Paramiko to persistent connection of Ansible for cnos_vlan module. Along with this I am trying to take care of the new CLIs introuduced for CNOS. And I have added UT codes for this module. Once this module passes through review process and get merged, I will replicate the paradigm for rest of 14 modules.
2018-07-04
Summary of key points
Team status update
Actions
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-07-11/ansible_network_working_group.2018-07-11-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-07-11/ansible_network_working_group.2018-07-11-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-07-11/ansible_network_working_group.2018-07-11-16.00.log.html
Please merge PR: ansible/ansible#42164 Merged
~Review and test PR https://github.com/ansible/ansible/pull/42847 for community supported platforms~ Merged
Please merge PR ansible/ansible#43194
In this PR, I am moving the legacy code for ssh connectivity from Paramiko to persistent connection of Ansible for cnos_portchannel, cnos_vlag and cnos_interface module. Along with this I am trying to take care of the new CLIs introuduced for CNOS. And I have added UT codes for this module. This done exactly in line with PR#42500. This code has been tested on a CNOS Mars switch.
2018-07-25
Team status update
Actions
Qalthos to merge PR ansible/ansible#43194
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-07-25/network_working_group.2018-07-25-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-07-25/network_working_group.2018-07-25-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-07-25/network_working_group.2018-07-25-16.00.log.html
Please merge PR ansible/ansible#43534
In this PR, I am moving the legacy code for ssh connectivity from Paramiko to persistent connection of Ansible for cnos_bgp, cnos_factory, cnos_reload, cnos_save, and cnos_showrun module. Along with this I am trying to take care of the new CLIs introuduced for CNOS. And I have added UT codes for this module. This done exactly in line with PR#42500. This code has been tested on a CNOS Bacon switch. Edited cnos_bgp to move code from cnos.py in Util folder to module code itself. This was done based on one review comment that cnos.py is too large and Qalthos suggested to move its code from util file to module file. Tested it too
Contributors Summit, AnsibleFest Austin 2018
What would you like to see discussed at the Community Summit, this is your opportunity to to ask questions either related to Networking, or the Community in general, feel free to add any topics to:
https://etherpad.openstack.org/p/ansible-summit-october-2018
URGENT: unable to join meeting:
got this error:
[09:06] == Usermode change: +Ri
[09:06] == gateway/web/freenode/ip.209.116.155.178 is now your hidden host (set by syn.)
[09:06] == #ansible-network Cannot join channel (+r) - you need to be identified with services
please help
@samerd Due to increases in spam in IRC, most of them now require you register.
@samerd I've removed the requirement, please try and join again
@mattclay: thanks, I'll register ASAP
Hi,
I have registered as Anil_Lenovo. Now how can I join the chat ?
-Anil
From: John R Barker notifications@github.com
Sent: Wednesday, August 01, 2018 9:51 PM
To: ansible/community community@noreply.github.com
Cc: Anil Kumar Muraleedharan amuraleedhar@lenovo.com; Comment comment@noreply.github.com
Subject: [External] Re: [ansible/community] Network Working Group Meeting Agenda 2 (#247)
@samerdhttps://github.com/samerd I've removed the requirement, please try and join again
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ansible/community/issues/247#issuecomment-409633027, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANdCcnKUKh5hFgmV1MQpbs1BN8Y__Ryxks5uMdVQgaJpZM4PdkPJ.
As part of http://www.ansible.com/ansiblefest
Ansible succeeds because of its community, and this contributor summit is for you. If you have any topics to discuss, large or small, please be sure to post those topics on the etherpad. For topics that other had added please vote by adding +1
https://etherpad.openstack.org/p/ansible-summit-october-2018
Please see https://groups.google.com/forum/#!topic/ansible-devel/ivrpVbLya8g for further info
We realize that not everybody can attend in person, which is why we’ll again be arranging a video conference and a meeting time on IRC in #ansible-meeting to encourage remote participation. This has worked very well for the past Contributor Summits. We’ll be sending out an email with the information for remote participation about a week before the Contributor Summit.
If you want to attend in-person please sign up at: https://www.eventbrite.com/e/ansible-contributor-conference-austin-2018-tickets-48557063484
After signing up you'll receive an email from Eventbrite with a discount code for AnsibleFest Austin.
Please add your name at the bottom of this etherpad: https://etherpad.openstack.org/p/ansible-summit-october-2018
1) Add ideas to https://etherpad.openstack.org/p/ansible-summit-october-2018
2) Attending in person: https://www.eventbrite.com/e/ansible-contributor-conference-austin-2018-tickets-48557063484 then use discount code to sign up to AnsibleFest
3) Attending in person OR virtually: Add your name to the end of https://etherpad.openstack.org/p/ansible-summit-october-2018
4) Happy automating!
Thanks for all of your contributions to Ansible, and we hope to see you in Austin! And if you have any questions at all, please let us know.
Speak to gundalow on Freenode, or anyone else in #ansible-devel
ios_config fails when creating macro "timeout trying to send command:" https://github.com/ansible/ansible/issues/38420
Please review and merge the following PRs
ansible/ansible#44398 MERGED
ansible/ansible#44559 MERGED
Thanks
-Anil
Would appreciate it if you could also look these.
https://github.com/ansible/ansible/issues/43861
https://github.com/ansible/ansible/issues/43996
Please merge ansible/ansible#44527
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-08-29/network_working_group.2018-08-29-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-08-29/network_working_group.2018-08-29-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-08-29/network_working_group.2018-08-29-16.00.log.html
Please Merge the following PR
ansible/ansible#44953 Merged
Thanks
-Anil
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-09-05/ansible_network_working_group.2018-09-05-15.59.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-09-05/ansible_network_working_group.2018-09-05-15.59.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-09-05/ansible_network_working_group.2018-09-05-15.59.log.html
Please Merge the following PR
ansible/ansible#45278 Merged
CLIs used in cnos_facts, cnos_command, its UT test modules and few CLI template files are in old format. This is getting corrected to new CLI format. Though we are having backward compatibility for this CLIs, it will get deprecated from 10.9 onwards. So we want to change them all to new CLI. Hence this PR is.
Please merge this so that Customers will have all latest CLI experience.
Thanks
-Anil
Please Merge the following PR
ansible/ansible#45483
Thanks
-Anil
nothing to repeat issue with nxos terminal plugin on Python 2.6Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-09-12/ansible_network_working_group.2018-09-12-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-09-12/ansible_network_working_group.2018-09-12-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-09-12/ansible_network_working_group.2018-09-12-16.00.log.html
Network session as part of Austin AnsibleFest Contributors Summit
We will be having a Network session which you can join in person or remotely via IRC/VideoCall
Network Agenda: https://etherpad.openstack.org/p/ansible-summit-october-2018-network
Zuul Agenda: https://etherpad.openstack.org/p/ansible-summit-october-2018-zuul
Also there are two Zuul talks at Ansible Fest:
Top level Agenda: https://etherpad.openstack.org/p/ansible-summit-october-2018
Please feel free to add comments and vote (+1) on topics.
NOTE time and location of sessions may change
For more information about the Contributors Summit see https://github.com/ansible/community/issues/329#issuecomment-410040811
If you have questions feel free to ask here, in IRC, or ping gundalow.
Hope to see you there either in person or online.
IRC logs
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-09-19/ansible_network_working_group.2018-09-19-16.02.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-09-19/ansible_network_working_group.2018-09-19-16.02.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-09-19/ansible_network_working_group.2018-09-19-16.02.log.html
Team Updates
Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-09-26/ansible_network_working_group.2018-09-26-16.00.html
Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2018-09-26/ansible_network_working_group.2018-09-26-16.00.txt
Log: https://meetbot.fedoraproject.org/ansible-network/2018-09-26/ansible_network_working_group.2018-09-26-16.00.log.html
Please welcome @anasbadaha (https://github.com/anasbadaha) a new team member of Mellanox-Onyx
His first PR: ansible/ansible#46218
Please review this PR and merge if OK with you
Thanks a lot @samerd
Hi, now that 2.7.0 is soon to be out I've created a fresh agenda.
Please subscribe to #363
As being discussed at AnsibleFest 2018, we would like to learn how we can extend httpapi with (very) specific needs for different connection methods (all different types of REST), for Cisco this is different technologies like IMC, UCS, ACI, MSC, Tetration, ...
One problem is the lack of documentation or code annotations that help understand how we ought to extend it. The second problem is that by extending the httpapi framework you may break the original use-cases without the ability to test.
So the easiest path is everybody does their own implementation, but even for Cisco only there's a benefit of having a single framework.