| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-14 | |||
| 08:19:23 | opendevreview | ribaudr proposed openstack/nova master: Add managed flag to PCI device specification https://review.opendev.org/c/openstack/nova/+/937649 | |
| 09:22:35 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP:Reproducer for cold migration on shared storage https://review.opendev.org/c/openstack/nova/+/940304 | |
| 10:36:53 | tkajinam | gibi, thanks ! | |
| 10:38:34 | tkajinam | ugh it seems we have to fix unit tests now... | |
| 10:47:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| 10:48:04 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add unit test coverage of get_machine_ips https://review.opendev.org/c/openstack/nova/+/936287 | |
| 11:12:59 | gibi | tkajinam: thanks for fixing it | |
| 14:53:28 | MengyangZhang[m] | do we have a nova meeting today? | |
| 15:21:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/941673 | |
| 15:23:25 | dansmith | sean-k-mooney: can you +2 this so we can get this fix landed? https://review.opendev.org/c/openstack/nova/+/940604/8 | |
| 15:24:37 | bauzas | dansmith: sean said he was on PTO but I can take a look | |
| 15:29:50 | dansmith | oh right | |
| 15:30:22 | dansmith | bauzas: well, it was specifically because he had comments already, but I think they're addressed and just wanted that ack so we could move on | |
| 19:06:34 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 19:08:02 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 20:53:08 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 20:53:08 | opendevreview | Michael Still proposed openstack/nova master: Don't calculate the minimum compute version repeatedly. https://review.opendev.org/c/openstack/nova/+/940848 | |
| 20:53:09 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 20:53:09 | opendevreview | Michael Still proposed openstack/nova master: Protect older compute managers from sound model requests. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 21:22:10 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 21:31:53 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| #openstack-nova - 2025-02-15 | |||
| 00:51:31 | vsaienko | hello nova community, do you know if depends-on to nova patch added on neutron-tempest-plugin commit is working? | |
| 03:32:32 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 03:37:28 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | |
| 05:30:25 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 05:30:43 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 05:37:26 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 07:10:13 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 08:18:24 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 10:46:37 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 12:06:17 | opendevreview | Ivan Anfimov proposed openstack/placement master: Skip installation to speed up pep8 https://review.opendev.org/c/openstack/placement/+/941792 | |
| 12:06:34 | opendevreview | Ivan Anfimov proposed openstack/placement master: Skip installation to speed up pep8 https://review.opendev.org/c/openstack/placement/+/941792 | |
| 12:22:03 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]default_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940194 | |
| 12:22:03 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 12:22:04 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 12:22:04 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 12:22:05 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Ensure secret security policy for TPM instances https://review.opendev.org/c/openstack/nova/+/941062 | |
| 12:22:05 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migration for `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 12:22:06 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Modify vTPM LM block to block legacy instances https://review.opendev.org/c/openstack/nova/+/941483 | |
| 12:22:07 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 19:39:51 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 19:47:02 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 21:23:46 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 22:40:48 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| #openstack-nova - 2025-02-16 | |||
| 08:06:07 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 10:07:28 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 10:23:11 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 10:34:31 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 11:06:05 | vsaienko | sean-k-mooney[m]: can you please restore this patch https://review.opendev.org/c/openstack/nova-specs/+/471815, I'll updated it | |
| 19:09:32 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 19:42:15 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 20:46:30 | opendevreview | Ghanshyam proposed openstack/nova master: api-ref update: Move the service APIs in a separate section https://review.opendev.org/c/openstack/nova/+/864655 | |
| 20:46:57 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role context in policy tests https://review.opendev.org/c/openstack/nova/+/864594 | |
| 21:10:19 | opendevreview | Vasyl Saienko proposed openstack/nova master: Add trunk subports to network metadata https://review.opendev.org/c/openstack/nova/+/941227 | |
| 21:10:20 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 21:26:03 | opendevreview | Ghanshyam proposed openstack/nova master: api-ref update: Move the service APIs in a separate section https://review.opendev.org/c/openstack/nova/+/864655 | |
| 21:26:37 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role in server_external_events API https://review.opendev.org/c/openstack/nova/+/892633 | |
| 21:27:00 | opendevreview | Ghanshyam proposed openstack/nova master: Default os-assisted-volume-snapshots APIs to service role https://review.opendev.org/c/openstack/nova/+/892635 | |
| 22:39:42 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 22:54:53 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| #openstack-nova - 2025-02-17 | |||
| 03:26:20 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 03:58:13 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Deprecate linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941585 | |
| 11:23:43 | gibi | dansmith, sean-k-mooney (maybe bauzas) can we get this over the line https://review.opendev.org/q/topic:%22bug/2097359%22 It is very closem | |
| 11:24:40 | sean-k-mooney | so the only reason i didnt +w the bottom patch is i would like all 3 to merge togehter ot for the latter two to be squashed | |
| 11:24:57 | sean-k-mooney | i am +2 on all 3 | |
| 11:25:19 | gibi | I see, thanks. Then we need dansmith | |
| 11:26:31 | sean-k-mooney | the second patch does not account for instnaces partly migrated by my intail fix patch which is why the thrid patch is imporant IMO | |
| 11:31:12 | gibi | sean-k-mooney: yes 3rd is important, dansmith asked me to do it separately as it has bit more invasive changes hence the 3 separate patch | |
| 11:31:21 | gibi | I will keep them separate but I'm OK to land them as a whole | |
| 11:31:32 | sean-k-mooney | ack | |
| 12:38:29 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 12:42:02 | sean-k-mooney | stephenfin: gibi https://review.opendev.org/c/openstack/nova/+/939476 is the change to disable the heal info cache perodic | |
| 12:42:27 | sean-k-mooney | stephenfin: bauzas: https://review.opendev.org/c/openstack/nova/+/938523 is the review for the schduler discover host perodic change | |
| 12:42:56 | sean-k-mooney | can we try and get those approved today? | |
| 12:47:59 | stephenfin | sure | |
| 13:21:07 | bauzas | sean-k-mooney: sent to the gate, thanks | |
| 13:21:57 | bauzas | btw. can some other core like gibi look at disabling heal_instance_info cache https://review.opendev.org/c/openstack/nova/+/939476 ? | |
| 14:28:51 | dansmith | gibi: sorry didn't realize I hadn't circled back to that. looks like the bottom two are in the gate right? I'll get the third one in a bit after coffee | |
| 14:36:09 | gibi | dansmith: thanks | |
| 14:36:49 | gibi | dansmith: the first patch also needs a +w, but it is OK to get it after you reviewed the 3rd patch | |
| 14:37:36 | tkajinam | sean-k-mooney, hi could you re-review https://review.opendev.org/c/openstack/nova/+/931582 when you have time ? | |
| 14:37:56 | tkajinam | I had to add the ugly mock to fix failures caused by openstacksdk (which internally calls the same psutil interface) | |
| 14:38:29 | sean-k-mooney | i tought we merged that already so sure | |
| 14:39:05 | tkajinam | thanks ! | |
| 14:39:13 | tkajinam | and also thanks for the discussions in os-vif patch | |
| 14:39:48 | sean-k-mooney | i still need to create the deprecation patch for nova that disable the linux bridge job | |
| 14:39:55 | sean-k-mooney | i assume you dont have one for that | |
| 14:40:03 | sean-k-mooney | i added the topic to the irc meeting for tomorrow | |
| 14:40:04 | tkajinam | you are correct | |
| 14:40:14 | sean-k-mooney | ack ill push one later today | |
| 14:40:32 | sean-k-mooney | im hoping we can get them merged tomorrow or wednesday and then do a release of os-vif | |
| 14:40:45 | tkajinam | that sounds nice ! | |
| 16:46:15 | opendevreview | Vasyl Saienko proposed openstack/nova master: Do not review https://review.opendev.org/c/openstack/nova/+/941823 | |
| 17:20:59 | bauzas | dansmith: you had good concerns on https://review.opendev.org/c/openstack/nova/+/941601 | |
| 17:21:07 | bauzas | can I try to clarify what I wrote ? | |
| 17:21:46 | bauzas | because I stupidely copy/pasted a lot of docstrings from the metrics conf option which led to confusion | |
| 17:22:59 | bauzas | basically, with https://review.opendev.org/c/openstack/nova/+/941601/1/nova/scheduler/weights/image_props.py the weight is a sum of subweights dependent on the per-property weigh | |
| 17:23:32 | bauzas | that weight would then be multiplied by the main conf option | |
| 17:24:11 | bauzas | so, yes, properties would be weighed between themselves, but then the main image_props_weight_multiplier would balance the weighed value against other weighers | |
| 17:25:39 | bauzas | https://review.opendev.org/c/openstack/nova/+/941601/1/nova/tests/unit/scheduler/weights/test_weights_image_props.py shows you that | |