| opendevreview | Merged openstack/nova master: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923257 | 03:06 |
|---|---|---|
| *** bauzas_ is now known as bauzas | 03:28 | |
| opendevreview | Merged openstack/nova stable/2023.1: Fix disk_formats in ceph job tempest config https://review.opendev.org/c/openstack/nova/+/923344 | 06:13 |
| opendevreview | Maxime Lubin proposed openstack/nova-specs master: USB over IP https://review.opendev.org/c/openstack/nova-specs/+/920687 | 07:19 |
| frickler | more gate resets happening all the time. I'll regate https://review.opendev.org/c/openstack/nova/+/923258 now, but not promote in order to give the glance stack a chance first | 07:25 |
| abhishekk | thank you! | 07:28 |
| opendevreview | Maxime Lubin proposed openstack/nova-specs master: USB over IP https://review.opendev.org/c/openstack/nova-specs/+/920687 | 07:32 |
| amorin | hey team, just wondering if there is any way to start nova-conductor when nova-placement is down? It seems nova-conductor refused to start if placement API is not up | 07:49 |
| amorin | openstack.exceptions.NotSupported: The placement service for keystone.x.y.z xists but does not have any supported versions. | 07:49 |
| frickler | amorin: what would be the use case for that? I don't think the conductor can do anything useful without placement? | 07:55 |
| amorin | I mean, maybe it can start anyway, and try to reconnect later, at least it will start and answer to computes that need some data | 07:56 |
| frickler | amorin: hmm, maybe, but then let me ask a different question: in what scenario would you expect your placement service to be down for an extended period? or would just retrying a couple of times at startup be enough? | 08:06 |
| amorin | agree that this scenario of placement down is wrong | 08:19 |
| amorin | I am just surprised that this is the behavior of nova-conductor. | 08:20 |
| amorin | my good old stein openstack was not complaining about this :) | 08:27 |
| *** bauzas_ is now known as bauzas | 08:45 | |
| frickler | did that even have the split placement service yet? ;) | 08:55 |
| amorin | yes, placement was splitted (at least in our deployment) | 08:57 |
| gibi | amorin: I think by https://github.com/openstack/nova/commit/232684b44022f1bc4d72b07045900780de456e63 | 09:01 |
| gibi | it shoud not happen any more | 09:01 |
| bauzas | this is indeed an expected behaviour | 09:02 |
| amorin | perfect! | 09:11 |
| amorin | will try cherry-picking that to my antelope openstack then | 09:12 |
| amorin | but wait, it should be there | 09:12 |
| amorin | the exeption raised is the following: exceptions.NotSupported | 09:15 |
| amorin | which is not in the first except block, so it's conesidered as a fatal one | 09:16 |
| amorin | my service is stop and returning http 503 | 09:16 |
| gibi | amorin: do you have placement in openstack endpoint list? | 09:26 |
| frickler | amorin: that sounds like the keystone lib might be mishandling that error | 09:32 |
| amorin | yes gibi, it's all set correctly, as soon as I start placement, everything works fine :) | 09:35 |
| frickler | frickler: can you try to run something like "openstack --debug resource provider list" while placement is down and show the output on paste.opendev.org? (and possibly let's move this to #openstack-sdks) | 09:37 |
| frickler | ehm amorin ^^ | 09:37 |
| sean-k-mooney | amorin: this is a similar but slightly diffent issue | 09:51 |
| sean-k-mooney | the orginal fix was to allow it to start when there was a keyston outaage not placment actully down | 09:51 |
| sean-k-mooney | i think we have a diffent patch to make loading the placement client lazy | 09:51 |
| sean-k-mooney | so that provided you dont make an api call that need it to talk to palcment it wont try | 09:52 |
| sean-k-mooney | we worked on that for the schduler too but i dont recall whicih serviecs got that change and whihc did not | 09:52 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/878238 that was the scheduler change | 09:53 |
| sean-k-mooney | i guess it thet same apprch https://github.com/openstack/nova/commit/232684b44022f1bc4d72b07045900780de456e63 make the report clint a property and lazy loads it too | 09:56 |
| sean-k-mooney | my guess is that a "503 Service Unavailable" responce does not count as ConnectionFailure and we just need to add that to the first expection block to cover your edgecase | 09:58 |
| frickler | seems there is another different set of error classes for that, like in this case keystoneauth1.exceptions.http.ServiceUnavailable | 10:10 |
| sean-k-mooney | ya so we just need to add them | 10:15 |
| sean-k-mooney | with that said nova does expect placment ot be aviabel when wee start up we just added this lazy loading to make it more graceful | 10:15 |
| sean-k-mooney | if placement is down for a protacted period of tiem we want to allow you to manage things that dont require interaction with placement but obvioulsy you wont be abel to boot new vms, evaucate them or move them in any way | 10:16 |
| sean-k-mooney | start/stop/snapshot/attache port ectra is ok | 10:16 |
| sean-k-mooney | but we would consdier the openstack to be in a degreated state and expect some operations ot fail | 10:17 |
| opendevreview | Stephen Finucane proposed openstack/placement master: tox: Simplify functional testenv definitions https://review.opendev.org/c/openstack/placement/+/919578 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/placement master: Integrate pre-commit https://review.opendev.org/c/openstack/placement/+/919579 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Add sphinx-lint https://review.opendev.org/c/openstack/placement/+/919580 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Add autopep8 https://review.opendev.org/c/openstack/placement/+/923490 | 10:20 |
| opendevreview | Stephen Finucane proposed openstack/placement master: Fix typos https://review.opendev.org/c/openstack/placement/+/923491 | 10:20 |
| sean-k-mooney | sigh.... https://github.com/al45tair/netifaces netifaces is not buildable on debian testin on python 3.12... we need to drop that depency form nova | 11:37 |
| sean-k-mooney | it works on 3.11 | 11:37 |
| opendevreview | sean mooney proposed openstack/nova master: port format insepctor tests from glance https://review.opendev.org/c/openstack/nova/+/923504 | 11:42 |
| opendevreview | sean mooney proposed openstack/nova master: port format inspector tests from glance https://review.opendev.org/c/openstack/nova/+/923504 | 11:42 |
| frickler | hmm, seems at least canonical managed to make it work https://launchpad.net/ubuntu/+source/netifaces | 11:52 |
| sean-k-mooney | it works on 3.11 but it raise an error where it say "you must add supprot for your plathform" | 11:52 |
| sean-k-mooney | ill get the exact message in a sec | 11:52 |
| sean-k-mooney | it might be because im using debian testing | 11:53 |
| sean-k-mooney | but the project is archivbed and not maintaned anymore | 11:53 |
| sean-k-mooney | so we need to replace it anyway | 11:53 |
| sean-k-mooney | i dont thnk nova really should have thsi dep anymore anyway it was mainly for nova-net before and os-vif on windows i think | 11:54 |
| frickler | ack. I'm just wondering why the py312 tests don't seem to notice it | 11:54 |
| sean-k-mooney | so we can likely jsut remove the code that is using it | 11:54 |
| sean-k-mooney | https://paste.opendev.org/show/bzSoczPxEGhAlGJPfcCc/ | 11:55 |
| sean-k-mooney | thats what im seeing on 3.12 | 11:56 |
| opendevreview | sean mooney proposed openstack/nova master: port format inspector tests from glance https://review.opendev.org/c/openstack/nova/+/923504 | 11:56 |
| sean-k-mooney | ok this is what is firing https://github.com/al45tair/netifaces/blob/master/netifaces.c#L206-L211 | 11:57 |
| frickler | hmm, can you check why pip is wanting to build the wheel, are you on arm? on noble I just get "Using cached netifaces-0.11.0-cp312-cp312-linux_x86_64.whl" | 11:59 |
| sean-k-mooney | debian testing on x86 | 11:59 |
| sean-k-mooney | x86_64 to be precise | 12:00 |
| sean-k-mooney | how would i check that | 12:00 |
| frickler | good question, add some "-v"'s? | 12:01 |
| sean-k-mooney | sure i can add some to tox | 12:01 |
| sean-k-mooney | well the install line in tox.ini | 12:01 |
| sean-k-mooney | https://paste.opendev.org/show/bbCcX0QUpd3RdqD5jUTt/ | 12:02 |
| sean-k-mooney | i think all that is wrong is i dont have the required bindep or dev libs installed to allow it to build | 12:05 |
| sean-k-mooney | im not sure why it not finding the prebuilt wheel | 12:05 |
| sean-k-mooney | given it is using weels for most of the other deps | 12:05 |
| sean-k-mooney | https://etherpad.opendev.org/p/netifaces-py312 | 12:07 |
| sean-k-mooney | thats the full output | 12:07 |
| sean-k-mooney | it might be somethign to do with the version of pip or settuptools and pyproject.toml or something like that | 12:08 |
| sean-k-mooney | it looks like its beign pull in by oslo.utils and barbican-client | 12:09 |
| sean-k-mooney | anyway that a problem for another day im going to go back to fixing iso supprot which means we need 2 new format detectors, for http://wiki.osdev.org/ISO_9660 and http://wiki.osdev.org/UDF although one is an exteion of the other | 12:13 |
| frickler | fwiw I can also build from source on noble without issues (after running devstack, so that may have installed some necessary deps) | 12:15 |
| sean-k-mooney | yep proably | 12:15 |
| sean-k-mooney | if i was using one of my dev vms it would be fine i suspect | 12:16 |
| sean-k-mooney | ill likel move to one of the shortly as eventrually ill need to do some testing with devstack too | 12:16 |
| frickler | bauzas: Uggla: auniyal: please check https://review.opendev.org/c/openstack/releases/+/923118 which is due today | 12:19 |
| opendevreview | sean mooney proposed openstack/nova master: repoduce iso regression with deep format inspection https://review.opendev.org/c/openstack/nova/+/923507 | 12:57 |
| *** iurygregory_ is now known as iurygregory | 13:02 | |
| sahid | o/ | 13:31 |
| sahid | any chance that you re-approve https://review.opendev.org/c/openstack/nova/+/921665 | 13:31 |
| sahid | i had to rebase | 13:32 |
| sean-k-mooney | done | 14:14 |
| opendevreview | Bence Romsics proposed openstack/os-vif master: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/923036 | 14:43 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923273 | 15:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923274 | 15:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923275 | 15:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923276 | 15:22 |
| *** bauzas_ is now known as bauzas | 15:23 | |
| opendevreview | Merged openstack/nova master: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923258 | 15:32 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923284 | 15:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923285 | 15:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923286 | 15:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923287 | 15:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923288 | 15:44 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923289 | 15:44 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923290 | 15:44 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923291 | 15:44 |
| opendevreview | Andrei Yachmenev proposed openstack/nova master: Fix processing uniqueness of instance host names https://review.opendev.org/c/openstack/nova/+/923395 | 16:45 |
| frickler | two failures on one of the 2024.1 cve backports, not sure if someone's still around to check these and also possibly approve the whole stack? | 17:52 |
| frickler | https://review.opendev.org/c/openstack/nova/+/923274/2 | 17:52 |
| dansmith | frickler: looks like the instance booted fine to me, so I'd expect that's typical "sometimes we can't ssh to the guest" problems | 17:55 |
| dansmith | want me to recheck? | 17:56 |
| frickler | frickler: either recheck or if you want to approve the check, I can also enqueue into gate directly, the V-1 is ignored then | 17:56 |
| frickler | s/check/stack/ | 17:57 |
| dansmith | oh I see it's missing votes | 17:58 |
| dansmith | frickler: green all the way down on 2024.1 now, I didn't recheck if you want to promote, but left the reason there too | 18:01 |
| frickler | I won't promote, as that would reset the glance stack currently merging, but just enqueue into gate, which isn't too full right now I'd hope | 18:05 |
| opendevreview | Mohammed Naser proposed openstack/nova master: Fix port group network metadata generation https://review.opendev.org/c/openstack/nova/+/923530 | 18:11 |
| dansmith | I meant s/promote/treat specially/ but yeah, cool | 18:13 |
| opendevreview | Andrei Yachmenev proposed openstack/nova master: Fix processing uniqueness of instance host names https://review.opendev.org/c/openstack/nova/+/923395 | 18:22 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] add iso file format inspector https://review.opendev.org/c/openstack/nova/+/923533 | 19:11 |
| *** bauzas_ is now known as bauzas | 19:30 | |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] add iso file format inspector https://review.opendev.org/c/openstack/nova/+/923533 | 20:03 |
| opendevreview | Merged openstack/nova master: port format inspector tests from glance https://review.opendev.org/c/openstack/nova/+/923504 | 20:51 |
| opendevreview | Merged openstack/nova stable/2024.1: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923273 | 21:35 |
| opendevreview | Merged openstack/nova stable/2024.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923274 | 21:35 |
| opendevreview | Merged openstack/nova stable/2024.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923275 | 21:35 |
| opendevreview | Merged openstack/nova stable/2024.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923276 | 21:36 |
| opendevreview | Merged openstack/nova master: api: Migrate to JSON Schema Draft 2020-12 https://review.opendev.org/c/openstack/nova/+/915845 | 21:36 |
| fungi | looks like all the ossa-2024-001 nova patches for master and stable/2024.1 have merged now, changes for older branches are still lacking approvals | 22:14 |
| fungi | glance will be in a similar state as soon as the one change that bounced out earlier merges (i've tossed it back into the gate) | 22:15 |
| fungi | cinder's are all merged at this point | 22:15 |
| opendevreview | Merged openstack/nova master: api: Correct bug in flavors schema https://review.opendev.org/c/openstack/nova/+/923144 | 23:06 |
| opendevreview | Merged openstack/nova master: api: Add 'removed' decorator https://review.opendev.org/c/openstack/nova/+/915736 | 23:12 |
| opendevreview | Merged openstack/nova master: api: Don't do version check if nothing required https://review.opendev.org/c/openstack/nova/+/915737 | 23:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!