| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-19 | |||
| 18:36:35 | sean-k-mooney | the reason it snot that way, beyond legacy | |
| 18:36:55 | sean-k-mooney | is as a end user you dont knwo if the strage backend is accable from the current host | |
| 18:37:41 | sean-k-mooney | its tricky | |
| 18:38:35 | sshmlbrg | ok, i hope i got it, need to summarize it. Is it bug or ...? | |
| 18:49:35 | sean-k-mooney | its proably a but im just not sure how to fix it | |
| 18:50:06 | sean-k-mooney | im inclided to say pass false and then requrie pepoel to unshlve to the same az fi thats how the clodu is confiugred | |
| 18:51:08 | sean-k-mooney | we shoudl have a better solution to this, that the problem with config driven api bevhior and why we dont add it anymore if we can avoid it at all costs | |
| 18:53:30 | sshmlbrg | I was thinking about storing the old AZ, writing the AZ to it when shelving, and releasing it when unshelving. Then, when attaching the volume, we could check that specific availability zone. | |
| 18:54:04 | sean-k-mooney | so that not correct | |
| 18:54:25 | sean-k-mooney | instnace that dont request a az shoudl not be pinned to one by default | |
| 18:54:32 | sshmlbrg | understand why | |
| 18:55:20 | sean-k-mooney | if they have a cidner volume and land on a host with cross_az_attach=false and it does not violate the requirement nova will pin it | |
| 18:56:34 | sean-k-mooney | sshmlbrg: did the instance have a voluem before it was shelved? | |
| 18:56:50 | sshmlbrg | yes, volume-backed instance | |
| 18:57:03 | sean-k-mooney | ok then it should be pinned already | |
| 18:57:21 | sean-k-mooney | so the bug is that its proably not usign the az form the request spec if its not set in the host | |
| 18:57:28 | sean-k-mooney | * in the instance table | |
| 18:58:07 | sshmlbrg | yes | |
| 18:58:37 | sean-k-mooney | ok so tha that specific edge case is fixable | |
| 18:59:22 | sean-k-mooney | we proably cant fix the case where it did not have a volume before it was shelved eaisly | |
| 18:59:32 | sean-k-mooney | but we coudl fix this case. | |
| 18:59:46 | sean-k-mooney | can you write that up as a bug | |
| 18:59:58 | sean-k-mooney | sshmlbrg: im not sure if its somethign you feel comfortable trying to fix | |
| 19:00:20 | sshmlbrg | get availability_zone from RequestSpec for instance? | |
| 19:01:27 | sean-k-mooney | if its shelve offloaded | |
| 19:01:47 | sshmlbrg | I would like to fix it under your guidance :) I will create a bug tomorrow. | |
| 19:01:51 | sean-k-mooney | and if its not pinend then just proceed as if it passed i think | |
| 19:03:26 | sshmlbrg | sean-k-mooney: Thanks a lot for your attention. Stay in touch | |
| 19:04:34 | sean-k-mooney | sshmlbrg: no worreis, https://github.com/openstack/nova/blob/88a36a5a0099aa52f9ea93905c7c56911b2bf0b3/nova/objects/request_spec.py#L781-L791 is your starting point | |
| 19:05:01 | sean-k-mooney | you can use Requestspec.get_by_instance_uuid() to get the request spec object if its shelve offloaded | |
| 19:05:32 | sean-k-mooney | and then grab the az https://github.com/openstack/nova/blob/88a36a5a0099aa52f9ea93905c7c56911b2bf0b3/nova/objects/request_spec.py#L76 | |
| 19:05:56 | sean-k-mooney | if that is not none then you need to compare to the volume | |
| 19:06:09 | sean-k-mooney | if it is then i think returnign form the function might be ok | |
| 19:06:16 | sean-k-mooney | we will need to see what others think | |
| 19:11:46 | mikal | Morning | |
| 19:12:34 | sean-k-mooney | mikal: o/ | |
| 19:30:03 | opendevreview | Mengyang Zhang proposed openstack/nova-specs master: Add Burst Length Support to Cinder QoS https://review.opendev.org/c/openstack/nova-specs/+/932653 | |
| 20:58:10 | opendevreview | sean mooney proposed openstack/nova master: Deprecate linux bridge support https://review.opendev.org/c/openstack/nova/+/942291 | |
| 21:00:07 | sean-k-mooney | bauzas: ^ thats the nova side fo the deprecation | |
| #openstack-nova - 2025-02-20 | |||
| 02:47:14 | opendevreview | Merged openstack/os-vif master: Drop kuryr-kubernetes-tempest https://review.opendev.org/c/openstack/os-vif/+/941568 | |
| 02:51:43 | opendevreview | sean mooney proposed openstack/nova master: Add a new ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/940642 | |
| 02:51:44 | opendevreview | sean mooney proposed openstack/nova master: per-property ImageMetaPropsWeigher https://review.opendev.org/c/openstack/nova/+/941601 | |
| 03:00:17 | opendevreview | sean mooney proposed openstack/nova master: Per-Property ImageMetaPropsWeigher https://review.opendev.org/c/openstack/nova/+/941601 | |
| 04:27:10 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942334 | |
| 04:28:55 | melwitt | sean-k-mooney, dansmith: partial revert is proposed ^. I wasn't sure if we should have a release note as I think none of the branches have been released yet with the regression? | |
| 04:30:57 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942334 | |
| 04:35:18 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Fix regression of listDevices() return type https://review.opendev.org/c/openstack/nova/+/942334 | |
| 11:21:20 | sean-k-mooney | melwitt: i think your right that we have not had a release with it. its possibel someone picked it up from master or stabel in a downstream or directly but its unlikely. | |
| 14:54:03 | dansmith | gibi: did you want to re-review the imageprops weigher or should I +W? | |
| 15:11:56 | opendevreview | Johannes Kulik proposed openstack/nova master: libvirt: fix maxphysaddr passthrough dom parsing https://review.opendev.org/c/openstack/nova/+/942371 | |
| 15:26:37 | gibi | dansmith: go ahead I will have no time today to get back to it. I did not review the original impl either just discussed the high level design. | |
| 15:26:46 | dansmith | okay | |
| 15:27:07 | gibi | dansmith: I think we are in agreement of the design, and I trust your review about the impl | |
| 15:27:14 | dansmith | gibi: ack | |
| 15:27:36 | dansmith | bauzas: if you want to +W the patch below the first two can land, waiting on a recheck of the third | |
| 15:27:48 | bauzas | dansmith: sure | |
| 15:45:53 | sean-k-mooney | while we are talking about merging things can someone +2w this https://review.opendev.org/c/openstack/os-vif/+/941585 so i can propose a os-vif release | |
| 15:46:57 | sean-k-mooney | we need to do that today so we can bump it in uc before the non clinet lib freeze | |
| 15:47:42 | sean-k-mooney | also once https://review.opendev.org/c/openstack/nova/+/940873 is merged we can merge the spice direct series | |
| 15:50:50 | sean-k-mooney | well almost we also need to merge https://review.opendev.org/c/openstack/tempest/+/942023 but thats approved | |
| 15:51:01 | sean-k-mooney | it just hitting random ci failure that are not related to it | |
| 16:28:53 | melwitt | sean-k-mooney: thanks for looking at the patch, I'll check out your comments | |
| 17:35:05 | opendevreview | Merged openstack/nova master: Add fill_metadata() to InstanceList https://review.opendev.org/c/openstack/nova/+/942214 | |
| 17:35:26 | melwitt | sean-k-mooney: I added a comment on the nova-ovs-hybrid-plug patch, few release note things left | |
| 17:36:58 | sean-k-mooney | ya... so i fixed those in my editor but didnt hit save before i commited and pushed. | |
| 17:41:10 | sean-k-mooney | melwitt: would you like me to respin or would you be happy with a followup patch on top | |
| 17:42:05 | melwitt | sean-k-mooney: yeah I guess put a follow up so we don't lose bauzas +2 | |
| 17:42:42 | sean-k-mooney | that was my concern i was hoping ot let this and perhasp the speic sereise merge/recheck over the weekend | |
| 17:43:45 | melwitt | ++ | |
| 17:44:42 | melwitt | I think it's good, just waiting for the tempest change to merge to get that last sanity check on the series, then I can go through and +W them | |
| 17:44:49 | opendevreview | sean mooney proposed openstack/nova master: FUP for reno issues https://review.opendev.org/c/openstack/nova/+/942386 | |
| 17:45:00 | melwitt | CI has been extra bad on the tempest patch for some reason | |
| 17:45:25 | sean-k-mooney | ya so with ci changes the last patch cant merge until tempest is fixed | |
| 17:45:51 | sean-k-mooney | but i was also going to wait for that to happen before putting +w on the bottom of the seriese | |
| 17:46:18 | bauzas | I'm done now for the week | |
| 17:47:07 | sean-k-mooney | melwitt: im not sure why the tempest patch has been so unfortunete | |
| 17:47:25 | sean-k-mooney | im hoping it maybe load related | |
| 17:47:56 | sean-k-mooney | and it will stableise before next week and FF propper | |
| 17:48:02 | melwitt | yeah. I think that has been a big factor | |
| 17:48:13 | sean-k-mooney | bauzas: Uggla how is offically PTL/release liason currently | |
| 17:48:26 | melwitt | sean-k-mooney: agree with waiting for tempest sanity check before +W the bottom of the series. I guess that's what I really meant | |
| 17:49:11 | sean-k-mooney | im keen ot get this over the line so i was plannign to check in on it over the weekend, of couse if ti merges before then even better | |
| 17:49:52 | sean-k-mooney | bauzas: Uggla the reason im asking is i need to do the os-vif release later today and ill need ptl/release liason approval on that patch | |
| 17:50:22 | melwitt | +1 to getting this done | |
| 17:50:34 | sean-k-mooney | i guess bauzas is ptl in the governace repo and Uggla or amit is a release liason | |
| 17:51:22 | melwitt | yeah I think it won't swap in Uggla until this cycle closes probably. I don't remember what day of the cycle they use for that tho | |
| 17:52:25 | sean-k-mooney | i think your right i belive its after the offial release but ill jsut check what is in git | |
| 17:53:21 | sean-k-mooney | so ya rene and ammit are release liasons and can approve https://github.com/openstack/releases/blob/master/data/release_liaisons.yaml#L90-L96 | |
| 17:53:41 | Uggla | sean-k-mooney, I'm not sure it is official yet. And at least I'm not ready for that yet. :) | |
| 17:54:17 | sean-k-mooney | the osvif patch need to be recheck again anyay it failed in gate... | |
| 18:02:28 | opendevreview | Merged openstack/nova master: Add a new ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/940642 | |
| 19:12:27 | mikal | Morning | |
| 19:17:44 | sean-k-mooney | mikal: o/ | |
| 19:17:58 | sean-k-mooney | mikal: we are stuggallign to get your tempest change merged | |
| 19:18:44 | sean-k-mooney | it keeps hitting random unrelated isseus altough im partly concerd about the ironic failure | |
| 19:19:02 | sean-k-mooney | it snot related to our change that i cna see but the ipa job failed twice now | |
| 19:19:20 | sean-k-mooney | mikal: im hoping it just load related and a recheck at a less busy time will pass | |
| 19:24:19 | mikal | Yeah, I am not sure I see how my tempest test would affect ironic at all. | |
| 19:24:24 | mikal | Thank you for chasing it though. | |
| 19:25:25 | sean-k-mooney | most of us are off tomorrow but im goign to t check in perodicaly over the weekend | |