| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-03-20 | |||
| 10:29:38 | bauzas | eventually, we said meh | |
| 10:30:26 | 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:31:02 | bauzas | that has to be confirmed, I'm a bit unsure of that | |
| 10:37:26 | 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:48 | sean-k-mooney | it would be more useful if we recored the orginal request id but ti probably predates that? | |
| 10:38:23 | 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:39:07 | sean-k-mooney | bauzas: the resuqest id is set in the header of the request by the user so it is public | |
| 10:39:18 | sean-k-mooney | bauzas: we generate one if its not present using middelware | |
| 10:39:54 | 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:44:08 | 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:53 | 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:45:32 | 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:47:54 | sean-k-mooney | bauzas: by the way i think we have our own variant of teh grenade multi node job | |
| 10:48:18 | sean-k-mooney | bauzas: so if we wanted we could overried the base branch temporally until it get changed for everything else | |
| 10:48:48 | sean-k-mooney | we jsut need to export the wirte varible in the job def | |
| 10:49:11 | sean-k-mooney | or wait like 2 weeks until it gets fixed in greneade either is fine | |
| 12:17:37 | opendevreview | Mina Shahsavan pour proposed openstack/nova master: Volume Affinity Filter https://review.opendev.org/c/openstack/nova/+/945031 | |
| 12:57:49 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945095 | |
| 12:57:50 | 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:59:13 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945098 | |
| 12:59:14 | 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:55 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945100 | |
| 12:59:56 | 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 | |
| 13:01:26 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945102 | |
| 13:01:27 | 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:32:22 | 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:47 | sean-k-mooney | s/cores/course/ | |
| 13:33:45 | 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:38:14 | dansmith | sean-k-mooney: cool | |
| 14:03:03 | opendevreview | sean mooney proposed openstack/nova master: raise_forever on wsgi app init https://review.opendev.org/c/openstack/nova/+/945028 | |
| 14:10:13 | 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:20 | dansmith | I've got it open | |
| 17:21:30 | 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:33:11 | bauzas | gmann: cool excellent | |
| 17:57:21 | 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:59:01 | 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:03 | sean-k-mooney | zigo: its in | |
| 17:59:12 | zigo | Oh !!! :) | |
| 17:59:17 | sean-k-mooney | zigo: the osc patches for it missed however | |
| 17:59:41 | zigo | sean-k-mooney: Is there anything I should do to support it at the packaging level ? | |
| 17:59:51 | zigo | Or "it just works" (tm) ? | |
| 17:59:58 | 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 | |
| 18:00:46 | 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:50 | 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:01:24 | zigo | Maybe I should ask in the -discuss list about this topic? | |
| 18:01:32 | 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:02:14 | zigo | I'd love it if they were backported to Epoxy officially. | |
| 18:02:56 | 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:03:32 | 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:04:58 | 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:05:34 | 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:55 | 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 | |
| 19:06:28 | 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. | |
| #openstack-nova - 2025-03-21 | |||
| 00:57:50 | opendevreview | benlei proposed openstack/nova master: optimize process of get connector for swap volume https://review.opendev.org/c/openstack/nova/+/945177 | |
| 00:57:50 | opendevreview | benlei proposed openstack/nova master: Abort live migration task when stop nova compute service https://review.opendev.org/c/openstack/nova/+/938223 | |
| 06:22:11 | ishanwar[m] | Hi Folks... (full message at <https://matrix.org/oftc/media/v1/media/download/AfIljfPE94kqyEbidICMKCnOaBBquL9y8M186dozy6czq9FKuNXkU37cf0hQWWGf7ByL_Gxne4_JEcywIr82jMNCeV_3bP0QAG1hdHJpeC5vcmcvc2NSRlZCQUlsclpPc2lZWUZuZE9Ramdt>) | |
| 06:22:30 | ishanwar[m] | Can anyone explain how I can find the correct constant | |
| 06:22:35 | ishanwar[m] | * correct constant ? | |
| 08:38:57 | opendevreview | benlei proposed openstack/nova master: FIX: optimize process of get connector for swap volume https://review.opendev.org/c/openstack/nova/+/945177 | |
| 09:38:04 | zigo | It's been now 5 years that I opened a merge request for the Nova project to add a /healthcheck from oslo.middleware in the default api-paste.ini. Is this enough time so that the team delcares a "game over" from the team that wanted to improve the oslo.middleware healthcheck before merging ? | |
| 09:48:33 | frickler | zigo: it might help if you include a link to your review (we don't talk about merge requests with gerrit). and yes, without interacting with the community regularly, getting things done is sometimes difficult | |
| 09:48:43 | zigo | https://review.opendev.org/c/openstack/nova/+/724684 | |
| 09:49:24 | gibi | zigo: I think the nova team wants to do https://specs.openstack.org/openstack/nova-specs/specs/2025.1/approved/per-process-healthchecks.html | |
| 09:50:07 | zigo | That's a good thing, though I still need to patch Nova to be able to hook an haproxy to its API, that's kind of annoying. | |
| 09:52:52 | zigo | Especially considering the oslo.middleware healthcheck was implemented in all other projects. | |
| 11:02:22 | sean-k-mooney | zigo: we dont plan to ever use oslo.middleware for healthchefcs as we have stated in the past given its a security risk | |
| 11:02:34 | sean-k-mooney | and doe snot actully do anything useful for nova | |
| 11:04:59 | sean-k-mooney | specificlly the detailed mode is unsafe https://github.com/openstack/oslo.middleware/blob/master/oslo_middleware/healthcheck/__init__.py#L159-L160 | |
| 11:05:48 | sean-k-mooney | zigo: for any of the apis you can and should hit the microverion endpoint instead | |
| 11:06:00 | sean-k-mooney | which si the root of the api | |
| 11:58:28 | zigo | sean-k-mooney: That's a very silly answer. How do you explain that all other projects do have the /healthcheck and that it works for me? | |
| 12:00:35 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | |
| 12:05:31 | opendevreview | Ishan Shanware proposed openstack/nova master: [FEAT]: using quota_details to optimize validate networks https://review.opendev.org/c/openstack/nova/+/945204 | |
| 12:07:48 | ishanwar[m] | sean-k-mooney dansmith I have raised a PR based on our discussion in the previous meeting | |
| 12:08:11 | ishanwar[m] | let me know your thoughts | |
| 12:27:43 | sean-k-mooney | zigo: if it works for you your free to ignore out advice and change ti in your downstream packageing, i dont think any project should ever enable it by default | |
| 13:02:29 | zigo | sean-k-mooney: *ALL* projects enabled it by default... | |
| 13:02:34 | zigo | Only Nova refuses to do so. | |
| 13:44:02 | dansmith | sean-k-mooney: I updated this for some verbiage after your last +2, would you mind re-applying? should be easy: https://review.opendev.org/c/openstack/nova-specs/+/943486 | |
| 13:49:26 | opendevreview | Dan Smith proposed openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| 14:02:23 | sean-k-mooney | sorrry was doing somethign else but sure i can look now | |
| 14:03:52 | sean-k-mooney | i did skim v7 the comment gibi had on how you were usign burned we actully my intall reacitno too but then i deciced that since you did not defien the teim i would jsut follow your uasage instead of my internal definition but i generally agreed with that change | |
| 14:04:49 | sean-k-mooney | dansmith: by the way related to the latch term, i starged using bruned because i think of the single use devices like eFueses | |
| 14:05:27 | dansmith | burned is mine! | |
| 14:05:32 | dansmith | but yeah, new rev defines that clearly-er | |
| 14:08:24 | sean-k-mooney | +2w, v8 looks good to me. as an aside are you plannig to move on to the placement swap issue next or complte the integration with the glance api for multiple locations or your change for making glance the enforcer? if the next priortiy is tbd thats fine im just wonderign what you planned ot do next | |
| 14:09:26 | sean-k-mooney | dansmith: im kind of aware thaty you expressed interest in a a few differnet semi high priority issues and it felt like you had a lot on your plate | |
| 14:09:27 | dansmith | sean-k-mooney: so I missed that the locations stuff ever finally landed, and that's partially because I expected the glance people to iterate on my nova patch until it was passing | |
| 14:09:30 | dansmith | but apparently they just merged without it working | |
| 14:09:44 | dansmith | so I just refreshed that and will look at the test results | |
| 14:09:48 | gibi | it is Friday, I'm burned already, needs an external tool applied on me during the weekend to unburn me before I can be used again next week | |
| 14:10:01 | dansmith | gibi: ++ :) | |
| 14:10:04 | sean-k-mooney | lol | |
| 14:10:13 | dansmith | gibi: I'll allow you to use my trademarked term under license :) | |
| 14:10:32 | sean-k-mooney | gibi: can i recommend good food and a audio or phsyical book | |
| 14:10:32 | gibi | understood, send me the necessary invoice | |
| 14:10:40 | dansmith | sean-k-mooney: tbh, I'm pretty steamed about the gpt stuff. I've had that up with cinder, nova, glance changes since last year and I have run out of energy to push people to do simple reviews on it | |
| 14:10:47 | gibi | sean-k-mooney: physical book for sure along with the book :) | |
| 14:10:59 | gibi | sorry along with the food | |
| 14:11:06 | dansmith | sean-k-mooney: so at the moment we're in a reasonable state with image inspection, so I'm not really motivated to push on it at the moment | |