Wednesday, 2023-04-19

opendevreviewTerry Wilson proposed openstack/ovsdbapp master: Add if_exists and **kwargs columns to db_set  https://review.opendev.org/c/openstack/ovsdbapp/+/88068700:44
ralonsohhi slaweq, if you have 1 min, 3 patches07:37
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/88049607:37
ralonsohhttps://review.opendev.org/c/openstack/neutron-lib/+/88010307:38
ralonsohhttps://review.opendev.org/c/openstack/neutron-lib/+/88010207:38
ralonsohthanks!07:38
ralonsohah, please approve https://review.opendev.org/c/openstack/releases/+/87820207:43
opendevreviewRodolfo Alonso proposed openstack/neutron master: Improve "sync_ha_chassis_group" method  https://review.opendev.org/c/openstack/neutron/+/87202307:44
slaweqralonsoh done07:45
ralonsohthanks!07:45
ralonsohbtw, I think is time to approve https://review.opendev.org/c/openstack/python-neutronclient/+/871711 too07:46
slaweqralonsoh and regarding train eol patch I have same comment as bcafarel 07:46
slaweqit's now 4 projects, not 2 which aren't EOL'ed07:46
ralonsohslaweq, I'll push a new PS07:46
slaweqis that correct?07:46
ralonsohyes 407:46
slaweqdo we want to keep os-ken and python-neutronclient alive?07:46
ralonsohslaweq, yes, that was proposed by Elod07:47
ralonsohhttps://review.opendev.org/c/openstack/releases/+/878202/comments/52c84c8b_b832d67b07:48
ralonsohbecause these are not stadium projects07:48
slaweqralonsoh I'm not sure I understand that reason07:53
ralonsohwe can consider these projects as libraries07:54
slaweqthose projects are also governed by neutron team, same as other stadium projects and IMO we can EOL them07:54
slaweqok07:54
slaweqI see07:54
ralonsohI know yes, in any case, we don't need to continue supporting them07:54
ralonsohjust best effort and we'll EOL all of them in 1 cycle07:54
slaweqif we will treat them as libraries maybe it's indeed better to keep them open07:55
slaweq++07:55
opendevreviewSlawek Kaplonski proposed openstack/neutron master: WIP [S-RBAC] Switch to new policies by default  https://review.opendev.org/c/openstack/neutron/+/87982708:55
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Apply admin_state_up on a new member creation  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88073709:00
opendevreviewliuxie proposed openstack/neutron-lib master: New api-def: allowedaddresspairs-atomic  https://review.opendev.org/c/openstack/neutron-lib/+/88061509:08
opendevreviewMerged openstack/neutron-lib stable/zed: Return properly elevated context by get_admin_context() helper  https://review.opendev.org/c/openstack/neutron-lib/+/88010309:36
opendevreviewMerged openstack/neutron stable/2023.1: [sqlalchemy-20] Add reader context to ``get_ports_on_host_by_subnet``  https://review.opendev.org/c/openstack/neutron/+/88049609:43
opendevreviewliuxie proposed openstack/neutron-lib master: New api-def: allowedaddresspairs-atomic  https://review.opendev.org/c/openstack/neutron-lib/+/88061509:43
racostaHey folks, can someone please provide some review here? https://review.opendev.org/c/openstack/neutron/+/86751310:48
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add physical network information to LS  https://review.opendev.org/c/openstack/neutron/+/87949610:51
ralonsohracosta, the testing part is quite poor, almost nothing of "ovn_client.py" is tested10:53
ralonsohand not in UTs but also in FTs10:54
ralonsohracosta, please provide FTs testing the creation, update and deletion of a router10:55
ralonsohand how the nat entries are udpated10:55
racostathanks fir the feedback. Let me check again but I put the router creation, update with inclusion and removal of ports and tests of NAT rules in FT func test_dvr_ipv6_router_port (test_maintenance.py file).11:07
ralonsohracosta, why did you implement that code there? this should be in test_mech_driver11:16
opendevreviewDmitrii Shcherbakov proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref  https://review.opendev.org/c/openstack/neutron-lib/+/87088711:24
racostaralonsoh, because FTs for floating IPs is in maintenance and I based it on that. I know, it was a bad design decision, sorry.11:26
dmitriiso/ core devs, if anybody has time to review https://review.opendev.org/c/openstack/neutron-lib/+/870887 it would be much appreciated to unblock further testing and review of this series https://review.opendev.org/q/topic:2023-aa-l3-gw-multihoming. Brian is away this week so maybe somebody else is able to do it. 11:27
ralonsohI'll check it11:29
dmitriisralonsoh: tyvm11:30
opendevreviewMerged openstack/ovn-octavia-provider master: Use ovsdbapp commands to add/del backends to ip_port_mappings  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87438411:39
ralonsohdmitriis, please check my question there11:40
dmitriisralonsoh: ack, I'll loop around soon11:41
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Apply admin_state_up on a new member creation  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88073711:44
ykarelobondarev, hi11:51
ykarelcan you please check https://bugs.launchpad.net/neutron/+bug/1976323/comments/8 related to local_ip behavior11:52
obondarevykarel: hi, I'll check11:55
ykarelThanks obondarev 11:56
obondarevykarel: can you please point to the failed run?11:56
opendevreviewDmitrii Shcherbakov proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref  https://review.opendev.org/c/openstack/neutron-lib/+/87088712:03
dmitriisralonsoh: https://review.opendev.org/c/openstack/neutron-lib/+/870887/17..18 - done12:05
ykarelobondarev_, https://d03ae6c510e94a36ef14-ea5b6d57baa1d0c2bb40e5e53ebb92ee.ssl.cf5.rackcdn.com/843245/15/check/neutron-fullstack-with-uwsgi-fips/81f70e2/testr_results.html12:28
opendevreviewEdward Hope-Morley proposed openstack/neutron stable/zed: OVN: Always try and create a metadata port on subnets  https://review.opendev.org/c/openstack/neutron/+/88082212:57
opendevreviewEdward Hope-Morley proposed openstack/neutron stable/yoga: OVN: Always try and create a metadata port on subnets  https://review.opendev.org/c/openstack/neutron/+/88082312:57
slaweqlajoskatona obondarev mlavalle ykarel haleyb and others: hi, as we discussed yesterday on the neutron team meeting, with  ralonsoh we wrote today some initial drafts of the 2 sessions we would like to propose during summit. Please check it https://docs.google.com/document/d/13DTXS3i4FNIfoiyDLHkVzLj5H5kFQTB7A5MKcAhg4wo/edit, comment and tell us what do You think12:58
slaweqalso if any of You is interested in moderating session(s) - please let us know12:59
ralonsohat 14UTC, as commented, yesterday, we can have a quick meeting12:59
ralonsohor you can directly provide the feedback on the doc and here in IRC12:59
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp stable/2023.1: Use OVN's OVS submodule for functional tests  https://review.opendev.org/c/openstack/ovsdbapp/+/88083013:01
lajoskatonaslaweq: cool, thanks, I check it13:13
lajoskatonaslaweq, ralonsoh: I can check it now, but t 1400UTC I will be "outofoffice" ( I bring my sons and myself to the swimmingpool)13:14
ralonsohno problem13:14
ralonsohyou can provide feedback on the docs13:14
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Add physical network information to LS  https://review.opendev.org/c/openstack/neutron/+/87949613:17
opendevreviewTerry Wilson proposed openstack/ovsdbapp master: Add if_exists and **kwargs columns to db_set  https://review.opendev.org/c/openstack/ovsdbapp/+/88068713:36
opendevreviewFrode Nordahl proposed openstack/ovsdbapp master: Add support for manipulating BFD entries  https://review.opendev.org/c/openstack/ovsdbapp/+/87304213:50
mlavalleo/14:00
ralonsohhi folks14:09
ralonsohI'll use the meeting ping14:10
mlavalleo/14:10
ralonsohPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, sahid, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki 14:10
ralonsohsorry for the broad ping14:10
ralonsohdid you have time to review the docs slaweq shared?14:10
ralonsoh#link https://docs.google.com/document/d/13DTXS3i4FNIfoiyDLHkVzLj5H5kFQTB7A5MKcAhg4wo/edit14:11
bcafarelo/14:11
ralonsohlajoskatona cannot attend but he agreed on both proposals14:11
* mlavalle did14:11
slaweqo/14:11
ralonsohand mlavalle 14:11
opendevreviewliuxie proposed openstack/neutron-lib master: New api-def: allowedaddresspairs-atomic  https://review.opendev.org/c/openstack/neutron-lib/+/88061514:12
ralonsohif you have any feedback, we'll be here for a couple of hours more14:12
ralonsohyou have also the link provided14:12
ykarelboth proposals lgtm14:12
bcafarelnot a lot to add except LGTM14:12
ralonsohso far, I think we can propose both then14:13
ralonsohI hope both have traction and he have attendees14:13
slaweqthx14:13
slaweqI will submit both proposals today then14:13
ralonsohso please slaweq, go on yeah14:13
ralonsohand thanks!14:14
slaweqanyone would like to be moderator?14:14
slaweqif not, I will put me and ralonsoh there, at least for now14:14
ralonsohperfect14:14
mlavalleI can help moderating14:14
slaweqbut those are forum sessions so of course everyone is welcome there :)14:14
slaweqthx mlavalle I will add You there too14:14
mlavalleand I can also conduct the on boarding zession, or a piece of it14:15
mlavallesession^^14:15
slaweqsure, thx a lot14:15
mlavalleslaweq, ralonsoh: for the on boarding session, we can reuse (after updating / upgrading) we can use https://docs.google.com/presentation/d/1Eqi3Kmr94FOuTOTmTXnqYu3s_ZQcLdppusfm-l03Kss/edit#slide=id.p14:19
mlavallewhich we used in Shanghai14:19
mlavalleI'm going to mention it in the proposal comments14:20
slaweq++14:20
ralonsohok but this is a forum session, this presentation could be a tool for new comers 14:20
ralonsohactually we can have something like this in our repo docs14:21
ralonsohwith these examples14:21
ralonsohwe need indeed to translate this presentation to the docs14:22
ralonsohmlavalle, ^14:22
ralonsohthat will be very helpful for sure14:22
mlavalleralonsoh: you want me to do it?14:22
ralonsohbut only if you have time, and that can be done in steps14:23
ralonsohnot in one shot14:23
mlavalleralonsoh: will do. I'll do it slowly14:23
mlavalle:-)14:23
*** Guest9750 is now known as atmark14:32
sahidguys what about a session presenting our testing frameworks? unittests, functinal, fullstacks, grenade...14:33
opendevreviewTerry Wilson proposed openstack/ovsdbapp master: Add if_exists and **kwargs columns to db_set  https://review.opendev.org/c/openstack/ovsdbapp/+/88068714:34
opendevreviewSlawek Kaplonski proposed openstack/neutron master: WIP [S-RBAC] Switch to new policies by default  https://review.opendev.org/c/openstack/neutron/+/87982714:55
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Fix dns_integration and ml2 plugin unit tests modules  https://review.opendev.org/c/openstack/neutron/+/88085514:57
opendevreviewRoberto Acosta proposed openstack/neutron master: [RFE] [OVN] - IPv6 Distributed routing  https://review.opendev.org/c/openstack/neutron/+/86751315:01
opendevreviewFrode Nordahl proposed openstack/ovsdbapp master: Add support for manipulating BFD entries  https://review.opendev.org/c/openstack/ovsdbapp/+/87304215:30
opendevreviewRodolfo Alonso proposed openstack/neutron master: [DNM] Skip the MySQL and Postgree user definition  https://review.opendev.org/c/openstack/neutron/+/88086715:46
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/ovsdbapp/+/87609116:44
opendevreviewRodolfo Alonso proposed openstack/ovsdbapp stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/ovsdbapp/+/87609216:45
opendevreviewLajos Katona proposed openstack/python-neutronclient master: OSC: Remove FWAAS V2 calls to neutronclient  https://review.opendev.org/c/openstack/python-neutronclient/+/88062916:46
opendevreviewRodolfo Alonso proposed openstack/neutron master: [DNM] Skip the MySQL and Postgree user definition  https://review.opendev.org/c/openstack/neutron/+/88086716:46
ralonsohsee you tomorrow16:47
opendevreviewFrode Nordahl proposed openstack/ovsdbapp master: Add support for manipulating BFD entries  https://review.opendev.org/c/openstack/ovsdbapp/+/87304217:14
opendevreviewMerged openstack/ovsdbapp master: Add 'no timeout' option to wait_for_change  https://review.opendev.org/c/openstack/ovsdbapp/+/85780617:19
opendevreviewMerged openstack/ovsdbapp stable/2023.1: Use OVN's OVS submodule for functional tests  https://review.opendev.org/c/openstack/ovsdbapp/+/88083017:19
opendevreviewMerged openstack/ovsdbapp stable/2023.1: Update .gitreview for stable/2023.1  https://review.opendev.org/c/openstack/ovsdbapp/+/87609119:18
opendevreviewMerged openstack/ovsdbapp stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1  https://review.opendev.org/c/openstack/ovsdbapp/+/87609219:18
opendevreviewIhar Hrachyshka proposed openstack/neutron-tempest-plugin master: gate: bump ovn to the latest LTS release (22.03)  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88088920:36
opendevreviewIhar Hrachyshka proposed openstack/neutron master: gate: bump ovn to the latest LTS release (22.03)  https://review.opendev.org/c/openstack/neutron/+/88089020:45
opendevreviewIhar Hrachyshka proposed openstack/neutron master: gate: bump ovn to the latest LTS release (22.03)  https://review.opendev.org/c/openstack/neutron/+/88089021:00
opendevreviewMiro Tomaska proposed openstack/neutron master: Handle "no such process" during keepalived process cleanup  https://review.opendev.org/c/openstack/neutron/+/88089321:00
opendevreviewIhar Hrachyshka proposed openstack/neutron-tempest-plugin master: gate: bump ovn to the latest LTS release (22.03)  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/88088921:00
-opendevstatus- NOTICE: The Etherpad service on etherpad.opendev.org will be offline for the next 90 minutes for a server replacement and operating system upgrade21:58
opendevreviewMiguel Lavalle proposed openstack/neutron master: Add rate-limiting to metadata agents  https://review.opendev.org/c/openstack/neutron/+/85887923:23

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!