| opendevreview | Miro Tomaska proposed openstack/neutron master: [DHCP agent] Add route to OVN metadata port if exists https://review.opendev.org/c/openstack/neutron/+/886988 | 01:19 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Add basical functionalities for metadata path extension https://review.opendev.org/c/openstack/neutron/+/881535 | 06:03 |
| opendevreview | liuyulong proposed openstack/neutron master: Add metadata path extension openflows https://review.opendev.org/c/openstack/neutron/+/888097 | 06:03 |
| opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 06:03 |
| *** ralonsoh_away is now known as ralonsoh | 07:09 | |
| lajoskatona | ralonsoh, slaweq, yatin: Hi, could please check this small CI fix for bagpipe when you have few free mins: https://review.opendev.org/c/openstack/networking-bagpipe/+/893709 ? | 07:29 |
| slaweq | lajoskatona +2 | 07:31 |
| ralonsoh | sure | 07:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894008 | 07:51 |
| lajoskatona | thanks | 07:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894009 | 07:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894010 | 07:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894011 | 07:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894012 | 07:58 |
| ralonsoh | folks ^^^ to unblock the CI | 07:58 |
| ralonsoh | ah folks, please check lajoskatona's patches in neutronclient: | 08:08 |
| ralonsoh | --> https://review.opendev.org/q/topic:bug/1999774+project:openstack/python-neutronclient+status:open | 08:08 |
| opendevreview | Merged openstack/neutron master: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/893962 | 08:11 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority https://review.opendev.org/c/openstack/neutron/+/893653 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities https://review.opendev.org/c/openstack/neutron/+/893654 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis https://review.opendev.org/c/openstack/neutron/+/893655 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue https://review.opendev.org/c/openstack/neutron/+/893656 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler https://review.opendev.org/c/openstack/neutron/+/893657 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis https://review.opendev.org/c/openstack/neutron/+/893658 | 08:25 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling https://review.opendev.org/c/openstack/neutron/+/893659 | 08:25 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Fix bindep.txt for python 3.11 job(Debian Bookworm) https://review.opendev.org/c/openstack/networking-bgpvpn/+/894024 | 08:50 |
| opendevreview | Merged openstack/networking-bagpipe master: CI: Change focal nodeset to jammy https://review.opendev.org/c/openstack/networking-bagpipe/+/893709 | 09:07 |
| opendevreview | Daniel Alvarez proposed openstack/neutron master: [WIP][ovn] Add support for IPv6 metadata https://review.opendev.org/c/openstack/neutron/+/894026 | 09:08 |
| opendevreview | Daniel Alvarez proposed openstack/neutron-tempest-plugin master: [ovn] Enable IPv6 metadata tests for OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/894027 | 09:12 |
| noonedeadpunk | hey folks! is it some known issue with iptables-restore on Yoga in neutron-openvswitch-agent (and lxb agent) | 09:19 |
| noonedeadpunk | on centos 8 stream | 09:20 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: DNM: test py311 job https://review.opendev.org/c/openstack/neutron-fwaas/+/894028 | 09:20 |
| noonedeadpunk | as it fails quite weirdly | 09:20 |
| ralonsoh | noonedeadpunk, yes, there is a bug open | 09:21 |
| ralonsoh | it is missing he iptables-restore alternative configuration | 09:21 |
| noonedeadpunk | aha, ok. Is that centos bug? | 09:21 |
| ralonsoh | no, a tripleo on | 09:21 |
| ralonsoh | one* | 09:21 |
| ralonsoh | this is a deployment issue | 09:22 |
| noonedeadpunk | huh, interesting.... As that used to work until some latest package update | 09:22 |
| noonedeadpunk | and we're not using tripleo | 09:22 |
| ralonsoh | in any case, this is part of the deployment configuration | 09:23 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2033683 | 09:23 |
| noonedeadpunk | yeah, already found that | 09:23 |
| ralonsoh | so you need to configure this binary and use update-alternatives | 09:24 |
| frickler | we did this in kolla fwiw https://review.opendev.org/c/openstack/kolla/+/893423 | 09:24 |
| noonedeadpunk | aha, ok, I bet that smth has changed in centos8 stream though.... | 09:24 |
| noonedeadpunk | yeah, already checking that | 09:24 |
| noonedeadpunk | so basically centos 8 no more links iptables to iptables/legacy? | 09:26 |
| ralonsoh | yes but you need to configure it | 09:26 |
| * noonedeadpunk loves centos stream | 09:26 | |
| noonedeadpunk | seems there's actually a bug in bz? https://bugzilla.redhat.com/show_bug.cgi?id=2236501 | 09:27 |
| noonedeadpunk | this matches with timeline when we've first noticed the failure | 09:28 |
| noonedeadpunk | And I see that kolla has pinned iptables back frickler, so I bet this is real fix of the issue I see: https://review.opendev.org/c/openstack/kolla/+/893364/2/docker/neutron/neutron-base/Dockerfile.j2 | 09:29 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/os-vif/+/894060 | 09:37 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/os-vif/+/894062 | 09:37 |
| opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/os-vif/+/894064 | 09:37 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894129 | 09:40 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894130 | 09:40 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894131 | 09:40 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/os-ken/+/894132 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/os-ken/+/894133 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/os-ken/+/894134 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/ovsdbapp/+/894135 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/ovsdbapp/+/894136 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/ovsdbapp/+/894137 | 09:41 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894138 | 09:42 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894139 | 09:42 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894140 | 09:42 |
| ralonsoh | lajoskatona, slaweq please check https://review.opendev.org/q/I2f8130dc3cf3244be2a44a4ecbdbaa9c7f865731 | 10:02 |
| ralonsoh | thanks in advance! | 10:02 |
| opendevreview | Fiorella Yanac proposed openstack/neutron-tempest-plugin master: Add username in resource_setup https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/893967 | 10:25 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: DNM: test py311 job https://review.opendev.org/c/openstack/neutron-fwaas/+/894028 | 11:21 |
| lajoskatona | ralonsoh: done | 11:29 |
| ralonsoh | thanks! | 11:33 |
| opendevreview | Merged openstack/python-neutronclient stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894138 | 11:41 |
| opendevreview | Merged openstack/neutron-lib stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894129 | 11:45 |
| opendevreview | Merged openstack/neutron-lib stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894130 | 11:45 |
| opendevreview | Merged openstack/os-ken stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/os-ken/+/894132 | 11:49 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Fix bindep.txt for python 3.11 job(Debian Bookworm) https://review.opendev.org/c/openstack/networking-bagpipe/+/893987 | 11:51 |
| opendevreview | Merged openstack/os-ken stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/os-ken/+/894133 | 11:55 |
| opendevreview | Merged openstack/neutron-lib master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/neutron-lib/+/894131 | 11:58 |
| opendevreview | Merged openstack/python-neutronclient master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894140 | 11:59 |
| opendevreview | Merged openstack/ovsdbapp stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/ovsdbapp/+/894135 | 12:01 |
| opendevreview | Merged openstack/ovsdbapp stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/ovsdbapp/+/894136 | 12:01 |
| opendevreview | Merged openstack/python-neutronclient stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/python-neutronclient/+/894139 | 12:03 |
| opendevreview | Fiorella Yanac proposed openstack/neutron-tempest-plugin master: Add username in resource_setup https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/893967 | 12:33 |
| opendevreview | Daniel Alvarez proposed openstack/neutron-tempest-plugin master: [ovn] Enable IPv6 metadata tests for OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/894027 | 13:00 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/893839 | 13:21 |
| opendevreview | Daniel Alvarez proposed openstack/neutron master: [WIP][ovn] Add support for IPv6 metadata https://review.opendev.org/c/openstack/neutron/+/894026 | 13:42 |
| opendevreview | Daniel Alvarez proposed openstack/neutron-tempest-plugin master: [ovn] Enable IPv6 metadata tests for OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/894027 | 13:53 |
| opendevreview | Merged openstack/neutron stable/2023.1: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894008 | 15:02 |
| opendevreview | Merged openstack/neutron stable/yoga: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894010 | 15:02 |
| opendevreview | Merged openstack/neutron stable/wallaby: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894012 | 15:33 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Cover the use case of a member non existing https://review.opendev.org/c/openstack/ovn-octavia-provider/+/893839 | 15:33 |
| opendevreview | Merged openstack/neutron stable/xena: Call the "tc qdisc" command for ingress qdisc without parent https://review.opendev.org/c/openstack/neutron/+/894011 | 15:34 |
| opendevreview | Ghanshyam proposed openstack/networking-bagpipe master: Fix bindep.txt for python 3.11 job(Debian Bookworm) https://review.opendev.org/c/openstack/networking-bagpipe/+/893987 | 15:56 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove ovn.ini example file https://review.opendev.org/c/openstack/neutron/+/894210 | 16:46 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: [sqlalchemy-20] Remove subtransactions=True https://review.opendev.org/c/openstack/networking-bagpipe/+/879463 | 18:37 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: SQLAlchemy: add context wrappers to SFC driver https://review.opendev.org/c/openstack/networking-bagpipe/+/891325 | 18:37 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: [sqlalchemy-20] Remove subtransactions=True https://review.opendev.org/c/openstack/networking-bagpipe/+/879463 | 18:38 |
| opendevreview | Daniel Alvarez proposed openstack/neutron master: [ovn] Add support for IPv6 metadata https://review.opendev.org/c/openstack/neutron/+/894026 | 20:38 |
| JayF | /win 26 | 20:46 |
| zigo | Hi. networking-baremetal doesn't work with the newer neutron-lib from bobcat. My package FTBFS (fail to build from source) with: | 21:16 |
| zigo | AttributeError: module 'neutron_lib.db.model_base' has no attribute 'HasProjectPrimaryKeyIndex' | 21:16 |
| zigo | Is there a patch available for that in master for that pb in networking-baremetal? | 21:18 |
| JayF | networking-baremetal is an Ironic deliverable | 21:18 |
| JayF | Did some digging on this with zigo in -ironic. Looks like a neutron bug. | 21:32 |
| JayF | https://github.com/openstack/neutron/blob/master/neutron/db/qos/models.py#L135 still exists even though https://github.com/openstack/neutron-lib/commit/673e48a1890c721654ce0de9cd9e0897c791bd6a is merged | 21:32 |
| JayF | details in https://bugs.launchpad.net/neutron/+bug/2034761 | 21:32 |
| JayF | I'm not sure I understand how that neutron-lib change is nonbreaking, tbh | 21:34 |
| JayF | it looks like it'd remove project_id from any models that were subclassing that; but I obviously only have pointed my brain this way for 15 mins | 21:35 |
| haleyb | JayF: i don't see any occurrence of HasProjectPrimaryKeyIndex anywhere in tree, or with codesearch | 21:37 |
| JayF | oh, that's HasProjectPrimaryKey, with no Index | 21:37 |
| JayF | hmm, so zigo is likely on the wrong version of neutron then | 21:38 |
| JayF | and I missed the distinction | 21:38 |
| JayF | zigo: please confirm you were testing 2023.2 neutron-lib with master neutron; otherwise that failure reported is not valid | 21:38 |
| haleyb | i do see neutron is still at neutron-lib 3.7.0 and not 3.8.0, don't know if that matters wrt neutron not being broken | 21:39 |
| JayF | oh, nice call | 21:39 |
| JayF | So I think this is probably just a bad bugreport, not any broken code | 21:39 |
| zigo | Well, my build in debian/experimental with neutron and neutron-lib from antelope ran the tests without issue. | 21:39 |
| JayF | it only got this far because I didn't read Index | 21:39 |
| zigo | My build in bookworm-backports with latest neutron-lib (but still older neutron) failed. | 21:40 |
| JayF | that's never really going to work | 21:40 |
| JayF | neutron and neutron-lib are tightly bound, for the most part | 21:40 |
| JayF | same goes for ironic and ironic-lib, fwiw | 21:40 |
| zigo | Right, but I haven't packaged yet neutron for bobcat (I'm waiting for the rc to pop ...). | 21:40 |
| JayF | (for Ironic, at least) the only reason the -lib exists is so other repos released alongside can share code | 21:40 |
| zigo | I can try packaging the latest neutron b3 release, but I'm kind of sure it wont fix networking-baremetal... | 21:41 |
| JayF | So I'm telling you, that test failure is invalid then :D it'll be fixed when you test against an updated neutron; and I'd suggest against testing neutron-lib (or ironic-lib or dependencies of it) until you get a new neutron release | 21:41 |
| zigo | ok, doing the b3 release to try then. | 21:42 |
| JayF | zigo: you are right that strictly speaking pypi released version of networking-baremetal is broken until neutron releases as well | 21:42 |
| haleyb | zigo: b3 will probably fix it, we've had to bump some other related projects to that lately for other things | 21:42 |
| JayF | zigo: just because we don't require a specific version of neutron/neutron-lib so you're getting newest of both (which are incompatible versions during this small window) | 21:42 |
| JayF | haleyb: ty for setting me straight, going to mark the bug invalid | 21:43 |
| JayF | https://bugs.launchpad.net/neutron/+bug/2034761/comments/3 /me done | 21:45 |
| haleyb | JayF: np. i did just check the neutron experimental jobs using neutron-lib master are passing, so things are working :) | 21:45 |
| JayF | Yeah this is awkward though | 21:45 |
| JayF | anything released that depends on neutron-lib is just ... broken for now | 21:46 |
| haleyb | yeah i guess short of limiting to an older neutron-lib, guess we didn't think of that | 21:48 |
| JayF | might be worth holding a change like that a cycle for deprecation in the future | 21:48 |
| JayF | but from an Ironic standpoint, I doubt anyone is running pypi-released networking-baremetal ... | 21:49 |
| haleyb | it was part of a large sqlalchemy-20 cleanup, and we believed codesearch | 21:50 |
| JayF | haleyb: https://github.com/openstack/networking-baremetal/blob/master/requirements.txt#L20 yeah we dep on both -lib and neutron ... I have no idea how unique that is, but for anyone in this setup you'd need neutron-lib to keep the removed code through a neutron release to prevent this kinda break in the future | 21:52 |
| JayF | because Ironic can't (by policy and also it's a bad idea) version lock neutron{,-lib} and there's no way to express to pip "these two packages must have identical versions" | 21:53 |
| haleyb | JayF: makes sense and we'll be more careful in the future. networking-bgpvpn does that, but we bumped to use b3 | 21:54 |
| JayF | ack; I think we'd probably be OK to take patches like that too in the future | 21:54 |
| JayF | gonna be honest, always thought it was weird that networking-baremetal was in Ironic and not in Neutron; but I guess we both have the ability to break it and we probably care more :D | 21:55 |
| JayF | (that might have come off wrong; I just mean more: we know and care more about the real world, you all know and care more about the spooky virtual world) | 21:55 |
| haleyb | and we did the same in neutron-dynamic-routing - we don't look at the networking-baremetal gate afaik | 21:55 |
| JayF | might be worth thinking about having a simple unit test job running cross on both sides | 21:56 |
| JayF | which would've caught this, I don't think we even need tempest just unit tests | 21:56 |
| JayF | if you think you all would be amenable to that, I can add it to my list | 21:57 |
| haleyb | yes we are mostly virtual... and where would the job live? i'm assuming in -baremetal? | 21:59 |
| JayF | I was thinking more -lib because you are more likely to break us than the other way around (and non-voting woudl be fine) | 22:00 |
| JayF | the Ironic job being configured to use master/master of neutron+neutron-lib is why this doesn't happen in the gate, fwiw | 22:00 |
| JayF | do you all have a special job that sets up the old/new versions to find this? | 22:01 |
| * zigo needs one of these gamers laptop with 16 amd cores to build Neutron faster... | 22:02 | |
| haleyb | JayF: our experimental job is at https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L1215 but it also uses master/master | 22:07 |
| JayF | hmm | 22:07 |
| haleyb | wonder if we should have an experimental queue in -lib for such things, and can run before releasing | 22:07 |
| JayF | I'm going to stick a pin in this for now | 22:07 |
| JayF | I have enough this-cycle stuff and this is next-cycle | 22:08 |
| JayF | but I'll put it on the Ironic PTG etherpad so it doesn't get missed | 22:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!