| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Add 2025.1 (Epoxy) stable jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/944954 | 00:38 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron stable/2025.1: [2025.1 only] Switch to 2025.1 neutron-tempest-plugin jobs https://review.opendev.org/c/openstack/neutron/+/944955 | 00:44 |
| opendevreview | Merged openstack/neutron master: Remove workaround for old python versions https://review.opendev.org/c/openstack/neutron/+/943679 | 01:54 |
| opendevreview | Merged openstack/neutron master: Add missing Neutron milestones https://review.opendev.org/c/openstack/neutron/+/944803 | 03:02 |
| opendevreview | Merged openstack/neutron master: Open the 2025.2 (Flamingo) DB branch https://review.opendev.org/c/openstack/neutron/+/944804 | 03:02 |
| opendevreview | Merged openstack/neutron master: Revert "[eventlet-removal] Remove eventlet from DHCP agent" https://review.opendev.org/c/openstack/neutron/+/944826 | 03:02 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Exclude VlanQinq tests in ovn ubuntu jammy job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/944965 | 06:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Make the Logical_Switch_Port UP event more robust https://review.opendev.org/c/openstack/neutron/+/943404 | 06:52 |
| ralonsoh | ykarel, ^ thanks for finding the issue in Jammy ^^ | 07:02 |
| ykarel | ralonsoh, yw. I was already suspecting that ovn bug but just didn't know those new tests triggered it until digged further | 07:08 |
| opendevreview | Merged openstack/neutron master: [OVN] Isolate ``test_ovsdb_monitor.TestNBDbMonitor*`` FTs https://review.opendev.org/c/openstack/neutron/+/944858 | 07:56 |
| opendevreview | Merged openstack/neutron master: [OVN][FT] Use MySQL backend for ``TestNBDbMonitor*`` classes https://review.opendev.org/c/openstack/neutron/+/944860 | 07:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: [OVN] Isolate ``test_ovsdb_monitor.TestNBDbMonitor*`` FTs https://review.opendev.org/c/openstack/neutron/+/944968 | 07:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: [OVN][FT] Use MySQL backend for ``TestNBDbMonitor*`` classes https://review.opendev.org/c/openstack/neutron/+/944969 | 07:58 |
| ralonsoh | bcafarel, hello! I have a series of catastrophic errors along different releases... | 08:00 |
| ralonsoh | https://review.opendev.org/q/topic:%22neutron_milestones%22 | 08:00 |
| ralonsoh | if you have a couple of mins | 08:00 |
| bcafarel | ralonsoh: sure thing! let me take a look (no meetings for a bit I can check these) | 08:01 |
| bcafarel | ralonsoh: done, do you think it's worth adding a step in https://docs.openstack.org/neutron/latest/contributor/policies/release-checklist.html#major-release ? | 08:13 |
| bcafarel | we mention CURRENT_RELEASE in neutron.db.migration.cli but not __init__ NEUTRON_MILESTONES | 08:13 |
| ralonsoh | bcafarel, I opened a bug yesterday about this. The doc change should be included too, along with the code tests | 08:15 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2103531 | 08:15 |
| bcafarel | nice :) | 08:18 |
| opendevreview | Merged openstack/neutron stable/2025.1: [stable-only] Fix the Neutron milestones list https://review.opendev.org/c/openstack/neutron/+/944880 | 08:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [doc] Update the release doc to create a new database branch https://review.opendev.org/c/openstack/neutron/+/944971 | 09:14 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: Revert "[eventlet-removal] Remove the usage of eventlet in the DHCP agent" https://review.opendev.org/c/openstack/neutron/+/944972 | 09:24 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2025.1: Revert "[eventlet-removal] Remove eventlet from DHCP agent" https://review.opendev.org/c/openstack/neutron/+/944974 | 09:28 |
| lajoskatona | ralonsoh: I pushed the backport for the DHCP eventlet revert patches: https://review.opendev.org/q/I8bd0580bb830e0317ca6663c67b58842c34b87b4 & https://review.opendev.org/q/Ifd4f10a65935d9779ae4b5329258aef6bd5f871b | 09:30 |
| ralonsoh | lajoskatona, perfect | 09:30 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: [DNM] Test metadata rate limiting - master/antelope devstack ovn https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944921 | 09:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944979 | 10:35 |
| ralonsoh | ^^ something new found implementing the DB migration unit tests | 10:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944980 | 10:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944981 | 10:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944982 | 10:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944983 | 10:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add database migration tests https://review.opendev.org/c/openstack/neutron/+/944984 | 10:55 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: [DNM] Test metadata rate limiting - master/antelope devstack ovn https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944921 | 11:39 |
| lajoskatona | ykarel, bcafarel: Hi, a stable/2025.1 review (actually 2 patches in a row): https://review.opendev.org/c/openstack/neutron/+/944972 | 11:58 |
| ykarel | done, only have +1 there | 12:00 |
| lajoskatona | ykarel: thanks, I thought you have +2 on stable branches | 12:07 |
| ralonsoh | ^ I think we need to solve this | 12:13 |
| ralonsoh | I mean, adding those permissions to Yatin | 12:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944979 | 12:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add database migration tests https://review.opendev.org/c/openstack/neutron/+/944984 | 12:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944980 | 12:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944981 | 12:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944982 | 12:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944983 | 12:17 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: [DNM] Test metadata rate limiting - master/antelope devstack ovn https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944921 | 12:23 |
| ralonsoh | haleyb, lajoskatona please check https://review.opendev.org/q/Ic2d471978ad12ef69d6324bef006aadda0ab8c13 | 12:23 |
| ralonsoh | and https://review.opendev.org/c/openstack/neutron/+/944984/2 | 12:23 |
| ralonsoh | actually I found this issue because of the new test cases | 12:23 |
| ralonsoh | so they work! | 12:24 |
| opendevreview | Maor Blaustein proposed x/whitebox-neutron-tempest-plugin master: [DNM] Test metadata rate limiting - master/antelope devstack ovn https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944921 | 12:26 |
| opendevreview | Merged openstack/neutron stable/2024.1: [stable-only] Fix the Neutron milestones list https://review.opendev.org/c/openstack/neutron/+/944877 | 12:29 |
| opendevreview | Merged openstack/neutron unmaintained/2023.1: [stable-only] Fix the Neutron milestones list https://review.opendev.org/c/openstack/neutron/+/944875 | 12:29 |
| opendevreview | Merged openstack/neutron stable/2023.2: [stable-only] Fix the Neutron milestones list https://review.opendev.org/c/openstack/neutron/+/944876 | 12:29 |
| opendevreview | Merged openstack/neutron master: Remove flaky batch notifier unit test https://review.opendev.org/c/openstack/neutron/+/944950 | 12:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Make the Logical_Switch_Port UP event more robust https://review.opendev.org/c/openstack/neutron/+/943404 | 12:49 |
| haleyb | ralonsoh: interesting, again i'm surprised none of this broke anything | 13:11 |
| ralonsoh | haleyb, maybe just a lucky shot: the first milestone loaded (by alphabetical order) was the correct one | 13:47 |
| ralonsoh | but yes, makes no sense | 13:47 |
| haleyb | ykarel: can you take a look at my 2025.x job changes? https://review.opendev.org/q/topic:%222025-1%22 and https://review.opendev.org/q/topic:%222025.2-neutron-jobs%22 - not sure they can all merge yet based on openstack-zuul-jobs change but wanted to be ready to go | 14:06 |
| haleyb | ralonsoh: ^^ you too i guess, think you looked at one of them | 14:06 |
| ralonsoh | sure, let me check | 14:07 |
| haleyb | i copied from previous versions mostly of course | 14:07 |
| ralonsoh | haleyb, I going to comment that in the n-t-p patch | 14:10 |
| ralonsoh | we can add 3 extensions, one recently added to OVN (missing one) and 2 new ones added this cycle | 14:10 |
| ralonsoh | empty_string_filtering | 14:11 |
| ralonsoh | qinq | 14:11 |
| ralonsoh | uplink-status-propagation-updatable | 14:11 |
| haleyb | ralonsoh: sure, will take a look, figured i'd miss something like that even looking at the master jobs defs | 14:11 |
| priteau | ralonsoh: Would you be ok to approve https://review.opendev.org/c/openstack/neutron/+/930001? | 14:17 |
| haleyb | ralonsoh: uplink-status-propagation-updatable is even missing from the master job def, I might add that in a separate change to master and 2025.1 | 14:19 |
| ralonsoh | haleyb, and the other 2 too | 14:19 |
| ralonsoh | priteau, let me check | 14:20 |
| haleyb | ralonsoh: the empty string one is there, but i need to add qinq i tihnk | 14:20 |
| ralonsoh | yes | 14:20 |
| ralonsoh | priteau, I don't have +2 permissions in um branches. bcafarel can you push https://review.opendev.org/c/openstack/neutron/+/930001? | 14:21 |
| ralonsoh | thanks in advance | 14:21 |
| ralonsoh | haleyb, I think the tests implemented for the DB branches are effective | 14:21 |
| ralonsoh | at least I found this issue!! | 14:21 |
| haleyb | :) | 14:22 |
| priteau | ralonsoh: sorry, I didn't realise that | 14:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add database migration tests https://review.opendev.org/c/openstack/neutron/+/944984 | 14:33 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Add 2025.1 (Epoxy) stable jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/944954 | 14:39 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Add uplink-status-propagation-updatable extension https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/945012 | 14:39 |
| opendevreview | Brian Haley proposed openstack/neutron unmaintained/yoga: Enhance IptablesFirewallDriver with remote address groups https://review.opendev.org/c/openstack/neutron/+/945013 | 14:44 |
| haleyb | ralonsoh: thanks for the reviews. we will have a job explosion in n-t-p until 2023.2 goes unmaintained, now at 54 jobs :( | 15:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == [OVN][Placement] Placement init config with Chassis create event https://review.opendev.org/c/openstack/neutron/+/945017 | 15:30 |
| opendevreview | Merged openstack/neutron unmaintained/zed: Correct logic error when associating FIP with OVN LB https://review.opendev.org/c/openstack/neutron/+/930001 | 16:16 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Add new default policy for device_id field on ports https://review.opendev.org/c/openstack/neutron/+/861169 | 16:19 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Add new default policy for device_id field on ports https://review.opendev.org/c/openstack/neutron/+/861169 | 16:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Placement init config with Chassis create event https://review.opendev.org/c/openstack/neutron/+/945017 | 16:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Placement init config with Chassis create event https://review.opendev.org/c/openstack/neutron/+/945017 | 16:47 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2025.1: Remove flaky batch notifier unit test https://review.opendev.org/c/openstack/neutron/+/945027 | 16:52 |
| ykarel | haleyb, ack | 16:59 |
| haleyb | ykarel: oh, jammy for slurp, but i guess we can remove that on master branch now? | 17:01 |
| ykarel | haleyb, ohkk i was thinking of dropping that post epoxy release, but yes seems we can do that now too considering stable/epoxy branches already cut | 17:02 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Add 2025.1 (Epoxy) stable jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/944954 | 17:06 |
| haleyb | ykarel: updated, thanks. and i guess we have to wait for the n-t-p patch to merge before my neutron patch will pick it up and actually run the jobs | 17:08 |
| ykarel | haleyb, i think that's not due to depends-on not merged | 17:10 |
| ykarel | i vaguely remember it was due so some project stable not cut yet | 17:10 |
| ykarel | likely devstack | 17:10 |
| haleyb | ah | 17:10 |
| ykarel | but can check tomorrow if it's something else, need to drop now | 17:11 |
| haleyb | np, thanks for review | 17:11 |
| ykarel | i can see devstack not yet cut 2025.1, so it could be that only | 17:11 |
| ykarel | https://github.com/openstack/devstack/branches | 17:12 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/935090 | 17:26 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/935090 | 17:30 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP] Just prototype for making sure that neutron:cidrs and device id is set. https://review.opendev.org/c/openstack/neutron/+/943501 | 17:46 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/935090 | 19:19 |
| opendevreview | Merged openstack/neutron stable/2025.1: Revert "[eventlet-removal] Remove the usage of eventlet in the DHCP agent" https://review.opendev.org/c/openstack/neutron/+/944972 | 19:25 |
| opendevreview | Merged openstack/neutron stable/2025.1: Revert "[eventlet-removal] Remove eventlet from DHCP agent" https://review.opendev.org/c/openstack/neutron/+/944974 | 19:25 |
| opendevreview | Merged openstack/neutron stable/2025.1: Remove the duplicated YOGA database milestone https://review.opendev.org/c/openstack/neutron/+/944980 | 19:25 |
| opendevreview | Merged openstack/neutron master: Disable skip-level jobs in check queue https://review.opendev.org/c/openstack/neutron/+/944808 | 19:26 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Avoid race condition by using persist_uuid https://review.opendev.org/c/openstack/neutron/+/923971 | 19:46 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/935090 | 20:18 |
| opendevreview | Brian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/935090 | 21:33 |
| opendevreview | Brian Haley proposed openstack/ovn-octavia-provider unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ovn-octavia-provider/+/935107 | 21:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!