Friday, 2022-07-29

opendevreviewZhouHeng proposed openstack/neutron-fwaas master: Bump pyroute2 version to 0.7.2  https://review.opendev.org/c/openstack/neutron-fwaas/+/84683400:29
opendevreviewOpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84686404:09
ralonsohhi folks, how do I trigger the periodic jobs in https://review.opendev.org/c/openstack/neutron/+/851433 ?07:27
slaweqralonsoh: IIRC all periodic jobs should also be in the experimental queue 07:28
ralonsohahhh yes07:28
ralonsohthanks!07:28
slaweqyw 07:28
opendevreviewRodolfo Alonso proposed openstack/neutron master: ``Session.autocommit`` parameter is removed  https://review.opendev.org/c/openstack/neutron/+/85143307:33
opendevreviewRodolfo Alonso proposed openstack/neutron master: ``Session.autocommit`` parameter is removed  https://review.opendev.org/c/openstack/neutron/+/85143307:35
opendevreviewliuxie proposed openstack/neutron-fwaas master: Support l3 firewall for ovn driver  https://review.opendev.org/c/openstack/neutron-fwaas/+/84575608:56
slaweqralonsoh: when You will have some time, please check https://review.opendev.org/c/openstack/neutron/+/844589 and hopefully approve it :)10:06
slaweqthx in advance10:06
ralonsohsure10:07
opendevreviewRodolfo Alonso proposed openstack/neutron master: Implement specific tracked resource count method per quota driver  https://review.opendev.org/c/openstack/neutron/+/85135710:11
ralonsohlajoskatona, https://review.opendev.org/c/openstack/neutron-fwaas/+/84683410:17
ralonsohjust missing a +W10:17
ralonsohI'm reviewing the patches that are ready to be merged10:17
opendevreviewMerged openstack/neutron-fwaas master: Add weekly jobs  https://review.opendev.org/c/openstack/neutron-fwaas/+/85139810:27
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job  https://review.opendev.org/c/openstack/neutron/+/85150610:43
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job  https://review.opendev.org/c/openstack/neutron/+/85150610:45
opendevreviewliuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver  https://review.opendev.org/c/openstack/neutron/+/85150911:03
opendevreviewMerged openstack/neutron-fwaas master: Add libczmq4 to bindep for testing on jammy  https://review.opendev.org/c/openstack/neutron-fwaas/+/85139711:06
opendevreviewMerged openstack/networking-ovn stable/train: Optimize queries for port operations  https://review.opendev.org/c/openstack/networking-ovn/+/85073811:10
opendevreviewArnau Verdaguer proposed openstack/neutron master: Migration revert plan  https://review.opendev.org/c/openstack/neutron/+/83563811:25
slaweqlajoskatona: ralonsoh please check https://review.opendev.org/c/openstack/neutron/+/851506 when You will have few minutes11:55
slaweqit fixes neutron-grenade job: https://zuul.opendev.org/t/openstack/build/aa1dbf7dac6b4da7b562c9529367235911:56
slaweqin stable/train11:56
lajoskatonaslaweq: thanks12:01
lajoskatonaslaweq: we just stopped it :-)12:01
slaweqlajoskatona: it isn't run anywhere?12:02
lajoskatonaslaweq: https://review.opendev.org/c/openstack/neutron/+/85133012:03
lajoskatonaTakashi made it non-voting early this week12:04
slaweqlajoskatona: ah, yeah12:11
slaweqbut I though it will be temporary12:11
slaweqand my patch makes it voting again12:11
lajoskatonaslaweq: sure, it was just a susrpise that you have solution so quickly :-)12:12
lajoskatonabcafarel, slaweq, all: elodilles sent out mail for starting pike EOL : https://lists.openstack.org/pipermail/openstack-discuss/2022-July/029769.html12:14
elodillesneutron has already pike-eol :)12:14
bcafarelah thanks for confirmation I thought I remembered we had it12:16
elodillesbut sorry, lajoskatona , go on o:)12:16
lajoskatonaelodilles: you are right, I always mess up these branches, just ignore me12:16
bcafarelalso for stadium projects IIRC?12:17
bcafareloh yes I see sfc has eol tags up to stein12:17
elodillesbcafarel: as i remember, those, too12:17
lajoskatonayeah we have, so no work with this :-) Good job :-)12:18
elodilles:]12:18
elodillesanyway, i'm just about to generate the pike-eol patches, so we will see if something was left out ;)12:21
slaweqlajoskatona: that was an easy one :)12:29
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job  https://review.opendev.org/c/openstack/neutron/+/85150612:32
slaweqlajoskatona: bcafarel ^^ I just updated it adding it back to the gate queue12:33
bcafarelslaweq++ I still wonder what is going wrong in the old job, but not high prio if updated job definition works fine12:36
slaweqbcafarel: the think is that legacy job is using devstack-gate scripts and new one not12:36
slaweqand it is hanging somewhere in that devstack-gate scripts12:36
slaweqI could try to investigate more but I don't think it's really worth to spend time on it :)12:37
bcafarelyeah most probably not :)12:38
lajoskatona+112:38
ralonsohslaweq, sure, let me check12:41
*** amoralej is now known as amoralej|lunch12:50
opendevreviewMerged openstack/tap-as-a-service master: Py3: Remove six  https://review.opendev.org/c/openstack/tap-as-a-service/+/84662412:54
opendevreviewBernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3  https://review.opendev.org/c/openstack/neutron/+/85151213:16
opendevreviewBernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3  https://review.opendev.org/c/openstack/neutron/+/85151213:21
opendevreviewBernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3  https://review.opendev.org/c/openstack/neutron/+/85151213:21
slaweqbcafarel: ^^ I see You are following up on my patch :)13:22
bcafarelslaweq: yes you motivated me to do some summer cleaning :) (and also limiting periodic job failure mails)13:23
slaweq++13:23
opendevreviewMerged openstack/neutron master: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/84912213:45
opendevreviewMerged openstack/neutron master: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85052913:45
*** dasm|off is now known as dasm13:59
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/yoga: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85138513:59
mlavalleslaweq, ralonsoh, lajoskatona: when you have a chance, please review https://review.opendev.org/c/openstack/neutron/+/85043013:59
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/xena: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85138613:59
ralonsohmlavalle, ok13:59
lajoskatona#startmeeting neutron_drivers14:00
opendevmeetMeeting started Fri Jul 29 14:00:11 2022 UTC and is due to finish in 60 minutes.  The chair is lajoskatona. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'neutron_drivers'14:00
lajoskatonao/14:00
mlavalleo/14:00
yamamotohi14:00
slaweqo/14:00
ralonsohhi14:00
obondarevhi14:00
lajoskatonaThanks for joining, let's start the meeting14:01
lajoskatonaWe have one RFE for today14:02
lajoskatona[RFE] OVN E/W routing for external (baremetal) VLAN ports (#link https://bugs.launchpad.net/neutron/+bug/1982541 )14:02
lajoskatonayesterday I tried to read about it and my understanding is that we miss things in core OVN for this, am I wrong?14:03
*** amoralej|lunch is now known as amoralej14:03
lajoskatonaoh, I see now comment 2 mentions these from ovs dev list14:03
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/wallaby: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85152314:03
slaweqI was reading it today and I was trying to understand what is this RFE about really and what we would need to do in Neutron with it14:04
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/victoria: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85138814:04
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/ussuri: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85138914:05
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: Port update will trigger less notifications to the DHCP agents  https://review.opendev.org/c/openstack/neutron/+/85153014:05
ralonsohI would need to confirm that, but the core OVN support is for VLAN networks in DVR14:06
ralonsohcurrently we support VLAN with a centralized chassis14:06
ralonsohok, and this is why it doesn't work: https://bugs.launchpad.net/neutron/+bug/1982541/comments/114:08
ralonsohbecause SRIOV traffic is vlan tagged 14:08
ralonsohand must have "reside-on-redirect-chassis"14:08
lajoskatonaok, so it is not somthing missing in Neutron but in OVN itself14:10
ralonsohand maybe neutron14:10
ralonsohbut I don't have context or more info14:10
lajoskatonaok, thanks14:10
slaweqbut is this really RFE for us?14:11
lajoskatonaI am not sure if we have "process" for such features, we have previously the smartnic for OVN from canonnical14:11
lajoskatonaI mean when first something has to be done in OVN14:11
ralonsohI'm pinging dalvarez and lucasagomes 14:12
ralonsohif they can provide more info on this14:12
lajoskatona+1. thanks14:12
ralonsohin any case, the description is done in https://bugs.launchpad.net/neutron/+bug/187585214:13
ralonsohthere are two alternative imlpementations14:13
ralonsohthe 2nd is the preferred 14:14
ralonsohand implies supporting VLAN DVR traffic14:14
ralonsoh(yes, core OVN support)14:14
ralonsohthis, IMO, I will say +1 to this RFE **once** we can fully supported this ^^ in core OVN14:15
lajoskatonaI agree, let's wait for the OVN version which supports this14:16
lajoskatonaI add rfe-postponed tag for it in lp with comment about the discussion, what do you think  ?14:17
mlavallesounds good to me14:17
ralonsohperfect14:17
slaweq+114:17
lucasagomesralonsoh, reading14:18
lajoskatonaand we can say that this is the process for such features: "Wait for the release of the feature of 3rd party product, like OVN"14:18
mlavalleupstream dependency, I would say14:19
lajoskatonamlavalle: perfect wording14:19
dalvarezralonsoh: afaict dvr and vlan works in core ovn i know that jlibosva verified that it works (with the ovn mac mappings in each chassis) but it has the issue of flooding the packets to every node connected to the ToR14:20
mlavalleI'm also reading http://dani.foroselectronica.es/ovn-external-ports-604/ as background info14:20
dalvarezralonsoh: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html14:21
ralonsohyeah, right14:21
dalvarezwe can route with sriov, not sure if that's a valid use case though. Also is it working on ml2ovs?14:21
dalvarezi guess it is, traffic will go to the gw and that's it... we can achieve that perhaps with the ovn external port too14:22
ralonsohno, we don't route sriov traffic with OVS14:22
lajoskatonaok, so we need more investigation if this is supported by OVN or not?14:25
ralonsohI think so or at least the problems present in the current implementation14:26
lajoskatonaok, so anyway I ask in comment for more details if it is really supported now, and if those present we can discuss it again14:27
lajoskatonais that ok for everybody?14:27
ralonsoh+114:27
obondarev+114:27
mlavalle+114:27
yamamoto+114:27
slaweq+114:28
lajoskatonaok ,thanks for the discussion14:30
lajoskatonado you have anything which we can discuss today?14:30
mlavallenot from me14:30
slaweqnope14:30
ralonsohno14:30
lajoskatonaJust a reminder:14:30
lajoskatonahttps://lists.openstack.org/pipermail/openstack-discuss/2022-July/029765.html14:30
slaweqjust wish You great holidays lajoskatona :)14:30
lajoskatonaI cancelled team and drivers meeting for the next 2 weeks14:30
ralonsoh+1 to holidays 14:31
lajoskatonaslaweq: thanks :-)14:31
mlavalleyeah, have a restful and happy vacation14:31
lajoskatonathanks you all14:31
lajoskatonaif nothing more we can close the meeting14:31
mlavallewe will wreck the place in your absence14:31
slaweqLOL14:31
lajoskatonamlavalle: please do it, I have to do something when I am back14:31
lajoskatona#endmeeting14:32
opendevmeetMeeting ended Fri Jul 29 14:32:00 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:32
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.html14:32
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.txt14:32
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.log.html14:32
lajoskatonaBye14:32
ralonsohhave a nice weekend14:32
slaweqo/14:32
obondarevo/14:32
slaweqthx, You too ralonsoh 14:32
yamamotoo/14:32
opendevreviewBernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3  https://review.opendev.org/c/openstack/neutron/+/85151214:35
opendevreviewliuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver  https://review.opendev.org/c/openstack/neutron/+/85150914:55
opendevreviewMiro Tomaska proposed openstack/neutron stable/yoga: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153215:30
opendevreviewMiro Tomaska proposed openstack/neutron stable/xena: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153315:31
opendevreviewMiro Tomaska proposed openstack/neutron stable/wallaby: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153415:31
opendevreviewMiro Tomaska proposed openstack/neutron stable/victoria: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153515:32
opendevreviewMiro Tomaska proposed openstack/neutron stable/ussuri: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153615:32
*** iurygregory_ is now known as iurygregory15:57
opendevreviewMiro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153716:01
opendevreviewAkihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84008816:13
*** amoralej is now known as amoralej|off16:21
opendevreviewRodolfo Alonso proposed openstack/neutron master: [DNM] [OVN][QoS] Add minimum bandwidth rule support to ML2/OVN  https://review.opendev.org/c/openstack/neutron/+/84229216:29
opendevreviewMiro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153716:30
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84008816:47
opendevreviewAkihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Update master for stable/xena  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/81041816:47
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84686416:52
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925716:54
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Replace deprecated django.conf.urls.url()  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/84925816:55
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Update master for stable/xena  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/81041817:13
opendevreviewAkihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/78223117:15
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/85083417:16
opendevreviewAkihiro Motoki proposed openstack/neutron-vpnaas-dashboard stable/victoria: Drop lower-constraints (victoria)  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/85159617:21
opendevreviewMerged openstack/neutron master: Make iptables rules diff to be case insensitive  https://review.opendev.org/c/openstack/neutron/+/84458917:37
opendevreviewMerged openstack/neutron-vpnaas-dashboard master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/78223117:39
opendevreviewTerry Wilson proposed openstack/ovsdbapp master: nb: add support for lb health checks API  https://review.opendev.org/c/openstack/ovsdbapp/+/83283518:17
opendevreviewMerged openstack/neutron stable/train: Optimize queries for port operations  https://review.opendev.org/c/openstack/neutron/+/85073518:24
opendevreviewMerged openstack/ovsdbapp master: nb: add methods to modify the lrp.networks  https://review.opendev.org/c/openstack/ovsdbapp/+/83215818:42
opendevreviewMerged openstack/ovsdbapp master: Provide base classes for {Get,Set}Options commands  https://review.opendev.org/c/openstack/ovsdbapp/+/84956919:17
opendevreviewAnthony proposed openstack/neutron-specs master: FWaaS Group Ordering Spec  https://review.opendev.org/c/openstack/neutron-specs/+/85160719:20
opendevreviewMerged openstack/ovsdbapp master: nb: add support for lb health checks API  https://review.opendev.org/c/openstack/ovsdbapp/+/83283520:14
*** dasm is now known as dasm|off20:46
opendevreviewMerged openstack/neutron-specs master: Add spec for Network cascade deletion  https://review.opendev.org/c/openstack/neutron-specs/+/81082221:41
opendevreviewMiro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153722:53
opendevreviewMiro Tomaska proposed openstack/neutron stable/ussuri: Add workaround for eventlet.greendns bug  https://review.opendev.org/c/openstack/neutron/+/85153622:54
opendevreviewMiguel Lavalle proposed openstack/neutron master: Avoid race condition when deleting trunk bridges  https://review.opendev.org/c/openstack/neutron/+/83778023:37

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