Network Working Group Meeting Agenda 2

gundalow commented 7 years ago

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

gundalow commented 7 years ago

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.

gundalow commented 7 years ago

~~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

gundalow commented 7 years ago

Proposals for 2.5 that we'd like your input on:

https://github.com/ansible/proposals/issues/71 Hold off and redesign Aggregates
https://github.com/ansible/proposals/issues/70 Hold off and redesign Declarative Intent

We welcome all of your input on this, making sure we implement something that's useful and easy to understand.

gundalow commented 7 years ago

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

gundalow commented 7 years ago

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.

gundalow commented 7 years ago

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

privateip commented 7 years ago

https://github.com/ansible/proposals/issues/76

caphrim007 commented 7 years ago

~~https://github.com/ansible/ansible/pull/30998~~ MERGED
~~https://github.com/ansible/ansible/issues/29241 (to close)~~ CLOSED
~~https://github.com/ansible/ansible/pull/31205~~ MERGED
~~https://github.com/ansible/ansible/pull/31207~~ MERGED

eoprede commented 7 years ago

~~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.

gundalow commented 7 years ago

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 ?

amuraleedhar commented 7 years ago

Please merge ansible/ansible#31401 This moves files from lenovo folder to cnos folder

caphrim007 commented 7 years ago

~~https://github.com/ansible/ansible/pull/30391~~ MERGED
~~https://github.com/ansible/ansible/pull/31207~~ MERGED

amuraleedhar commented 7 years ago

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

amuraleedhar commented 7 years ago

Updated my PRs to be merged List shown above

calfonso commented 7 years ago

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.

dathelen commented 7 years ago

~~https://github.com/ansible/ansible/pull/32162 (irc nick dt-arista)~~ MERGED & CP'd into 2.4

gundalow commented 7 years ago

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

gundalow commented 7 years ago

Review wanted on https://github.com/ansible/ansible/pull/31807 Docs: Network getting started example

saichint commented 7 years ago

I would like to to merge these 2 PRs. https://github.com/ansible/ansible/pull/29143 ~~https://github.com/ansible/ansible/pull/28933~~ MERGED

amuraleedhar commented 7 years ago

I would like to to merge these 2 PRs. ~~ansible/ansible#31696~~ MERGED ansible/ansible#31401

rahushen commented 7 years ago

PRs: ~~ansible/ansible#29030~~ MERGED ansible/ansible#28939

gundalow commented 7 years ago

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

jmighion commented 7 years ago

PR: ~~https://github.com/ansible/ansible/pull/32172~~ MERGED

gundalow commented 7 years ago

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

ganeshrn to review 32172, merge and CP into 2.4
EVERYBODY If you think anything is missing from https://github.com/ansible/ansible/issues?q=is%3Aopen%20label%3Anetworking%20milestone%3A2.5.0 please let us know in #ansible-network
Gundalow to add facts discussion to internal team agenda

amuraleedhar commented 7 years ago

~~Please merge ansible/ansible#31696~~ MERGED

jmighion commented 7 years ago

PR: ~~https://github.com/ansible/ansible/pull/32602~~ MERGED

amuraleedhar commented 7 years ago

~~Please merge ansible/ansible#32782~~ MERGED

amuraleedhar commented 7 years ago

~~Please merge ansible/ansible#32895~~ MERGED

rahushen commented 7 years ago

ansible/ansible#28939

gundalow commented 7 years ago

From Core discussions "Deprecate using jinja tests as filters"

https://github.com/ansible/proposals/issues/83 Proposal
https://github.com/ansible/ansible/pull/32361 WIP implementation

Comments welcome on the proposal

amuraleedhar commented 7 years ago

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

gundalow commented 6 years ago

nmap dynamic inventory https://github.com/ansible/ansible/pull/32857

gundalow commented 6 years ago

Now that https://github.com/ansible/ansible/pull/32825 is merged the Persistent Connection is terminated at the end of each play.

gundalow commented 6 years ago

network_cli developer documentation https://github.com/ansible/community/blob/master/group-network/network_dev_network_cli.rst

amuraleedhar commented 6 years ago

~~Please merge ansible/ansible#33081~~ MERGED

mikewiebe commented 6 years ago

Need to discuss the following PR's

https://github.com/ansible/ansible/pull/31524 and impact to ISSU capability
https://github.com/ansible/ansible/pull/33376
- Would like to understand the history here of why it's being deprecated and impact to existing users.

amuraleedhar commented 6 years ago

~~Please merge ansible/ansible#33857~~ MERGED

samerd commented 6 years ago

Hi, Please merge all mlnxos modules:

I finished pushing all mlnxos_modules for this ansible version, (except maybe bug fixes)

Thanks, Samer

gundalow commented 6 years ago

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

amuraleedhar commented 6 years ago

~~Please merge a bug fix in cnos.py module_util file. 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.~~ MERGED

amuraleedhar commented 6 years ago

~~ansible/ansible#34888~~ MERGED ~~These are pertaining to update of license information. This is pending on approval from Ganesh~~

grastogi23 commented 6 years ago

Discuss use of VCR for acceptance test. https://pypi.python.org/pypi/vcrpy

gundalow commented 6 years ago

DELETED

amuraleedhar commented 6 years ago

~~Please merge the following documentation / license PR ansible/ansible#36840 ~~ MERGED

grastogi23 commented 6 years ago

https://github.com/ansible/ansible/pull/36468 discuss vcr as shown in above pull request

gundalow commented 6 years ago

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.

This is likely to be the final Release Candidate, so please test, and raise issues on GitHub
Lots of new network documentation http://docs.ansible.com/ansible/devel/network/
No more unable to open shell!
First class plugins for 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.
Loads of bug fixes.
Thank you to all that have tested and provided feedback and PRs so far, can't do this without you.
Happy automating!

samerd commented 6 years ago

ansible/ansible#37310 please review and merge if OK. I have another bug fix that based on it

Qalthos commented 6 years ago

A proposal for eAPI connection type: https://github.com/ansible/proposals/issues/102

calfonso commented 6 years ago

Minutes: https://meetbot.fedoraproject.org/ansible-network/2018-04-04/ansible_network_workgroup.2018-04-04-16.02.html

Log: https://meetbot.fedoraproject.org/ansible-network/2018-04-04/ansible_network_workgroup.2018-04-04-16.02.log.html

ansible / community

Network Working Group Meeting Agenda 2 #247