| opendevreview | Merged openstack/neutron-fwaas stable/2026.1: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984758 | 01:51 |
|---|---|---|
| opendevreview | Merged openstack/neutron-fwaas stable/2025.2: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984759 | 01:51 |
| opendevreview | Merged openstack/neutron-fwaas stable/2025.1: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984760 | 01:51 |
| opendevreview | Merged openstack/neutron-fwaas stable/2024.2: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984762 | 01:51 |
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 02:18 |
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 03:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add missing Router ID and AZ hits in the HCG.external_ids https://review.opendev.org/c/openstack/neutron/+/985363 | 06:05 |
| opendevreview | Merged openstack/neutron-vpnaas master: Start using context.project_id https://review.opendev.org/c/openstack/neutron-vpnaas/+/984948 | 06:43 |
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 07:16 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Remove trailing space from cls.OC attribute https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/984707 | 08:15 |
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 08:19 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add neutron-ovn-db-sync-util tests for log and repair modes https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/983699 | 08:26 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add neutron-ovn-db-sync-util tests for log and repair modes https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/983699 | 08:34 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add neutron-ovn-db-sync-util tests for log and repair modes https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/983699 | 08:46 |
| ralonsoh | hi folks, please check https://review.opendev.org/c/openstack/neutron/+/985363 if you have time. This is a bug related to the HA_Chassis_Group migration | 09:46 |
| ralonsoh | the issue is only in master, not in stable releases | 09:46 |
| opendevreview | Stephen Finucane proposed openstack/neutron-lib master: objects: Prepare for oslo.versionedobjects 3.10.0 https://review.opendev.org/c/openstack/neutron-lib/+/985636 | 09:58 |
| stephenfin | ralonsoh: Dead simple fix to unblock o.vo 3.10.0 there ☝️ if you have a moment? | 09:58 |
| stephenfin | Maybe lajoskatona also? | 09:58 |
| ralonsoh | sure, checking now | 10:02 |
| lajoskatona | checking also | 10:11 |
| stephenfin | lajoskatona: While I have you, is https://review.opendev.org/c/openstack/python-openstackclient/+/977974 ready to go now? I still see a lot of unresolved comments so I have been holding off review until they were resolved | 10:22 |
| opendevreview | Lajos Katona proposed openstack/neutron master: DNN: Use SDK for Nova notifications https://review.opendev.org/c/openstack/neutron/+/983905 | 10:22 |
| ralonsoh | stephenfin, pep8 and py3 don't look happy | 10:23 |
| ralonsoh | https://zuul.opendev.org/t/openstack/status?change=985636 | 10:23 |
| stephenfin | hmm | 10:23 |
| opendevreview | Lajos Katona proposed openstack/neutron master: DNN: Use SDK for Nova notifications https://review.opendev.org/c/openstack/neutron/+/983905 | 10:23 |
| stephenfin | I'll be honest I didn't even check those since it was such a small change 😅 Looking now | 10:23 |
| ralonsoh | np | 10:24 |
| stephenfin | weird: autopep8 wants a newline before the method if its an instance method but not if it's a static method... | 10:25 |
| lajoskatona | stephenfin: as I remember all resolved, but today I check it again to be sure, and leave a comment | 10:26 |
| stephenfin | great, thanks | 10:27 |
| opendevreview | Stephen Finucane proposed openstack/neutron-lib master: objects: Prepare for oslo.versionedobjects 3.10.0 https://review.opendev.org/c/openstack/neutron-lib/+/985636 | 10:42 |
| stephenfin | Okay, that's fixed and both unit and linter checks are passing. It can obviously wait for a successful CI run though: I just wanted to make sure you were aware of it | 10:44 |
| ralonsoh | nice, you found a bug in the tests | 10:53 |
| opendevreview | Merged openstack/neutron-lib master: Add "l3-agent-scheduler-ha-chassis-priority" API extension definition https://review.opendev.org/c/openstack/neutron-lib/+/982781 | 11:47 |
| tafkamax | I don't know if this is a neutron question. | 12:38 |
| tafkamax | How can I delete the dns_domain and dns_name values from an floating ip? | 12:38 |
| ralonsoh | tafkamax, https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/dns.py#L74-L86 | 12:47 |
| ralonsoh | allow_put=False | 12:47 |
| ralonsoh | it is not possible. It is for ports, not for FIPs | 12:48 |
| tafkamax | Aha interesting, I am getting 15:18:35.392 1092 ERROR neutron.api.v2.resource designateclient.exceptions.NotFound: Name REDACTED.in-addr.arpa. didn't resolve when trying to delete a floating ip with those fields set. I suppose designate should have some entries there. | 12:50 |
| tafkamax | I seem to be having an issue with deleting floating IP-s that have dns_name and dns_domain defined. I am getting the error above or I am getting Managed records may not be deleted | 13:24 |
| tafkamax | I wonder if I should go look in the database? | 13:24 |
| ralonsoh | stephenfin, I don't know why openstack-tox-py313-with-neutron is failing, in both executions, the same test | 14:05 |
| * stephenfin looks | 14:06 | |
| stephenfin | Oh, I think I know what's going on. Give me 2 | 14:07 |
| ralonsoh | stephenfin, these objects are used inside OVOs | 14:11 |
| stephenfin | yeah | 14:12 |
| ralonsoh | so that would change the hash | 14:12 |
| stephenfin | this is what causes the hash change https://review.opendev.org/c/openstack/neutron-lib/+/985636/2/neutron_lib/objects/common_types.py#328 | 14:14 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: tests: Handle change in hash for neutron-lib field fixes https://review.opendev.org/c/openstack/neutron/+/985682 | 14:29 |
| opendevreview | Stephen Finucane proposed openstack/neutron-lib master: objects: Prepare for oslo.versionedobjects 3.10.0 https://review.opendev.org/c/openstack/neutron-lib/+/985636 | 14:39 |
| stephenfin | ralonsoh: That ought to fix it ^ | 14:40 |
| 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 | 14:50 |
| ralonsoh | stephenfin, yes, now seems to work (at least locally) | 14:55 |
| stephenfin | Great. I left some more notes on the neutron review also, since I appreciate this may be the first time we've done something this | 14:56 |
| ralonsoh | yeah, I saw them | 14:56 |
| stephenfin | ack | 14:56 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Print `ovsdb-client monitor` command output https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985685 | 14:59 |
| haleyb | cid: are we meeting in neutron or ironic room? | 15:02 |
| cid | In Ironic room | 15:02 |
| cid | We are rounding up with Nova at the moment | 15:02 |
| cid | https://meetpad.opendev.org/ironic-ptg-2026.2 | 15:03 |
| cid | haleyb, now in session \o/ | 15:05 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Print `ovsdb-client monitor` command output https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/985685 | 15:23 |
| opendevreview | Merged openstack/neutron master: Use project_id in agent DB code for AZs https://review.opendev.org/c/openstack/neutron/+/984202 | 15:41 |
| opendevreview | Merged openstack/neutron master: bgp: Use NIC types when obtaining peer NIC https://review.opendev.org/c/openstack/neutron/+/984555 | 15:50 |
| haleyb | slaweq: hey, if you're still around can you look at https://review.opendev.org/c/openstack/neutron-fwaas/+/984949 ? then we can merge the neturon-lib change :) | 16:05 |
| rm_work[m] | ralonsoh: ok so the patch I was testing fixes the issue in the batchnotifier in theory, and I think is probably "correct", but did not solve my thread exhaustion issue at all, so there must be somewhere else. trying to track it down, in the meantime that bug report is... not exactly invalid, but not really useful | 16:06 |
| rm_work[m] | I'll update it when I figure out wtf is causing all these threads to show up and never go away | 16:06 |
| rm_work[m] | so if you hadn't looked yet, don't bother until I poke you again :P | 16:07 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Add scenario test for VM connectivity with allowed_address_pairs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984133 | 16:15 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 16:42 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [WIP] PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 16:42 |
| otherwiseguy | haleyb: do you know how strongly we support "actual mysql" as opposed to mariadb? There are things I would find useful like the RETURNING construct which is supported by MariaDB, Postggres, and SQLite no problem, but actual MySQL not so much. | 17:44 |
| haleyb | otherwiseguy: was having double vision with the two channel ping :) And i can't say i know the answer, but assume you are asking for tests? | 17:59 |
| -opendevstatus- NOTICE: The wiki.openstack.org service will be offline for a few minutes while we perform a database upgrade | 18:07 | |
| opendevreview | Merged openstack/neutron master: tests: Handle change in hash for neutron-lib field fixes https://review.opendev.org/c/openstack/neutron/+/985682 | 18:23 |
| jssfr | just be careful that INSERT INTO .. ON CONFLICT .. RETURNING behaves a bit different between SQLite and Postgres at least | 19:23 |
| jssfr | (SQLite says "on conflict is not an insert, so returning returns null", while Postgres will give you the data from post conflict resolution) | 19:24 |
| opendevreview | Adam Harwell proposed openstack/neutron master: Fix unbounded thread pool in OVN MaintenanceThread https://review.opendev.org/c/openstack/neutron/+/985722 | 20:12 |
| rm_work[m] | ok ^^ that is the fix for https://bugs.launchpad.net/neutron/+bug/2149830 and does seem to solve my thread exhaustion issue in our env | 20:14 |
| otherwiseguy | haleyb: yeah, sorry. This would just be for being able to do things like "give me a random VNI that is not currently in the DB" in a single query w/o having to pre-populate rows in the db or do the thing like the ha_vr_id stuff that we do where it does racy things like SELECT all ha_vr_ids, find a random one locally, then do an insert based on that result and hope that nothing else took it". | 20:22 |
| otherwiseguy | The RETURNING feature makes it pretty easy to have the DB generate the random number and get back a result in every case (real result, range exhausted, or collision). Without it, might have to generate the random number locally and send it initially and rely on "big number, unlikely to collide, but retry if it does". | 20:24 |
| otherwiseguy | jssfr: thanks for the heads-up | 20:25 |
| otherwiseguy | There are also some issues with random() on sqlite not being normalized to [0, 1) which I could work around. | 20:26 |
| otherwiseguy | If I can come up with a way to do everything in a single query reliably across supported DBs, I'd be pretty happy. | 20:27 |
| opendevreview | Adam Harwell proposed openstack/neutron master: Fix unbounded thread creation in BatchNotifier https://review.opendev.org/c/openstack/neutron/+/985729 | 20:51 |
| rm_work[m] | ^^ submitted a reworked patch for the other one for completeness, but it wasn't actually what was wrecking my thread count, likely isn't hugely important | 20:52 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Handle chassis scale in/out https://review.opendev.org/c/openstack/neutron/+/985730 | 20:55 |
| opendevreview | Julia Kreger proposed openstack/neutron master: ml2: Add PortBindingError for explicit binding failure https://review.opendev.org/c/openstack/neutron/+/985732 | 21:06 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Handle chassis scale in/out https://review.opendev.org/c/openstack/neutron/+/985730 | 23:34 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!