Qalthos commented 6 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

ujwalkomarla commented 5 years ago

Please review and merge ~~https://github.com/ansible/ansible/pull/53762~~ Merged

rdelcampog commented 5 years ago

Please review and merge https://github.com/ansible/ansible/pull/53210

Qalthos commented 5 years ago

2019-03-20

Team status and updates

Community freeze is in 8 (eight!) days (2019-03-28)

Actions

@UjwalKomarla split exos_vlan from ansible/ansible#53762 into a new PR

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-03-20/ansible_network_working_group.2019-03-20-15.59.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-03-20/ansible_network_working_group.2019-03-20-15.59.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-03-20/ansible_network_working_group.2019-03-20-15.59.log.html

amuraleedhar commented 5 years ago

Please review and merge ~~ansible/ansible#53763~~ Merged

lvrfrc87 commented 5 years ago

Please review and merge. asa_og module. https://github.com/ansible/ansible/pull/52925

Qalthos commented 5 years ago

2019-03-27

Team status and updates

Community freeze has slipped to 2019-04-04
https://docs.ansible.com/ansible/devel/roadmap/ROADMAP_2_8.html should always have the latest dates
NIOS updates to use connection: httpapi instead of local in progress

Actions

@Qalthos report back on restconf_config with method: post

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/[date...] Minutes (text): https://meetbot.fedoraproject.org/ansible-network/[date...] Log: https://meetbot.fedoraproject.org/ansible-network/[date...]

amuraleedhar commented 5 years ago

Please review and merge ansible/ansible#54188 MERGED ansible/ansible#54623

lvrfrc87 commented 5 years ago

~~https://github.com/ansible/ansible/pull/52925~~ Merged

mikewiebe commented 5 years ago

Need to discuss options for https://github.com/ansible/ansible/pull/54691

ujwalkomarla commented 5 years ago

Need to discuss usage of restconf and jsonrpc for exos_* modules. First module using it is exos_vlan @ https://github.com/ansible/ansible/pull/54500

Qalthos commented 5 years ago

2019-04-03

Team status and updates

Freeze has slipped again (https://docs.ansible.com/ansible/devel/roadmap/ROADMAP_2_8.html)

Actions

@Qalthos figure out how mixed-protocol modules might work

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-04-03/ansible_network_working_group.2019-04-03-16.00.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-04-03/ansible_network_working_group.2019-04-03-16.00.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-04-03/ansible_network_working_group.2019-04-03-16.00.log.html

amuraleedhar commented 5 years ago

Please review and merge ~~ansible/ansible#54970~~ Merged This is a bug fix for cnos_vrf module. I fails, if the port is a switchport. So a check is added if its switch port module fails. Made modifications accordingly in unit tests and integration tests as well.

grastogi23 commented 5 years ago

Please review and merge https://github.com/ansible/ansible/pull/54894 This is a fork lift change to remove dependency on external Avi API SDK module. We do realize 2.8 is about to ship. We have extensively tested it but we are okay to wait for this change in 2.9.

Qalthos commented 5 years ago

2019-04-10

Team status and updates

Community Freeze is tomorrow (2019-04-11)
Alpha was released last week, so we mean it this time

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-04-10/ansible_network_working_group.2019-04-10-16.01.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-04-10/ansible_network_working_group.2019-04-10-16.01.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-04-10/ansible_network_working_group.2019-04-10-16.01.log.html

mikewiebe commented 5 years ago

Discuss https://github.com/ansible/ansible/issues/55381 causing many of our NX-OS tests to fail.

Update: No longer need to discuss this given that PR is open with a fix (https://github.com/ansible/ansible/pull/55384)

@Qalthos Thanks for the quick turn around!

chaitanyaavi commented 5 years ago

Please review and merge ansible/ansible#54894

ujwalkomarla commented 5 years ago

Can we discuss this? https://github.com/ansible/ansible/pull/54500 https://github.com/ujwalkomarla/ansible/compare/EXOS_VLAN...ruturajvy:EXOS_VLAN

Qalthos commented 5 years ago

2019-04-17

Team status and updates

2.8 is well on its way now, 2.8.0b1 was released on Monday

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-04-17/ansible_network_working_group.2019-04-17-16.00.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-04-17/ansible_network_working_group.2019-04-17-16.00.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-04-17/ansible_network_working_group.2019-04-17-16.00.log.html

kbreit commented 5 years ago

Additional PR reviewers for Meraki modules
Review https://github.com/ansible/ansible/pull/55528
Feedback and thoughts on https://github.com/ansible/ansible/pull/53891

samerd commented 5 years ago

Please review the following onyx PRs by @anasbadaha and approve if OK:

ansible/ansible#55127
ansible/ansible#55503
ansible/ansible#55513
ansible/ansible#55577

Thanks for your efforts.

twbowman commented 5 years ago

Can we discuss these for eos:

ansible/ansible#55298 ansible/ansible#54863

Thanks.

ujwalkomarla commented 5 years ago

Please Review and Merge: https://github.com/ansible/ansible/pull/51364 https://github.com/ansible/ansible/pull/54500

Qalthos commented 5 years ago

2019-04-24

Team status and updates

2.8 is branched, devel is now open for 2.9

Actions

Qalthos follow up on ansible/ansible#54894

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/https://meetbot.fedoraproject.org/ansible-network/2019-04-24/ansible_network_working_group.2019-04-24-16.01.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/https://meetbot.fedoraproject.org/ansible-network/2019-04-24/ansible_network_working_group.2019-04-24-16.01.txt Log: https://meetbot.fedoraproject.org/ansible-network/https://meetbot.fedoraproject.org/ansible-network/2019-04-24/ansible_network_working_group.2019-04-24-16.01.log.html

amuraleedhar commented 5 years ago

Please Review and Merge: ~~ansible/ansible#55954~~ Merged This is a bug fix for modules after running coverity scan. I need to backport this to 2.8 as well.

kbreit commented 5 years ago

Agenda item:

Review of https://github.com/ansible/ansible/pull/48820
Review of https://github.com/ansible/ansible/pull/49015

IRC nick is mrproper

twbowman commented 5 years ago

Follow up on ansible/ansible#55298. I want to confirm i followed the right procedures and it is good to go.

kbreit commented 5 years ago

IRC nick: mrproper

~~Review https://github.com/ansible/ansible/pull/56199 - Simple PR.~~ Merged
- Review https://github.com/ansible/ansible/pull/53576 - This PR is a little more extensive than a simple PR so it may make sense to take the review conversation away from the meeting.
- Review https://github.com/ansible/ansible/pull/56201 - Decent complexity, may make sense to take review conversation away from the meeting.

grastogi23 commented 5 years ago

Please review https://github.com/ansible/ansible/pull/54894

Qalthos commented 5 years ago

2019-05-15

Team status and updates

Ansible 2.8 is scheduled to release... tomorrow!
2.9 roadmap should be getting populated soon

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-05-15/ansible_network_working_goup.2019-05-15-15.59.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-05-15/ansible_network_working_goup.2019-05-15-15.59.txt https://meetbot.fedoraproject.org/ansible-network/2019-05-15/ansible_network_working_goup.2019-05-15-15.59.log.html

kbreit commented 5 years ago

IRC: mrproper

Review https://github.com/ansible/ansible/pull/48971
Review https://github.com/ansible/ansible/pull/56525
~~Review https://github.com/ansible/ansible/pull/54275~~ Merged
Assistance with mock/patch fetch_url to do unit tests in https://github.com/ansible/ansible/pull/55251

yuandongx commented 5 years ago

Please Review and Merge:

ansible/ansible#55473

Qalthos commented 5 years ago

2019-05-22

Team status and updates

Ansible 2.8 has been released!

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-05-22/ansible_network_working_group.2019-05-22-16.00.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-05-22/ansible_network_working_group.2019-05-22-16.00.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-05-22/ansible_network_working_group.2019-05-22-16.00.log.html

amuraleedhar commented 5 years ago

Please Review and Merge: ~~ansible/ansible#56810~~ Merged This is back porting of 55954 to stable-2.8 Please merge it for 2.8.1

kbreit commented 5 years ago

IRC: mrproper

~~Review of https://github.com/ansible/ansible/pull/48394~~ Merged
Review of https://github.com/ansible/ansible/pull/56818
~~Review of https://github.com/ansible/ansible/pull/48971~~ Merged
~~Review of https://github.com/ansible/ansible/pull/54502~~ Merged
Review of https://github.com/ansible/ansible/pull/56929

yuandongx commented 5 years ago

Please Review and Merge:

ansible/ansible#57123
~~ansible/ansible#56974~~ Merged
~~ansible/ansible#56976~~ Merged
ansible/ansible#57025
~~ansible/ansible#57057~~ Merged

samerd commented 5 years ago

Added support for container management in onyx switches: could you please review and merge: ansible/ansible#57066

chaitanyaavi commented 5 years ago

Please review and merge:

Added new modules https://github.com/ansible/ansible/pull/56995 https://github.com/ansible/ansible/pull/56997 https://github.com/ansible/ansible/pull/56998 https://github.com/ansible/ansible/pull/56999 https://github.com/ansible/ansible/pull/57001 https://github.com/ansible/ansible/pull/57002 https://github.com/ansible/ansible/pull/57003 https://github.com/ansible/ansible/pull/57004 https://github.com/ansible/ansible/pull/57116
Split Avi Credentials into two parts - secrets and login_info https://github.com/ansible/ansible/pull/57114

chrisvanheuveln commented 5 years ago

Please review and merge several NXOS BFD related PRs:

ansible/ansible#56317 (new module: nxos_bfd_global, new NxosCmdRef utility library)
ansible/ansible#56748 (adds bfd option to existing module)
ansible/ansible#56807 (adds bfd option to existing module)
ansible/ansible#56748 (adds bfd option to existing module)
ansible/ansible#56932 (adds bfd option to existing module)
ansible/ansible#57133 (adds bfd option to existing module)

Qalthos commented 5 years ago

2019-05-29

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-05-29/ansible_network_working_group.2019-05-29-15.59.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-05-29/ansible_network_working_group.2019-05-29-15.59.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-05-29/ansible_network_working_group.2019-05-29-15.59.log.html

kbreit commented 5 years ago

IRC: mrproper

~~Review https://github.com/ansible/ansible/pull/54499~~ Merged
~~Review https://github.com/ansible/ansible/pull/54638~~ Merged
~~Review https://github.com/ansible/ansible/pull/56818~~ Merged
~~Review https://github.com/ansible/ansible/pull/56929~~ Merged

yuandongx commented 5 years ago

Please review and merge.

backport-2.8 ansible/ansible#57170
backport-2.8 ansible/ansible#57175
ansible/ansible#57183
ansible/ansible#57187
ansible/ansible#57257
ansible/ansible#57264

mikewiebe commented 5 years ago

Please merge. Reviewed internally and shipit label added.

ansible/ansible#56317 (new module: nxos_bfd_global, new NxosCmdRef utility library)
ansible/ansible#56748 (adds bfd option to existing module)
ansible/ansible#56807 (adds bfd option to existing module)
ansible/ansible#56932 (adds bfd option to existing module)
ansible/ansible#57133 (adds bfd option to existing module)

mikewiebe commented 5 years ago

Please review and merge: Already reviewed by @trishnaguha but she had one additional comment which has been resolved.

https://github.com/ansible/ansible/pull/55993

Please merge. Reviewed internally and shipit label added.

Qalthos commented 5 years ago

2019-06-05

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-06-05/ansible_network_working_group.2019-06-05-16.00.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-06-05/ansible_network_working_group.2019-06-05-16.00.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-06-05/ansible_network_working_group.2019-06-05-16.00.log.html

kbreit commented 5 years ago

IRC: mrproper Review - https://github.com/ansible/ansible/pull/57289 Review - https://github.com/ansible/ansible/pull/54627 (merged) Review - https://github.com/ansible/ansible/pull/57286 (merged) Review - https://github.com/ansible/ansible/pull/53891 Review - https://github.com/ansible/ansible/pull/57562 (merged) Review - https://github.com/ansible/ansible/pull/56201 (merged)

chaitanyaavi commented 5 years ago

Updated as per review comments by @Qalthos https://github.com/ansible/ansible/pull/57531 https://github.com/ansible/ansible/pull/57116

yuandongx commented 5 years ago

Please review and merge.

ansible/ansible#57620
ansible/ansible#57615
ansible/ansible#57625
ansible/ansible#57025
ansible/ansible#57264
ansible/ansible#57268

Qalthos commented 5 years ago

2019-06-12

Team status and updates

Ansible 2.8.1 should be coming very soon now

Logs

Minutes: https://meetbot.fedoraproject.org/ansible-network/2019-06-12/ansible_network_working_group.2019-06-12-16.01.html Minutes (text): https://meetbot.fedoraproject.org/ansible-network/2019-06-12/ansible_network_working_group.2019-06-12-16.01.txt Log: https://meetbot.fedoraproject.org/ansible-network/2019-06-12/ansible_network_working_group.2019-06-12-16.01.log.html

kbreit commented 5 years ago

IRC: mrproper

Feature Enhancements: Review - https://github.com/ansible/ansible/pull/57708 Review - https://github.com/ansible/ansible/pull/54625

New Module: Review - https://github.com/ansible/ansible/pull/55485 Review - https://github.com/ansible/ansible/pull/52889

yuandongx commented 5 years ago

Please review and merge. #

ansible/ansible#57317
ansible/ansible#57903
ansible/ansible#57904
ansible/ansible#57905
ansible/ansible#57907
ansible/ansible#57916
ansible/ansible#57947 #

ansible / community

Network Working Group Meeting Agenda 3 #363

ansible-network on Freenode IRC

2019-03-20

Team status and updates

Actions

Logs

2019-03-27

Team status and updates

Actions

Logs

2019-04-03

Team status and updates

Actions

Logs

2019-04-10

Team status and updates

Logs

2019-04-17

Team status and updates

Logs

2019-04-24

Team status and updates

Actions

Logs

2019-05-15

Team status and updates

Logs

2019-05-22

Team status and updates

Logs

2019-05-29

Logs

2019-06-05

Logs

2019-06-12

Team status and updates

Logs