| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-20 | |||
| 16:21:22 | Uggla | sean-k-mooney, sure I think approving the spec is good. So author can "check" that box. | |
| 16:21:25 | sean-k-mooney | Uggla: the vtpm spec is on my radar to review this week. if the cinder one actully makes progress i dont have issues with that either but i have not reviewd it since last cycle. im not sure if we really want to go into details on those in real time | |
| 16:21:50 | gibi | is it a general rule that we should not +2 a spec if we are not commiting to review the impl? | |
| 16:22:12 | dansmith | sean-k-mooney: agreed, I think we need to see a little sniff of the implementation in CI | |
| 16:22:12 | sean-k-mooney | in the past no. | |
| 16:22:31 | dansmith | gibi: not a hard rule, I just think for something like this I'm going to avoid putting my name on such an effort | |
| 16:22:39 | sean-k-mooney | but there is at least some expecation i guess that if you do +2 and the authour has questions you will at least try to guide them | |
| 16:22:59 | gibi | dansmith: ack | |
| 16:23:23 | sean-k-mooney | dansmith: i thinik if you know you wont have time then signaling that is also valuable | |
| 16:23:28 | gibi | sean-k-mooney: yeah by voting on the spec I agree I will be around to answer questions about the spec | |
| 16:25:09 | Uggla | Something to add ? | |
| 16:25:36 | sean-k-mooney | i think its good to call out the spec that are ready for review but i think we need to actully do the review async | |
| 16:25:59 | sean-k-mooney | so unless you have specific question i would suggest we move on and if we want to have a spec review day | |
| 16:26:28 | gibi | +1 | |
| 16:26:32 | sean-k-mooney | we can do that but do that at/around the soft spec freeeze in june | |
| 16:26:45 | Uggla | sure this topic is just to highlight that specs are waiting and that we should start to review them not to be overwhelmed. | |
| 16:27:09 | sean-k-mooney | masahito: your sepc also has a poc implemantion if i recall correct | |
| 16:27:21 | sean-k-mooney | i belive gibi has alrady started to look at it too | |
| 16:27:24 | masahito | yup. | |
| 16:27:40 | masahito | is it okay to add it into the priority? | |
| 16:27:41 | sean-k-mooney | masahito: since you are here did you have any questions for us that you wanted input on | |
| 16:28:03 | sean-k-mooney | you can certenlly put it in the etherpad as ready for review if you think it is | |
| 16:28:05 | gibi | sean-k-mooney: yeah I looked at the PoC and I liked what I saw | |
| 16:28:25 | masahito | nothing now. i think the spec and poc need first review round now. | |
| 16:28:58 | sean-k-mooney | gibi: ack i opened it but didnt get to read it yet. masahito: ack | |
| 16:29:21 | Uggla | and so moving on. | |
| 16:29:23 | Uggla | #topic OpenAPI | |
| 16:29:30 | Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | |
| 16:29:35 | sean-k-mooney | masahito: please add it here https://etherpad.opendev.org/p/nova-2025.2-status#L34 | |
| 16:29:40 | Uggla | 21 patches remaining. | |
| 16:30:04 | sean-k-mooney | actully slightly more | |
| 16:30:19 | sean-k-mooney | stephen as a branch on his github fork with all of them | |
| 16:30:42 | sean-k-mooney | but is propsoing them in batches. i was intended to review them today but didnt get to it so i will be reviewign them tomrrow | |
| 16:31:13 | masahito | sean-k-mooney: got it. thanks | |
| 16:31:16 | sean-k-mooney | im hoping gmaan or other will also have time in the next week or two to reviwe some of them | |
| 16:31:17 | Uggla | This topic is to follow progress on openapi. Because we said in PTG we would try to close it. | |
| 16:31:40 | gmaan | yeah, I will try if I can do during end of this week otherwise next week sometime | |
| 16:32:16 | Uggla | I have reviewed 2 of them. | |
| 16:32:52 | gmaan | I would not be able to review all of them together as it exhaust eyes checking all fields for many APIs :) but I will target some set | |
| 16:33:18 | Uggla | gmaan, I expect to progress of them week after week. | |
| 16:33:28 | sean-k-mooney | ya i can ususlly do 5-10 over the course of an hour or two and then i need to clear my mind | |
| 16:33:43 | sean-k-mooney | they are not hard persay but its hard to maintain concentration on them for long periods of time | |
| 16:33:59 | gmaan | yeah | |
| 16:34:33 | Uggla | yep my idea is to do a first pass on them and then if gmaan and sean-k-mooney you can review and merge that would be great. | |
| 16:35:25 | Uggla | any as time is flying, I'd like to move on the next topic. | |
| 16:35:33 | Uggla | #topic Stable Branches | |
| 16:35:43 | Uggla | elodilles, the floor is yours. | |
| 16:35:59 | elodilles | ack, thanks, not so much things this time | |
| 16:36:06 | elodilles | #info stable branches (stable/2025.1 and stable/2024.*) seem to be in OK state | |
| 16:36:11 | elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | |
| 16:36:17 | elodilles | that's all from me | |
| 16:36:27 | elodilles | Uggla: back to you | |
| 16:36:37 | Uggla | thx elodilles | |
| 16:36:52 | elodilles | np | |
| 16:37:10 | Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | |
| 16:37:23 | Uggla | fwiesel, anything to say ? | |
| 16:37:25 | fwiesel | Hi, no updates from my side. | |
| 16:37:35 | Uggla | ok | |
| 16:37:44 | Uggla | #topic Gibi's news about eventlet removal. | |
| 16:37:50 | Uggla | #link Series: https://gibizer.github.io/categories/eventlet/ | |
| 16:38:32 | Uggla | gibi, do you want to say something ? I think you are chasing for reviews. | |
| 16:38:45 | gibi | yepp | |
| 16:38:56 | gibi | the nova-scheduler series is ready for core review | |
| 16:39:10 | gibi | (I've added it to the status etherpad as well) | |
| 16:39:28 | gibi | the series starts here https://review.opendev.org/c/openstack/nova/+/947966 | |
| 16:39:56 | gibi | the first 11 patches are ready to go, the last two depends on oslo.service to land and release the threading backend support | |
| 16:40:26 | gibi | that is it | |
| 16:40:56 | Uggla | cool thx gibi | |
| 16:41:05 | Uggla | #topic Open discussion | |
| 16:41:19 | Uggla | I can see topic on the agenda. | |
| 16:41:27 | Uggla | s/can/can't/ | |
| 16:42:01 | Uggla | so unless if we have points, maybe we can try to triage bugs. | |
| 16:43:20 | Uggla | #topic Bug scrubbing | |
| 16:43:27 | Uggla | #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4 | |
| 16:44:01 | Uggla | First one : https://bugs.launchpad.net/nova/+bug/2110222 | |
| 16:45:07 | Uggla | Nova server image uploaded to S3 uses singlepart instead of multipart upload | |
| 16:45:26 | gibi | that feels like a feature request | |
| 16:45:37 | Uggla | yep I was about to say the same. | |
| 16:46:34 | Uggla | it is more an optimization request. | |
| 16:47:11 | Uggla | So I could probably set the bug to invalid and ask for a blueprint. | |
| 16:47:13 | gibi | it is a small feature even a specless one if we can assume that we can pass the size to the glance client at uploat | |
| 16:48:28 | Uggla | ok I'll do that for this one. | |
| 16:48:37 | Uggla | Next one: https://bugs.launchpad.net/nova/+bug/2109727 - removing an host from an aggregate should be accepted even if the host no longer exists | |
| 16:48:54 | Uggla | I think it is a valid one as opened by Sylvain | |
| 16:50:09 | Uggla | bauzas, ^ | |
| 16:51:11 | gibi | I hope bauzas could push a functional reproducer for that | |
| 16:51:41 | bauzas | sorry was taking a break | |
| 16:52:02 | bauzas | not sure if I have time for the reproducer but I can try | |
| 16:52:39 | sean-k-mooney | bauzas: i guess that only matteres if your going to work on the bug | |
| 16:52:40 | Uggla | bauzas, at least this is something valid. | |
| 16:52:49 | sean-k-mooney | if not then the peroson who picks it up can | |
| 16:54:08 | Uggla | So I'll move the bug to valid. | |
| 16:54:16 | Uggla | Next one : https://bugs.launchpad.net/nova/+bug/2108974 - Keypairs lost during cross-cell resize in instance_extra | |
| 16:54:26 | opendevreview | Merged openstack/nova master: Revert "Support glance's new location API" https://review.opendev.org/c/openstack/nova/+/950336 | |
| 16:55:42 | gibi | it has reproducer steps so somebody should run them and if reproducible mark it valid | |
| 16:56:29 | Uggla | Is resizing cross-cell supported ? | |
| 16:56:53 | dansmith | yeah | |
| 16:57:06 | dansmith | it uses shelve on the backend | |
| 16:57:22 | dansmith | and it has to copy between databases, so it might be missing something | |
| 16:58:46 | gibi | The config drive associated with the server, if there is one, will be re-generated on the destination host in the target cell. Therefore if the server was created with personality files they will be lost. However, this is no worse than evacuating a server that had a config drive when the source and destination compute host are not on shared storage or when shelve offloading and unshelving a server | |