| cardoe | When is the driver meeting? | 00:42 |
|---|---|---|
| opendevreview | Arnau Verdaguer proposed x/whitebox-neutron-tempest-plugin master: Modify location of nb/sb db unix socket https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985684 | 07:41 |
| opendevreview | Arnau Verdaguer proposed x/whitebox-neutron-tempest-plugin master: Modify location of nb/sb db unix socket https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985684 | 08:56 |
| opendevreview | Arnau Verdaguer proposed x/whitebox-neutron-tempest-plugin master: Modify location of nb/sb db unix socket https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985684 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: typing: Add initial type annotations https://review.opendev.org/c/openstack/os-vif/+/745162 | 11:06 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: typing: Add typing to vif_plug_noop module https://review.opendev.org/c/openstack/os-vif/+/976362 | 11:06 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: WIP: typing: Add typing to vif_plug_ovs module https://review.opendev.org/c/openstack/os-vif/+/976363 | 11:06 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 11:52 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 11:57 |
| haleyb | cardoe: driver meeting canceled as we had ptg | 13:14 |
| cardoe | haleyb: okay. When is it regularly? I'll set a reminder on my calendar. I just saw a note on the EVPN type-5 spec about discussing vxlan-evpn and more vni pools in the Friday meeting so I assumed it was the driver meeting. | 13:15 |
| haleyb | https://meetings.opendev.org/#Neutron_drivers_Meeting - but be aware it moves when dst moves | 13:16 |
| cardoe | So it'd go on now or in 45 minutes usually? | 13:18 |
| haleyb | today it would have been 20 minutes ago | 13:20 |
| cardoe | Thanks | 13:40 |
| haleyb | so i have a general question about our CI - i'm looking at a neutron change in stable/2026.1 that's running neutron-tempest-plugin-openvswitch-2026-1, but failing one test. I would have assumed we'd have used n-t-p tag 3.1.0 for this, but it shows 3.1.1dev4. Same on other stable branches. What am I missing? | 14:31 |
| haleyb | so it's basically using a breaking change | 14:32 |
| ykarel_ | haleyb, we use master ntp even for stable branches, similar to tempest | 14:33 |
| haleyb | ykarel_: sigh, so what are the tags for? | 14:33 |
| ykarel_ | we do use them like when we transition to unmaintained release atleast | 14:34 |
| haleyb | ykarel_: ack, that makes sense... seems i broke our stable branches then | 14:35 |
| ykarel_ | basically we try to keep ntp supporting both master and atleast some stable branches | 14:35 |
| ykarel_ | haleyb, with which patch? | 14:35 |
| *** ykarel_ is now known as ykarel | 14:35 | |
| haleyb | ykarel_: https://review.opendev.org/c/openstack/neutron/+/985850 but it affects anything running an *openvswitch* test | 14:36 |
| frickler | haleyb: do you know which n-t-p patch caused this? | 14:38 |
| haleyb | ykarel: so i have another question since you're around - in this change i tried to address the issue https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984291 but now i realize it did not run an openvswitch tests except the master one, but it ran all the ovn ones. i might need help tracking down why as it seemed it should have | 14:38 |
| haleyb | frickler: it was a combo of a neutron-lib patch and ^^ - https://review.opendev.org/c/openstack/neutron-lib/+/982598 - i should maybe just revert things it's a little messy | 14:39 |
| haleyb | and https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/978603 | 14:40 |
| haleyb | i really made a mess of things | 14:40 |
| ykarel | haleyb, but in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984291 i see ovs jobs ran | 14:41 |
| ykarel | neutron-tempest-plugin-openvswitch-2026-1 https://zuul.opendev.org/t/openstack/build/3b7262ff59b848a18e2629db7831b177 : SUCCESS in 1h 36m 21s | 14:42 |
| ykarel | neutron-tempest-plugin-openvswitch-iptables_hybrid-2026-1 https://zuul.opendev.org/t/openstack/build/bdc95f5bb55c46b192ee0147a3412717 : SUCCESS in 1h 48m 53s | 14:42 |
| haleyb | 💩 | 14:42 |
| frickler | those jobs ran in check, but not in gate. so probably a question of pipeline setup? | 14:42 |
| haleyb | it's friday, right? i was looking at gate since it was the last reported | 14:43 |
| haleyb | doesn't explain the stable/2026.1 etc failures, maybe i just need to debug how to workaround that | 14:44 |
| frickler | but also these passed on all changes afaict, so if you really broke stuff, the question becomes: why did these jobs not fail | 14:44 |
| haleyb | they passed in n-t-p but not in the neutron stable branches, i maybe just need to do some debugging with a dependency | 14:46 |
| haleyb | thanks for helping me to keep my sanity, maybe things aren't as bad as they seem | 14:48 |
| haleyb | well, except for stable branches being broken | 14:48 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2026.1: DNM: Testing 2026.1 CIo https://review.opendev.org/c/openstack/neutron/+/986107 | 14:50 |
| ykarel | from what i see issue started after https://review.opendev.org/c/openstack/neutron-lib/+/982598 got released and updated in upper-constraints | 14:50 |
| haleyb | ykarel: yup, that was where i changed a constant | 14:51 |
| haleyb | i guess it's getting pulled into the stable testing? | 14:51 |
| ykarel | yes in 2026.1 jobs we using master constraints for tempest and ntp setup | 14:52 |
| ykarel | so that's being pulled | 14:52 |
| ykarel | https://github.com/openstack/devstack/blob/stable/2026.1/stackrc#L308 | 14:53 |
| haleyb | right, so i would have thought my n-t-p workaround would help with any mis-match but it's not working | 14:54 |
| ykarel | yes seems so you need to handle both case in that test | 14:58 |
| haleyb | ykarel: i'll keep hacking, i thought i had even if i marked it temporary | 15:00 |
| ykarel | also not just 2026.1 but all stable branches impacted | 15:01 |
| haleyb | ykarel: yeah, i broke them all :( | 15:01 |
| ykarel | haleyb, instead of failing can't we just handle tenant in except IndexError? | 15:02 |
| haleyb | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984291/8/neutron_tempest_plugin/api/admin/test_routers_ha.py - thought i did that there, it worked on master branch at least | 15:03 |
| haleyb | or my check just might be wrong | 15:04 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2026.1: DNM: Testing 2026.1 CI https://review.opendev.org/c/openstack/neutron/+/986107 | 15:08 |
| ykarel | yes it worked because in tempest venv released neutron lib was used not the patch one | 15:09 |
| opendevreview | Arnau Verdaguer proposed x/whitebox-neutron-tempest-plugin master: Modify location of nb/sb db unix socket https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985684 | 15:09 |
| ykarel | haleyb, commented https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984291/8#message-72c5b8140ec5fd98f04a39684eb19c8a779d5e94 | 15:15 |
| ykarel | that way should work, wdyt? | 15:16 |
| opendevreview | Arnau Verdaguer proposed x/whitebox-neutron-tempest-plugin master: Modify location of nb/sb db unix socket https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985684 | 15:18 |
| haleyb | ykarel: yeah, let me try that | 15:18 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 15:22 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Fix router HA test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/986116 | 15:26 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2026.1: DNM: Testing 2026.1 CI https://review.opendev.org/c/openstack/neutron/+/986107 | 15:26 |
| haleyb | ykarel: i should have put a suggested-by: yatin there, thanks for looking i was going crazy | 15:36 |
| opendevreview | Helen Chen proposed openstack/neutron-specs master: Propose spec for BGP EVPN Type-5 Route Support https://review.opendev.org/c/openstack/neutron-specs/+/982256 | 15:44 |
| ichen | I updated the EVPN spec, hopefully for the last time. The new update proposes to put the EVPN changes in a new EVPN Service Plugin and a new OVN Agent EVPN Extension, instead of extension the existing BGP Service Plugin and OVN Agent BGP Extension. This change doesn’t really affect the design, since these two features largely co-exist in parallel as already documented in the spec. | 15:47 |
| ichen | https://review.opendev.org/c/openstack/neutron-specs/+/982256 | 15:47 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove NEUTRON_DEPLOY_MOD_WSGI from zuul jobs https://review.opendev.org/c/openstack/neutron/+/986214 | 20:09 |
| opendevreview | Brian Haley proposed openstack/neutron master: Start raising exception with invalid port binding attrs https://review.opendev.org/c/openstack/neutron/+/986217 | 20:15 |
| opendevreview | Merged openstack/neutron stable/2025.2: Fix unbounded thread pool in OVN MaintenanceThread https://review.opendev.org/c/openstack/neutron/+/985816 | 20:15 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Use project_id key in api and scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/982736 | 22:10 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Fix router HA test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/986116 | 22:19 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Use project_id key in api and scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/982736 | 22:19 |
| opendevreview | Merged openstack/neutron stable/2025.1: Fix unbounded thread pool in OVN MaintenanceThread https://review.opendev.org/c/openstack/neutron/+/985817 | 22:37 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!