| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-17 | |||
| 23:26:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Ensure secret security policy for TPM instances https://review.opendev.org/c/openstack/nova/+/941062 | |
| 23:26:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 23:26:38 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migration for `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 23:26:38 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Modify vTPM LM block to block legacy instances https://review.opendev.org/c/openstack/nova/+/941483 | |
| 23:26:39 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 23:32:43 | mikal | sean-k-mooney: ummm, that will require a new tempest change as well then. That's very disappointing. | |
| 23:33:20 | sean-k-mooney | ya i really didnt want ot merge that until your patch was in | |
| 23:40:31 | mikal | https://review.opendev.org/c/openstack/tempest/+/942023 is the tempest renumbering. gmann you'll need to take a look at that one soonish please. | |
| 23:41:00 | mikal | We wont see an end-to-end run though until I get a chance to renumber the nova API changes as well. | |
| 23:41:13 | sean-k-mooney | strictly speakign that should have a depends on against the nova change | |
| 23:41:24 | sean-k-mooney | the tempest changes are only ment to merge after the nova api change | |
| 23:41:41 | sean-k-mooney | i was orginally surrpised to find the tempest patch meged eailier | |
| 23:43:40 | mikal | Ok, I added that. | |
| #openstack-nova - 2025-02-18 | |||
| 00:13:05 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 00:13:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 00:13:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 00:13:33 | mikal | ^--- the third patch will fail because there's some subtle API testing thing I need to figure out, but I don't have time right now. | |
| 00:23:08 | sean-k-mooney | i think its https://review.opendev.org/c/openstack/nova/+/924844/37/nova/api/openstack/compute/console_auth_tokens.py#93 | |
| 00:23:18 | sean-k-mooney | that shoudl be 2.98 not 2.97 | |
| 00:26:10 | sean-k-mooney | im comemntign on a few more | |
| 00:28:38 | sean-k-mooney | mikal: ok comment left on the thing i think that need to be updated but im not 100% sure | |
| 04:28:38 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/939643 | |
| 04:39:05 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/939643 | |
| 09:38:50 | opendevreview | Merged openstack/nova master: allow discover host to be enabled in multiple schedulers https://review.opendev.org/c/openstack/nova/+/938523 | |
| 10:18:00 | tkajinam | sean-k-mooney gibi, could one of you nudge https://review.opendev.org/c/openstack/nova/+/936287 to gate ? its parent was already merged | |
| 10:18:12 | tkajinam | (it already has two +2s but no +A | |
| 10:18:36 | tkajinam | I had to rebase it, and that wiped out +A :-( | |
| 10:19:23 | sean-k-mooney[m] | done | |
| 10:20:01 | tkajinam | sean-k-mooney[m], thx ! | |
| 10:43:08 | mikal | sean-k-mooney: thanks for the pointer on the review, I'm uploading a new set of patches nowish that should pass tests. | |
| 10:43:41 | mikal | I am not sure what timezone gmann is in, but we might want to make sure they're aware of the tempest SNAFU. | |
| 10:48:40 | opendevreview | Merged openstack/nova master: Add unit test coverage of get_machine_ips https://review.opendev.org/c/openstack/nova/+/936287 | |
| 10:49:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 10:49:20 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 10:49:20 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 10:51:47 | bauzas | mikal: I need to leave now but I left some comments on your series | |
| 10:52:16 | mikal | bauzas: the spice-direct or the USB / sound one? | |
| 10:52:27 | bauzas | mikal: nothing critical as most of them were related to the 2.99 fixes you need to do, but there is also another question about the returned value of tls_port in case of a rolling upgrade | |
| 10:52:32 | bauzas | mikal: the former | |
| 10:53:34 | bauzas | tl;dr: if the API calls some old compute for getting the ConsoleAuthToken object, it will obtain a None value for the tls_port field | |
| 10:54:04 | bauzas | which basically works but returns 'null' (IIRC) as an API returned value | |
| 10:54:05 | mikal | I think that is ok. That would also happen if you had spice.require_secure set to false. | |
| 10:54:23 | mikal | i.e. a user of the API already has to handle that case. | |
| 10:54:30 | bauzas | mikal: cool, I wanted you to be aware of such value, feel free to leave a comment then to reply you're good with that | |
| 10:54:39 | mikal | Ok, will do. | |
| 11:09:16 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 11:09:17 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 11:09:17 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 11:48:58 | sean-k-mooney | bauzas: can you review https://review.opendev.org/c/openstack/nova/+/940835 and https://review.opendev.org/c/openstack/nova/+/940873/6 in preperation for the spice series, that prepares the jobs to use spice, the devstack change to make multi node sspeice work proeprly is already merged so i coudl technially squash those but i think its simpler just ot merge them as is | |
| 11:49:38 | sean-k-mooney | im likely going to rename that job in a followup but i wont get to that for a while | |
| 11:50:19 | sean-k-mooney | im thinking nova-alt (i.e. for testing non default config that we do not plan to make the default, like nova-next but subtley differnt | |
| 11:50:21 | sean-k-mooney | ) | |
| 11:51:15 | sean-k-mooney | i plan to eventully renable the numa testing in that job and ill also try and add nova on nfs to it too | |
| 11:52:05 | sean-k-mooney | but i need to do some homework on exactly how to do that properly | |
| 12:33:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Reproduce bug/2097359 https://review.opendev.org/c/openstack/nova/+/942085 | |
| 12:33:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Update InstanceNUMACell version after data migration https://review.opendev.org/c/openstack/nova/+/942086 | |
| 12:33:03 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: Update InstanceNUMACell version in more cases https://review.opendev.org/c/openstack/nova/+/942087 | |
| 12:47:00 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Reproduce bug/2097359 https://review.opendev.org/c/openstack/nova/+/942088 | |
| 12:47:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Update InstanceNUMACell version after data migration https://review.opendev.org/c/openstack/nova/+/942089 | |
| 12:47:02 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Update InstanceNUMACell version in more cases https://review.opendev.org/c/openstack/nova/+/942090 | |
| 12:49:59 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: repoduce post liberty pre vicoria instance numa db issue https://review.opendev.org/c/openstack/nova/+/942091 | |
| 12:50:00 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: allow upgrade of pre-victoria InstanceNUMACells https://review.opendev.org/c/openstack/nova/+/942092 | |
| 12:50:53 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Reproduce bug/2097359 https://review.opendev.org/c/openstack/nova/+/942093 | |
| 12:50:54 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Update InstanceNUMACell version after data migration https://review.opendev.org/c/openstack/nova/+/942094 | |
| 12:50:55 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: Update InstanceNUMACell version in more cases https://review.opendev.org/c/openstack/nova/+/942095 | |
| 13:23:47 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for cold migration on shared storage https://review.opendev.org/c/openstack/nova/+/940304 | |
| 13:23:47 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/939643 | |
| 13:35:34 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for cold migration on shared storage https://review.opendev.org/c/openstack/nova/+/940304 | |
| 13:35:35 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/939643 | |
| 14:35:39 | danfai | hi nova, I'm currently trying to understand live migration in a bit more detail, especially implications on the network model. My current use case is moving from Linuxbridge to OVN, and initial tests were promising to be able to do this with live migration in our context. When doing so nova seems to convert the vif correctly but in our case uses the bridge name from the source | |
| 14:35:41 | danfai | hypervisor, which then breaks. If I hardcode/override the vif_type/bridge name on the destination hypervisor, most things work (in our case). I see that part of it comes from neutron when asking about the vif type and it would get bridge. Another one I'm not sure where it comes from, but I guess from the rpc call from the source hypervisor. What I was wondering is, should | |
| 14:35:43 | danfai | bridge names not be independent of the source hypervisor and rather be calculated based on the destination hypervisors information? (our case libvirt/kvm, public network only, still yoga) | |
| 14:39:51 | danfai | (best part of it is, that if you migrate twice, it will use the correct bridge name as the vif_type would change) | |
| 15:18:23 | opendevreview | Sylvain Bauza proposed openstack/nova master: Support multiple allocations for vGPUs https://review.opendev.org/c/openstack/nova/+/845757 | |
| 15:18:24 | opendevreview | Sylvain Bauza proposed openstack/nova master: document how to ask for more a single vGPU https://review.opendev.org/c/openstack/nova/+/906151 | |
| 15:33:29 | bauzas | dansmith: sean-k-mooney: thanks for the reviews on the weigher, I got an interesting race condition that I need to fix fwiw https://review.opendev.org/c/openstack/nova/+/940642/9 | |
| 15:33:41 | dansmith | I saw | |
| 15:33:46 | bauzas | I'll probably try/catch the call | |
| 15:34:00 | bauzas | any other opinion ? | |
| 15:34:27 | sean-k-mooney | huh i see | |
| 15:34:34 | sean-k-mooney | that form looking upt the isntnaces on the host right | |
| 15:34:58 | sean-k-mooney | or rather there image properites | |
| 15:35:19 | sean-k-mooney | so ya wraping it in a try and ignoring it if the lookup fails makes sense | |
| 15:35:53 | sean-k-mooney | im not sure if you want to explcity handlel the detelete case or also skip if there is a general exception | |
| 15:36:08 | sean-k-mooney | like the cell db was not accsabel at the time | |
| 15:37:23 | sean-k-mooney | you shoudl handel nova.exception.InstanceNotFound for sure | |
| 15:48:37 | bauzas | sean-k-mooney: yeah, I'll go this way | |
| 15:48:49 | bauzas | because users can delete their instances anyway | |
| 15:48:59 | bauzas | even when scheduling | |
| 15:55:45 | sean-k-mooney | yep and in this case the instnace that could raise this migh belong to another tenant | |
| 16:01:06 | bauzas | #startmeeting nova | |
| 16:01:06 | opendevmeet | Meeting started Tue Feb 18 16:01:06 2025 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:06 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:06 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:12 | bauzas | howdy folks | |
| 16:01:17 | fwiesel | o/ | |
| 16:01:26 | r-taketn | o/ | |
| 16:01:46 | bauzas | wiki again is super slow | |
| 16:02:00 | bauzas | I updated it in advance but the update failed | |
| 16:02:28 | elodilles | o/ | |