| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-10 | |||
| 11:08:20 | opendevreview | Merged openstack/nova master: Fix add/remove SecurityGroup action json schemas https://review.opendev.org/c/openstack/nova/+/934823 | |
| 11:24:25 | opendevreview | Dr. Jens Harbott proposed openstack/nova master: DNM: Test eventlet bump https://review.opendev.org/c/openstack/nova/+/938879 | |
| 11:26:46 | frickler | sean-k-mooney: gibi: ^^ maybe better to have a look at this before we merge the u-c bump. also the nova test fix is the only remaining blocker now, even masakari is passing with the new oslo.log, so it'd be really good if you could come to terms with that | |
| 11:28:41 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:29:19 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:29:46 | sean-k-mooney | frickler: ill see if i can respin the nova patch shortly | |
| 11:30:51 | sean-k-mooney | Uggla: i look forward to reviewing the code for the spec soon :) let me know when you think its getting close to being ready and ill take a look | |
| 11:30:56 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:37:46 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:39:49 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:41:06 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 11:44:25 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 12:07:09 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 12:28:38 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 12:51:08 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 12:51:27 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 12:52:34 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: Adding link for RabbitMQ installation during nova deployment on controller node. https://review.opendev.org/c/openstack/nova/+/938702 | |
| 13:51:05 | Uggla | sean-k-mooney, yep thanks, I'll ping you when it will be almost ready. Right now I think it still needs some refinements. | |
| 13:54:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Fail when file-backed memory is used with reservation https://review.opendev.org/c/openstack/nova/+/906161 | |
| 13:58:34 | opendevreview | sean mooney proposed openstack/nova master: allow discover host to be enabeld in multiple schedulers https://review.opendev.org/c/openstack/nova/+/938523 | |
| 13:58:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Fail when file-backed memory is used with reservation https://review.opendev.org/c/openstack/nova/+/906161 | |
| 14:04:15 | opendevreview | sean mooney proposed openstack/nova master: allow discover host to be enabled in multiple schedulers https://review.opendev.org/c/openstack/nova/+/938523 | |
| 14:27:19 | sean-k-mooney | gibi: dansmith: i think ^ is sufficent to complete https://blueprints.launchpad.net/nova/+spec/distributed-host-discovery if ye have time to review but no rush | |
| 14:42:42 | artom | dansmith, sean-k-mooney, I can amend the vTPM spec to refer to the image props in server show spec, and start a stub for the latter, but I won't be able to have that up today and would need someone else to cover it :( I proposed auniyal or ratailor in a Gerrit comment, but maybe just for the spec itself it'd be faster if someone else wrote it - because of the TZ if nothing else | |
| 14:43:22 | dansmith | ack, or we ask bauzas for an exception | |
| 14:43:30 | dansmith | or could it be specless? | |
| 14:48:47 | gibi | I vote for asking for an exception | |
| 14:49:43 | gibi | probably not a specless as it is an API impact | |
| 14:51:35 | sean-k-mooney | ya it will need a spec because of the microversion impact | |
| 14:52:13 | gibi | I can put up a spec for it | |
| 14:52:26 | sean-k-mooney | we could combine it with the work dviroel is doing to add the schduler hits to server show | |
| 14:52:33 | sean-k-mooney | and use one microverion for both | |
| 14:53:48 | dansmith | I know it will require a microversion, it's just fairly straightforward I think, but fair enough | |
| 14:54:20 | artom | gibi, oh, I have that in progress already - let me put that up in a few minutes | |
| 14:54:26 | gibi | artom: ack | |
| 14:55:18 | gibi | bauzas: fyi, there will be a small API impact spec asking for an exception, related to vTPM work, but formulated in a way that both implementation can land independently | |
| 14:56:27 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: WIP: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | |
| 15:02:02 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | |
| 15:02:24 | artom | OK, that should do it. | |
| 15:03:27 | gibi | artom: thanks. I will fill the spec | |
| 15:03:38 | artom | gibi++ cheers! | |
| 15:04:36 | gibi | artom: what is the gameplan, do I just +2 both spec in the vTPM spec series? Or you will squash? | |
| 15:05:01 | artom | I've already squashed into The Final Version (almost said Solution there) | |
| 15:08:26 | gibi | cool | |
| 15:14:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Fail when file-backed memory is used with reservation https://review.opendev.org/c/openstack/nova/+/906161 | |
| 15:15:33 | bauzas | sorry folks I had a problem at home | |
| 15:16:32 | bauzas | artom: want me to accept some exception for your spec ? | |
| 15:17:41 | artom | bauzas, I think that can wait until Monday? | |
| 15:17:47 | gibi | bauzas: https://review.opendev.org/c/openstack/nova-specs/+/938910 we will ask for an exception for this | |
| 15:17:55 | gibi | I'm filling out the spec as we speak | |
| 15:18:01 | artom | I think vTPM proper will me approved in time, but ^^ will take a bit of time to fill out | |
| 15:18:08 | gibi | yepp | |
| 15:18:33 | bauzas | artom: LGTM | |
| 15:18:41 | gibi | bauzas: think about as a spinoff of the vTPM spec | |
| 15:18:57 | bauzas | I don't have a lot of context, I trust you folks | |
| 15:18:58 | gibi | we are filling it independently as it is useful independently and can be landed independently | |
| 15:43:35 | gibi | artom: sean-k-mooney: dansmith: do we want to put the new properties field into the existing image dict in server show or a top level image_properties field? | |
| 15:44:01 | sean-k-mooney | same | |
| 15:44:40 | gibi | today we have id and links there | |
| 15:44:42 | sean-k-mooney | im thinking new filed but let me check what it looks like today | |
| 15:44:43 | dansmith | ah "properties" inside existing image seems good.. would be weird to have image.. and image_properties I think | |
| 15:45:14 | gibi | in the past we had the same for flavor but we dropped the links and added the embedded fields | |
| 15:45:50 | sean-k-mooney | as long as properies is the key in image and its a map of string to string that work for me | |
| 15:46:01 | gibi | OK, that sounds good to me too | |
| 15:47:22 | sean-k-mooney | thats what we do for flavor extra specs i think too but we dont have an exampel in the api ref | |
| 15:47:37 | sean-k-mooney | well we have | |
| 15:47:39 | sean-k-mooney | "flavor": { | |
| 15:47:41 | sean-k-mooney | "disk": 1, | |
| 15:47:43 | sean-k-mooney | "ephemeral": 0, | |
| 15:47:45 | sean-k-mooney | "extra_specs": {}, | |
| 15:47:47 | sean-k-mooney | "original_name": "m1.tiny", | |
| 15:47:49 | sean-k-mooney | "ram": 512, | |
| 15:47:51 | sean-k-mooney | "swap": 0, | |
| 15:47:53 | sean-k-mooney | "vcpus": 1 | |
| 15:47:55 | sean-k-mooney | }, | |
| 15:48:00 | sean-k-mooney | so ya properties under image would be consistent with that | |
| 15:48:35 | sean-k-mooney | just an fyi im going to finish for the week shortly | |
| 15:49:52 | gibi | sean-k-mooney: OK. Thanks. Have a nice weekend | |
| 15:58:38 | sean-k-mooney | o/ catch ye next week | |
| 16:20:24 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938932 | |
| 16:21:12 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938934 | |
| 16:21:51 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | |
| 16:23:10 | gibi | artom: dansmith: ^^ that is my first stab at the image properties spec | |
| 16:26:51 | dansmith | ack, will look in a sec | |
| 16:51:15 | gibi | dansmith: thanks for the review, I will respin it | |
| 16:51:30 | dansmith | cool, thanks | |
| 16:51:49 | dansmith | I hate that we have to make more work, but I also think this is long overdue, so I'm glad for the excuse :) | |
| 16:57:23 | gibi | yeah this feels like a low effort high value change regardless of the vTPM work | |
| 16:57:42 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | |
| 16:57:55 | gibi | I've fixed the comments ^^ | |
| 17:00:23 | dansmith | I've +2d :) | |
| 17:01:34 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.2: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938939 | |
| 17:01:35 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.2: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938941 | |
| 17:01:35 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.2: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938940 | |
| 17:01:38 | gibi | dansmith: thansk | |
| 17:01:43 | gibi | thanks even () | |
| 17:01:51 | gibi | :) | |
| 17:03:04 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.1: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938943 | |
| 17:03:04 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.1: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938942 | |
| 17:03:05 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | |