| opendevreview | Eunkyung Kim proposed openstack/tooz master: zookeeper: Add retry parameters' parsing logic https://review.opendev.org/c/openstack/tooz/+/984292 | 01:50 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/debtcollector master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/debtcollector/+/1008374 | 02:47 |
| opendevreview | Takashi Kajinami proposed openstack/debtcollector master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/debtcollector/+/1008374 | 02:47 |
| opendevreview | Takashi Kajinami proposed openstack/debtcollector master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/debtcollector/+/1008374 | 02:48 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/castellan/+/1008375 | 02:50 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: Drop redundant test requirement https://review.opendev.org/c/openstack/castellan/+/1008376 | 02:51 |
| opendevreview | Takashi Kajinami proposed openstack/cursive master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/cursive/+/1008377 | 02:52 |
| opendevreview | Takashi Kajinami proposed openstack/etcd3gw master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/etcd3gw/+/1008378 | 02:53 |
| opendevreview | Takashi Kajinami proposed openstack/tooz master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/tooz/+/1008379 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/etcd3gw master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/etcd3gw/+/1008378 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/taskflow master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/taskflow/+/1008380 | 02:58 |
| opendevreview | Takashi Kajinami proposed openstack/automaton master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/automaton/+/1008381 | 02:59 |
| opendevreview | Takashi Kajinami proposed openstack/stevedore master: Add cover target https://review.opendev.org/c/openstack/stevedore/+/1008382 | 03:00 |
| opendevreview | Takashi Kajinami proposed openstack/debtcollector master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/debtcollector/+/1008374 | 03:01 |
| opendevreview | Takashi Kajinami proposed openstack/pycadf master: Add coverage options to pyproject.toml https://review.opendev.org/c/openstack/pycadf/+/1008383 | 03:03 |
| opendevreview | Takashi Kajinami proposed openstack/whereto master: Declare Python 3.15 support https://review.opendev.org/c/openstack/whereto/+/1008384 | 03:05 |
| opendevreview | Takashi Kajinami proposed openstack/whereto master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/whereto/+/1008385 | 03:05 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: Reduce trigger for tempest test https://review.opendev.org/c/openstack/castellan/+/1008387 | 03:46 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: Reduce trigger for tempest test https://review.opendev.org/c/openstack/castellan/+/1008387 | 03:52 |
| opendevreview | Merged openstack/whereto master: Declare Python 3.15 support https://review.opendev.org/c/openstack/whereto/+/1008384 | 03:53 |
| opendevreview | Merged openstack/whereto master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/whereto/+/1008385 | 03:59 |
| opendevreview | Merged openstack/castellan master: Drop redundant test requirement https://review.opendev.org/c/openstack/castellan/+/1008376 | 04:22 |
| opendevreview | Merged openstack/oslo.cache master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/oslo.cache/+/1008186 | 04:33 |
| opendevreview | Merged openstack/tooz master: Add version-less job with etcd3gw https://review.opendev.org/c/openstack/tooz/+/1007587 | 04:36 |
| opendevreview | Takashi Kajinami proposed openstack/castellan master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/castellan/+/1008375 | 06:52 |
| opendevreview | Takashi Kajinami proposed openstack/taskflow master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/taskflow/+/1008380 | 06:53 |
| opendevreview | Takashi Kajinami proposed openstack/etcd3gw master: Add cross tests with taskflow https://review.opendev.org/c/openstack/etcd3gw/+/1008422 | 08:33 |
| opendevreview | Merged openstack/etcd3gw master: Add cross tests with oslo.cache and tooz https://review.opendev.org/c/openstack/etcd3gw/+/993867 | 09:30 |
| opendevreview | Stephen Finucane proposed openstack/tooz master: ruff: Enable more linters https://review.opendev.org/c/openstack/tooz/+/1008313 | 09:36 |
| opendevreview | Stephen Finucane proposed openstack/tooz master: Migrate requirements to pyproject.toml https://review.opendev.org/c/openstack/tooz/+/1000832 | 09:45 |
| opendevreview | Stephen Finucane proposed openstack/tooz master: trivial: Change indentation of pyproject.toml https://review.opendev.org/c/openstack/tooz/+/1008431 | 09:45 |
| opendevreview | Merged openstack/castellan master: Reduce trigger for tempest test https://review.opendev.org/c/openstack/castellan/+/1008387 | 10:36 |
| opendevreview | Merged openstack/castellan master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/castellan/+/1008375 | 11:09 |
| opendevreview | Merged openstack/castellan master: Do not run functional tests by default https://review.opendev.org/c/openstack/castellan/+/1004910 | 11:09 |
| damani[m] | tkajinam, when you have time can you check the patch please https://review.opendev.org/c/openstack/oslo.service/+/1006180 | 11:12 |
| opendevreview | Merged openstack/tooz master: Get rid of ddt https://review.opendev.org/c/openstack/tooz/+/1008318 | 11:14 |
| opendevreview | Merged openstack/etcd3gw master: Add cross tests with taskflow https://review.opendev.org/c/openstack/etcd3gw/+/1008422 | 11:16 |
| opendevreview | Merged openstack/pycadf master: Add coverage options to pyproject.toml https://review.opendev.org/c/openstack/pycadf/+/1008383 | 11:19 |
| opendevreview | Merged openstack/debtcollector master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/debtcollector/+/1008374 | 11:29 |
| opendevreview | Merged openstack/etcd3gw master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/etcd3gw/+/1008378 | 11:47 |
| opendevreview | Merged openstack/automaton master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/automaton/+/1008381 | 11:47 |
| opendevreview | Merged openstack/taskflow master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/taskflow/+/1008380 | 11:59 |
| opendevreview | Merged openstack/cursive master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/cursive/+/1008377 | 12:12 |
| opendevreview | Merged openstack/tooz master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/tooz/+/1008379 | 12:14 |
| opendevreview | Stephen Finucane proposed openstack/tooz master: trivial: Change indentation of pyproject.toml https://review.opendev.org/c/openstack/tooz/+/1008431 | 12:17 |
| opendevreview | Stephen Finucane proposed openstack/tooz master: Migrate requirements to pyproject.toml https://review.opendev.org/c/openstack/tooz/+/1000832 | 12:17 |
| opendevreview | Merged openstack/stevedore master: Add cover target https://review.opendev.org/c/openstack/stevedore/+/1008382 | 12:27 |
| *** haleyb_ is now known as haleyb | 14:07 | |
| opendevreview | zehao proposed openstack/oslo.utils master: Fix mask_password corrupting valid JSON payloads https://review.opendev.org/c/openstack/oslo.utils/+/1008265 | 16:07 |
| opendevreview | zehao proposed openstack/oslo.utils master: Treat iso plus gpt as a bootable ISO, not an ambiguous image https://review.opendev.org/c/openstack/oslo.utils/+/1008287 | 16:07 |
| opendevreview | zehao proposed openstack/oslo.utils master: Fix mypy errors in StringToBytesTest scenario lists https://review.opendev.org/c/openstack/oslo.utils/+/1008497 | 16:07 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.rootwrap master: DNM: Run tests with Ubuntu 26.04 https://review.opendev.org/c/openstack/oslo.rootwrap/+/1007437 | 16:08 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.concurrency master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/oslo.concurrency/+/1008498 | 16:13 |
| rm_work | tkajinam: looking at your review, thanks | 16:28 |
| tkajinam | rm_work, ;-) | 16:31 |
| tkajinam | we might need that eventually, but I'd like to understand the current problem correctly (especially what is affected within current usage) to avoid adding more hacks for "threading in eventlet" | 16:31 |
| rm_work | Yeah I’m removing as much as I can actually | 16:31 |
| rm_work | Some day I can maybe show you exactly what I’m doing if you’d like to spend a few minutes | 16:32 |
| rm_work | Most of it revolves around HTTP-RPC | 16:32 |
| tkajinam | we spent some effort to it for heartbeat_in_thread but finally gave up that due to multiple problems there, as you may know | 16:32 |
| rm_work | it is lessons learned from both lots of testing with synthetic harnesses (but actually exercising the code against real tooz backends) and from real deployment testing in our staging cloud | 16:33 |
| rm_work | We are on 2026.1 mostly, with oslo.messaging and other oslo deps from master, which seems to work | 16:34 |
| rm_work | Will be on 2026.2 in like a week | 16:34 |
| tkajinam | sounds great | 16:34 |
| rm_work | I guess Furukawa-san is now out for a while so I am trying not to bother him for a bit 😇 | 16:41 |
| rm_work | but I am working on a much simplified branch of the messaging driver and a new approach for the consul driver because the complexity of both was getting out of hand and reviewing the code myself I was a little concerned with maintainability | 16:41 |
| rm_work | I think the third attempt looks much much better 😆 | 16:41 |
| tkajinam | yeah I hear him taking leave | 16:42 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Drop version check for eventlet < 0.38.2 https://review.opendev.org/c/openstack/oslo.log/+/1008500 | 16:44 |
| rm_work | Things will be much simpler when eventlet is truly gone everywhere, this intermediate state is painful 😓 | 16:45 |
| rm_work | I think his cloud runs an older version slightly, so I have been trying to keep eventlet working for him until they can upgrade, my own cloud is eventlet-free now I believe | 16:46 |
| rm_work | So a lot of this stuff is just compatibility to not disrupt his deployment | 16:46 |
| tkajinam | I think that's why they kept it inactive for a while and their plan at that time was to wait until eventlet is gone | 16:50 |
| rm_work | Yeah | 16:52 |
| rm_work | OK so this change you reviewed is required by version two of the RPC driver but not actually with my new v3 chain, which drops eventlet almost entirely | 16:53 |
| rm_work | So if you don’t want to merge this I’m happy to put it WIP and just use it as a depends-on for v2 until that chain dies | 16:53 |
| rm_work | Though it IS a real bug and I can update the CR, fix a few things per your comment, and file the bug | 16:54 |
| rm_work | but I don’t think I’ll strictly need it fixed for my work | 16:54 |
| tkajinam | ok | 16:54 |
| tkajinam | could you add a reference to oslo.messaging chain as an example of affected use case to that oslo.log fix ? | 16:56 |
| tkajinam | and any other use of threading you are aware of | 16:57 |
| tkajinam | (adding these references to the bug is also fine | 16:57 |
| tkajinam | we may want to review the current status of eventlet removal in this cycle and decide how much we may prioritize adding that hack | 16:57 |
| tkajinam | the discussion reminds me of large TODO of eventlet removal from masakari :-P | 16:58 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Drop workaround for Python < 3.11 https://review.opendev.org/c/openstack/oslo.log/+/1008505 | 17:16 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Drop workaround for Python < 3.11 https://review.opendev.org/c/openstack/oslo.log/+/1008505 | 17:19 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Drop version check for eventlet < 0.38.2 https://review.opendev.org/c/openstack/oslo.log/+/1008500 | 17:21 |
| opendevreview | Adam Harwell proposed openstack/oslo.log master: Make eventlet logging lock safe for native threads https://review.opendev.org/c/openstack/oslo.log/+/1007444 | 17:24 |
| stephenfin | dansmith: jfyi, I've requested your input on https://review.opendev.org/c/openstack/oslo.utils/+/1008287 when you get a chance. Not hugely urgent, but you know that code best | 17:34 |
| stephenfin | please and thanks 🙏 | 17:34 |
| JayF | stephenfin: I linked that in #openstack-ironic, I suspect we may have seen images shaped like this | 17:35 |
| dansmith | hrm, I thought we were already allowing those | 17:37 |
| dansmith | oh we are they just want to change the behavior of format(), but the goal was for people to use formats() and make their own determination of how to handle the ones it found | 17:38 |
| dansmith | stephenfin: replied, see if that makes sense | 17:48 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.config master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/oslo.config/+/1008510 | 17:54 |
| opendevreview | Merged openstack/oslo.log master: Fix missing context manager interface for _AsyncioMutex https://review.opendev.org/c/openstack/oslo.log/+/1007415 | 17:55 |
| opendevreview | Merged openstack/oslo.log master: Declare Python 3.15 support https://review.opendev.org/c/openstack/oslo.log/+/1007381 | 17:55 |
| opendevreview | Takashi Kajinami proposed openstack/debtcollector master: Reuse test commands in cover target https://review.opendev.org/c/openstack/debtcollector/+/1008511 | 17:58 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: Drop workaround for Python < 3.11 https://review.opendev.org/c/openstack/oslo.log/+/1008505 | 18:01 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.log master: opts: Remove deprecated aliases https://review.opendev.org/c/openstack/oslo.log/+/971272 | 18:04 |
| rm_work | tkajinam: Sure, I added them to the bug: https://bugs.launchpad.net/oslo.log/+bug/2169320 | 18:06 |
| rm_work | Our own case is the v2 driver, which will likely go away with v3. The rest came from an automated code search, so it may not be complete. Agreed on checking the Eventlet removal status first. | 18:06 |
| rm_work | to be honest I would ignore that oslo.log fix I guess if we're close to being fully free of eventlet | 18:29 |
| rm_work | rather you spend your time on more important stuff | 18:29 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: tooz service discovery https://review.opendev.org/c/openstack/oslo.messaging/+/1007620 | 19:17 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: wire protocol and client https://review.opendev.org/c/openstack/oslo.messaging/+/1007621 | 19:17 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: server side and entry point https://review.opendev.org/c/openstack/oslo.messaging/+/1007622 | 19:17 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: fanout broadcaster https://review.opendev.org/c/openstack/oslo.messaging/+/1007623 | 19:17 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: DevStack plugin, CI and docs https://review.opendev.org/c/openstack/oslo.messaging/+/1007624 | 19:17 |
| opendevreview | Adam Harwell proposed openstack/oslo.messaging master: HTTP-RPC driver: interop with the original driver https://review.opendev.org/c/openstack/oslo.messaging/+/1007625 | 19:17 |
| opendevreview | Merged openstack/oslo.concurrency master: Use pyproject.toml to maintain coverage options https://review.opendev.org/c/openstack/oslo.concurrency/+/1008498 | 19:26 |
| opendevreview | Merged openstack/oslo.utils master: Fix mypy errors in StringToBytesTest scenario lists https://review.opendev.org/c/openstack/oslo.utils/+/1008497 | 19:37 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul driver with distributed locking https://review.opendev.org/c/openstack/tooz/+/1007613 | 19:58 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul group membership and capabilities https://review.opendev.org/c/openstack/tooz/+/1007614 | 19:58 |
| opendevreview | Adam Harwell proposed openstack/tooz master: Add Consul service catalog reads https://review.opendev.org/c/openstack/tooz/+/1007615 | 19:58 |
| opendevreview | Merged openstack/oslo.log master: opts: Remove deprecated aliases https://review.opendev.org/c/openstack/oslo.log/+/971272 | 20:28 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!