| opendevreview | Adam McArthur proposed openstack/placement master: Placement spec for bulk read API of resource providers https://review.opendev.org/c/openstack/placement/+/990761 | 00:10 |
|---|---|---|
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Add hot attach/detach support for virtiofs shares https://review.opendev.org/c/openstack/nova/+/1008154 | 02:14 |
| opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | 07:32 |
| opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | 07:34 |
| opendevreview | Juan Larriba proposed openstack/nova master: Add OpenTelemetry tracing middleware for Nova https://review.opendev.org/c/openstack/nova/+/982871 | 07:39 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | 08:11 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | 08:16 |
| opendevreview | Fabian Wiesel proposed openstack/nova master: Clean up stuck/leaked volumes in BFV scenarios on failed build or early deletion https://review.opendev.org/c/openstack/nova/+/1008215 | 09:19 |
| bauzas | gibi: thanks gibi for your excellent comments on https://review.opendev.org/c/openstack/nova/+/1006786/3 currently working on another revision | 09:29 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Guard against deleting a concurrent request's BDM on timeout https://review.opendev.org/c/openstack/nova/+/1005727 | 10:16 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Prevent concurrent attach_volume with task_state guard https://review.opendev.org/c/openstack/nova/+/1006786 | 10:16 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Reject stale reserve_block_device_name RPC on compute https://review.opendev.org/c/openstack/nova/+/1006011 | 10:16 |
| gibi | bauzas: ack | 10:31 |
| gibi | sorry for not spotting that sooner | 10:31 |
| bauzas | don't be sorry, I never saw it | 10:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2026.2: CellDatabases: replace shared lock with thread-local cell routing https://review.opendev.org/c/openstack/nova/+/1008257 | 11:55 |
| gibi | melwitt: bauzas: elodilles: ^^ this helped avoiding the functional / cover job hanging so lets add it to 2026.2 as well ^^ | 11:56 |
| gibi | I +Ad https://review.opendev.org/c/openstack/nova/+/1007996 making tox cover voting again, I will propose a backport for that as well once it lands on master | 11:57 |
| elodilles | gibi: +2'd | 12:33 |
| gibi | cool, thanks | 12:36 |
| elodilles | np o/ | 12:36 |
| gibi | bauzas: there is a new ~bug in https://review.opendev.org/c/openstack/nova/+/1006786 sorry :) | 12:57 |
| bauzas | doh | 12:57 |
| gibi | but we are getting there | 12:57 |
| bauzas | sorry I'm neither a notification nor a BDM SME :) | 12:58 |
| Uggla | dansmith, adamcarthur5, I agree Dan, we already had such kind of request and this is tricky regarding performances. So we might not need a X session with Ironic, but discussing the use case at the PTG sounds a good start. | 12:58 |
| gibi | no worries this new one is just a try block restructuring issue :) | 12:58 |
| bauzas | gibi: thanks, I'll provide a new rev :) | 13:00 |
| bauzas | gibi++ | 13:00 |
| gibi | and I will re-review it :) | 13:01 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Prevent concurrent attach_volume with task_state guard https://review.opendev.org/c/openstack/nova/+/1006786 | 13:19 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Reject stale reserve_block_device_name RPC on compute https://review.opendev.org/c/openstack/nova/+/1006011 | 13:19 |
| gibi | bauzas: thanks +2 | 13:31 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | 15:08 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run periodics outside compute test callbacks https://review.opendev.org/c/openstack/nova/+/1007150 | 15:36 |
| gibi | gmaan: smabork: I respin https://review.opendev.org/c/openstack/nova/+/1007150 with the run_periodics improvement | 16:06 |
| gibi | I've respun (?) | 16:07 |
| gibi | anyhow I fixed the oustanding comment in that patch :) | 16:08 |
| *** EugenMayer4401800 is now known as EugenMayer440180 | 16:26 | |
| gmaan | gibi: ack | 16:57 |
| opendevreview | Merged openstack/nova master: Add pyroute2 in doc dependencies https://review.opendev.org/c/openstack/nova/+/1004781 | 17:25 |
| opendevreview | Merged openstack/nova stable/2026.2: CellDatabases: replace shared lock with thread-local cell routing https://review.opendev.org/c/openstack/nova/+/1008257 | 17:49 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Stop hanging the test if threadpool executor is stuck in cleanup https://review.opendev.org/c/openstack/nova/+/1006351 | 17:49 |
| dansmith | melwitt: re-reading your comment about naming it V2, I thought you had maybe not grokked (or not grokked when you wrote that) that this'd be a permanent replacement eventually | 18:36 |
| dansmith | but maybe you _did_ understand that and still don't think that it needs "v2" or "new" or "now better than ever" in the name | 18:37 |
| dansmith | it seems like it might be hard to explain or confuse people if we don't, but still valid if that's your preference | 18:37 |
| dansmith | I'd think it becomes one of those things where it's hard for them to remember which is the newer one they should be building on when they just look at the name. obviously if they look at the microversion history it'll be clear, but... | 18:38 |
| melwitt | dansmith: yes I did understand it to be an eventual replacement. I don't have a rigid opinion, I was just thinking about when it _is_ the only available API it being named "V2" in the rest API is kinda eh ... but again not feeling that certain about it. it's true we already have like "block_device_mapping_v2" and it would be like that | 18:39 |
| dansmith | yes, I both hate the "only one and it's v2" and I also hate block_device_mapping_v2 as well | 18:41 |
| melwitt | if the API were not intended to be an eventual replacement I would not be supportive of the new API idea heh | 18:41 |
| dansmith | it's just one of those things where the v2 is an ugly wart but "block_device_mapping" and device_mapping_block" would be way too confusing trying to remember which one was new | 18:41 |
| dansmith | anyway, just wanted to make sure we're on the same page, and sounds like we are | 18:42 |
| melwitt | sure | 18:42 |
| opendevreview | Michael Sherman proposed openstack/nova master: ironic: Reproduce node leak after local delete https://review.opendev.org/c/openstack/nova/+/1008334 | 19:21 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | 19:47 |
| opendevreview | Merged openstack/nova master: Revert "Make tox cover non voting to unblock RC1" https://review.opendev.org/c/openstack/nova/+/1007996 | 20:06 |
| adamcarthur5 | Uggla dansmith sean-k-mooney thank you for your comments, here and on the review. I will work through them. In terms of the PTG, is there an ideal date/time that I should suggest in the etherpad? In terms of performance, I agree with you. I am happy to work on performance testing for such a change, I would just need some help for making the right | 20:41 |
| adamcarthur5 | assumptions when I benchmark. | 20:41 |
| adamcarthur5 | If benchmarking potential changes would be something useful pre-PTG, I can work on that? I.e. maybe the implementation is not quite right, but we can get a sense of performance? Does that make sense? | 20:42 |
| JayF | adamcarthur5: I can't speak for how nova does things, but if you're getting reviews on your spec and making design progress async, a PTG session may be less needed. I'd suggest prioritizing quick roundtrips on those reviews. | 20:54 |
| adamcarthur5 | JayF I was basing it off what they said early in the chat ^^^^ | 20:55 |
| adamcarthur5 | But yes quick turn around on the reviews is also the plan | 20:55 |
| JayF | I mainly wanted you to know if your spec is getting reviewed, you're already making good progress :) | 20:55 |
| adamcarthur5 | 👍 | 20:56 |
| opendevreview | Adam McArthur proposed openstack/nova master: Use safe allocation deletion on instance delete https://review.opendev.org/c/openstack/nova/+/1003208 | 21:13 |
| *** seebaer is now known as seba | 21:15 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!