| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-28 | |||
| 16:49:07 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 16:49:07 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 16:49:08 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migrations for admins https://review.opendev.org/c/openstack/nova/+/925771 | |
| 16:49:50 | r-taketn | ok. I'll rethink the strategy. It would be helpful if you could let us discuss it on the mailing list, etc. | |
| 16:50:34 | sean-k-mooney | you are definetly free too | |
| 16:50:37 | bauzas | sure, r-taketn are you familiar with our release cadence ? | |
| 16:51:10 | sean-k-mooney | this could also be a good topic for a future PTG | |
| 16:51:30 | sean-k-mooney | i know other are also interested in confindcial computing in general | |
| 16:51:53 | r-taketn | I have read and understand the Eproxy Release Schedule. | |
| 16:53:21 | bauzas | r-taketn: the PTG dates for the next release were recently announced | |
| 16:53:47 | bauzas | PTG is a virtual event occuring a couple of days where we usually chat about feature requests | |
| 16:54:13 | bauzas | as sean-k-mooney said, it could be one item worth speaking about | |
| 16:54:41 | bauzas | anyway, we're almost at time, so we need to wrap | |
| 16:54:45 | r-taketn | I would like to attend PTG and discuss it. | |
| 16:55:28 | fwiesel | ls | |
| 16:57:30 | bauzas | r-taketn: then please continue to look at the mailing-list, you'll see some emails about the next steps | |
| 16:58:22 | bauzas | our agenda is pretty free-form, as soon as we draft some ptg etherpad you should be able to add your topic | |
| 16:58:32 | bauzas | ok, that's it for now | |
| 16:58:34 | r-taketn | Ok. Thank you for discussing. | |
| 16:58:41 | bauzas | any other item to cover ? | |
| 16:59:08 | gibi | - | |
| 16:59:21 | bauzas | okay then thanks all | |
| 16:59:26 | bauzas | #endmeeting | |
| 16:59:26 | opendevmeet | Meeting ended Tue Jan 28 16:59:26 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:59:26 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-01-28-16.00.html | |
| 16:59:26 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-01-28-16.00.txt | |
| 16:59:26 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-01-28-16.00.log.html | |
| 16:59:46 | fwiesel | Thanks, and sorry for the noise. | |
| 16:59:56 | masahito | thank you | |
| 17:02:31 | gibi | thanks | |
| 17:12:47 | gibi | bauzas, dansmith, melwitt, sean-k-mooney: I just discovered that we missed the rebuild API action response when we considered the image properties in server details https://review.opendev.org/c/openstack/nova/+/939649/4#message-7d57a602cdd025a52c15c2470091baa6f0ee0340 | |
| 17:12:58 | gibi | would you be OK to amend the spec with this impact? | |
| 17:16:18 | gibi | I need to drop soon but I will read back | |
| 17:21:09 | priteau | bauzas and sean-k-mooney, could you please review this backport? Same as yesterday but for Bobcat: https://review.opendev.org/c/openstack/nova/+/940272 | |
| 17:21:59 | dviroel | gibi: i hit the same issue in https://review.opendev.org/c/openstack/nova/+/938604 - so the same decision should also impact the scheduler_hints spec... | |
| 17:25:53 | bauzas | gibi: I need to reload the context but if you ask for updating a spec post-approval-freeze, that's absolutely fine to me | |
| 17:29:45 | sean-k-mooney | priteau: +2 | |
| 17:53:46 | dansmith | gibi: yep, commented | |
| 18:05:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 18:05:11 | stephenfin | sean-k-mooney: took a different approach ^ | |
| 18:06:06 | sean-k-mooney | ya that could work as well. although if your only checkign the min version why is it not positional | |
| 18:06:45 | stephenfin | It effectively is. Do you mean using the forward slash operator? | |
| 18:06:47 | sean-k-mooney | this https://review.opendev.org/c/openstack/nova/+/940325/1/nova/api/openstack/api_version_request.py#300 does not need to be a kw arg | |
| 18:07:17 | stephenfin | I don't understand the question? Are you thinking Python here or Go? :D | |
| 18:07:41 | sean-k-mooney | in python | |
| 18:07:48 | sean-k-mooney | if we always expect to pass version | |
| 18:07:52 | sean-k-mooney | instead of is_supported(req, version=_MIN_API_VERSION): | |
| 18:07:56 | stephenfin | Oh | |
| 18:08:01 | sean-k-mooney | it should be is_supported(req, version): | |
| 18:08:20 | stephenfin | Okay, you were asking why does it take a default | |
| 18:08:21 | sean-k-mooney | you just dont call this if your not enforcing a min right? | |
| 18:08:31 | sean-k-mooney | yes | |
| 18:08:32 | stephenfin | yup, 100%. Good call | |
| 18:08:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only check minimum API version https://review.opendev.org/c/openstack/nova/+/940325 | |
| 18:08:57 | stephenfin | Done. Thanks | |
| 18:09:41 | sean-k-mooney | ok so for make you just using not min | |
| 18:09:47 | sean-k-mooney | ya that works | |
| 18:10:03 | stephenfin | Like I said, if you really want a max check just negate the call | |
| 18:10:25 | sean-k-mooney | well i was just reviewing https://review.opendev.org/c/openstack/nova/+/940325/2/nova/api/openstack/compute/create_backup.py#68 | |
| 18:10:28 | sean-k-mooney | where your doing that | |
| 18:10:52 | stephenfin | I'm nothing if not consistent | |
| 18:11:38 | sean-k-mooney | +1 for now i looked at the first couple of files and ill loop back to it once i have reveiewd the previosu patches | |
| 18:14:49 | sean-k-mooney | sicne your here why did you add a bunch of commans an reflow things in https://review.opendev.org/c/openstack/nova/+/936369/7/nova/api/validation/parameter_types.py | |
| 18:16:39 | stephenfin | sean-k-mooney: literally leaving a comment on that now. I ran ruff by mistake on the file. I thought I had undone most of it manually in a respin but apparently not (or I've undone the undo) | |
| 18:17:20 | stephenfin | If it's impossible to review, I can respin. If not, this code is *very* rarely touched in backports and therefore shouldn't cause issues down the line. I didn't do it in any other change that I'm aware of | |
| 18:17:21 | sean-k-mooney | ack, can ruff be configure to not add the trailing commas im pretty sure black can | |
| 18:17:39 | stephenfin | It can now. It was a preview feature until recently | |
| 18:17:49 | sean-k-mooney | its not imposibel to revieww it just dislike the stypel which i normally dont -1 for | |
| 18:18:13 | stephenfin | https://docs.astral.sh/ruff/settings/#format_skip-magic-trailing-comma | |
| 18:18:38 | sean-k-mooney | oh i dont mind the backport impact its trivial, its just annoying to read when it expand the enum in particalar | |
| 18:19:14 | sean-k-mooney | we are not actully using ruff, at least right now but it might be nice ot add that to our pyproject.toml anyway | |
| 19:19:03 | sean-k-mooney | stephenfin: i belive we have a bug in the openapi schma serise. you are asusmign that project id and user id have to be uuids but as afar as i cna tell looking at keystone and our own code they are not required to eb uuids | |
| 19:22:05 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 19:22:05 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 19:22:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 19:22:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 19:22:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 19:22:22 | mikal | sean-k-mooney: ^--- is a rebase and the tweaked USB controller patch. | |
| 20:11:40 | opendevreview | Merged openstack/nova master: Fix typo in release note https://review.opendev.org/c/openstack/nova/+/935681 | |
| 20:11:50 | opendevreview | Merged openstack/nova stable/2023.2: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/940272 | |
| 22:32:51 | opendevreview | Merged openstack/nova master: zuul: Add missing context comments for nova-next https://review.opendev.org/c/openstack/nova/+/935682 | |
| #openstack-nova - 2025-01-29 | |||
| 02:14:07 | opendevreview | melanie witt proposed openstack/nova master: pwr mgmt: power down free PCPUS when updating compute node https://review.opendev.org/c/openstack/nova/+/932926 | |
| 08:09:37 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Extend image-properties-in-server-show spec with rebuild https://review.opendev.org/c/openstack/nova-specs/+/940360 | |
| 08:12:11 | gibi | dansmith, melwitt, sean-k-mooney: thanks for the quick response. Here are the spec update ^^ | |
| 08:17:30 | masahito | gibi: hi, could you review the small bug fixes for the v2.96 or later API bug if you have time? It has been +2ed already so it needs one more W+1. https://review.opendev.org/c/openstack/nova/+/939673 and https://review.opendev.org/c/openstack/nova/+/939658 | |
| 08:21:46 | gibi | masahito: looking... | |
| 08:48:42 | gibi | masahito: I've approved the reproducer and left some API concerns in the fix. We probably need other cores to express opinion about the API impact. | |
| 08:49:03 | gibi | sean-k-mooney: ^^ | |
| 09:33:22 | masahito | gibi: thanks for the quick review :) | |
| 09:34:00 | mikal | sean-k-mooney: that SPICE patch series has now been updated and passes CI. | |
| 09:57:37 | opendevreview | Mark Goddard proposed openstack/nova master: ironic: Let Ironic handle deployment cleanup actions during destroy https://review.opendev.org/c/openstack/nova/+/883411 | |
| 10:02:55 | stephenfin | sean-k-mooney: feck :( | |
| 10:04:12 | stephenfin | sean-k-mooney: Yeah, that checks out. Let me think on it. If you're doing further reviews, maybe just ignore/call out the places I've made those assumption and move on. I'll rejig | |
| 10:05:17 | sean-k-mooney[m] | sure we have merged one already the flavor acess schema | |
| 10:05:23 | sean-k-mooney[m] | we should fix that | |
| 10:05:44 | sean-k-mooney[m] | but there are very few apis that take either a project id or user id | |
| 10:15:10 | stephenfin | sean-k-mooney[m]: Potential proposal. I'm going to drop https://review.opendev.org/c/openstack/nova/+/937013 from the stack and see how clean the rebase is | |
| 10:15:40 | stephenfin | If it's horrific, I might keep it and rejig the commit message to indicate that no, these things aren't UUIDs but they often look like UUIDs so this is a little more realistic?? | |
| 10:16:17 | sean-k-mooney[m] | sure, i would jsut assert that its a stiring between 1 and 64 printable characters | |
| 10:16:17 | stephenfin | Hopefully not an issue but I'd _love_ to avoid a multi-hour rebase if possible 😇 | |