| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-08 | |||
| 17:21:50 | sean-k-mooney | so they are not really familary with workign in openstack and i think git | |
| 17:22:06 | sean-k-mooney | but ya the commit message is not forllwoing or normal convetion | |
| 17:22:25 | melwitt | yeah. it's such a tiny thing to fix I thought might as well + backport it. I agree with you that the long term thing will be to stop formatting eph disks | |
| 17:22:25 | sean-k-mooney | is more like what you do on mailing list where for each revision to expalin what you changed form the last one | |
| 17:22:34 | melwitt | sean-k-mooney: the commit message claims stuff that isn't in the patch though, like what is that | |
| 17:22:46 | melwitt | huh ok | |
| 17:22:57 | sean-k-mooney | melwitt: so it explainging what has change over time | |
| 17:23:06 | sean-k-mooney | whic is fine in a mailing list btu weired in gerrit | |
| 17:23:24 | sean-k-mooney | also the lost the change id so https://review.opendev.org/c/openstack/nova/+/945204/6 | |
| 17:23:31 | sean-k-mooney | is actully the review they shoudl of updated | |
| 17:23:31 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks [FIX]: added error handling for .get() statements [FIX]: pep8 formatting fixed for changes [FIX]: added additional unit tests changes to mock validate_networks [FIX]: fixes using autopep8 https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:23:49 | melwitt | I see | |
| 17:24:31 | sean-k-mooney | ishanwar[m]: so you now have two reviews for the same thing because you lost the change id at one point | |
| 17:25:34 | sean-k-mooney | ishanwar[m]: i think you ment to locally squash the two pathces before doing git review | |
| 17:25:47 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks [FIX]: added error handling for .get() statements [FIX]: pep8 formatting fixed for changes [FIX]: added additional unit tests changes to mock validate_networks [FIX]: fixes using autopep8 https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:27:03 | sean-k-mooney | ishanwar[m]: ok ^ is better but as i was dissusing with melwitt above you shoudl update the commit message to expalin what the current revion of the patch is changing not how the patch changed over time | |
| 17:27:32 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks [FIX]: added error handling for .get() statements [FIX]: pep8 formatting fixed for changes [FIX]: added additional unit tests changes to mock validate_networks [FIX]: fixes using autopep8 https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:27:55 | sean-k-mooney | ishanwar[m]: https://wiki.openstack.org/wiki/GitCommitMessages document the convetions we try to follow | |
| 17:27:59 | fungi | sean-k-mooney: i can update it, though separately i recommend the nova team consider adding some channel operators at https://opendev.org/openstack/project-config/src/branch/master/accessbot/channels.yaml#L239 | |
| 17:28:18 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks [FIX]: added error handling for .get() statements [FIX]: pep8 formatting fixed for changes [FIX]: added additional unit tests changes to mock validate_networks [FIX]: fixes using autopep8 https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:28:49 | sean-k-mooney | fungi: oh i didnt know we could do that. i assuem we cant just update the topic via a git reivew? | |
| 17:29:10 | sean-k-mooney | fungi: i htink we used to be able to do this in the chanel before we moved to oftc | |
| 17:29:19 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks [FIX]: added error handling for .get() statements [FIX]: pep8 formatting fixed for changes [FIX]: added additional unit tests changes to mock validate_networks [FIX]: fixes using autopep8 https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:29:26 | sean-k-mooney | but its been so long i kind of forget the detials | |
| 17:30:18 | ishanwar[m] | sean-k-mooney: i have fixes the pep8 issue kindly have a look whenver you can | |
| 17:30:28 | fungi | sean-k-mooney: you'd add people to an ops list for the channel in that file, then any of them can ask chanserv to set the channel topic like the example in https://docs.opendev.org/opendev/system-config/latest/irc.html#basic-channel-operator-commands | |
| 17:31:03 | sean-k-mooney | fungi: ah cool ill add that to the irc metting adgenda for next week so | |
| 17:31:19 | sean-k-mooney | and we can see what folks think | |
| 17:31:26 | ishanwar[m] | sean-k-mooney: yep changing that | |
| 17:31:51 | fungi | but also if Uggla wants the topic updated in the meantime, i'm happy to do that myself too | |
| 17:34:39 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: This feat optimises the validate_networks method by reducing networks calls to the database. Here we utilise the quota_details extension to gather the list of uses ports using the neutron extension, this is a smarter query as we don't have to list down all the ports which are in use. Instead we now utilise the quota details extension to get the count of used ports directly. ht | |
| 17:39:36 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:40:28 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: This feat optimises the validate_networks method by reducing networks calls to the database. Here we utilise the quota_details extension to gather the list of uses ports using the neutron extension, this is a smarter query as we don't have to list down all the ports which are in use. Instead we now utilise the quota details extension to get the count of used ports directly. ht | |
| 17:41:15 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | |
| 17:41:30 | ishanwar[m] | Sorry for the spam everyone | |
| 17:41:56 | ishanwar[m] | i didn't realize opendev has been sending a notification here all this time | |
| 17:42:23 | sean-k-mooney | ya it does it also send an email to those on the review | |
| 17:43:02 | sean-k-mooney | its not that distruptive but its goit to do those iteration locally and then push a reivew | |
| 17:43:09 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: This feat optimises the validate_networks method by reducing networks calls to the database. Here we utilise the quota_details extension to gather the list of uses ports using the neutron extension, this is a smarter query as we don't have to list down all the ports which are in use. Instead we now utilise the quota details extension to get the count of used ports directly. ht | |
| 17:43:28 | sean-k-mooney | it avoids the spam and the ci churn but that comes with practice | |
| 17:44:31 | ishanwar[m] | yeah sorry about that again, will be more careful from now on. | |
| 17:44:31 | ishanwar[m] | I think i have made all the changes i had to right now | |
| 17:44:31 | ishanwar[m] | you can have a look, whenever you can | |
| 17:57:38 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Use common naming convention for ephemeral disk labels https://review.opendev.org/c/openstack/nova/+/947541 | |
| 17:58:29 | melwitt | sean-k-mooney: added code comment to the test and a release note ^ | |
| 18:00:13 | opendevreview | Merged openstack/nova master: wsgi: Don't create, use lock in same line https://review.opendev.org/c/openstack/nova/+/949161 | |
| 22:52:07 | opendevreview | sean mooney proposed openstack/nova master: update pre-commit version pins https://review.opendev.org/c/openstack/nova/+/949291 | |
| 22:52:07 | opendevreview | sean mooney proposed openstack/nova master: Allow autopep8 to fix more things https://review.opendev.org/c/openstack/nova/+/949292 | |
| #openstack-nova - 2025-05-09 | |||
| 00:35:17 | opendevreview | sean mooney proposed openstack/nova master: reorder and extend pre-commit hooks https://review.opendev.org/c/openstack/nova/+/949299 | |
| 00:35:18 | opendevreview | sean mooney proposed openstack/nova master: enable bandit and fix high and medium severity issues https://review.opendev.org/c/openstack/nova/+/949300 | |
| 06:17:01 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | |
| 07:02:37 | 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:20:55 | 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:32:59 | frickler | Uggla: ^^ please check | |
| 07:34:55 | Uggla | hi frickler ! Thx that looks good. | |
| 08:27:17 | ishanwar[m] | sean-k-mooney: dansmith updated the commit title and description as requested, got a +1 from Zuul as well. | |
| 08:27:17 | ishanwar[m] | https://review.opendev.org/c/openstack/nova/+/945204 Can you review ? | |
| 08:27:17 | 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:39:53 | 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. | |
| 10:12:47 | 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:21:51 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove openSUSE/SLES from install guide https://review.opendev.org/c/openstack/nova/+/949324 | |
| 10:22:20 | sean-k-mooney | hertrste: ah i see you set up a ppa to provide the libvirt supprot for cloud hypervioer | |
| 10:23:20 | 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? | |
| 13:27:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 13:27:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 13:27:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 13:27:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 13:27:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 13:27:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 13:27:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 13:27:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 13:27:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 13:27:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 13:27:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 13:27:51 | 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:12:27 | ishanwar[m] | Hey sean-k-mooney | |
| 14:12:27 | ishanwar[m] | can you have a look for this https://review.opendev.org/c/openstack/nova/+/945204 | |
| 14:39:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 14:39:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 14:39:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 14:39:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 14:39:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 14:39:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 14:39:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 14:39:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 14:39:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 14:39:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 14:39:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 14:39: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 | |
| 14:39:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 17:20:43 | 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 | |
| #openstack-nova - 2025-05-12 | |||
| 08:24:47 | mikal | sean-k-mooney: you around? I realised today that there hasn't yet been an os-traits release for the USB and sound compute traits. I can haz release plz? | |
| 11:01:33 | stephenfin | mikal: Sean will be around later today, alright. However, you can propose a release yourself if there isn't one already. You want the openstack/releases repo | |
| 11:02:08 | stephenfin | bauzas: gibi: In Sean's absence, have either of you any idea what the workaround is that they're talking about here? https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/2065685 | |
| 11:02:45 | stephenfin | I'm trying to bump the Ubuntu versions used for both Gophercloud and k8s cloud-provider-openstack and am running into that issue, e.g. https://github.com/gophercloud/gophercloud/actions/runs/14969772718/job/42047885458?pr=3374 | |
| 11:07:47 | gibi | stephenfin: https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/2065685/comments/9 I think this manual pathing what Sean refers to | |
| 11:08:46 | stephenfin | Do you know if we're doing that though? codesearch.o.o doesn't turn up any relevant hints (fwict) for the bug ID or the packages | |
| 11:09:50 | stephenfin | No worries if not. I was expecting to find something in our .zuul.yaml or the DevStack git log but no dice | |
| 11:10:04 | gibi | I dont know | |
| 11:10:39 | gibi | but I guess we have noble jobs so we should either see the issue or have some fix | |