| opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/python-novaclient/+/935125 | 00:59 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/938053 | 03:02 |
| opendevreview | Merged openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | 03:02 |
| opendevreview | Merged openstack/nova master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/nova/+/929395 | 03:02 |
| opendevreview | Merged openstack/nova master: Modify the url of openstack client commands. https://review.opendev.org/c/openstack/nova/+/851197 | 03:02 |
| opendevreview | Merged openstack/nova master: Fix the command to list hw_machine_type unset instances https://review.opendev.org/c/openstack/nova/+/913652 | 03:02 |
| opendevreview | Merged openstack/nova master: Drop compatibility code for Windows https://review.opendev.org/c/openstack/nova/+/937503 | 03:17 |
| opendevreview | Merged openstack/nova master: Adapt tests to new messages from jsonschema 4.23.0 https://review.opendev.org/c/openstack/nova/+/937556 | 03:17 |
| opendevreview | Merged openstack/placement master: Adapt tests to new messages from jsonschema 4.23.0 https://review.opendev.org/c/openstack/placement/+/937601 | 03:17 |
| opendevreview | Merged openstack/nova master: docs: Show the recommended way to PXE boot an instance https://review.opendev.org/c/openstack/nova/+/928309 | 03:17 |
| opendevreview | Merged openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | 04:33 |
| opendevreview | Merged openstack/nova master: db: Retrieve VirtualInterface objects by UUID, not address https://review.opendev.org/c/openstack/nova/+/828819 | 04:33 |
| *** kernelpanic77[m] is now known as IshanShanware[m] | 05:34 | |
| opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | 07:30 |
| opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | 07:41 |
| Uggla | sean-k-mooney, thank for reviewing the spec yesterday evening. And thanks for your help and availability to move forward this spec. | 07:45 |
| opendevreview | Merged openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | 08:15 |
| *** tosky_ is now known as tosky | 09:33 | |
| *** ralonsoh_ is now known as ralonsoh | 09:58 | |
| opendevreview | Merged openstack/nova master: Fix add/remove SecurityGroup action json schemas https://review.opendev.org/c/openstack/nova/+/934823 | 11:08 |
| opendevreview | Dr. Jens Harbott proposed openstack/nova master: DNM: Test eventlet bump https://review.opendev.org/c/openstack/nova/+/938879 | 11:24 |
| 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:26 |
| 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:28 |
| 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 |
| sean-k-mooney | frickler: ill see if i can respin the nova patch shortly | 11:29 |
| 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 |
| 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:30 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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:51 |
| 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:54 |
| 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 |
| 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 |
| 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:04 |
| 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:27 |
| 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:42 |
| dansmith | ack, or we ask bauzas for an exception | 14:43 |
| dansmith | or could it be specless? | 14:43 |
| gibi | I vote for asking for an exception | 14:48 |
| gibi | probably not a specless as it is an API impact | 14:49 |
| sean-k-mooney | ya it will need a spec because of the microversion impact | 14:51 |
| gibi | I can put up a spec for it | 14:52 |
| sean-k-mooney | we could combine it with the work dviroel is doing to add the schduler hits to server show | 14:52 |
| sean-k-mooney | and use one microverion for both | 14:52 |
| * gibi goes and creates the stup bp and spec so artom can link to it | 14:53 | |
| dansmith | I know it will require a microversion, it's just fairly straightforward I think, but fair enough | 14:53 |
| artom | gibi, oh, I have that in progress already - let me put that up in a few minutes | 14:54 |
| gibi | artom: ack | 14:54 |
| 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:55 |
| opendevreview | Artom Lifshitz proposed openstack/nova-specs master: WIP: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | 14:56 |
| opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | 15:02 |
| artom | OK, that should do it. | 15:02 |
| gibi | artom: thanks. I will fill the spec | 15:03 |
| artom | gibi++ cheers! | 15:03 |
| gibi | artom: what is the gameplan, do I just +2 both spec in the vTPM spec series? Or you will squash? | 15:04 |
| artom | I've already squashed into The Final Version (almost said Solution there) | 15:05 |
| gibi | cool | 15:08 |
| 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:14 |
| bauzas | sorry folks I had a problem at home | 15:15 |
| bauzas | artom: want me to accept some exception for your spec ? | 15:16 |
| artom | bauzas, I think that can wait until Monday? | 15:17 |
| gibi | bauzas: https://review.opendev.org/c/openstack/nova-specs/+/938910 we will ask for an exception for this | 15:17 |
| gibi | I'm filling out the spec as we speak | 15:17 |
| artom | I think vTPM proper will me approved in time, but ^^ will take a bit of time to fill out | 15:18 |
| gibi | yepp | 15:18 |
| bauzas | artom: LGTM | 15:18 |
| gibi | bauzas: think about as a spinoff of the vTPM spec | 15:18 |
| bauzas | I don't have a lot of context, I trust you folks | 15:18 |
| gibi | we are filling it independently as it is useful independently and can be landed independently | 15:18 |
| 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:43 |
| * dansmith pulls up the api ref | 15:43 | |
| sean-k-mooney | same | 15:44 |
| gibi | today we have id and links there | 15:44 |
| sean-k-mooney | im thinking new filed but let me check what it looks like today | 15:44 |
| dansmith | ah "properties" inside existing image seems good.. would be weird to have image.. and image_properties I think | 15:44 |
| gibi | in the past we had the same for flavor but we dropped the links and added the embedded fields | 15:45 |
| 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:45 |
| gibi | OK, that sounds good to me too | 15:46 |
| 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 |
| sean-k-mooney | well we have | 15:47 |
| sean-k-mooney | "flavor": { | 15:47 |
| sean-k-mooney | "disk": 1, | 15:47 |
| sean-k-mooney | "ephemeral": 0, | 15:47 |
| sean-k-mooney | "extra_specs": {}, | 15:47 |
| sean-k-mooney | "original_name": "m1.tiny", | 15:47 |
| sean-k-mooney | "ram": 512, | 15:47 |
| sean-k-mooney | "swap": 0, | 15:47 |
| sean-k-mooney | "vcpus": 1 | 15:47 |
| sean-k-mooney | }, | 15:47 |
| sean-k-mooney | so ya properties under image would be consistent with that | 15:48 |
| sean-k-mooney | just an fyi im going to finish for the week shortly | 15:48 |
| gibi | sean-k-mooney: OK. Thanks. Have a nice weekend | 15:49 |
| sean-k-mooney | o/ catch ye next week | 15:58 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938932 | 16:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938934 | 16:21 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | 16:21 |
| gibi | artom: dansmith: ^^ that is my first stab at the image properties spec | 16:23 |
| dansmith | ack, will look in a sec | 16:26 |
| gibi | dansmith: thanks for the review, I will respin it | 16:51 |
| dansmith | cool, thanks | 16:51 |
| 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:51 |
| gibi | yeah this feels like a low effort high value change regardless of the vTPM work | 16:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | 16:57 |
| gibi | I've fixed the comments ^^ | 16:57 |
| dansmith | I've +2d :) | 17:00 |
| 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 |
| 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 |
| 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 |
| gibi | dansmith: thansk | 17:01 |
| gibi | thanks even () | 17:01 |
| gibi | :) | 17:01 |
| 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 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | 17:03 |
| 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 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/placement stable/2023.2: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938947 | 17:04 |
| opendevreview | Balazs Gibizer proposed openstack/nova-specs master: Image properties in server show https://review.opendev.org/c/openstack/nova-specs/+/938910 | 21:51 |
| 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 | 21:54 |
| sean-k-mooney | im also +2 on that too but ya we can chat about it in the team meeting on tuesday | 22:05 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | 22:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!