opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: Bump pyroute2 version to 0.7.2 https://review.opendev.org/c/openstack/neutron-fwaas/+/846834 | 00:29 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/846864 | 04:09 |
ralonsoh | hi folks, how do I trigger the periodic jobs in https://review.opendev.org/c/openstack/neutron/+/851433 ? | 07:27 |
slaweq | ralonsoh: IIRC all periodic jobs should also be in the experimental queue | 07:28 |
ralonsoh | ahhh yes | 07:28 |
ralonsoh | thanks! | 07:28 |
slaweq | yw | 07:28 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: ``Session.autocommit`` parameter is removed https://review.opendev.org/c/openstack/neutron/+/851433 | 07:33 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: ``Session.autocommit`` parameter is removed https://review.opendev.org/c/openstack/neutron/+/851433 | 07:35 |
opendevreview | liuxie proposed openstack/neutron-fwaas master: Support l3 firewall for ovn driver https://review.opendev.org/c/openstack/neutron-fwaas/+/845756 | 08:56 |
slaweq | ralonsoh: when You will have some time, please check https://review.opendev.org/c/openstack/neutron/+/844589 and hopefully approve it :) | 10:06 |
slaweq | thx in advance | 10:06 |
ralonsoh | sure | 10:07 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Implement specific tracked resource count method per quota driver https://review.opendev.org/c/openstack/neutron/+/851357 | 10:11 |
ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/neutron-fwaas/+/846834 | 10:17 |
ralonsoh | just missing a +W | 10:17 |
ralonsoh | I'm reviewing the patches that are ready to be merged | 10:17 |
opendevreview | Merged openstack/neutron-fwaas master: Add weekly jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/851398 | 10:27 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job https://review.opendev.org/c/openstack/neutron/+/851506 | 10:43 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job https://review.opendev.org/c/openstack/neutron/+/851506 | 10:45 |
opendevreview | liuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 11:03 |
opendevreview | Merged openstack/neutron-fwaas master: Add libczmq4 to bindep for testing on jammy https://review.opendev.org/c/openstack/neutron-fwaas/+/851397 | 11:06 |
opendevreview | Merged openstack/networking-ovn stable/train: Optimize queries for port operations https://review.opendev.org/c/openstack/networking-ovn/+/850738 | 11:10 |
opendevreview | Arnau Verdaguer proposed openstack/neutron master: Migration revert plan https://review.opendev.org/c/openstack/neutron/+/835638 | 11:25 |
slaweq | lajoskatona: ralonsoh please check https://review.opendev.org/c/openstack/neutron/+/851506 when You will have few minutes | 11:55 |
slaweq | it fixes neutron-grenade job: https://zuul.opendev.org/t/openstack/build/aa1dbf7dac6b4da7b562c95293672359 | 11:56 |
slaweq | in stable/train | 11:56 |
lajoskatona | slaweq: thanks | 12:01 |
lajoskatona | slaweq: we just stopped it :-) | 12:01 |
slaweq | lajoskatona: it isn't run anywhere? | 12:02 |
lajoskatona | slaweq: https://review.opendev.org/c/openstack/neutron/+/851330 | 12:03 |
lajoskatona | Takashi made it non-voting early this week | 12:04 |
slaweq | lajoskatona: ah, yeah | 12:11 |
slaweq | but I though it will be temporary | 12:11 |
slaweq | and my patch makes it voting again | 12:11 |
lajoskatona | slaweq: sure, it was just a susrpise that you have solution so quickly :-) | 12:12 |
lajoskatona | bcafarel, slaweq, all: elodilles sent out mail for starting pike EOL : https://lists.openstack.org/pipermail/openstack-discuss/2022-July/029769.html | 12:14 |
elodilles | neutron has already pike-eol :) | 12:14 |
bcafarel | ah thanks for confirmation I thought I remembered we had it | 12:16 |
elodilles | but sorry, lajoskatona , go on o:) | 12:16 |
lajoskatona | elodilles: you are right, I always mess up these branches, just ignore me | 12:16 |
bcafarel | also for stadium projects IIRC? | 12:17 |
bcafarel | oh yes I see sfc has eol tags up to stein | 12:17 |
elodilles | bcafarel: as i remember, those, too | 12:17 |
lajoskatona | yeah we have, so no work with this :-) Good job :-) | 12:18 |
elodilles | :] | 12:18 |
elodilles | anyway, i'm just about to generate the pike-eol patches, so we will see if something was left out ;) | 12:21 |
slaweq | lajoskatona: that was an easy one :) | 12:29 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: [stable/train] Convert neutron-grenade job to the zuul v3 job https://review.opendev.org/c/openstack/neutron/+/851506 | 12:32 |
slaweq | lajoskatona: bcafarel ^^ I just updated it adding it back to the gate queue | 12:33 |
bcafarel | slaweq++ I still wonder what is going wrong in the old job, but not high prio if updated job definition works fine | 12:36 |
slaweq | bcafarel: the think is that legacy job is using devstack-gate scripts and new one not | 12:36 |
slaweq | and it is hanging somewhere in that devstack-gate scripts | 12:36 |
slaweq | I could try to investigate more but I don't think it's really worth to spend time on it :) | 12:37 |
bcafarel | yeah most probably not :) | 12:38 |
lajoskatona | +1 | 12:38 |
ralonsoh | slaweq, sure, let me check | 12:41 |
*** amoralej is now known as amoralej|lunch | 12:50 | |
opendevreview | Merged openstack/tap-as-a-service master: Py3: Remove six https://review.opendev.org/c/openstack/tap-as-a-service/+/846624 | 12:54 |
opendevreview | Bernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3 https://review.opendev.org/c/openstack/neutron/+/851512 | 13:16 |
opendevreview | Bernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3 https://review.opendev.org/c/openstack/neutron/+/851512 | 13:21 |
opendevreview | Bernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3 https://review.opendev.org/c/openstack/neutron/+/851512 | 13:21 |
slaweq | bcafarel: ^^ I see You are following up on my patch :) | 13:22 |
bcafarel | slaweq: yes you motivated me to do some summer cleaning :) (and also limiting periodic job failure mails) | 13:23 |
slaweq | ++ | 13:23 |
opendevreview | Merged openstack/neutron master: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/849122 | 13:45 |
opendevreview | Merged openstack/neutron master: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/850529 | 13:45 |
*** dasm|off is now known as dasm | 13:59 | |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/yoga: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851385 | 13:59 |
mlavalle | slaweq, ralonsoh, lajoskatona: when you have a chance, please review https://review.opendev.org/c/openstack/neutron/+/850430 | 13:59 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851386 | 13:59 |
ralonsoh | mlavalle, ok | 13:59 |
lajoskatona | #startmeeting neutron_drivers | 14:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:00 |
lajoskatona | o/ | 14:00 |
mlavalle | o/ | 14:00 |
yamamoto | hi | 14:00 |
slaweq | o/ | 14:00 |
ralonsoh | hi | 14:00 |
obondarev | hi | 14:00 |
lajoskatona | Thanks for joining, let's start the meeting | 14:01 |
lajoskatona | We have one RFE for today | 14:02 |
lajoskatona | [RFE] OVN E/W routing for external (baremetal) VLAN ports (#link https://bugs.launchpad.net/neutron/+bug/1982541 ) | 14:02 |
lajoskatona | yesterday 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 amoralej | 14:03 | |
lajoskatona | oh, I see now comment 2 mentions these from ovs dev list | 14:03 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851523 | 14:03 |
slaweq | I 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 it | 14:04 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851388 | 14:04 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851389 | 14:05 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Port update will trigger less notifications to the DHCP agents https://review.opendev.org/c/openstack/neutron/+/851530 | 14:05 |
ralonsoh | I would need to confirm that, but the core OVN support is for VLAN networks in DVR | 14:06 |
ralonsoh | currently we support VLAN with a centralized chassis | 14:06 |
ralonsoh | ok, and this is why it doesn't work: https://bugs.launchpad.net/neutron/+bug/1982541/comments/1 | 14:08 |
ralonsoh | because SRIOV traffic is vlan tagged | 14:08 |
ralonsoh | and must have "reside-on-redirect-chassis" | 14:08 |
lajoskatona | ok, so it is not somthing missing in Neutron but in OVN itself | 14:10 |
ralonsoh | and maybe neutron | 14:10 |
ralonsoh | but I don't have context or more info | 14:10 |
lajoskatona | ok, thanks | 14:10 |
slaweq | but is this really RFE for us? | 14:11 |
lajoskatona | I am not sure if we have "process" for such features, we have previously the smartnic for OVN from canonnical | 14:11 |
lajoskatona | I mean when first something has to be done in OVN | 14:11 |
ralonsoh | I'm pinging dalvarez and lucasagomes | 14:12 |
ralonsoh | if they can provide more info on this | 14:12 |
lajoskatona | +1. thanks | 14:12 |
ralonsoh | in any case, the description is done in https://bugs.launchpad.net/neutron/+bug/1875852 | 14:13 |
ralonsoh | there are two alternative imlpementations | 14:13 |
ralonsoh | the 2nd is the preferred | 14:14 |
ralonsoh | and implies supporting VLAN DVR traffic | 14:14 |
ralonsoh | (yes, core OVN support) | 14:14 |
ralonsoh | this, IMO, I will say +1 to this RFE **once** we can fully supported this ^^ in core OVN | 14:15 |
lajoskatona | I agree, let's wait for the OVN version which supports this | 14:16 |
lajoskatona | I add rfe-postponed tag for it in lp with comment about the discussion, what do you think ? | 14:17 |
mlavalle | sounds good to me | 14:17 |
ralonsoh | perfect | 14:17 |
slaweq | +1 | 14:17 |
lucasagomes | ralonsoh, reading | 14:18 |
lajoskatona | and 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 |
mlavalle | upstream dependency, I would say | 14:19 |
lajoskatona | mlavalle: perfect wording | 14:19 |
dalvarez | ralonsoh: 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 ToR | 14:20 |
mlavalle | I'm also reading http://dani.foroselectronica.es/ovn-external-ports-604/ as background info | 14:20 |
dalvarez | ralonsoh: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-September/050678.html | 14:21 |
ralonsoh | yeah, right | 14:21 |
dalvarez | we can route with sriov, not sure if that's a valid use case though. Also is it working on ml2ovs? | 14:21 |
dalvarez | i guess it is, traffic will go to the gw and that's it... we can achieve that perhaps with the ovn external port too | 14:22 |
ralonsoh | no, we don't route sriov traffic with OVS | 14:22 |
lajoskatona | ok, so we need more investigation if this is supported by OVN or not? | 14:25 |
ralonsoh | I think so or at least the problems present in the current implementation | 14:26 |
lajoskatona | ok, so anyway I ask in comment for more details if it is really supported now, and if those present we can discuss it again | 14:27 |
lajoskatona | is that ok for everybody? | 14:27 |
ralonsoh | +1 | 14:27 |
obondarev | +1 | 14:27 |
mlavalle | +1 | 14:27 |
yamamoto | +1 | 14:27 |
slaweq | +1 | 14:28 |
lajoskatona | ok ,thanks for the discussion | 14:30 |
lajoskatona | do you have anything which we can discuss today? | 14:30 |
mlavalle | not from me | 14:30 |
slaweq | nope | 14:30 |
ralonsoh | no | 14:30 |
lajoskatona | Just a reminder: | 14:30 |
lajoskatona | https://lists.openstack.org/pipermail/openstack-discuss/2022-July/029765.html | 14:30 |
slaweq | just wish You great holidays lajoskatona :) | 14:30 |
lajoskatona | I cancelled team and drivers meeting for the next 2 weeks | 14:30 |
ralonsoh | +1 to holidays | 14:31 |
lajoskatona | slaweq: thanks :-) | 14:31 |
mlavalle | yeah, have a restful and happy vacation | 14:31 |
lajoskatona | thanks you all | 14:31 |
lajoskatona | if nothing more we can close the meeting | 14:31 |
mlavalle | we will wreck the place in your absence | 14:31 |
slaweq | LOL | 14:31 |
lajoskatona | mlavalle: please do it, I have to do something when I am back | 14:31 |
lajoskatona | #endmeeting | 14:32 |
opendevmeet | Meeting ended Fri Jul 29 14:32:00 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.html | 14:32 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.txt | 14:32 |
opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-29-14.00.log.html | 14:32 |
lajoskatona | Bye | 14:32 |
ralonsoh | have a nice weekend | 14:32 |
slaweq | o/ | 14:32 |
obondarev | o/ | 14:32 |
slaweq | thx, You too ralonsoh | 14:32 |
yamamoto | o/ | 14:32 |
opendevreview | Bernard Cafarelli proposed openstack/neutron stable/train: [stable/train] Convert neutron-tempest-postgres-full to zuul v3 https://review.opendev.org/c/openstack/neutron/+/851512 | 14:35 |
opendevreview | liuxie proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 14:55 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/yoga: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851532 | 15:30 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/xena: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851533 | 15:31 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/wallaby: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851534 | 15:31 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/victoria: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851535 | 15:32 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/ussuri: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851536 | 15:32 |
*** iurygregory_ is now known as iurygregory | 15:57 | |
opendevreview | Miro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851537 | 16:01 |
opendevreview | Akihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/840088 | 16:13 |
*** amoralej is now known as amoralej|off | 16:21 | |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] [OVN][QoS] Add minimum bandwidth rule support to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/842292 | 16:29 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851537 | 16:30 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/840088 | 16:47 |
opendevreview | Akihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Update master for stable/xena https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/810418 | 16:47 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/846864 | 16:52 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Replace deprecated ugettext_lazy and ungettext_lazy https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/849257 | 16:54 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Replace deprecated django.conf.urls.url() https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/849258 | 16:55 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Update master for stable/xena https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/810418 | 17:13 |
opendevreview | Akihiro Motoki proposed openstack/neutron-vpnaas-dashboard master: Update master for stable/wallaby https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/782231 | 17:15 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/850834 | 17:16 |
opendevreview | Akihiro Motoki proposed openstack/neutron-vpnaas-dashboard stable/victoria: Drop lower-constraints (victoria) https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/851596 | 17:21 |
opendevreview | Merged openstack/neutron master: Make iptables rules diff to be case insensitive https://review.opendev.org/c/openstack/neutron/+/844589 | 17:37 |
opendevreview | Merged openstack/neutron-vpnaas-dashboard master: Update master for stable/wallaby https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/782231 | 17:39 |
opendevreview | Terry Wilson proposed openstack/ovsdbapp master: nb: add support for lb health checks API https://review.opendev.org/c/openstack/ovsdbapp/+/832835 | 18:17 |
opendevreview | Merged openstack/neutron stable/train: Optimize queries for port operations https://review.opendev.org/c/openstack/neutron/+/850735 | 18:24 |
opendevreview | Merged openstack/ovsdbapp master: nb: add methods to modify the lrp.networks https://review.opendev.org/c/openstack/ovsdbapp/+/832158 | 18:42 |
opendevreview | Merged openstack/ovsdbapp master: Provide base classes for {Get,Set}Options commands https://review.opendev.org/c/openstack/ovsdbapp/+/849569 | 19:17 |
opendevreview | Anthony proposed openstack/neutron-specs master: FWaaS Group Ordering Spec https://review.opendev.org/c/openstack/neutron-specs/+/851607 | 19:20 |
opendevreview | Merged openstack/ovsdbapp master: nb: add support for lb health checks API https://review.opendev.org/c/openstack/ovsdbapp/+/832835 | 20:14 |
*** dasm is now known as dasm|off | 20:46 | |
opendevreview | Merged openstack/neutron-specs master: Add spec for Network cascade deletion https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 21:41 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/train: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851537 | 22:53 |
opendevreview | Miro Tomaska proposed openstack/neutron stable/ussuri: Add workaround for eventlet.greendns bug https://review.opendev.org/c/openstack/neutron/+/851536 | 22:54 |
opendevreview | Miguel Lavalle proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!