| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-28 | |||
| 11:46:18 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | |
| 11:48:03 | Callum027 | sean-k-mooney: Created bug: https://bugs.launchpad.net/nova/+bug/2100588 - The current patch *should* be able to be cherry-picked into 2024.2 cleanly, I think 2024.1 will need some changes though | |
| 11:52:48 | sean-k-mooney | Callum027: ya we can asses that when we get to that point | |
| 12:56:19 | opendevreview | Tobias Urdin proposed openstack/nova master: wip: Reserve volumes when doing instance resize https://review.opendev.org/c/openstack/nova/+/942985 | |
| 14:43:30 | dansmith | sean-k-mooney: bauzas are you around to +W this? https://review.opendev.org/c/openstack/nova/+/845660 | |
| 14:43:50 | sean-k-mooney | sure | |
| 14:46:07 | sean-k-mooney | by the way i am around but ill be finishing early today i.e. in the next hour or so | |
| 14:46:24 | sean-k-mooney | dansmith: i assume your glance series is not going to be there this cycle | |
| 14:46:34 | sean-k-mooney | so we dont need to land the gpt change? | |
| 14:46:58 | sean-k-mooney | i.e. this https://review.opendev.org/c/openstack/nova/+/933928 | |
| 14:47:02 | dansmith | sean-k-mooney: yeah >:( .. the glance part of that has been sitting since november | |
| 14:47:28 | sean-k-mooney | this just future proofs us for when it is aviable | |
| 14:48:18 | sean-k-mooney | i assume there is no point in merging that in nova first? | |
| 14:48:54 | sean-k-mooney | i know you have a wip patch follwoing it | |
| 14:49:55 | dansmith | yeah I mean, if that other thing never merges (or someone complains about the format name or something) it would be kinda silly to have pre-merged that in nova | |
| 14:50:12 | dansmith | and it would be an easy backport if we needed to for some reason, so no real need to jump the gun I think | |
| 15:14:27 | opendevreview | Tobias Urdin proposed openstack/nova master: Retry more volume API functions on server error https://review.opendev.org/c/openstack/nova/+/942981 | |
| 15:14:28 | opendevreview | Tobias Urdin proposed openstack/nova master: wip: Reserve volumes when doing instance resize https://review.opendev.org/c/openstack/nova/+/942985 | |
| 17:06:19 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.96 for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942875 | |
| 17:13:12 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.98 doc/tests for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942878 | |
| 17:14:00 | opendevreview | Ghanshyam proposed openstack/nova master: Fix microversion 2.98 doc/tests for update/rebuild APIs https://review.opendev.org/c/openstack/nova/+/942878 | |
| #openstack-nova - 2025-03-01 | |||
| 06:16:04 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop direct dependency on iso8601 https://review.opendev.org/c/openstack/nova/+/943083 | |
| 07:58:07 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop direct dependency on iso8601 https://review.opendev.org/c/openstack/nova/+/943083 | |
| 14:09:11 | opendevreview | Merged openstack/nova master: libvirt: Add new option to enforce multipath volume connections https://review.opendev.org/c/openstack/nova/+/845660 | |
| 14:47:41 | tkajinam | \o/ | |
| 15:08:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 15:08:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 15:10:13 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 15:10:22 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| #openstack-nova - 2025-03-02 | |||
| 23:49:31 | opendevreview | ribaudr proposed openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 23:49:32 | opendevreview | ribaudr proposed openstack/nova master: Add live_migratable flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/942144 | |
| 23:49:32 | opendevreview | ribaudr proposed openstack/nova master: Augment the LiveMigrateData object https://review.opendev.org/c/openstack/nova/+/942143 | |
| 23:49:33 | opendevreview | ribaudr proposed openstack/nova master: Update conductor and filters allowing migration with SR-IOV devices https://review.opendev.org/c/openstack/nova/+/942146 | |
| 23:49:33 | opendevreview | ribaudr proposed openstack/nova master: Update manager to allow vfio pci device live migration https://review.opendev.org/c/openstack/nova/+/942145 | |
| 23:49:34 | opendevreview | ribaudr proposed openstack/nova master: FUP improve comment accuracy and variable naming for tag removal https://review.opendev.org/c/openstack/nova/+/943124 | |
| 23:49:34 | opendevreview | ribaudr proposed openstack/nova master: Update driver to map the targeted address for SR-IOV PCI devices https://review.opendev.org/c/openstack/nova/+/942147 | |
| #openstack-nova - 2025-03-03 | |||
| 00:09:49 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | |
| 03:00:27 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 05:03:53 | opendevreview | Callum Dickinson proposed openstack/nova master: Add more flavor metadata to libvirt guest XML https://review.opendev.org/c/openstack/nova/+/942974 | |
| 08:36:16 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop quota options from the DEFAULT section https://review.opendev.org/c/openstack/nova/+/943131 | |
| 09:04:33 | opendevreview | Tobias Urdin proposed openstack/nova master: wip: Reserve volumes when doing instance resize https://review.opendev.org/c/openstack/nova/+/942985 | |
| 09:08:23 | tobias-urdin | when testing above^ patch i just realized that you cannot reserve a cinder volume when status is in-use, based on the issue described there i guess the only two alternatives to solve the problem in that patch is either 1) add a simple check for volume status and live with the potential race condition between check of volume status and actual _terminate_volume_connections call | |
| 09:08:25 | tobias-urdin | or 2) catch InvalidVolume exceptions directly in _terminate_volume_connections and try to rollback volume attachments and then raise a InstanceFaultRollback exception, anybody else has any better ideas? | |
| 09:37:07 | opendevreview | ribaudr proposed openstack/nova master: Update driver to map the targeted address for SR-IOV PCI devices https://review.opendev.org/c/openstack/nova/+/942147 | |
| 09:37:07 | opendevreview | ribaudr proposed openstack/nova master: FUP improve comment accuracy and variable naming for tag removal https://review.opendev.org/c/openstack/nova/+/943124 | |
| 09:44:11 | sean-k-mooney | tobias-urdin: i have not looked into your patches yet but waht was the problem you were tryign to resolve? | |
| 09:44:34 | sean-k-mooney | tobias-urdin: i vagely saw them coming in last week but didnt have time to read them | |
| 09:46:02 | sean-k-mooney | tobias-urdin: oh ok... so thisi is kind of a cinder bug | |
| 09:46:34 | sean-k-mooney | tobias-urdin: so the problem is that cinder does not knwo that nova is in the middle of doing a resize and nova does not know that cidner wants to do a backup | |
| 09:46:35 | tobias-urdin | sean-k-mooney: if i try to resize and instance it goes to ERROR and needs operator to restore it (shutdown has already been called) because attachment_create() volume API raises InvalidVolume when an attached volume has backing-up status | |
| 09:46:40 | sean-k-mooney | so we can get in a broken state | |
| 09:47:05 | sean-k-mooney | tobias-urdin: well you can resize instnace with cinder volumes | |
| 09:47:21 | sean-k-mooney | that not the problem its because you also happen to have a cinder backup at the same time | |
| 09:48:42 | sean-k-mooney | tobias-urdin: both nova and cidner backup exepct to be the only thing modifying the volume state when doign a resize or backup | |
| 09:49:25 | tobias-urdin | yea, i could just loop volumes and check status in the context and raise InstanceFaultRollback i guess for resize to not hit it, but there would still be a race between me checking the status and attachment code being called in resize | |
| 09:49:46 | sean-k-mooney | that not really the fix | |
| 09:49:58 | tobias-urdin | i was hoping i could reserve the volumes before starting the operation, but my research failed because it didn't accommodate for in-use not being possible on in-use volume :( | |
| 09:50:06 | sean-k-mooney | we either need nova to annotate the volume with somethjing to indicate that we are migrating | |
| 09:50:21 | sean-k-mooney | or we need to have cidner check the status of attached instnace before doing a backup | |
| 09:51:09 | sean-k-mooney | tobias-urdin: is cidner backup changing the state of the volume to something other then inuse | |
| 09:51:22 | tobias-urdin | yes "backing-up" is set | |
| 09:51:30 | sean-k-mooney | right os that also a bug | |
| 09:52:17 | sean-k-mooney | once a volume is attached to a nova instnace we do not exect the voluem status to change unless its via a call to the nova api | |
| 09:52:42 | sean-k-mooney | so form a nova perspecitve it should remain in use | |
| 09:53:02 | sean-k-mooney | it would be fine to have a seperate task_state that backing-up | |
| 09:54:05 | sean-k-mooney | but if cidner want to modify it they need to make the attchment api treat backing-up the same as in-use | |
| 09:54:41 | tobias-urdin | that actually makes more sense | |
| 09:55:24 | sean-k-mooney | we have a bunch of task_state that we translate to "active" because semanticly external service should treat them the same | |
| 09:56:31 | sean-k-mooney | i dont know how to express that in cidner api but i think if they have internal states liek backing-up they may need a similar mechanisum | |
| 09:59:08 | sean-k-mooney | like this https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L46-L63 | |
| 10:03:21 | tobias-urdin | sean-k-mooney: thanks for the directions! i will investigate further and see if i can find a better path forward | |
| 10:04:03 | sean-k-mooney | ill try an comment on the bug and maybe a good dicusion for the ptg in a nova/cinder cross project if we dont have a path forward by then | |
| 10:08:49 | tobias-urdin | sean-k-mooney: ack, looks like they account for some migration logic in _attachment_reserve() https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L2409 | |
| 10:09:31 | tobias-urdin | so imo handling backing-up (if "previous status", that it sets back when backup is complete, is in-use) as if it were in-use makes sense | |
| 10:44:59 | sean-k-mooney | tobias-urdin: https://bugs.launchpad.net/nova/+bug/2077512/comments/5 i tried to summeriese my toughts and i added cinder to the bug | |
| 13:10:48 | tobias-urdin | sean-k-mooney: this is the real path forward https://review.opendev.org/c/openstack/cinder-specs/+/915973 instead of my new hack approach https://review.opendev.org/c/openstack/cinder/+/943176 | |
| 13:54:58 | opendevreview | Merged openstack/nova master: Update driver to deal with managed flag https://review.opendev.org/c/openstack/nova/+/938405 | |
| 14:04:29 | sean-k-mooney | tobias-urdin: ah yes that does seam to be adressign this edgecase | |
| 15:41:38 | cardoe | hello. hoping to get https://review.opendev.org/c/openstack/nova/+/942019 into 2025.1. This fixes an issue in 2024.1 and 2024.2 when you use Ironic and Nova is unable to deploy the error message is just the name of the OpenStack SDK object and the memory address instead of the actual error message. | |
| 15:49:34 | sean-k-mooney | cardoe: ah i skimmed that. im not sure if we want to merge that in the RC period but i think its mostly a vailid change | |
| 15:49:51 | sean-k-mooney | cardoe: my main question is did it report "No Error" | |
| 15:50:01 | sean-k-mooney | before when usign ironic client? | |
| 15:50:33 | cardoe | It printed an empty string | |
| 15:50:49 | cardoe | It felt awkward so I added that. | |
| 15:51:08 | sean-k-mooney | ok so im not sure if we would want to maintain the old behivor or not | |
| 15:51:27 | cardoe | It said "(deploy: power: some message storage: )" | |
| 15:51:28 | sean-k-mooney | i think no error is ok | |
| 15:51:40 | sean-k-mooney | oh | |
| 15:51:51 | sean-k-mooney | they ya its better with teh current version | |
| 15:52:09 | sean-k-mooney | wha ti was going to suggest is only inluciding the one that had an error | |
| 15:52:22 | sean-k-mooney | btu that could break people if they depeneded on all 3 being there | |
| 15:52:25 | cardoe | Now it says "(deploy: <openstack.response 0xDEADBEEF> power: <openstack.response 0xBEEFDEAD> storage: <openstack.response 0xFFFFFFFF)" | |
| 15:52:46 | cardoe | My patch makes it say "(deploy: No Error power: some message storage: No Error)" | |
| 15:53:03 | sean-k-mooney | ya which is an improvmenet over the old and current behavior | |
| 15:53:44 | cardoe | I was originally gonna drop the "deploy:" and "storage:" section if there wasn't a problem. But like you said, if people are expecting it I didn't wanna break them. | |
| 15:54:45 | sean-k-mooney | i have not looked at this in deaile but the other question i was wondering about is is this only looged or can this be returned to an end user | |
| 15:55:04 | cardoe | It's only logged today. I wanna return it to the user. | |
| 15:55:28 | sean-k-mooney | ok so the warning i was oging ot give is these mesagges come from ironic | |
| 15:55:35 | cardoe | I'm refactoring a few parts of that because today the user gets "Failed to prepare block device" as the error for nearly everything. | |
| 15:55:42 | sean-k-mooney | it many not be ok to show that to a user unless they are admin | |