| opendevreview | Adam Kijak proposed openstack/nova stable/2024.1: Fix race condition when attaching a volume and deleting the instance https://review.opendev.org/c/openstack/nova/+/945067 | 07:48 |
|---|---|---|
| bauzas | dansmith: sean-k-mooney: yeah the reservation id is some older field related to ec2 API, nothing related to blazar | 09:17 |
| bauzas | it was existing *before* climate :) (ie. blazar older name) | 09:17 |
| bauzas | gmann: dansmith: when you're up, as a reminder, there is some work to be done on grenade as we can't merge https://review.opendev.org/c/openstack/nova/+/944018 yet for our non-SLURP support enveloppe | 10:09 |
| sean-k-mooney | bauzas: ack, i use horizon im just rarely logged in as admin so i was like what is this feild i never see | 10:27 |
| bauzas | more than a decade ago when we incepted climate, we were wondering the interest of that field and whether it could be useful | 10:29 |
| bauzas | eventually, we said meh | 10:29 |
| bauzas | but afaik, you get the same reservation id for multicreate, that's one way for you to match the relationship (particularly given the request_id isn't public) | 10:30 |
| bauzas | that has to be confirmed, I'm a bit unsure of that | 10:31 |
| sean-k-mooney | ya apprently its just a way to corralate instance create by multi create (and happens to be set on all instnaces probly because of common code path) im not really sure why that useful | 10:37 |
| sean-k-mooney | it would be more useful if we recored the orginal request id but ti probably predates that? | 10:37 |
| sean-k-mooney | at least with the resquest id we could easilly corralate the request to logs if they still existed and it would work across all services | 10:38 |
| sean-k-mooney | bauzas: the resuqest id is set in the header of the request by the user so it is public | 10:39 |
| sean-k-mooney | bauzas: we generate one if its not present using middelware | 10:39 |
| sean-k-mooney | but you as a normal user are free to set it, you just dont have access to the logs to make use of it. the only api that takes it as an input it the instance action log | 10:39 |
| sean-k-mooney | dansmith: by the way my ci run with the invokeation fo kill took the sys.exit path... so either the mod_wsgi docs are wrong about the constant i need to check or it does not work for some other reason | 10:44 |
| sean-k-mooney | dansmith: on the pluse side the raise_forever version worked as expected and cause the k8s pods to fail the ready check and that got reported back up into the CRs | 10:44 |
| sean-k-mooney | so im going to rework https://review.opendev.org/c/openstack/nova/+/945028 with a workaorund config flag to opt into that behavior if you have external monitoring that will detect the error | 10:45 |
| sean-k-mooney | bauzas: by the way i think we have our own variant of teh grenade multi node job | 10:47 |
| sean-k-mooney | bauzas: so if we wanted we could overried the base branch temporally until it get changed for everything else | 10:48 |
| sean-k-mooney | we jsut need to export the wirte varible in the job def | 10:48 |
| sean-k-mooney | or wait like 2 weeks until it gets fixed in greneade either is fine | 10:49 |
| opendevreview | Mina Shahsavan pour proposed openstack/nova master: Volume Affinity Filter https://review.opendev.org/c/openstack/nova/+/945031 | 12:17 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945095 | 12:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/945096 | 12:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945098 | 12:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/945099 | 12:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945100 | 12:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/945101 | 12:59 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945102 | 13:01 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Ignore metadata tags in pci/stats _find_pool logic https://review.opendev.org/c/openstack/nova/+/945103 | 13:01 |
| sean-k-mooney | feck it anyway, of cores i cant use a workaround config flag to decied how to handel invalid config errors | 13:32 |
| sean-k-mooney | s/cores/course/ | 13:32 |
| sean-k-mooney | i wrote some nice help text and everything, oh well i guess i can use some of that for the release note | 13:33 |
| dansmith | sean-k-mooney: cool | 13:38 |
| opendevreview | sean mooney proposed openstack/nova master: raise_forever on wsgi app init https://review.opendev.org/c/openstack/nova/+/945028 | 14:03 |
| sean-k-mooney | dansmith: ^ i think is in a pretty good state ill kick off one more recheck via the operator repo to to get fresh result with mod_wsgi but it should be good to review | 14:10 |
| dansmith | I've got it open | 14:10 |
| *** priteau2 is now known as priteau | 15:01 | |
| gmann | bauzas: yes, today I got go ahead from release team about all repo are branched now and I will start the QA tooling (including grenade) setup for old/new releases | 17:21 |
| bauzas | gmann: cool excellent | 17:33 |
| zigo | sean-k-mooney: Hey there! Do you happen to know how far we've been from having the virtiofs driver in Manila? Is it in Epoxy, or should we wait another release ?!? | 17:57 |
| opendevreview | sean mooney proposed openstack/nova master: wrap wsgi_app.init_application with latch_error_on_raise https://review.opendev.org/c/openstack/nova/+/945028 | 17:59 |
| sean-k-mooney | zigo: its in | 17:59 |
| zigo | Oh !!! :) | 17:59 |
| sean-k-mooney | zigo: the osc patches for it missed however | 17:59 |
| zigo | sean-k-mooney: Is there anything I should do to support it at the packaging level ? | 17:59 |
| zigo | Or "it just works" (tm) ? | 17:59 |
| sean-k-mooney | so the osc command to attach/detach the share were not merged on time as they are waiting for the sdk patch for the new nova api to land | 17:59 |
| sean-k-mooney | zigo: i dont think it changes any deps or requries packaging work beyond making sure you have the right version of openstacksdk ectra | 18:00 |
| zigo | sean-k-mooney: At this point, I probably will try to get Epoxy in Debian 13, so if there's a (small) series of patches to add, it's probably worth it ! | 18:00 |
| zigo | Maybe I should ask in the -discuss list about this topic? | 18:01 |
| sean-k-mooney | they are proposed we proablly will see if we can do a release early in 2025.2 to get the client out there | 18:01 |
| zigo | I'd love it if they were backported to Epoxy officially. | 18:02 |
| sean-k-mooney | https://review.opendev.org/c/openstack/python-openstackclient/+/881540 and https://review.opendev.org/c/openstack/openstacksdk/+/880056 are the pending patches | 18:02 |
| sean-k-mooney | they need a little more work to get over the line hopefully Uggla will have time to work on them again soon | 18:03 |
| sean-k-mooney | zigo: but sure start a mail thread if you want we know completing hte client side fo the various api we added this cycle is a priority | 18:04 |
| sean-k-mooney | we have a couple of gaps for spice-direct, manilla-shared, image properties in server show and server groups in server show | 18:05 |
| sean-k-mooney | all i think have pathces up but we need to work with the clinet fould to get them in shape and landed | 18:05 |
| mikal | zigo: yeah, it would be good to know if Debian 13 had any discussions around dropping SPICE support too. Debian is currently the recommended host OS for hypervisors with spice-direct. | 19:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!