| opendevreview | Takashi Kajinami proposed openstack/ovn-octavia-provider master: Update default envlist https://review.opendev.org/c/openstack/ovn-octavia-provider/+/939028 | 05:12 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/ovn-bgp-agent master: Update default envlist https://review.opendev.org/c/openstack/ovn-bgp-agent/+/939029 | 05:12 |
| opendevreview | Takashi Kajinami proposed openstack/networking-bgpvpn master: Update default envlist https://review.opendev.org/c/openstack/networking-bgpvpn/+/939034 | 05:18 |
| opendevreview | Takashi Kajinami proposed openstack/networking-sfc master: Update default envlist https://review.opendev.org/c/openstack/networking-sfc/+/939035 | 05:18 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-vpnaas master: Update default envlist https://review.opendev.org/c/openstack/neutron-vpnaas/+/939036 | 05:18 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-vpnaas-dashboard master: Update default envlist https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/939037 | 05:19 |
| opendevreview | Merged openstack/neutron master: Fix some pylint issues in the test tree https://review.opendev.org/c/openstack/neutron/+/938853 | 07:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Neutron FWAAS vArmour driver was removed 8 years ago https://review.opendev.org/c/openstack/neutron/+/939053 | 08:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Replace ``eventlet.spawn_n`` usage https://review.opendev.org/c/openstack/neutron/+/938411 | 09:04 |
| opendevreview | Bodo Petermann proposed openstack/neutron-vpnaas master: Use OvnPeriodicWorker for VPN agent check https://review.opendev.org/c/openstack/neutron-vpnaas/+/938915 | 09:28 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 09:29 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: use greenlet os-ken implementation not monkey-patched https://review.opendev.org/c/openstack/neutron/+/938487 | 09:29 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: use native os-ken implementation https://review.opendev.org/c/openstack/neutron/+/938487 | 09:29 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: use greenlet os-ken implementation not monkey-patched https://review.opendev.org/c/openstack/neutron/+/939057 | 09:35 |
| vsaienko | hello neutron team, please review https://review.opendev.org/c/openstack/neutron/+/938657 | 10:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Check if the LRP exists in ``check_provider_distributed_ports`` https://review.opendev.org/c/openstack/neutron/+/938889 | 10:41 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2024.2: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937489 | 11:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Use non-eventlet metadata proxy in OVN metadata agent https://review.opendev.org/c/openstack/neutron/+/938393 | 11:21 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to MAC flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 11:22 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to MAC flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 11:26 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2024.1: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937490 | 11:35 |
| opendevreview | Merged openstack/ovn-bgp-agent stable/2023.2: Make in_port consistent type https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937491 | 11:35 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Update default envlist https://review.opendev.org/c/openstack/ovn-bgp-agent/+/939029 | 11:35 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Split _ensure_ovn_network_link function https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936409 | 11:35 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Use is_cidr_host utils to detect if AAP ip is host in l3_dvr_db https://review.opendev.org/c/openstack/neutron/+/939075 | 11:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Use non-eventlet metadata proxy in OVN metadata agent https://review.opendev.org/c/openstack/neutron/+/938393 | 11:35 |
| ralonsoh | hi folks, we need https://review.opendev.org/c/openstack/neutron/+/938319 to be merged in order to avoid the issues with WSGI initialization | 11:40 |
| ralonsoh | please, if you have some time, review the patch | 11:40 |
| ralonsoh | thanks in advance! | 11:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-*" for OVN with WSGI https://review.opendev.org/c/openstack/neutron/+/931842 | 11:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP - DNM - Test ML2/OVN plugin without eventlet https://review.opendev.org/c/openstack/neutron/+/931252 | 11:46 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Use is_cidr_host utils to detect if AAP ip is host in l3_dvr_db https://review.opendev.org/c/openstack/neutron/+/939075 | 12:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove eventlet usage in ``ProcessMonitor`` https://review.opendev.org/c/openstack/neutron/+/939078 | 12:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove eventlet usage in ``ProcessMonitor`` https://review.opendev.org/c/openstack/neutron/+/939078 | 12:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove eventlet usage in ``ProcessMonitor`` https://review.opendev.org/c/openstack/neutron/+/939078 | 12:08 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check pyroute2 master https://review.opendev.org/c/openstack/neutron/+/938641 | 12:33 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 13:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove ``common.utils.spawn`` https://review.opendev.org/c/openstack/neutron/+/939090 | 13:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove eventlet usage in ``ProcessMonitor`` https://review.opendev.org/c/openstack/neutron/+/939078 | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove ``common.utils.spawn`` https://review.opendev.org/c/openstack/neutron/+/939090 | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Reimplement ``common.utils.spawn_n`` https://review.opendev.org/c/openstack/neutron/+/939095 | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Synchronize the network segment range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 14:16 |
| ralonsoh | haleyb, hello! Please, if you have time ^^ | 14:17 |
| ralonsoh | I know this is not a trivial patch, but it should fix a recurrent bug in the CI (and in the wsgi API) | 14:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Remove ``subprocess_popen`` https://review.opendev.org/c/openstack/neutron/+/939097 | 14:22 |
| sahid | ralonsoh: o/ I have noticed your comment regarding async_process, it was just not clear for me whether you know if there is an already work in progress to remove it or if you are asking me to clean this part? | 14:43 |
| sahid | btw, os-ken starts the agent in a separate thread, right? and the agent is using signal | 14:44 |
| sahid | we will have to find a different implementation for that i guess | 14:45 |
| ralonsoh | about async_process, this is used in OVS (polling) but the code is not updated | 14:45 |
| ralonsoh | sahid, what do you mean with "agent is using signal"? | 14:45 |
| sahid | https://zuul.opendev.org/t/openstack/build/51a39b72d08e4014a155e6361b115e06/log/controller/logs/screen-q-agt.txt?severity=4 | 14:46 |
| ralonsoh | yeah... that could be a problem | 14:46 |
| sahid | btw to ensure i have all the key in my head, the aim it at the end to completly remove usage of eventlet, but as a first step, it is ok to just remove the monkey patching part? | 14:48 |
| ralonsoh | why that was not happening in the current code? | 14:48 |
| sahid | ralonsoh: i guess because that was using greenthread | 14:48 |
| ralonsoh | sahid, you can't remove the monkey_patch if you are still using eventlet | 14:48 |
| ralonsoh | right, signal applies, if I'm not wrong, to the main kernel thread | 14:48 |
| sahid | you mean envent let can't work without monkey_patch? even if we use directly its API? | 14:50 |
| sahid | eventlet | 14:50 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [WIP] Workaround ipv6 metadata test failure https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/939104 | 14:50 |
| ralonsoh | sahid, no, you need to moneky patch the kernel threads to use eventlet | 14:50 |
| ralonsoh | at least this is needed because we are not completely using eventlet only methods, we are mixing implementations (we are using socket, threading, etc) | 14:51 |
| sahid | ok i thought the monkey_aptch was just to replace threading and all the standard interfaces to use evenlet | 14:51 |
| ralonsoh | yes but because we are mixing implementations, we cannot directly use eventlet methods and kernel ones | 14:52 |
| ralonsoh | at least without a proper design (and we don't have it) | 14:52 |
| ralonsoh | the OVS agent (and any other one) is using oslo-service launh | 14:52 |
| ralonsoh | that uses ServiceLauncher to spawn the agent | 14:52 |
| ralonsoh | this class controls the signals | 14:52 |
| ralonsoh | we can maybe (I don't know) use the handlers to catch the sihgup and to the reconfig | 14:53 |
| ralonsoh | ServiceLauncher has a add_handler() method | 14:54 |
| sahid | Yes I guess we can implement something with queue | 14:55 |
| ralonsoh | but I don't think we need to re-implement anything | 14:56 |
| sahid | ok I will make some try and let you know anyway | 14:57 |
| ralonsoh | just adding the _check_and_handle_signal (maybe splitting it for HUP and TERM) to the add_handler() | 14:57 |
| ralonsoh | but this is just a suggestion | 14:57 |
| ralonsoh | I think this could be tested quickly | 14:57 |
| sahid | ack | 14:59 |
| sahid | btw, if you have a moment to start looking into the change in os-ken | 15:00 |
| ralonsoh | sure, I'll do it | 15:00 |
| sahid | because I guess we can't use was I had in my head which is to do not monkey_patch but continue using evenlet API | 15:00 |
| sahid | https://review.opendev.org/q/topic:%22bug/2087939%22 | 15:00 |
| sahid | cool thank you ! | 15:00 |
| slaweq | ykarel_ ralonsoh hi, do we have ci meeting today? | 15:01 |
| ralonsoh | yes, waiting (I don't know if video or IRC) | 15:02 |
| ykarel_ | slaweq, ralonsoh yes IRC | 15:02 |
| *** ykarel_ is now known as ykarel | 15:02 | |
| ykarel | #startmeeting neutron_ci | 15:02 |
| opendevmeet | Meeting started Mon Jan 13 15:02:37 2025 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:02 |
| ykarel | Ping list: bcafarel, lajoskatona, slawek, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:02 |
| ralonsoh | hello | 15:02 |
| lajoskatona | Hi | 15:03 |
| slaweq | o/ | 15:03 |
| ykarel | ok let's start with the topics | 15:04 |
| ykarel | #topic Actions from previous meetings | 15:04 |
| ykarel | there were none for this week | 15:04 |
| ykarel | #topic Stable branches | 15:04 |
| ykarel | bernard not around today, but all good on stable | 15:05 |
| ykarel | #topic Stadium projects | 15:05 |
| ykarel | neutron-dynamic-routing still down with wsgi switch https://bugs.launchpad.net/neutron/+bug/2085421 | 15:05 |
| lajoskatona | neutron-dynamic-routng is red, yes with the wsgi | 15:05 |
| lajoskatona | but am I right that the neutron patch will solve it? | 15:06 |
| ykarel | which neutron patch? | 15:06 |
| ykarel | have we already tested that job with the patch you referring? | 15:06 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Fix text from CrLrpChassisChangeWithdrawEvent docstring https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936427 | 15:06 |
| lajoskatona | let me check | 15:08 |
| lajoskatona | https://review.opendev.org/c/openstack/neutron/+/936829 | 15:08 |
| lajoskatona | no, a one for n-t-p | 15:08 |
| lajoskatona | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/938466 | 15:09 |
| lajoskatona | from last wee kmy memories tell me that we said that this one to limit the wsgi workers is a workaround but we can go with it temporarily, but after that other things just washed this topic from my mind | 15:10 |
| ykarel | yeap that was a workaround if we want to consider | 15:12 |
| ykarel | ralonsoh, do you already have some patch to fix ^ issue? | 15:13 |
| ralonsoh | no, sorry | 15:13 |
| ralonsoh | I didn't investigate the ndr issue | 15:13 |
| ralonsoh | but I can start checking it tomorrow | 15:13 |
| ralonsoh | you can assign it to me | 15:13 |
| ykarel | ralonsoh, ack behavior wise we seeing those lock wait timeout errors here | 15:14 |
| lajoskatona | yatin's patch with the wsgi worker number showed that the issue is related to the eventlet/wsgi topic | 15:14 |
| lajoskatona | ykarel:exactly | 15:15 |
| ykarel | #action ralonsoh to check https://bugs.launchpad.net/neutron/+bug/2085421 | 15:15 |
| lajoskatona | but I think it is fine to merge the workaround from ykarel, to not add to the piles of ralonsoh | 15:15 |
| ralonsoh | hmmmm let's wait a bit | 15:16 |
| ralonsoh | because reducing to 1 the concurrency will hide some problems | 15:16 |
| ykarel | yeap can wait | 15:16 |
| ralonsoh | so I would prefer not to merge this patch yet | 15:16 |
| lajoskatona | ack | 15:17 |
| ykarel | let's move | 15:17 |
| ykarel | #topic Rechecks | 15:17 |
| ykarel | so we still have good number of rechecks for these known issues with ovn | 15:18 |
| ykarel | bare recheck wise we had 5/34 | 15:18 |
| ykarel | Let's check failures | 15:18 |
| ykarel | #topic Unit tests | 15:18 |
| ykarel | bandit issue https://bugs.launchpad.net/neutron/+bug/2093849 fixed with new 1.8.2 release | 15:18 |
| ykarel | this already got fixed, thanks haleyb for checking | 15:19 |
| ykarel | #topic fullstack/functional | 15:19 |
| haleyb | :) | 15:19 |
| ykarel | we still seeing some oom-kill on raxflex node | 15:19 |
| ykarel | - https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_740/periodic/opendev.org/openstack/neutron/master/neutron-functional/740c0fc/controller/logs/syslog.txt | 15:19 |
| ykarel | - https://bcd85c4e99ffc7863f97-b20dd3fcccdb5478c1889ed1c3ad542c.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-sqlalchemy-master/294db5b/controller/logs/syslog.txt | 15:19 |
| ykarel | - https://47b74d2027b9966dbc6f-6fe938074b11dd62ac4ccd867c30657d.ssl.cf2.rackcdn.com/938853/2/gate/neutron-functional/32345b2/controller/logs/syslog.txt | 15:19 |
| ykarel | - https://bugs.launchpad.net/neutron/+bug/2091855 still happening less frequent though | 15:19 |
| ykarel | i will check that again if we can further improve it | 15:19 |
| ykarel | #action ykarel to check lp#2091855 | 15:20 |
| ykarel | #topic Tempest/Scenario | 15:20 |
| ykarel | Here we mainly seeing ovn wsgi switch related issues | 15:20 |
| ykarel | most of the failures that i saw were where lsp up event was missed | 15:20 |
| ykarel | ralonsoh, do we have a bug for ^ specifically | 15:21 |
| ralonsoh | yeah, this is killing me | 15:21 |
| ralonsoh | one related | 15:21 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/2093327 | 15:21 |
| ralonsoh | but no, I need to open the bug for the missing LSP up event | 15:21 |
| ykarel | hmm but that's different | 15:21 |
| ralonsoh | yes | 15:21 |
| ykarel | ok yes please let's have dedicated one for that issue | 15:21 |
| ralonsoh | about the LSP up event, we see that in the OVN logs | 15:22 |
| ralonsoh | but never received in the Neutron API | 15:22 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Drop dead code related to keystone-v3 in network segment ranges https://review.opendev.org/c/openstack/neutron/+/939107 | 15:22 |
| ykarel | #action ralonsoh to open bug for missing lsp up event with wsgi switch | 15:22 |
| ykarel | linking some of these failures here | 15:23 |
| ykarel | https://a6fbb86efdf5860e1b02-ce677b1d4d068789ef14672b1b5d2453.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-ovn-tempest-with-neutron-lib-master/fc27ce5/testr_results.html | 15:23 |
| ykarel | https://650bc3c55ab286ed9493-bc3c897401c8263f785d7df7482dd6e6.ssl.cf1.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-ovn-tempest-plugin-ovs-lts/cde340b/testr_results.html | 15:23 |
| ykarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_18d/periodic/opendev.org/openstack/neutron/master/neutron-tempest-plugin-ovn-enforce-scope-old-defaults/18dbe6f/testr_results.html | 15:24 |
| ykarel | https://2780d629a7a6d047ca0f-9244b66e59ac9fd5e3dbb52d3e6f0017.ssl.cf5.rackcdn.com/periodic/opendev.org/openstack/neutron/master/neutron-tempest-plugin-ovn-enforce-scope-old-defaults/091ece8/testr_results.html | 15:24 |
| ykarel | https://e76b8a042504872b0b7d-f18cd0734c7002742f0baf633d0573f2.ssl.cf1.rackcdn.com/938800/3/gate/neutron-tempest-plugin-ovn/548dc8b/testr_results.html | 15:24 |
| ralonsoh | yes, unfortunately we have several examples everyday | 15:24 |
| lajoskatona | :-) | 15:25 |
| ykarel | also nother tempest failure unrelated to wsgi switch | 15:25 |
| ykarel | test_metadata_ipv6_only_network failed with malformed check | 15:25 |
| ykarel | https://33d4ca1eb24f74c1535d-46fef9f34fd106c3774e2eeeca119539.ssl.cf5.rackcdn.com/938670/1/gate/neutron-tempest-plugin-openvswitch/878b8bb/testr_results.html | 15:25 |
| ykarel | somehow the console output was malformed like cloud-init[515]: latestQueried th]: e metadata service over IPv6ci-info | 15:26 |
| ykarel | so the expected string not found | 15:26 |
| ykarel | i will report the issue for it | 15:26 |
| ykarel | mtomaska1, fyi ^ | 15:26 |
| opendevreview | Vasyl Saienko proposed openstack/neutron-lib master: Remove tenant_id from segment rages responces https://review.opendev.org/c/openstack/neutron-lib/+/939108 | 15:26 |
| ykarel | #action ykarel to report issue for ipv6 metadata test | 15:26 |
| ykarel | #topic Periodic | 15:27 |
| mtomaska1 | ACK ykarel | 15:27 |
| ykarel | here we mostly hit ovn wsgi related issues | 15:27 |
| ykarel | pyroute2 functional issues are now resolved | 15:27 |
| ralonsoh | ykarel, but is this one related to wsgi? | 15:27 |
| ralonsoh | the metadata bug | 15:28 |
| ykarel | ralonsoh, no | 15:28 |
| ykarel | it's unrelated | 15:28 |
| ralonsoh | ahhhh sorry (I need to read better...) | 15:28 |
| ykarel | for pyroute2 Petr will likely be joining tomorrows neutron meeting | 15:29 |
| ykarel | to tell more about upcoming pyroute2 changes and removing some legacy bits and adoption in neutron for it | 15:30 |
| ykarel | https://github.com/svinota/pyroute2/issues/1225#issuecomment-2582071411 where he pointed it out | 15:30 |
| lajoskatona | +1, good news, always happy to hear/read from him | 15:30 |
| ralonsoh | and about using asyncio | 15:30 |
| ykarel | just to update i will not be around tomorrow | 15:31 |
| ykarel | ok that's it on failures | 15:31 |
| ykarel | #topic Grafana | 15:32 |
| ykarel | lets quicly look on https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:32 |
| ykarel | looks normal, check failures either known issues or related to patches | 15:35 |
| ykarel | anything to add? | 15:35 |
| ralonsoh | related to wsgi: https://review.opendev.org/c/openstack/neutron/+/938319 | 15:35 |
| ralonsoh | please review it | 15:35 |
| ralonsoh | that's all | 15:35 |
| ykarel | #topic On Demand | 15:37 |
| ykarel | anything else you would like to raise? | 15:37 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Make API policies for tags to be working with resource attributes https://review.opendev.org/c/openstack/neutron/+/938135 | 15:38 |
| lajoskatona | Nothing from me | 15:39 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to MAC flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 15:40 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Use is_cidr_host utils to detect if AAP ip is host in l3_dvr_db https://review.opendev.org/c/openstack/neutron/+/939075 | 15:41 |
| haleyb | ralonsoh: i am looking at that now, lgtm | 15:47 |
| ralonsoh | thanks | 15:47 |
| ralonsoh | ykarel, hi, are you still there? | 15:50 |
| ralonsoh | maybe he had a network disconnection | 15:51 |
| ralonsoh | I think we can close the meeting for today, right? | 15:51 |
| ykarel | #endmeeting | 15:51 |
| opendevmeet | Meeting ended Mon Jan 13 15:51:23 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-13-15.02.html | 15:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-13-15.02.txt | 15:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2025/neutron_ci.2025-01-13-15.02.log.html | 15:51 |
| lajoskatona | possible | 15:51 |
| lajoskatona | o/ | 15:51 |
| ralonsoh | heheheh ok | 15:51 |
| ykarel | thx ralonsoh for reminding :) | 15:51 |
| ralonsoh | heheheh no problem | 15:51 |
| ykarel | i was about to leave | 15:51 |
| slaweq | o/ | 15:53 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Add option to configure live migration activation strategy for OVN https://review.opendev.org/c/openstack/neutron/+/938106 | 16:22 |
| otherwiseguy | haleyb: on https://bugs.launchpad.net/neutron/+bug/2092133 / https://review.opendev.org/c/openstack/ovsdbapp/+/938004, I can't see any way that the patch is not a NoOp. | 16:28 |
| otherwiseguy | Without the patch: DEBUG ovsdbapp.backend.ovs_idl [None req-1d6c1ec4-f079-4a1c-8be6-269af4fff9c3 None None] Created schema index Logical_Router_Port.name {{(pid=304174) autocreate_indices /opt/stack/data/venv/lib/python3.12/site-packages/ovsdbapp/backend/ovs_idl/__init__.py:106}} | 16:28 |
| otherwiseguy | With the patch: DEBUG ovsdbapp.backend.ovs_idl [None req-c00d73c2-6929-49a8-9a53-7fcffa95413c None None] Created lookup_table index Logical_Router.name {{(pid=303874) autocreate_indices /opt/stack/data/venv/lib/python3.12/site-packages/ovsdbapp/backend/ovs_idl/__init__.py:87}} | 16:29 |
| otherwiseguy | In both cases, autocreate_indices() creates the index with the same code. | 16:29 |
| otherwiseguy | With the patch, it creates it because of the addition to lookup_table, without it, it creates the index based on the schema. | 16:29 |
| opendevreview | Lajos Katona proposed openstack/neutron master: If OVS Manager creation failes retry to set values https://review.opendev.org/c/openstack/neutron/+/939117 | 16:30 |
| otherwiseguy | haleyb: Do you know maxlamprecht? Not sure how to contact regarding the supposed performance improvement. I think something else must have been going on in their profiling setup. | 16:31 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove dibbler code from l3 agent and elsewhere https://review.opendev.org/c/openstack/neutron/+/934283 | 16:31 |
| opendevreview | Elod Illes proposed openstack/os-vif unmaintained/wallaby: [CI] Remove devstack-gate requirement https://review.opendev.org/c/openstack/os-vif/+/939119 | 16:44 |
| opendevreview | Elod Illes proposed openstack/os-vif unmaintained/wallaby: [CI] Remove devstack-gate requirement https://review.opendev.org/c/openstack/os-vif/+/939119 | 16:53 |
| haleyb | otherwiseguy: i don't know max, the best you can do is try through launchpad, https://launchpad.net/~maxlamprecht/+contactuser | 16:55 |
| otherwiseguy | haleyb: Ok, thanks. Sorry I didn't catch it earlier, was out on PTO for nearly a month. | 16:56 |
| haleyb | so was i, work sucks :) | 16:56 |
| otherwiseguy | :D Not rich enough to make enough w/o making money, so I guess the suckage will continue. | 16:58 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Revert "Add Northbound Logical_Router_Port name index" https://review.opendev.org/c/openstack/ovsdbapp/+/939122 | 16:59 |
| otherwiseguy | ralonsoh: It'd be nice to know how the author supposedly "noticed a performance improvement" with the patch, but it looks literally impossible to me. | 17:00 |
| otherwiseguy | I have to suspect that they just accidentally changed multiple things during testing. | 17:00 |
| ralonsoh | otherwiseguy, right, as you mentioned, LRP.name is already an index | 17:00 |
| ralonsoh | so this patch is no needed | 17:00 |
| ralonsoh | btw, I didn'tknow you were talking about this here | 17:01 |
| ralonsoh | I saw Jakub's comment in the releases patch | 17:01 |
| ralonsoh | otherwiseguy, https://review.opendev.org/c/openstack/releases/+/939069 | 17:03 |
| ralonsoh | btw, I need to leave now | 17:03 |
| ralonsoh | I'll read your comments tomorrow | 17:03 |
| ralonsoh | bye! | 17:04 |
| otherwiseguy | ralonsoh: yeah, w/ and w/o the patch, i see log messages for the index creation. And both just do the same "idx.add_column(col)" call. The only thing I could see is if somehow the value of "col" was different between the two calls. | 17:04 |
| otherwiseguy | o/ | 17:04 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 17:05 |
| MaxLamprecht[m] | <otherwiseguy> "haleyb: Do you know maxlamprecht..." <- Hey together, actually you're right :) But we found out something else... | 18:17 |
| MaxLamprecht[m] | I left a comment at the LP bug with more details | 18:17 |
| MaxLamprecht[m] | https://bugs.launchpad.net/neutron/+bug/2092133/comments/5 | 18:17 |
| haleyb | thanks Max! | 18:22 |
| opendevreview | Brian Haley proposed openstack/neutron master: Install DVR src to MAC flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 18:25 |
| opendevreview | Merged openstack/neutron master: Synchronize the network segment range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 18:50 |
| opendevreview | Merged openstack/neutron master: [eventlet-deprecation] Remove eventlet usage in ``ProcessMonitor`` https://review.opendev.org/c/openstack/neutron/+/939078 | 19:19 |
| opendevreview | Merged openstack/os-vif unmaintained/wallaby: [CI] Remove devstack-gate requirement https://review.opendev.org/c/openstack/os-vif/+/939119 | 19:37 |
| opendevreview | Elod Illes proposed openstack/os-vif unmaintained/victoria: [CI] Remove devstack-gate requirement https://review.opendev.org/c/openstack/os-vif/+/939160 | 19:48 |
| opendevreview | Kevin Carter proposed openstack/neutron master: fix: add try/except to allow maintenance to run on unmanaged segments https://review.opendev.org/c/openstack/neutron/+/939166 | 20:47 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 22:03 |
| opendevreview | Merged openstack/os-vif unmaintained/wallaby: Update .gitreview for unmaintained/wallaby https://review.opendev.org/c/openstack/os-vif/+/911281 | 22:12 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: doc: Remove description of CI jobs from documentation https://review.opendev.org/c/openstack/neutron/+/938858 | 22:28 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: doc: Remove description of CI jobs from documentation https://review.opendev.org/c/openstack/neutron/+/938858 | 22:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!