| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-12-03 | |||
| 16:16:51 | bauzas | thanks all | |
| 16:16:56 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-12-03-16.00.log.html | |
| 16:16:56 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-12-03-16.00.txt | |
| 16:16:56 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-12-03-16.00.html | |
| 16:16:56 | opendevmeet | Meeting ended Tue Dec 3 16:16:56 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:16:56 | bauzas | #endmeeting | |
| 16:18:28 | s3rj1k | Do we have something (spec, rfes, bugs) related to graceful shutdown of computes? | |
| 22:29:58 | gmann | stephenfin: ack | |
| #openstack-nova - 2024-12-04 | |||
| 02:14:53 | opendevreview | Masanori Kuroha proposed openstack/nova master: Retrieve the image metadata from instance.image_meta during resize https://review.opendev.org/c/openstack/nova/+/936910 | |
| 07:31:10 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Add support for preserve/delete port on vm delete https://review.opendev.org/c/openstack/nova-specs/+/936990 | |
| 10:10:25 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Add support for preserve/delete port on vm delete https://review.opendev.org/c/openstack/nova-specs/+/936990 | |
| 13:13:25 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | |
| 13:13:25 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 15:19:37 | ratailor | sean-k-mooney, could you please review https://review.opendev.org/c/openstack/nova/+/873901 , need one more +2 | |
| 15:19:49 | ratailor | gibi, sean-k-mooney please review this as well https://review.opendev.org/c/openstack/nova/+/911108 | |
| 15:20:47 | sean-k-mooney | the secodn one im not sure if we should restre the orginal AZ | |
| 15:21:30 | sean-k-mooney | i dont nessiarly agree with the epxecation in the bug reprot https://bugs.launchpad.net/nova/+bug/1978573 | |
| 15:22:35 | sean-k-mooney | you can argure that either is correct so i watn input form other on why we shoudl revert vs not | |
| 15:23:29 | sean-k-mooney | the first one im more ok with. ill see if i can review | |
| 15:24:04 | ratailor | sean-k-mooney, ack. Thanks! could you please comment on restore original AZ patch for initiating discussion ? | |
| 15:24:29 | sean-k-mooney | sure | |
| 15:24:35 | ratailor | Thanks! | |
| 15:51:30 | sean-k-mooney | ok i left comments on all 3 patches | |
| 17:50:14 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 17:50:14 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 18:14:26 | opendevreview | Merged openstack/osc-placement master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/osc-placement/+/935579 | |
| 18:27:49 | sean-k-mooney | gmann: that test_release job is on focal... | |
| 18:27:51 | sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4cd/935579/2/gate/test-release-openstack/4cd9a9d/job-output.txt | |
| 18:28:03 | sean-k-mooney | what even is that? | |
| 18:28:20 | sean-k-mooney | i assume its building a wheel or something but that not defiend in os-vif is it | |
| 18:28:31 | sean-k-mooney | its presumable part of a shared template | |
| 18:29:00 | gmann | sean-k-mooney: let me check | |
| 18:29:17 | gmann | I am not sure we moved test-release-openstack job to latest distro version | |
| 18:30:28 | sean-k-mooney | so its defiend in project-config | |
| 18:30:43 | gmann | sean-k-mooney: mainly it is running because of change in setup.cfg | |
| 18:30:55 | sean-k-mooney | https://github.com/openstack/project-config/blob/master/zuul.d/jobs.yaml#L50-L63 | |
| 18:31:00 | gmann | but it was running fine in other projects | |
| 18:31:02 | gmann | yeah | |
| 18:31:14 | sean-k-mooney | so the issue is that is pinned to ubuntu-focal | |
| 18:31:18 | gmann | let me move it to Noble and see but even on focal it should pass | |
| 18:31:21 | sean-k-mooney | it passed the second tim | |
| 18:31:31 | gmann | yeah | |
| 18:31:32 | sean-k-mooney | but ya we proablly shoudl update it | |
| 18:31:44 | gmann | yup, doing that | |
| 18:32:55 | sean-k-mooney | looking at the failure https://zuul.opendev.org/t/openstack/build/58b699c223834b00a2755ee35b473a2c/log/job-output.txt#832 | |
| 18:33:21 | sean-k-mooney | im guessing this si related to the versions aviable in the mirros | |
| 18:33:32 | gmann | yeah, seems so | |
| 18:33:59 | sean-k-mooney | it failled in ovh-gra1 and passed in rax-iad | |
| 18:41:23 | opendevreview | Ghanshyam proposed openstack/os-vif master: test test-release-openstack job on noble https://review.opendev.org/c/openstack/os-vif/+/937055 | |
| 18:41:50 | opendevreview | Ghanshyam proposed openstack/os-vif stable/2024.2: test test-release-openstack job on noble https://review.opendev.org/c/openstack/os-vif/+/937056 | |
| 18:42:10 | gmann | sean-k-mooney: https://review.opendev.org/c/openstack/project-config/+/937053 | |
| 18:44:47 | sean-k-mooney | oh you going ot keep focal for the older branches. | |
| 18:45:34 | sean-k-mooney | i woudl proably move for stable and only pin to focal on unmaintained | |
| 18:45:40 | sean-k-mooney | actully | |
| 18:45:58 | sean-k-mooney | unmaintained proably does not need that job? | |
| 18:46:36 | sean-k-mooney | i guess we still push to tarballs.openstack.org for unmaintiend we just dont ot the rest of the release processs | |
| 18:47:25 | sean-k-mooney | ok ya i think i agree with the approch you took in the patch | |
| 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 (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 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: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? | |