| opendevreview | liuyulong proposed openstack/neutron master: Add basical functionalities for metadata path extension https://review.opendev.org/c/openstack/neutron/+/881535 | 01:11 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Add metadata path extension openflows https://review.opendev.org/c/openstack/neutron/+/888097 | 01:11 |
| opendevreview | liuyulong proposed openstack/neutron master: Fullstack case for metadata path https://review.opendev.org/c/openstack/neutron/+/888098 | 01:11 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: OSC: Remove SFC V2 calls to neutronclient https://review.opendev.org/c/openstack/python-neutronclient/+/887388 | 07:34 |
| opendevreview | yatin proposed openstack/neutron master: [UT] Adjust autogen_process_directives for alembic-1.12.0+ https://review.opendev.org/c/openstack/neutron/+/893602 | 08:07 |
| ykarel | ralonsoh, slaweq lajoskatona gate failure fix ^, please check when get a chance | 08:07 |
| ykarel | first side effect for those job in check queue | 08:08 |
| slaweq | ykarel done :) | 08:12 |
| lajoskatona | ykarel: checking | 08:19 |
| opendevreview | yatin proposed openstack/neutron master: [UT] Adjust autogen_process_directives for alembic-1.12.0+ https://review.opendev.org/c/openstack/neutron/+/893603 | 08:23 |
| ralonsoh | ykarel, sure, let me check | 08:40 |
| ralonsoh | ykarel, can you add the [1] link please? | 08:41 |
| ykarel | ralonsoh, added in latest PS | 08:41 |
| ykarel | :( | 08:42 |
| ralonsoh | thanks! | 08:42 |
| ykarel | it created different review, fixing | 08:42 |
| ralonsoh | hehehehe | 08:42 |
| opendevreview | yatin proposed openstack/neutron master: [UT] Adjust autogen_process_directives for alembic-1.12.0+ https://review.opendev.org/c/openstack/neutron/+/893602 | 08:42 |
| ykarel | done ^ | 08:42 |
| opendevreview | Merged openstack/neutron master: [UT] Adjust autogen_process_directives for alembic-1.12.0+ https://review.opendev.org/c/openstack/neutron/+/893602 | 09:56 |
| mnaser | hi neutron team , appreciate reviews on https://review.opendev.org/q/I34caf7d0212ccb4bd7259c4414e7c3994bd8da4d — this is hitting hard on a cloud that’s running a newer ovn | 13:41 |
| ralonsoh | sure, let me chck | 13:42 |
| ralonsoh | check* | 13:42 |
| slaweq | ralonsoh hi, can You check https://review.opendev.org/c/openstack/neutron/+/890683 when You will have time? | 13:54 |
| ralonsoh | sure | 13:55 |
| ralonsoh | slaweq, sorry, I can't | 13:55 |
| slaweq | why? | 13:55 |
| ralonsoh | this is a new feature and we are in feature freeze now | 13:55 |
| slaweq | ahh, right | 13:56 |
| ralonsoh | but, for sure, that will land in C asap | 13:56 |
| slaweq | ok, lets wait for stable/2023.2 to be created first and then move on with this | 13:56 |
| slaweq | ++ | 13:56 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority https://review.opendev.org/c/openstack/neutron/+/893653 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities https://review.opendev.org/c/openstack/neutron/+/893654 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis https://review.opendev.org/c/openstack/neutron/+/893655 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue https://review.opendev.org/c/openstack/neutron/+/893656 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler https://review.opendev.org/c/openstack/neutron/+/893657 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis https://review.opendev.org/c/openstack/neutron/+/893658 | 14:07 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling https://review.opendev.org/c/openstack/neutron/+/893659 | 14:07 |
| felixhuettner[m] | fyi: all things i just pushed are related to https://bugs.launchpad.net/neutron/+bug/2023993 | 14:15 |
| frickler | mnaser: just being curious, is this a bug fix or just a performance thing? if bug fix, why is there no bug report? | 14:16 |
| frickler | doing stable backports without a bug reference is at least questionable practice | 14:19 |
| ralonsoh | frickler, that was my fault for not creating initially a LP bug | 14:19 |
| ralonsoh | we can create it now | 14:19 |
| ralonsoh | but this is indeed an issue with newer OVN releases | 14:20 |
| ralonsoh | felixhuettner[m], there is another patch right now modifying the L3 schedulers | 14:21 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/892604 | 14:21 |
| frickler | ralonsoh: ok, then I should likely need to apply this downstream, too, we're running with 23.06 on zed currently | 14:21 |
| ralonsoh | frickler, most probably | 14:21 |
| ralonsoh | felixhuettner[m], I will review your patches, for sure, but please consider that there would be merge conflicts between your patches and this one | 14:22 |
| felixhuettner[m] | ralonsoh: that looks interesting. While i don't think it fixes the bug i link it will definately conflict heavily with the fixes i built | 14:23 |
| felixhuettner[m] | allthough i guess first taking the newer schedulers and then rebasing mine on top of them is easier than the other way round | 14:25 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: [sqlalchemy-20] Add context wrapper to ppg operations https://review.opendev.org/c/openstack/networking-sfc/+/893660 | 14:31 |
| lajoskatona | mnaser: done, waiting for the patch on zed's zuul results | 14:32 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: [sqlalchemy-20] Remove subtransactions=True https://review.opendev.org/c/openstack/networking-bagpipe/+/879463 | 14:35 |
| mnaser | frickler: yeah sorry I just backported the initial patch | 14:40 |
| mnaser | I believe after nailing it down that was the root cause I was seeing for our issues, this was in a trunk heavy cloud so maybe it pops up more there | 14:40 |
| mnaser | lajoskatona, ralonsoh: thank you for reviews. I will ask my team to track it through gate :) | 14:43 |
| ralonsoh | yw | 14:43 |
| opendevreview | Stephen Finucane proposed openstack/os-ken master: Remove unused requirements file https://review.opendev.org/c/openstack/os-ken/+/893663 | 14:52 |
| opendevreview | Stephen Finucane proposed openstack/os-ken master: Remove duplicate requirements files https://review.opendev.org/c/openstack/os-ken/+/893664 | 14:52 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority https://review.opendev.org/c/openstack/neutron/+/893653 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities https://review.opendev.org/c/openstack/neutron/+/893654 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis https://review.opendev.org/c/openstack/neutron/+/893655 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue https://review.opendev.org/c/openstack/neutron/+/893656 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler https://review.opendev.org/c/openstack/neutron/+/893657 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis https://review.opendev.org/c/openstack/neutron/+/893658 | 15:21 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling https://review.opendev.org/c/openstack/neutron/+/893659 | 15:21 |
| opendevreview | Merged openstack/os-ken master: Remove unused requirements file https://review.opendev.org/c/openstack/os-ken/+/893663 | 15:41 |
| *** whoami-r- is now known as whoami-rajat | 15:54 | |
| opendevreview | Merged openstack/neutron stable/2023.1: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter https://review.opendev.org/c/openstack/neutron/+/893542 | 16:36 |
| opendevreview | Merged openstack/os-ken master: Remove duplicate requirements files https://review.opendev.org/c/openstack/os-ken/+/893664 | 16:39 |
| mnaser | lajoskatona, ralonsoh: https://review.opendev.org/c/openstack/neutron/+/893543 is ready for the +W if you're around | 16:47 |
| lajoskatona | mnaser: ack, frickler's comment about bug report is really good, but we can comment in worst case on the patches with the bug report anytime | 17:06 |
| opendevreview | Merged openstack/neutron stable/zed: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter https://review.opendev.org/c/openstack/neutron/+/893543 | 19:01 |
| -opendevstatus- NOTICE: Some Gerrit changes that update Zuul configuration may fail with no response from Zuul. A fix is in progress. | 20:06 | |
| *** damian___921836 is now known as damian___92183 | 20:13 | |
| -opendevstatus- NOTICE: Gerrit changes with updates to Zuul's configuration should now be handled correctly. Recheck any changes to Zuul configuration which did not report results. | 22:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!