*** ykarel_ is now known as ykarel | 04:18 | |
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 06:28 |
---|---|---|
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 06:36 |
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 07:19 |
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 07:23 |
opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Remove use of ddt https://review.opendev.org/c/openstack/python-novaclient/+/953193 | 07:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | 08:13 |
*** LarsErik1 is now known as LarsErikP | 10:30 | |
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 11:29 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | 13:24 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server usage audit log APIs https://review.opendev.org/c/openstack/nova/+/953209 | 13:25 |
Uggla | Nova meeting in ~1h | 15:01 |
opendevreview | Johannes Beisiegel proposed openstack/nova master: feat: compute created weigher https://review.opendev.org/c/openstack/nova/+/947503 | 15:24 |
gmaan | Uggla: Due to board meeting at same time, I will not be able to attend nova meeting today. | 15:52 |
Uggla | gmaan, ok so I'll skip "your" topic. | 15:53 |
Uggla | gmaan thx for warning me. | 15:53 |
Uggla | Meeting in ~5mn | 15:54 |
fungi | i'll be lurking here in case there are more questions about the btg topic from last week, but also mostly on the board ai policy discussion conference call | 15:56 |
Uggla | #startmeeting nova | 16:01 |
opendevmeet | Meeting started Tue Jun 24 16:01:44 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
opendevmeet | The meeting name has been set to 'nova' | 16:01 |
Uggla | Hello everyone | 16:01 |
bauzas | o/ | 16:02 |
elodilles | o/ | 16:02 |
* bauzas hopes that one day, AI bubble will just explode | 16:02 | |
andreykurilin | O/ | 16:02 |
bauzas | the quickier being the better | 16:02 |
gibi | o/ | 16:03 |
bauzas | (sorry, was a rant) | 16:03 |
gibi | bauzas: it will | 16:03 |
Uggla | bauzas ;) | 16:03 |
Uggla | #topic Bugs (stuck/critical) | 16:03 |
Uggla | #info No Critical bug | 16:03 |
gibi | the question is where we will be when that bubble bursts | 16:04 |
Uggla | #topic Gate status | 16:04 |
Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:04 |
Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:04 |
Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:04 |
Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:04 |
Uggla | #info Please try to provide meaningful comment when you recheck | 16:05 |
Uggla | jumping to next topic because gmaan is not available. | 16:05 |
Uggla | #topic Release Planning | 16:05 |
Uggla | #link https://releases.openstack.org/flamingo/schedule.html | 16:05 |
Uggla | #info Nova deadlines are set in the above schedule | 16:05 |
Uggla | #info Nova Spec review day was last week. | 16:06 |
bauzas | we have one week before the spec freeze | 16:06 |
bauzas | let's be productive :) | 16:06 |
Uggla | I'm not sure we managed to review a lot of them. As I think everybody was busy. | 16:07 |
Uggla | bauzas, You are right spec freeze is July 3rd | 16:07 |
gibi | do we want to impose a limit how much we approve based on how much bandwidth we have to review implementation? | 16:07 |
bauzas | good call, last time (Epoxy) I was figuring this wasn't needed due to the low level of approved ones | 16:07 |
bauzas | what's the current state of approved specs ? | 16:08 |
Uggla | bauzas, I'm not up2date with it. :( | 16:08 |
ratailor | How can I get reviews for https://review.opendev.org/q/topic:%22bp/show-instance-action-finish-time%22 which is already completed. | 16:08 |
fungi | that did seem like one of the upshots of the btg topic discussion too, approving a reasonable number of specs helps to not set unrealistic expectations for the people who might submit changes to implement them | 16:08 |
bauzas | https://specs.openstack.org/openstack/nova-specs/specs/2025.2/ | 16:10 |
bauzas | I see 7 approved | 16:10 |
bauzas | but I can't recall whether we approved specless bps | 16:10 |
sean-k-mooney | we have | 16:11 |
sean-k-mooney | not a lot but one or two | 16:11 |
gibi | eventlet removal is a specless bp | 16:11 |
sean-k-mooney | i think thsoe are all done | 16:11 |
Uggla | we have also some specless BP. | 16:11 |
sean-k-mooney | well expcte eventlet | 16:11 |
bauzas | that would put the number to a small 10 | 16:11 |
sean-k-mooney | im not sure we shoudl set a numerical limit | 16:11 |
bauzas | I'd assume that we have a room for not more than 5 or 6 but usually we only merge ~12 features per cycle | 16:11 |
sean-k-mooney | i prefered gibis suggestion last week or +1 isntead of +2 | 16:12 |
sean-k-mooney | if you dont plan to review | 16:12 |
bauzas | so that's really a messaging | 16:12 |
bauzas | messaging question | 16:12 |
gibi | yeah I applied that last week on some specs | 16:12 |
bauzas | the real ask here is whether we want to paperwork some capacity limits while we never did before | 16:13 |
bauzas | but about the btg discussion, it has a cost : people could consider that we're committed to review their implementation, which isn't the case | 16:13 |
sean-k-mooney | if we had one i woudl likely be forece to downgrade some of my +2s to +1s we can condier it next cycle | 16:13 |
sean-k-mooney | but i dont want to chagne things this cycle at this point | 16:14 |
gibi | I think if each core is only +2 a spec if they promise to review the impl then there will be a natural emergent limt | 16:14 |
bauzas | I still want to review a couple of left specs that I got interested by | 16:14 |
bauzas | gibi: we somehow approached that on the previous PTG but we never enforced such rule yet | 16:15 |
Uggla | I agree gibi, that's probably a good method. | 16:15 |
bauzas | but we're in the middle of the acceptance process | 16:15 |
gibi | sure I don't suggest to go back and change +2s (but could be if you want) but going forward we can be limiting +2s | 16:17 |
bauzas | I mean, I'm not opposed to that, but there are precedent specs that we approved | 16:17 |
bauzas | anyway, if you feel so, do so | 16:18 |
bauzas | I'll personnally follow that rule too | 16:18 |
sean-k-mooney | i really dont want to change the rules this cycle | 16:19 |
sean-k-mooney | this is all valid for next cycle but i think its unfair to change the critiria mid way. (removing +2s) | 16:20 |
sean-k-mooney | that up to each reviewer | 16:20 |
sean-k-mooney | but i htink taht would send lot of mixed messages | 16:21 |
sean-k-mooney | we only have 9? days till spec freeze | 16:21 |
sean-k-mooney | if folks want to not add a +2 or +1 to open spec that totally fair | 16:21 |
bauzas | that's why I'm saying : as we don't have quorum now, let's just leave that to each core's opinion | 16:22 |
sean-k-mooney | +1 | 16:22 |
Uggla | +1, I think it will be ok, based on the time left. | 16:23 |
Uggla | Moving on. | 16:23 |
gibi | OK | 16:23 |
Uggla | #topic Review priorities | 16:23 |
Uggla | #link https://etherpad.opendev.org/p/nova-2025.2-status | 16:23 |
Uggla | I think we should pay attention to this one: https://review.opendev.org/c/openstack/nova-specs/+/951636 | 16:24 |
Uggla | #topic OpenAPI | 16:26 |
Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:26 |
Uggla | #info 25 increase +5, there are new ones from Stephen. | 16:27 |
sean-k-mooney | yep | 16:27 |
sean-k-mooney | im making my way through them | 16:27 |
sean-k-mooney | several are fixu ups form minor issues earlier in the series | 16:28 |
Uggla | sean-k-mooney, you are right there are FUP in the list. | 16:28 |
sean-k-mooney | there are currnetly about 10 queued to merge | 16:29 |
sean-k-mooney | im kind of brunt on them for tosay but im planning to loop back again later in the week | 16:29 |
Uggla | sean-k-mooney 👍 | 16:30 |
Uggla | Moving on because time is flying. | 16:30 |
sean-k-mooney | i think gmaan has done a first pass on all the currently open ones or most of them and Uggla has reviewds many fo them too | 16:30 |
Uggla | #topic Stable Branches | 16:30 |
sean-k-mooney | so ya good progress but we can move on | 16:30 |
Uggla | elodilles, the mic is yours | 16:30 |
elodilles | thanks Uggla | 16:30 |
elodilles | as in the past weeks, there's nothing special to report | 16:31 |
elodilles | #info stable branches (stable/2025.1 and stable/2024.*) seem to be in OK state | 16:31 |
elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:31 |
elodilles | that's all, back to you Uggla | 16:31 |
Uggla | thx elodilles | 16:31 |
Uggla | Skipping next topic because fwiesel is not available | 16:32 |
Uggla | #topic Gibi's news about eventlet removal. | 16:32 |
Uggla | #link Blog: https://gibizer.github.io/categories/eventlet/ | 16:32 |
Uggla | #link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/947966 | 16:32 |
Uggla | gibi, something you want to share. | 16:32 |
gibi | hm | 16:33 |
gibi | sorry | 16:33 |
gibi | thanks for the feedback on the series | 16:33 |
gibi | I spent time on https://review.opendev.org/c/openstack/nova/+/953121 | 16:33 |
gibi | that was triggered by the discusson on the spawn_on patch | 16:33 |
gibi | I have local changes to apply some of those to the eventlet series itself | 16:33 |
gibi | but downstream priorities stole my time | 16:34 |
bauzas | gibi: I saw the -W, can I start to review it tho ? | 16:34 |
bauzas | (I mean the spawn_on patch) | 16:34 |
gibi | bauzas: you can it is not broken. There is a missing piece that I will add | 16:34 |
gibi | all noted in the review comments | 16:34 |
gibi | to be precise I will add a missing test piece | 16:35 |
gibi | not production code piece | 16:35 |
bauzas | ack | 16:35 |
gibi | and | 16:35 |
gibi | there is the governance discussion about when to drop eventlet | 16:35 |
gibi | honestly I'm a bit lost how to move that forward | 16:36 |
gibi | https://review.opendev.org/c/openstack/governance/+/952903/1/goals/selected/remove-eventlet.rst | 16:36 |
gibi | I hope the tc has plans to reconcile the discussion there ;) | 16:36 |
gibi | that is it form me | 16:36 |
Uggla | thx gibi | 16:36 |
Uggla | #topic Open discussion | 16:37 |
Uggla | We have 2 topics | 16:37 |
Uggla | #topic (andreykurilin) Expansion of "servers:detail:get_all_tenants" policy to other server read-only APIs (example: https://review.opendev.org/c/openstack/nova/+/952896) | 16:37 |
Uggla | andreykurilin, please go ahead | 16:37 |
andreykurilin | Hi folks! | 16:37 |
andreykurilin | I, as an operator, have a need to provide readonly access to compute resources for third-party teams | 16:38 |
andreykurilin | While list api works fine, get api provides much better UX | 16:38 |
andreykurilin | What do you think about extending current readonly apis to reuse existing policy? | 16:38 |
andreykurilin | Does it need new api micro version ? Spec? Blueprint? | 16:39 |
sean-k-mooney | one of the latter two but policy change generally dont need micro verions | 16:40 |
sean-k-mooney | we do have a spec, two in fact for the intorduction fo the manager role and service role to nova | 16:41 |
sean-k-mooney | if i recall form the mailing list dicussion you only wanted to expose 1 endpoint correct | 16:42 |
sean-k-mooney | if it was many then i would say it needs a spec | 16:42 |
andreykurilin | Yes, I need get server api exposed. Maybe server actions in future, but not at this moment | 16:42 |
Uggla | It seems gmann is not opposed to the change, which is a good sign. | 16:43 |
andreykurilin | I’m happy to commit some resources to cover more than is needed for my company, if you want, but not sure that I have enough resources to work on a big spec | 16:44 |
Uggla | I think the question is if we are ok to fix it as bug | 16:45 |
Uggla | sean-k-mooney any thought about that ? | 16:46 |
sean-k-mooney | i dont consider it a bug in teh backport sense. we could treat it as an RFE bug i.e. a bug with the rfe tag instead of a specless bluepirnt | 16:47 |
sean-k-mooney | i would suggest that if we want to do somethign this clcye we keep it small and targeted | 16:48 |
sean-k-mooney | and we can dicuss larger changes but in the context of next cycle | 16:48 |
sean-k-mooney | just being practical we hae several large api serises in flight | 16:49 |
sean-k-mooney | openapi scemea, service role and manger role | 16:49 |
sean-k-mooney | we may be ableo to review a targed patch but if you want to do many endpoint we will need more time to discss it then we have before m2 | 16:50 |
Uggla | Tbh, I would rather have a specless BP. Easier to track on my side. | 16:50 |
andreykurilin | I’m ok to have only one endpoint “fixed” this cycle | 16:50 |
andreykurilin | Creating a specless BP works for me | 16:50 |
Uggla | So andreykurilin, if you can open a specless BP so I could track it. | 16:50 |
sean-k-mooney | then i woudl suggest filing a specless blueprint for server show | 16:50 |
Uggla | 👍 | 16:51 |
sean-k-mooney | and then asking for it to be approved on the mailing list or next meeting | 16:51 |
sean-k-mooney | to give gmaan or other type to respond | 16:51 |
andreykurilin | Ok, got it. Thank you | 16:51 |
sean-k-mooney | s/type/time/ | 16:51 |
gmaan | I am ok for the spec-less BP and agree that it does not need microversion | 16:52 |
sean-k-mooney | im ok with that direction too | 16:52 |
sean-k-mooney | just for the metting ^ | 16:52 |
gmaan | I tried to search history why we did not do for SHOW server when doing for details and could not find any reason | 16:52 |
gmaan | it was just not needed by anyone till now | 16:53 |
andreykurilin | It was “workaroundable”. We lived with list + id filter for several years already and finally I found time to ask you guys about this | 16:53 |
gmaan | ++, thanks for bringing that | 16:54 |
Uggla | andreykurilin++ | 16:54 |
Uggla | moving on to next topic. | 16:55 |
Uggla | #topic (ratailor) spec and implementation for show-instance-action-finish-time (https://review.opendev.org/q/topic:%22bp/show-instance-action-finish-time%22) | 16:55 |
Uggla | ratailor please go ahead. | 16:55 |
ratailor | As the implementation is also completed, just a request to provide feedback on spec so that it gets merged before FF. | 16:56 |
opendevreview | Merged openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | 16:56 |
ratailor | That's all from my side re this. | 16:57 |
sean-k-mooney | ratailor: do you have tempest coverage showing that this works ? | 16:58 |
sean-k-mooney | specificly it would be nice to see the test that orginally demponstrated this did not work passing | 16:59 |
ratailor | sean-k-mooney, don't have tempest tests yet, but I can try to submit chagne for that. | 16:59 |
ratailor | sean-k-mooney, yes, sure. I will try with that one. | 16:59 |
sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/905130 | 16:59 |
sean-k-mooney | it woudl be nice to update that and have it depend on your seriese to show ti workign end to end | 17:00 |
ratailor | sean-k-mooney, sure. Thansk! | 17:00 |
Uggla | sean-k-mooney, I agree it might also help on an escalation we have internaly | 17:00 |
Uggla | We have 2 more topics, but we are also at the top of the hour. So gibi, jonnyb if you don't mind we would be able to discuss next week. Except if it's urgent. | 17:03 |
jonnyb | hi, fine for me | 17:03 |
Uggla | jonnyb, thx for your understanding | 17:03 |
Uggla | gibi ? | 17:03 |
sean-k-mooney | jonnyb: you wanted to add a tiny weigher right? | 17:04 |
jonnyb | yeah right | 17:04 |
sean-k-mooney | if you have the patch it woudl be good for other to at least condier it before the next meeting | 17:04 |
sean-k-mooney | or perhaps start a mailing list thread on the topic | 17:04 |
sean-k-mooney | tl;dr is it weighs host by how old they are | 17:05 |
sean-k-mooney | and prefers the newer ones | 17:05 |
jonnyb | its a pretty simple patch | 17:05 |
sean-k-mooney | which is similer but differnt to the hyperviors_version weigher and they both complement each other | 17:05 |
sean-k-mooney | im generally ok with this as a specless blueprint by the way but we can disscuss that outside teh meeting or next week | 17:06 |
gibi | Uggla: no worries my topic was touched during the eventlet block | 17:07 |
jonnyb | I can move this to the mailing list to not drag this out here. thanks | 17:07 |
Uggla | jonnyb, that will be ok next week, I guess it will not take long. | 17:07 |
Uggla | anyway time to close. | 17:07 |
Uggla | Thanks all | 17:08 |
Uggla | #endmeeting | 17:08 |
opendevmeet | Meeting ended Tue Jun 24 17:08:16 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:08 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-24-16.01.html | 17:08 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-24-16.01.txt | 17:08 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-24-16.01.log.html | 17:08 |
gibi | sorry I mixed up my topic | 17:30 |
gibi | we need to discuss https://review.opendev.org/c/openstack/nova/+/952966 | 17:30 |
gibi | this is pretty much needed by debian and ubuntu to be able to release on top of 3.13 while cpython 3.13 fixing a GC bug | 17:30 |
gibi | so I need core reviews, pretty please | 17:30 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!