| opendevreview | sean mooney proposed openstack/nova master: reorder and extend pre-commit hooks https://review.opendev.org/c/openstack/nova/+/949299 | 00:35 |
|---|---|---|
| opendevreview | sean mooney proposed openstack/nova master: enable bandit and fix high and medium severity issues https://review.opendev.org/c/openstack/nova/+/949300 | 00:35 |
| opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | 06:17 |
| Uggla | fungi, sean-k-mooney tbh I have just noticed the topic is incorrect. fungi it is ok to change it if you can. However there is no hurry it can wait Tuesday and we can ask gmaan too as he seems to be op on this channel. | 07:02 |
| gibi | sean-k-mooney: thank for the review on the pooling. I replied in the review, including your valid comment about the fixtures as well there. | 07:20 |
| *** ChanServ changes topic to "This channel is for Nova development | development-planning: https://etherpad.opendev.org/p/nova-2025.2-status | this channel is logged at https://meetings.opendev.org/irclogs/%23openstack-nova/" | 07:32 | |
| frickler | Uggla: ^^ please check | 07:32 |
| Uggla | hi frickler ! Thx that looks good. | 07:34 |
| ishanwar[m] | sean-k-mooney: dansmith updated the commit title and description as requested, got a +1 from Zuul as well. | 08:27 |
| ishanwar[m] | https://review.opendev.org/c/openstack/nova/+/945204 Can you review ? | 08:27 |
| ishanwar[m] | melwitt sorry about the spam earlier, I had lost the change ID of my patches, also I was unaware of the format of a commit message in gerrit. I am quite used to github, so this is a bit new to me | 08:27 |
| hertrste | Hi. Can we have another round of review for https://review.opendev.org/c/openstack/nova-specs/+/945549? I think I addressed all the issues discussed in the PTG. | 08:39 |
| sean-k-mooney | gibi: so having an absraction and not copy pasting was the motivation for using functools.cache. we can writeh our own but since one is provided in the std lib i was advocating for using it to minimise our techdebt. with that said a reusable pool manager is pretty trivial to write so if you want to do that then ok | 10:12 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Remove openSUSE/SLES from install guide https://review.opendev.org/c/openstack/nova/+/949324 | 10:21 |
| sean-k-mooney | hertrste: ah i see you set up a ppa to provide the libvirt supprot for cloud hypervioer | 10:22 |
| sean-k-mooney | hertrste: that was going to be my main quiestion. have you figroued out how you will enable devstack (in ci and or locally) to deploy cloud cloud hypervioer and libvirt? | 10:23 |
| *** haleyb is now known as haleyb|out | 12:44 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | 13:27 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | 13:27 |
| ishanwar[m] | Hey sean-k-mooney | 14:12 |
| ishanwar[m] | can you have a look for this https://review.opendev.org/c/openstack/nova/+/945204 | 14:12 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | 14:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | 14:39 |
| opendevreview | Merged openstack/nova stable/2025.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/948061 | 17:20 |
| *** __ministry is now known as Guest15503 | 22:12 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!