| opendevreview | Miguel Lavalle proposed openstack/neutron master: User defined router flavor driver with no LSP https://review.opendev.org/c/openstack/neutron/+/917800 | 00:19 |
|---|---|---|
| opendevreview | Ihar Hrachyshka proposed openstack/neutron unmaintained/wallaby: Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/927547 | 01:23 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.1: Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/927560 | 02:19 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: User defined router flavor driver with no LSP https://review.opendev.org/c/openstack/neutron/+/917800 | 03:38 |
| opendevreview | yatin proposed openstack/neutron-fwaas stable/2023.1: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/927076 | 07:54 |
| ykarel | lajoskatona, when you get chance please check https://review.opendev.org/c/openstack/neutron-fwaas/+/927463 | 07:55 |
| lajoskatona | ykaerl: sure | 07:55 |
| ykarel | lajoskatona, and also https://review.opendev.org/q/Ib2e412ed5e5d4c4fcb4f0abe71d4750fb78bce27 | 07:55 |
| opendevreview | David Pineau proposed openstack/neutron-specs master: Add spec to allow attaching SecurityGroups to Networks - RFE #2075955 https://review.opendev.org/c/openstack/neutron-specs/+/927520 | 08:05 |
| liushy | hi folks, please feel free to review ^-^ : https://review.opendev.org/c/openstack/neutron/+/925968 | 08:11 |
| opendevreview | Lajos Katona proposed openstack/neutron-specs master: Doc: remove sphinxcontrib-*diag from doc dependencies https://review.opendev.org/c/openstack/neutron-specs/+/927573 | 08:25 |
| opendevreview | Merged openstack/neutron-fwaas stable/2024.1: [stable-only] Switched to branched tempest plugin job https://review.opendev.org/c/openstack/neutron-fwaas/+/927463 | 09:09 |
| opendevreview | Tobias Urdin proposed openstack/os-ken master: Python3.12: do not use ssl.wrap_socket https://review.opendev.org/c/openstack/os-ken/+/927576 | 09:14 |
| opendevreview | Merged openstack/neutron-fwaas unmaintained/zed: Stable/2023.1 only: Fix failing jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/927509 | 09:19 |
| lajoskatona | liushy: I set back reiew-priority on your patch | 09:42 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: WIP Add trusted vif api extension for the port https://review.opendev.org/c/openstack/neutron/+/926068 | 09:46 |
| ralonsoh | haleyb|out, hi! if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/927236 | 09:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Fix port_hardware_offload_type ML2 extension https://review.opendev.org/c/openstack/neutron/+/927577 | 09:54 |
| liushy | lajoskatona, thx! | 09:54 |
| opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 09:59 |
| opendevreview | Merged openstack/neutron-fwaas stable/2024.1: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/927074 | 10:03 |
| joa_ | Hello ! As discussed during the meeting last week, I've pushed a spec for our securitygroup/network binding proposal. I've added the people showing an interest in it during the last drivers meeting as reviewers on the proposal: https://review.opendev.org/c/openstack/neutron-specs/+/927520 | 10:03 |
| *** joa_ is now known as joa | 10:03 | |
| opendevreview | Merged openstack/neutron master: Ensure latest revision number after commit port binding https://review.opendev.org/c/openstack/neutron/+/925968 | 11:41 |
| opendevreview | Merged openstack/neutron master: Add ovs-master job with neutron-plugin enabled https://review.opendev.org/c/openstack/neutron/+/927219 | 11:59 |
| opendevreview | Merged openstack/neutron master: Use ovs from main for -ovs-master jobs https://review.opendev.org/c/openstack/neutron/+/927502 | 12:07 |
| opendevreview | Merged openstack/neutron-fwaas stable/2023.1: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/927076 | 12:38 |
| opendevreview | Merged openstack/neutron-fwaas stable/2023.2: create default security group using an independent context https://review.opendev.org/c/openstack/neutron-fwaas/+/927075 | 12:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add trusted vif api extension for the port https://review.opendev.org/c/openstack/neutron/+/926068 | 13:04 |
| *** haleyb|out is now known as haleyb | 13:07 | |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.1: Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/927560 | 13:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib master: Add port trusted vif extension https://review.opendev.org/c/openstack/neutron-lib/+/923860 | 13:16 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron unmaintained/wallaby: Support nested SNAT for ml2/ovn https://review.opendev.org/c/openstack/neutron/+/927547 | 13:23 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: remove sphinxcontrib-*diag from doc dependencies https://review.opendev.org/c/openstack/tap-as-a-service/+/927584 | 13:25 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add trusted vif api extension for the port https://review.opendev.org/c/openstack/neutron/+/926068 | 13:26 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add trusted vif api extension for the port https://review.opendev.org/c/openstack/neutron/+/926068 | 13:28 |
| slaweq | haleyb hi, is drivers meeting today or are You going to cancel it maybe? | 13:30 |
| lajoskatona | haleyb, slaweq: Hi, today I can't join anyway :-( | 13:36 |
| haleyb | i guess we should cancel then. The only discussion would have been a follow-up for https://bugs.launchpad.net/neutron/+bug/2075955 - we discussed last week and asked for more info and i see a spec now. it was not approved since we didn't have quorum | 13:39 |
| ralonsoh | I think we can take a look at the spec, comment on the patch and leave to the next week the decision of accepting the RFE or not | 13:45 |
| ralonsoh | we'll have more info reviewing the patch | 13:45 |
| mlavalle | haleyb: so will there be meeting today? | 13:50 |
| haleyb | mlavalle: no, lets cancel and ask questions in the spec. can meet next friday | 13:50 |
| mlavalle | ack. Thanks | 13:50 |
| mlavalle | haleyb, slaweq, lajoskatona, ralonsoh: https://review.opendev.org/c/openstack/neutron/+/917800 is ready for the next round of reviews. Thanks! | 13:52 |
| slaweq | good, I will then use that time to do some reviews :) | 13:53 |
| ralonsoh | sure | 13:53 |
| slaweq | and speaking of which, please take a look at https://review.opendev.org/c/openstack/neutron/+/926068 when You will have time, I know it is very late in the cycle but still one week before feature freeze so maybe we could get this in | 13:54 |
| slaweq | thx in advance | 13:54 |
| ralonsoh | sure | 13:54 |
| ralonsoh | slaweq, but we can't | 13:54 |
| ralonsoh | there is a n-lib patch related | 13:54 |
| ralonsoh | and we no longer can merge n-lib patches | 13:55 |
| slaweq | but I added api extension definition to neutron too | 13:55 |
| slaweq | with comment to remove it once n-lib patch will be merged and released | 13:55 |
| slaweq | so we can possibly use definition from neutron now and "rehome" it in next cycle | 13:55 |
| ralonsoh | but then, same as in https://review.opendev.org/c/openstack/neutron/+/926725 and the related n-lib patch | 13:55 |
| ralonsoh | we'll have something in Neutron that is not documented in the n-lib API | 13:56 |
| slaweq | n-lib API ref is always rendered from master branch so it still can be available on the web page before 2024.2 release will be out | 13:56 |
| ralonsoh | but the latest n-lib version of 2024.2 has been already released | 13:57 |
| slaweq | because we can merge it as soon as we will cut stable/2024.2 branch in neutron-lib repo, which I think should be next week | 13:57 |
| ralonsoh | at this point, I don't know how it works, to be honest | 13:58 |
| ralonsoh | we should not merge anything in n-lib until the stable branch creation | 13:58 |
| ralonsoh | this is what I was told | 13:59 |
| *** obondarev_ is now known as obondarev | 14:01 | |
| ralonsoh | mlavalle, https://review.opendev.org/c/openstack/neutron/+/917800/comment/9d32889d_b22934ee/ | 14:07 |
| *** bpetermann is now known as Guest1902 | 14:11 | |
| mlavalle | ralonsoh: thanks! I'll submit a revision in a few minutes | 14:15 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Introduce ovs-lts jobs that run against the latest ovn LTS branch https://review.opendev.org/c/openstack/neutron/+/927221 | 14:26 |
| seba | ralonsoh, would it make sense to have another go at https://review.opendev.org/c/openstack/neutron/+/919590 before the feature freeze, as the neutron-lib change for that has already been merged? | 14:33 |
| ralonsoh | seba, sure, I'll check it | 14:34 |
| seba | tnx | 14:34 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Introduce ovs-lts jobs that run against the latest ovn LTS branch https://review.opendev.org/c/openstack/neutron/+/927221 | 14:36 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: docs: fix neutron-ovn-tempest-with-neutron-lib-master description https://review.opendev.org/c/openstack/neutron/+/927593 | 14:36 |
| opendevreview | Roberto Acosta proposed openstack/neutron master: [ML2/OVN] Add external_ids.neutron:is_snat key for SNAT rules https://review.opendev.org/c/openstack/neutron/+/907915 | 14:39 |
| opendevreview | Merged openstack/neutron master: Monkey patch the system libraries before calling them https://review.opendev.org/c/openstack/neutron/+/925376 | 14:39 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.1: DNM test nested_snat scenario with 2023.1 feature backport https://review.opendev.org/c/openstack/neutron/+/927597 | 14:49 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: User defined router flavor driver with no LSP https://review.opendev.org/c/openstack/neutron/+/917800 | 14:59 |
| mlavalle | ralonsoh: https://review.opendev.org/c/openstack/neutron/+/917800. for your revies. Thanks | 15:07 |
| ralonsoh | let me check | 15:10 |
| mlavalle | also fixed the unit tests | 15:11 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: User defined router flavor driver with no LSP https://review.opendev.org/c/openstack/neutron/+/917800 | 15:16 |
| mlavalle | ralonsoh: I just pushed another revision. I only updated 1 comment to reflect that now we check the existence of the LSP and not the revision number | 15:18 |
| ralonsoh | ok | 15:18 |
| ralonsoh | slaweq, commented on https://review.opendev.org/c/openstack/neutron/+/926068 | 15:18 |
| mlavalle | yeah, I'd rather pass on that one, unless there is a strong feeling it must be rafctored. He indicated it was a tiny nitt | 15:19 |
| *** ykarel__ is now known as ykarel | 15:25 | |
| slaweq | ralonsoh thx a lot | 15:29 |
| slaweq | I will address it as first thing on Monday morning | 15:29 |
| mlavalle | slaweq: thanks for the review. have a nice weekend | 15:35 |
| mlavalle | it's a long one for the US crowd | 15:36 |
| slaweq | thx mlavalle, you too :) | 15:36 |
| slaweq | so have an even better long weekend then :) | 15:36 |
| mlavalle | ykarel: the USA delegation won't make it to this coming Monday's CI meeting. It is Labor Day here \o/ | 15:37 |
| opendevreview | Merged openstack/neutron master: [OVN][FT] Ensure the non OVN agent is correctly created https://review.opendev.org/c/openstack/neutron/+/927236 | 15:49 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-tempest-plugin master: DNM: enable nested_snat scenario for ovn https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/927225 | 16:56 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-tempest-plugin master: Use ovs and ovn from packages https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/927626 | 17:03 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-tempest-plugin master: Use ovs and ovn from packages https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/927626 | 17:04 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: WIP create snat rules for every ipv4 subnet, not just the first one https://review.opendev.org/c/openstack/neutron/+/927631 | 17:44 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: Clean up code to handle old versions with no stateless acl support https://review.opendev.org/c/openstack/neutron/+/927634 | 17:58 |
| opendevreview | Merged openstack/neutron master: Fix port_hardware_offload_type ML2 extension https://review.opendev.org/c/openstack/neutron/+/927577 | 19:05 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-designate-scenario non voting https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/927648 | 20:02 |
| gmann | slaweq: haleyb ^^ I am still not figured out the root cause for designate job failure. we need to merge the requirement change for oslo.policy 4.4.0 and to avoid breaking the designate gate I am making it non voting | 20:04 |
| haleyb | gmann: ack, guess that's the best thing until it's figured out | 20:11 |
| gmann | haleyb: yeah, I will continue working on debugging | 20:12 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: Clean up code to handle old versions with no stateless acl support https://review.opendev.org/c/openstack/neutron/+/927634 | 20:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!