| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-12-04 | |||
| 18:47:45 | sean-k-mooney | your pinning ot the oldes ubuntu supproted in each set | |
| 18:48:10 | sean-k-mooney | more or less alinging the job to the testing runtime | |
| 18:48:40 | gmann | yeah. we can remove the job definition for branches where it is not running | |
| 18:50:54 | sean-k-mooney | cool well lest see how your test patches go | |
| 19:24:48 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | |
| #openstack-nova - 2024-12-05 | |||
| 04:12:23 | genekuo | Hi, does anyone have experience with incorrect cpu model reported by nova? | |
| 04:12:56 | genekuo | I'm having nodes reporting EPYC-Rome, but the actual cpu model is EPYC-Milan | |
| 11:04:00 | stephenfin | sean-k-mooney: gibi: bauzas: If you do 'GET /os-quota-sets/foo', where 'foo' is a project ID that either doesn't exist or no longer exists, you get a HTTP 400. I personally expected a HTTP 404 since the resource I'm fetching is a quota set, and that just happens to be identified by project ID | |
| 11:04:30 | stephenfin | Ideally, I'd like to change it, but am I correct in thinking I can't do that without a microversion? | |
| 11:05:51 | stephenfin | I can obviously catch HTTP 400, but the problem is that (a) that covers a whole load of other things including schema validation failures (i.e. actually bad or invalid requests) and (b) I can't narrow it down since we translate messages in error responses | |
| 12:34:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Return 404 on bad project ID for 'os-quota-sets' https://review.opendev.org/c/openstack/nova/+/937125 | |
| 12:39:50 | stephenfin | That's the fix for same ^ | |
| 14:05:06 | ildikov | bauzas: Hi, I know you are very busy, so I wanted to reach out with a reminder to the patch that updates the processes in the Nova contributor documentation: https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:05:52 | opendevreview | Merged openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 14:05:53 | ildikov | bauzas: Since it describes a process that you've been instrumental in creating, it would be very helpful to have your input whether or not the proposed documentation change accurately describes the process or not. | |
| 14:56:32 | ratailor | gmann, Could you please review https://review.opendev.org/c/openstack/nova/+/925207 , need one more +2 | |
| 14:59:32 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Distributed Locking for Host Discovery`. https://review.opendev.org/c/openstack/nova-specs/+/936389 | |
| 15:01:25 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 15:27:08 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 15:39:39 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 16:06:49 | s3rj1k | Hi all, please take a look at proposed spec above ^, thanks | |
| 16:15:41 | melwitt | stephenfin: sorry to bug you (also thank you for the +2 🙂) but I uploaded a new PS for the tpool.Proxy patch to reference a bug number filed at https://review.opendev.org/c/openstack/nova/+/932669 if you could pls look again at your convenience | |
| 16:16:16 | stephenfin | done | |
| 16:16:36 | melwitt | thank you! | |
| 16:29:46 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 17:30:31 | opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | |
| 18:33:50 | opendevreview | Merged openstack/os-traits master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/os-traits/+/935582 | |
| 18:36:59 | opendevreview | Merged openstack/os-resource-classes master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/os-resource-classes/+/935578 | |
| 20:58:21 | opendevreview | Merged openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 23:22:12 | opendevreview | Merged openstack/nova master: api: Add response body schemas for versions APIs https://review.opendev.org/c/openstack/nova/+/924587 | |
| #openstack-nova - 2024-12-06 | |||
| 02:54:07 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add conf option for live migration monitor interval https://review.opendev.org/c/openstack/nova/+/937223 | |
| 03:13:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |
| 03:15:20 | opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/placement/+/935734 | |
| 07:59:18 | opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | |
| 10:52:05 | amorin | hey all, am I wrong saying that the availability-zone param set on boot is immutable? E.G. if I forgot to set it, I will end-up without any AZ pinned, nova will schedule my instance on a specific az, but I have no way to pin that later without delete/recreate? | |
| 10:55:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 10:55:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 10:55:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 11:35:24 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |
| 13:56:46 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:18:58 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:36:43 | ildikov | sean-k-mooney: I saw your comment in the contributor doc patch about updating the README.rst file in the root of the nova repo. I'm a little confused in terms of having both the index.rst and contributing.rst with similar information and most places in the different READMEs point to contributing.rst. | |
| 14:38:04 | ildikov | I updated the README as you suggested, but it is inconsistent with other references now. Should it still point to contributing.rst and have a direct link to process.rst in addition? | |
| 14:46:39 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:48:59 | ildikov | sean-k-mooney: gmann has a patch that adds a new link to the tracking etherpad in contributing.rst. I think that would be a better place to update links and references, and make sure that the changes are consistent throughout the Nova repo: https://review.opendev.org/c/openstack/nova/+/936776/1/doc/source/contributor/contributing.rst | |
| 14:49:05 | ildikov | What do you all think? | |
| 14:49:54 | sean-k-mooney | yep we can do that too | |
| 14:50:19 | sean-k-mooney | but part of the problemis its quite hard to find the process doc in the contibutor docs | |
| 14:50:28 | sean-k-mooney | there is a lot of text too read through first | |
| 14:50:47 | sean-k-mooney | so that is why i wated to add riectrly links to the process doc in the readme | |
| 14:51:23 | sean-k-mooney | *direct links | |
| 14:51:44 | sean-k-mooney | i dont want to link directly to the etherpad there | |
| 14:53:04 | sean-k-mooney | ildikov: developer will generally look at the git repo before anythign else so that why i wante to have this clear in the readme | |
| 14:53:47 | sean-k-mooney | we can do that as a sperate chagne if you want but it took me some time to find the prcoess doc when i wanted to find the links for the comment | |
| 14:56:30 | ildikov | sean-k-mooney: I don't question the value to add links to the process doc more directly, I'm personally good with that direction | |
| 14:56:51 | sean-k-mooney | ok ill just submit a patch for that later myself | |
| 14:56:56 | ildikov | based on pervious discussions, it doesn't seem like there's an overall agreement on what should be linked from where | |
| 14:57:22 | ildikov | and I would like to avoid holding up landing the process documentation, because people don't agree on where to put what link :) | |
| 14:57:45 | sean-k-mooney | my main concern is peopel still wont find the procss doc | |
| 14:58:16 | sean-k-mooney | the feed back we got was there was very littel docusmation for contibutors and as you have seen that is not the case | |
| 14:58:29 | sean-k-mooney | so +1 for makeign the exitsing docs correct | |
| 14:58:36 | ildikov | I can add the process doc link back into my patch, and leave the decision to another path whether or not to update the link to contributing.rst vs contributor/index.rst | |
| 14:58:37 | sean-k-mooney | but we need to also fix the discoveray aspect | |
| 14:58:53 | ildikov | would that be a good middle ground? | |
| 14:59:07 | sean-k-mooney | sure | |
| 14:59:12 | ildikov | ok, cool | |
| 14:59:18 | sean-k-mooney | findig that process link was actully what took time | |
| 14:59:29 | ildikov | and yes, I totally agree on the discoverability aspect | |
| 14:59:30 | sean-k-mooney | so that the part i find hard to discover | |
| 14:59:39 | ildikov | +1 | |
| 15:01:32 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:02:35 | ildikov | sean-k-mooney: done ^^ | |
| 15:03:08 | ildikov | I also consolidated the template part in process.rst, I missed your comment on that earlier, apologies for the delay on that. | |
| 15:03:41 | sean-k-mooney | no worries | |
| 15:04:08 | sean-k-mooney | im ok with the current version | |
| 15:04:21 | ildikov | awesome, thank you for checking! | |
| 15:04:25 | sean-k-mooney | i was ok without consolidating them by the way | |
| 15:04:51 | ildikov | ah ok, I wasn't sure | |
| 15:05:14 | ildikov | with the comment from bauzas, I think it's better consolidated | |
| 15:05:27 | sean-k-mooney | i wanted the template but not pushed on which is more readable | |
| 15:05:48 | sean-k-mooney | so if sylvain has a prefence thats fine by me | |
| 15:13:31 | ildikov | sounds good | |
| 15:13:32 | ildikov | I'll keep an eye out for his review and see if there are any more updates he would like to have | |
| 15:16:44 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:17:35 | ildikov | it looks like I got the formatting wrong in rst, haven't done rst in a while so I'm a bit rusty :) | |
| 15:23:34 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 15:23:35 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 15:24:29 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 15:24:29 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 15:30:29 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:35:19 | gibi | melwitt: thanks for volunteering of looking at the placement a_c explosion fix. The first patch of the series in https://review.opendev.org/q/topic:%22bug/2070257%22 is ready for review. The second is also already working but I still intended to clean it up so that is WIP but any feedback is highly appreciated | |
| 15:37:09 | gibi | melwitt: I'm working on getting timing and memory numbers for comparision of before / after the fix and I will add that to the commit message as soon as I can convince my test node to build not just 4 but 8 devstack VMs | |
| 16:20:09 | opendevreview | Merged openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | |
| 17:39:55 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 17:39:56 | opendevreview | Balazs Gibizer proposed openstack/placement master: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/937273 | |
| 17:39:56 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | |
| 17:41:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | |
| 18:00:08 | gmann | ildikov: current version lgtm, thanks for updates | |
| 18:24:47 | ildikov | gmann: great, thank you for checking! | |
| 18:38:43 | opendevreview | Merged openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |