| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-10 | |||
| 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: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938940 | |
| 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: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: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938942 | |
| 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:05 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | |
| 17:04:15 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2023.2: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938945 | |
| 17:04:15 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2023.2: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938946 | |
| 17:04:16 | opendevreview | Balazs Gibizer proposed openstack/placement stable/2023.2: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938947 | |
| 21:51:15 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | |
| 21:54:58 | melwitt | gibi: I had +W ^ but then saw in the backscroll this is supposed to wait for a spec freeze exception, so I pulled it from the gate | |
| 22:05:22 | sean-k-mooney | im also +2 on that too but ya we can chat about it in the team meeting on tuesday | |
| 22:26:01 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| #openstack-nova - 2025-01-11 | |||
| 07:57:25 | gibi | melwitt: sean-k-mooney: thanks, yeah I will formally ask for an exception on Monday from bauzas and we can discuss it on the meeting Tuesday | |
| 15:43:55 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 16:15:44 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 16:49:47 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| 21:45:37 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: doc: nova service_user wrong auth_url https://review.opendev.org/c/openstack/nova/+/938680 | |
| #openstack-nova - 2025-01-13 | |||
| 04:55:50 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: Fix outdated envlist https://review.opendev.org/c/openstack/os-resource-classes/+/939018 | |
| 05:04:15 | opendevreview | Takashi Kajinami proposed openstack/python-novaclient master: Remove environment for Python 3.8 https://review.opendev.org/c/openstack/python-novaclient/+/939022 | |
| 05:11:14 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop environment for Python 3.8 https://review.opendev.org/c/openstack/nova/+/939027 | |