Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-02-04
16:13:54 bauzas I definitely want to go back reviewing this week
16:14:01 sean-k-mooney so if we proceed with merging those we shoudl ideally do another one this week
16:14:02 bauzas so I'll probably figure out when to release traits
16:14:17 bauzas sean-k-mooney: cool
16:14:28 bauzas moving on the status, because we could disucss that there
16:14:35 bauzas #topic Review priorities
16:14:39 fwiesel o/
16:14:42 bauzas #link https://etherpad.opendev.org/p/nova-2025.1-status
16:14:53 bauzas the etherpad is stale but I'll update it tomorrow
16:15:28 bauzas for traits, I could also look at the existing series and try to organize a list of traits changes to review
16:15:35 bauzas those things are small
16:16:05 bauzas sean-k-mooney: for the question you asked at the beginning, that's probably the right time to talk about it
16:16:35 bauzas sean-k-mooney: so you wanna want to prioritize all the API changes by a list ? I'm like fine with that, we did this previously IIRC
16:16:56 sean-k-mooney yes so we have 3 api changes competing for 2.98
16:17:21 sean-k-mooney i woudl like to merge the speic direct patches with that since it was there first effectivly
16:17:49 sean-k-mooney the other two serise are the two to add schduler hints to server show and image properties
16:18:13 sean-k-mooney im not sure about the sate of the later two series in terms fo which can merge first
16:18:33 gibi does the spice series close to land?
16:18:48 sean-k-mooney i am +2 on the first 3 patches which is up to the api change
16:19:02 bauzas I basically have no clear view on the mergeable state of those 3 series :(
16:19:05 sean-k-mooney the final too add sound devices and usb to spice
16:19:10 bauzas but we can try something
16:19:17 sean-k-mooney so those are optional and can merge later
16:19:37 sean-k-mooney gibi: so up to here https://review.opendev.org/c/openstack/nova/+/924844/29 yes
16:19:50 bauzas what we can do for THIS week is to try to merge spice-direct API microversion
16:19:51 sean-k-mooney thats the mvp for that feature
16:19:59 gibi ack
16:20:18 bauzas if we're not able to merge it, then we can discuss whether one of the two other API patches can merge before that one
16:20:29 bauzas let's say during the nova meeting
16:20:47 bauzas call it sprint, call it runway, I don't care
16:20:57 sean-k-mooney does anyone know the state of the other two api series?
16:20:58 bauzas but let's try to merge spice-direct first
16:21:24 sean-k-mooney im hopign to take a pass over both thise week but have not been following them closely
16:21:32 gibi sean-k-mooney: I have a one week old context on the image properties
16:21:35 ratailor sean-k-mooney, functional tests are failing on my series.
16:21:48 gibi there we needed extend the series for the rebuild response
16:21:58 gibi I think the same requirement is true for the scheduler hints
16:22:17 sean-k-mooney ack i think doug is working on that currently but i dont know the eta
16:22:31 sean-k-mooney so soudn like either of those could be 2.99 and the other will get 2.100
16:22:38 gibi yepp
16:22:44 ratailor sean-k-mooney, for this https://review.opendev.org/c/openstack/nova/+/939649 I am also not sure how to correctly write rebulid schema validation.
16:23:12 sean-k-mooney it shoudl be identical to the server show repsponce schema
16:23:45 gibi sean-k-mooney: there is no server show response schema yet in tree :)
16:23:53 bauzas is anyone merging 2.100 getting a perk ? :)
16:23:53 sean-k-mooney there is
16:23:56 gibi rebuild has it, it needs to be extended
16:24:05 ratailor sean-k-mooney, there is no server show response schema here https://opendev.org/openstack/nova/src/branch/master/nova/api/openstack/compute/schemas/servers.py
16:24:26 sean-k-mooney i tough ti linked to the chagne where it was added
16:24:44 sean-k-mooney maybe show is using the rebuild schdmea but i tought this was merrged in novmber
16:24:55 ratailor gibi, but the catch here is that we are adding 'image_properties' in 'properties' subkey and 'properties' has special meaning in schema validation, so we might need to change it to 'image_properties' in response.
16:24:56 sean-k-mooney we can follow up after the metting
16:25:15 bauzas are we then clear with what I proposed ?
16:25:22 ratailor sean-k-mooney, sure.
16:25:31 sean-k-mooney oh ok stephn responded so rebuild is there btu show is still pending
16:25:42 gibi sean-k-mooney: yep that is my view
16:26:25 gibi ratailor: even if properties is a keyword for jsonschema we should have a way to have a schema that matches a field called properties :) but I have to load context and dig
16:26:49 sean-k-mooney ok ya looking at https://review.opendev.org/c/openstack/nova/+/915743/8/nova/api/openstack/compute/schemas/servers.py that correct.
16:27:15 sean-k-mooney gibi: oh does it not like having a feild in the repsonce called properties
16:27:33 ratailor gibi, sure. Thanks! that would be helpful.
16:29:10 bauzas can we move on folks ?
16:29:14 sean-k-mooney yep
16:29:21 bauzas I think we agreed on the plan
16:29:32 sean-k-mooney we can pick this up on gerrit or after the meeting
16:29:36 bauzas we'll look where we are next week
16:29:46 bauzas cool
16:29:54 bauzas #topic Stable Branches
16:29:59 bauzas elodilles: welcome back
16:30:02 elodilles o/
16:30:06 elodilles #info nothing to report, stable gates seem to be healthy
16:30:13 elodilles #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:30:19 elodilles that's all i got
16:32:18 bauzas thanks
16:32:24 bauzas moving on
16:32:33 bauzas #topic vmwareapi 3rd-party CI efforts Highlights
16:32:40 bauzas fwiesel: hello
16:33:03 bauzas any news so far ?
16:33:06 fwiesel So, I fixed the "regression". As it turns out, I didn't clean properly up after each run in the network department, which eventually became too much.
16:33:41 fwiesel Looking at the past test runs, it looks fairly stable with the one know broken test.
16:33:43 bauzas bravo then
16:33:45 fwiesel That's from my side.
16:34:03 bauzas excellent news and thanks for the hard work on investigating the root cause and fixing it
16:34:26 bauzas #topic Open discussion
16:34:47 bauzas I have one topic
16:35:04 r-taketn bauzas: I want to add one topic (request) , is it ok?
16:36:08 bauzas r-taketn: sure, I'm just grabbing some email link, sec
16:36:52 r-taketn I have another topic to mention from last week..
16:37:08 bauzas there we go
16:37:21 bauzas #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/3DBNNHXAPGVUX44NHH43WTDFMIDJ3X5U/
16:37:30 bauzas disabling the instance info caching
16:37:47 bauzas sean-k-mooney: I guess we can move forward and propose the patch, then ?
16:38:13 bauzas we said to look at the results this week
16:38:23 bauzas and the only person who replied was very happy with it
16:38:44 bauzas sean-k-mooney: so I guess you'll propose the instance info cache disabling patch
16:38:55 sean-k-mooney ah well the patch is already up
16:39:00 sean-k-mooney so we can proceed with the review
16:39:10 bauzas oh my bad you're right
16:39:13 sean-k-mooney https://review.opendev.org/c/openstack/nova/+/939476
16:39:29 bauzas I'll then proceed with the review
16:40:04 sean-k-mooney i have at least one typo to correct so ill respin it this evnnign or tomorrow
16:40:12 bauzas ++

Earlier   Later