| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-09 | |||
| 17:56:03 | sean-k-mooney | masahito: ya so the least invaive change woudl be to allow server list to take a list of uuids | |
| 17:56:13 | sean-k-mooney | but im not sure if that somehtign others would supprot | |
| 17:57:50 | artom | melwitt, yeah, I guess the call was too early for you. The spec changed pretty drastically. It might actually be good that you read it with no further introduction by me, to see if I'm making sense to someone without the full context. | |
| 17:58:45 | masahito | Got it. Thank you for taking your time :) Let me update the spec tomorrow if there is no other comment following this talk. | |
| 18:00:38 | melwitt | artom: ah, gotcha. thanks, I'll read through | |
| 18:53:16 | sean-k-mooney | gibi: bauzas: Uggla i have +2'd https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 18:54:00 | sean-k-mooney | i have one comment inlien regardign the trait and how we report it but im ok to resolve that in a follow up when we update the spec for the object field defineiton | |
| 18:55:01 | artom | dansmith, so just to be clear, you're saying add a new property into ImageMetaProps, call it something like hw_can_live_migrate, so that we can then check instance.image_meta.properties[hw_can_live_migrate]? | |
| 18:55:32 | dansmith | no | |
| 18:55:40 | artom | Yeah I'm confused then. | |
| 18:55:43 | dansmith | we cache the user's image metadata on the instance yeah? | |
| 18:55:58 | sean-k-mooney | in instnace_system_metadata | |
| 18:56:03 | sean-k-mooney | with a img_ prefix | |
| 18:56:16 | artom | OK... | |
| 18:56:32 | sean-k-mooney | artom: but to your questiong | |
| 18:56:52 | sean-k-mooney | we shoudl be addeing a flavg to the instnace.system_metadata | |
| 18:56:53 | dansmith | right, so I'm proposing we update the cached image meta with a new "traits required" as if they had set it on their image in the past already | |
| 18:57:07 | dansmith | sean-k-mooney: he's asking about my challenge to that | |
| 18:57:08 | sean-k-mooney | we coudl do that too yes | |
| 18:57:34 | artom | Are the required traits already stored in there? | |
| 18:57:36 | dansmith | I'm suggesting we *not* create another thing we have to track that might be out of sync or different and just use the same thing the user would have used, had it existed before this | |
| 18:57:48 | dansmith | so that if we snapshot it's automatic both here and in the future | |
| 18:57:48 | sean-k-mooney | dansmith: altought we woudl have to merge that with any traits request actully from the image | |
| 18:57:54 | dansmith | sean-k-mooney: yep | |
| 18:58:41 | sean-k-mooney | i think we are mixing two diffent uescases | |
| 18:59:05 | sean-k-mooney | that or artoms naming is confusing | |
| 18:59:14 | sean-k-mooney | im just rereading the spec by the way | |
| 18:59:46 | dansmith | yeah maybe read the spec and my comments before litigating what we're talking about :) | |
| 18:59:55 | sean-k-mooney | when we chatted the other day i was saying that we coudl have a flag in instnace_system_matadata ot report if the instance is live migratable | |
| 19:00:05 | sean-k-mooney | btu that has nothign to do with the triat | |
| 19:00:06 | dansmith | that's not what we're talking about | |
| 19:00:11 | sean-k-mooney | ok | |
| 19:00:35 | sean-k-mooney | so "instance.image_meta.properties[hw_can_live_migrate]" was just confusing me then | |
| 19:01:07 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/936775/11/specs/2025.1/approved/vtpm-live-migration.rst#168 this? | |
| 19:01:51 | artom | Yeah, I think dansmith is saying - don't actually persist the security policy anywhere except as the required trait in the instance image meta | |
| 19:02:01 | artom | (Which I don't actually know where those are) | |
| 19:02:02 | sean-k-mooney | ya we can encode the tpm seciryt policy in the traits request rather then a seperate key in the system_metadata | |
| 19:02:10 | dansmith | I'm saying don't persist it in two places | |
| 19:02:29 | sean-k-mooney | to me its less clean to do it in the image metadata | |
| 19:02:33 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 19:02:35 | dansmith | it has to be persisted in the image meta cache, so no need to add a *second* place and always just do it in the one place it has to be | |
| 19:02:45 | sean-k-mooney | well as a fake request in the cached image data | |
| 19:02:56 | sean-k-mooney | dansmith: it does not need to be there at all | |
| 19:03:03 | artom | OK fine, so kill the system_metadata key | |
| 19:03:19 | artom | (Again, I'm assuming image required traits are already saved in there) | |
| 19:03:19 | sean-k-mooney | why do you say it has to be in the image meta cache? | |
| 19:03:37 | sean-k-mooney | artom: they are | |
| 19:03:49 | dansmith | that's my suggestion, because you'll have to look in both places if it's in two places | |
| 19:04:12 | artom | So then how do we indicate to owners - "the operator has chosen a policy for you, do something with your instance to confirm or do nothing/delete your instance to refuse" - IOW, L286 | |
| 19:04:20 | dansmith | once for initial boot, then the other place for the other, and if they ever get out of sync, you'll be doing the wrong thing, a snapshot will have the wrong thing, a snapshot-boot will behave differently than you expect, etc | |
| 19:04:25 | sean-k-mooney | but peopel wont normlly set this on an image | |
| 19:04:43 | sean-k-mooney | they could it they want to express a policy i guess | |
| 19:04:58 | dansmith | can we gmeet? my keyboard is starting to feel my frustration :) | |
| 19:05:05 | artom | Sure | |
| 19:05:22 | dansmith | meet.google.com/vub-zvsa-khm | |
| 19:42:00 | opendevreview | Merged openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 20:20:35 | sean-k-mooney | frickler: so based on https://review.opendev.org/c/openstack/requirements/+/933257 now that the oslo releas has been done it looks like we have a path forwoard on the eventlet bump | |
| 20:20:56 | sean-k-mooney | frickler: that does mean we need to get https://review.opendev.org/c/openstack/nova/+/933365 into a mergeable state however | |
| 20:21:54 | sean-k-mooney | ill see if i can sync with gibi tomorow or monday on how to proceed | |
| 20:32:12 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: Another approach for vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/938843 | |
| 20:32:57 | artom | dansmith, sean-k-mooney ^^ OK, that was quicker than I thought, but I'm afraid that I've probably missed things. I need to run errands and pick up kids now, but I'll be able to check IRC every so often. | |
| 20:37:17 | sean-k-mooney | artom: i also summerised my tought in a commont on the previous patch | |
| 20:37:24 | sean-k-mooney | but ill take a look shortly | |
| 20:37:36 | sean-k-mooney | im going to drop for the enving at the top of the hour | |
| 20:46:52 | frickler | sean-k-mooney: yes, this is looking like we really could make some progress now | |
| #openstack-nova - 2025-01-10 | |||
| 00:59:48 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/python-novaclient/+/935125 | |
| 03:02:22 | opendevreview | Merged openstack/nova master: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/938053 | |
| 03:02:29 | opendevreview | Merged openstack/nova master: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/932669 | |
| 03:02:36 | opendevreview | Merged openstack/nova master: Drop SQLALCHEMY_WARN_20 https://review.opendev.org/c/openstack/nova/+/929395 | |
| 03:02:44 | opendevreview | Merged openstack/nova master: Modify the url of openstack client commands. https://review.opendev.org/c/openstack/nova/+/851197 | |
| 03:02:51 | opendevreview | Merged openstack/nova master: Fix the command to list hw_machine_type unset instances https://review.opendev.org/c/openstack/nova/+/913652 | |
| 03:17:33 | opendevreview | Merged openstack/nova master: Drop compatibility code for Windows https://review.opendev.org/c/openstack/nova/+/937503 | |
| 03:17:42 | 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:45 | 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:50 | opendevreview | Merged openstack/nova master: docs: Show the recommended way to PXE boot an instance https://review.opendev.org/c/openstack/nova/+/928309 | |
| 04:33:04 | opendevreview | Merged openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | |
| 04:33:12 | opendevreview | Merged openstack/nova master: db: Retrieve VirtualInterface objects by UUID, not address https://review.opendev.org/c/openstack/nova/+/828819 | |
| 07:30:44 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 07:41:14 | opendevreview | Dmitriy Chubinidze proposed openstack/placement master: Modification of placement-api.conf https://review.opendev.org/c/openstack/placement/+/938664 | |
| 07:45:58 | Uggla | sean-k-mooney, thank for reviewing the spec yesterday evening. And thanks for your help and availability to move forward this spec. | |
| 08:15:32 | opendevreview | Merged openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 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 | |