| opendevreview | Aleksandr Chudinov proposed openstack/nova master: Fix instance snapshot min disk size after instance resize https://review.opendev.org/c/openstack/nova/+/931737 | 06:10 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | 07:24 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | 07:24 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | 07:25 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | 07:26 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Drop unused parameter type validation method https://review.opendev.org/c/openstack/nova/+/909189 | 07:29 |
| elodilles | bauzas: auniyal: Uggla: if you have some time could you please review (release liaison approve) the antelope final stable release? https://review.opendev.org/c/openstack/releases/+/932406 | 08:09 |
| mikal | sean-k-mooney: I've reproposed https://review.opendev.org/c/openstack/nova-specs/+/932387 (spice-direct spec), just wondering if there's anything else I should do apart from put it in the status etherpad? | 08:14 |
| sean-k-mooney[m] | mikal: you updated the history section so that all that was needed | 08:17 |
| sean-k-mooney[m] | gibi: ill fix the typos in https://review.opendev.org/c/openstack/nova/+/932931/1 once i have had coffee and am actually awake. thanks for the review | 08:21 |
| sean-k-mooney[m] | mikal: i assume the rest of the pataches are ready for review? | 08:22 |
| sean-k-mooney[m] | im not seeing any merge conflicts or direct requests form mel that need to be addressed but also have not looked closely at the current state | 08:24 |
| gibi | sean-k-mooney[m]: sure, just ping me after the respin and I will re-add my +2 | 08:55 |
| gibi | bauzas: when you are up, I'm wondering how much quorum the nova room will have on wednesday during the global eventlet removal discussion? | 09:02 |
| bauzas | I'm already up :) | 09:02 |
| gibi | o/ :) | 09:02 |
| bauzas | gibi: that's a good question | 09:02 |
| bauzas | that discussion will trample our own sessions | 09:03 |
| gibi | I would like to join to the eventlet removal discussion at least | 09:03 |
| bauzas | I was thinking it could be on Monday | 09:03 |
| gibi | but other cores might have the same plans | 09:03 |
| bauzas | :( | 09:03 |
| bauzas | let's discuss then the agenda today | 09:03 |
| bauzas | I'm currently looking at the etherpad | 09:03 |
| gibi | yeah I think the deeper dive will be on wednesday, yesterday we only covered high level things | 09:03 |
| bauzas | and I'll try to provide an agenda | 09:03 |
| gibi | cool, thanks | 09:04 |
| gibi | I have no other limitation during the week as far as I see | 09:04 |
| bauzas | gibi: my own limitation will be on Friday due to the TC sessions :( | 09:10 |
| bauzas | hence the agenda proposal | 09:11 |
| bauzas | gibi: I think we have a serious problem that we need to solve about Wed | 09:22 |
| sean-k-mooney | bauzas: gibi: i dont know if we want to discuss this at the ptg https://bugs.launchpad.net/nova/+bug/2076089 but it would be good if other could confirm or refute my assertion that we shoudl not be abel to force boot a new instance on a disabled compute | 09:41 |
| bauzas | sean-k-mooney: the agenda is now fully packed tbh | 09:42 |
| sean-k-mooney | as far as i am aware we made an intentional decsion to prevent this | 09:42 |
| sean-k-mooney | bauzas: that fine its a bug so we can discuss it async | 09:42 |
| bauzas | the eventlet-removal session just ate all the left time | 09:42 |
| sean-k-mooney | i just want to spread awareness of it | 09:42 |
| sean-k-mooney | im basically asking for a sanity check to ensure im not off base when i say this is not a bug. we can put it on the next team meeting adgenda too if we like | 09:44 |
| bauzas | well | 09:44 |
| bauzas | the forced AZ hack was the usual way for ops to test new computes | 09:44 |
| sean-k-mooney | right but not disabled hosts | 09:44 |
| bauzas | no, even disabled ones | 09:44 |
| sean-k-mooney | that was definetly a bug then | 09:44 |
| sean-k-mooney | i.e that that ever worked | 09:45 |
| bauzas | yeah, for me it looks to me a regression | 09:45 |
| sean-k-mooney | im saying we should not allow that to happen and that i dont condier this a regression | 09:45 |
| sean-k-mooney | if the az hack worked in the past tha that was a bug IMO | 09:46 |
| bauzas | because when you want to verify that your new compute works fine, then you just use the AZ hack for it | 09:46 |
| bauzas | you just make sure that when you create the new compute, it's disabled first | 09:46 |
| sean-k-mooney | your you regerster it as disabeld and add it to a host aggreate restricted to your tenent and then enable it | 09:46 |
| bauzas | hum, no, in general you just disable the host, and then you create new instances using the AZ hack | 09:48 |
| sean-k-mooney | bauzas: the az hack is not ment to force a host bypasssing all schdulign | 09:48 |
| bauzas | once you're sure it works, then you delete the instances and the enable the host, that's it | 09:48 |
| bauzas | sean-k-mooney: trust me, this is why I say this is a hack :) | 09:49 |
| sean-k-mooney | bauzas: right so i object to reintroducing that broken behavior if its what we had in the past | 09:49 |
| sean-k-mooney | as i said i condier it a bug | 09:49 |
| sean-k-mooney | so if we want to fix it i want to be conviced why that is accapable to bypass placment | 09:50 |
| bauzas | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L593 | 09:50 |
| sean-k-mooney | the reporter confirmed the existing behavior (where its rejected) exists as far back as zed | 09:51 |
| sean-k-mooney | meaning that all stable release have the current behavior | 09:51 |
| sean-k-mooney | bauzas: right i have used that in the past but that was nto inteded to ignore if its disabeld. | 09:52 |
| bauzas | that's because the forced host values are not checking the filters | 09:53 |
| bauzas | but... the pre-filters actually check them | 09:53 |
| sean-k-mooney | the prefilters run before the filters | 09:53 |
| bauzas | hence the change | 09:53 |
| bauzas | correct, hence the bug | 09:54 |
| sean-k-mooney | yep a change that was made in train | 09:54 |
| sean-k-mooney | as a result of a spec that was intened to prevent schduling to disable compute | 09:54 |
| sean-k-mooney | to me this was somethign we agree to block as part of the spec | 09:54 |
| sean-k-mooney | so not a bug as a result | 09:54 |
| sean-k-mooney | we also added the ablity to request a host without the az hack in train for what its worth | 09:56 |
| sean-k-mooney | this is the spec https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/pre-filter-disabled-computes.html | 09:56 |
| bauzas | I just think we forgot about this UX... | 10:01 |
| bauzas | and honestly I agree with the solution on https://bugs.launchpad.net/nova/+bug/2076089/comments/1 | 10:01 |
| sean-k-mooney | its possible but i dont think this shoudl be allowed wihtout a new microversion becuase honstely i think that is breaking the meaning of disbaled | 10:02 |
| bauzas | or indeed the workaround is to force live-migrate to the compute | 10:02 |
| sean-k-mooney | we have had this behavior since train | 10:02 |
| sean-k-mooney | so all cloud in produciton should expect this behavior | 10:02 |
| bauzas | I don't see why we need to add a microversion | 10:02 |
| bauzas | because this is only for an admin-only AZ hack value | 10:02 |
| sean-k-mooney | i do because as far as i am concerned disbaled mean you cannot schdule to this under any cuirmsutances | 10:03 |
| sean-k-mooney | so if you are changing the maning of disable by allowitn this we need to have a way to request that explcitly | 10:03 |
| sean-k-mooney | the az hack which is deprecated is not that IMO | 10:04 |
| bauzas | let's then agree to disagree :) | 10:04 |
| sean-k-mooney | if we actully want to supprot this we shoudl have --force or similar which we intentionally removed form other apis | 10:04 |
| sean-k-mooney | well we can but how do we move forward with this bug close it or reintoduce the broken pre train behavior | 10:05 |
| sean-k-mooney | this is why i wanted to add it to the ptg but we can disuss it in next weeks meeitng too | 10:05 |
| bauzas | yeah, let's discuss this then on next tuesday's meeting | 10:06 |
| sean-k-mooney | we have other mandaory prefiletrs too by the way, and we also include required/forbiden triats in other ways uncondtionally | 10:07 |
| sean-k-mooney | so fundementally you cant ignore the placement filtering the way you coudl before placemnt | 10:07 |
| opendevreview | sean mooney proposed openstack/nova master: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/932931 | 11:22 |
| sean-k-mooney | gibi: ^ sorry got distracted, should be good now i hope | 11:23 |
| sean-k-mooney | gibi: on a related note it would be good to merge https://review.opendev.org/c/openstack/nova/+/923738/2 to update to fix issues caught by the latest version of codespell and https://review.opendev.org/c/openstack/nova/+/923739/2 to bump to using that version | 11:27 |
| *** iurygregory_ is now known as iurygregory | 11:38 | |
| gibi | sean-k-mooney: looks good to me | 12:34 |
| bauzas | nova PTG sessions starting now : https://meetpad.opendev.org/nova-2025.1-ptg | 13:02 |
| bauzas | sean-k-mooney: want to join us ? | 13:07 |
| sean-k-mooney | bauzas: just finising a 1:1 ill join shortly | 13:07 |
| sean-k-mooney | start without me | 13:08 |
| bauzas | artom: will you be able to join us today ? we have your topic for around 2.30pm UTC | 13:19 |
| tkajinam | hmm ok so we undefine the secret here https://github.com/openstack/nova/blob/ed2bf3699ddc360fb646b895560fa9dbcfd6beba/nova/virt/libvirt/driver.py#L8074-L8075 | 14:46 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/933022 | 15:20 |
| jlejeune | gibi: hello, I propose that small patch to fix https://bugs.launchpad.net/nova/+bug/2085135 | 15:23 |
| gibi | jlejeune: awesome, thanks you. I will take a look | 15:23 |
| bauzas | as a reminder, today's nova meeting is CANCELLED | 15:50 |
| bauzas | please rather join the nova PTG sessions if you want to join us :) https://ptg.opendev.org/ptg.html | 15:51 |
| bauzas | as a reminder, nova sessions will start back tomorrow at *1400 UTC | 16:52 |
| bauzas | (and not 1300UTC) | 16:52 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: power down free PCPUS when updating compute node https://review.opendev.org/c/openstack/nova/+/932926 | 17:30 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | 17:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!