| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-13 | |||
| 13:06:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | |
| 13:06:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | |
| 13:06:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | |
| 13:06:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | |
| 13:06:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 13:06:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | |
| 13:06:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | |
| 13:06:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | |
| 13:06:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | |
| 13:06:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | |
| 13:06:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | |
| 13:06:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | |
| 13:06:34 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | |
| 13:39:48 | opendevreview | Merged openstack/nova stable/2024.1: Add ServersViewBuilderTestV296 unit test class https://review.opendev.org/c/openstack/nova/+/955304 | |
| 13:43:12 | dansmith | gibi: I assume you saw that I fixed a bug in your bottom patch and only pushed up the patches through the nova-next one | |
| 13:43:25 | dansmith | so the others will need to be rebased on that stack (or once they merge) | |
| 13:45:05 | gibi | dansmith: o/ I haven't been able to check the eventlet series yet but I will | |
| 13:45:47 | dansmith | okay I just rechecked the latest failure | |
| 14:18:00 | gibi | dansmith: thanks for te rebase and adding back the import. I will take care of the rest of the series later today | |
| 14:18:10 | dansmith | ack | |
| 19:09:53 | opendevreview | sean mooney proposed openstack/nova master: Remove Linux bridge VIF support from Nova https://review.opendev.org/c/openstack/nova/+/957040 | |
| 19:09:54 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add usage comment to vif model constants https://review.opendev.org/c/openstack/nova/+/957278 | |
| 19:22:27 | opendevreview | sean mooney proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | |
| 20:29:16 | opendevreview | sean mooney proposed openstack/nova master: only show standard image properties in server show. https://review.opendev.org/c/openstack/nova/+/942413 | |
| 20:29:17 | opendevreview | sean mooney proposed openstack/nova master: [tests] Add printing of sample and template paths https://review.opendev.org/c/openstack/nova/+/944054 | |
| 20:37:25 | sean-k-mooney | melwitt: ^ i think i just resovlev the previous conflict the revers way | |
| 20:50:21 | melwitt | sean-k-mooney: ok cool, I'll check it out | |
| 20:52:53 | sean-k-mooney | thanks im going to call it a day. as an fyi i shoudl have time to deploy your vtpm series on tuesday. im going to try and review the sev seris this week but the vtpm series it the next on my list to deploy locally | |
| 20:53:19 | sean-k-mooney | i need to add barbican because i forgot the last time i stacked ... | |
| 21:22:49 | melwitt | sean-k-mooney: cool thanks for thinking about it :) | |
| 21:52:54 | opendevreview | Merged openstack/nova stable/2024.2: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955310 | |
| 22:11:33 | opendevreview | Merged openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 22:11:56 | opendevreview | Merged openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| #openstack-nova - 2025-08-14 | |||
| 00:32:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 00:34:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 00:34:49 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 00:34:57 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 04:28:01 | opendevreview | Merged openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 09:44:56 | opendevreview | Elod Illes proposed openstack/os-vif master: DNM: gate health test https://review.opendev.org/c/openstack/os-vif/+/957374 | |
| 09:48:29 | opendevreview | Elod Illes proposed openstack/placement master: DNM: gate health test https://review.opendev.org/c/openstack/placement/+/957389 | |
| 10:23:58 | opendevreview | Kamil Sambor proposed openstack/nova master: Rename deadline parameter to more accurate timeout https://review.opendev.org/c/openstack/nova/+/957411 | |
| 11:35:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Reproduce bug/2116852 https://review.opendev.org/c/openstack/nova/+/957419 | |
| 11:40:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 11:40:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 11:40:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 11:40:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 11:40:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 11:40:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 11:40:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 11:40:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 11:40:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 11:40:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 11:40:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 11:40:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 11:45:28 | jssfr | hmm, what would be a good way to check for the libvirt version from within guest.py:Guest? As far as I can tell, the _host object from driver.py:LibvirtDriver is not reachable from there... | |
| 11:45:41 | jssfr | I need to know whether it's new enough to support the VIR_DOMAIN_UNDEFINE_KEEP_TPM flag. | |
| 11:45:49 | jssfr | (in Guest.delete_configuration) | |
| 11:58:52 | jssfr | adding something like `may_preserve_vtpm` to the Guest constructor causes quite a bit of havoc in unittests which create an ad-hoc Guest instance... | |
| 12:01:37 | jssfr | aha | |
| 12:01:51 | jssfr | self._domain.connect(), and I think I can manage from there. | |
| 12:21:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 12:21:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 12:21:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 12:21:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 12:21:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 12:21:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 12:21:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 12:21:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 12:21:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 12:21:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 12:21:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 12:21:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ask for pre-prod testing for native threading https://review.opendev.org/c/openstack/nova/+/957424 | |
| 15:03:49 | dansmith | gibi: not sure if you've already been pinged, but there were a few places in the vTPM series that we were looking for your opinon | |
| 15:03:56 | dansmith | such as: https://review.opendev.org/c/openstack/nova/+/941483/21/nova/compute/api.py | |
| 15:33:50 | melwitt | thanks dansmith, I had forgot to ping him | |
| 16:03:51 | gibi | ohh, I will check | |
| 16:49:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Reproduce bug/2116852 https://review.opendev.org/c/openstack/nova/+/957419 | |
| 17:12:02 | sean-k-mooney | jssfr: we do libvirt version check in driver.py mostly | |
| 17:12:34 | sean-k-mooney | so you can do the check in driver.py and call guest.py as appropat rather then doign the check in guest.py | |
| 17:16:21 | sean-k-mooney | jssfr: takign flags as an argumet to delete_configuration may also be a bterr option | |
| 19:23:29 | melwitt | 140 items in the zuul check queue 🤯 that might be a record | |
| 19:39:19 | sean-k-mooney | you say that but im pretty sure tripleo alow triger more then tha tbefore it got put in its own queue | |
| 19:39:34 | sean-k-mooney | but ya htat alot more then typical | |
| 19:40:04 | sean-k-mooney | i feel like they update the ui agian. not that im complaining | |
| 21:28:13 | opendevreview | Merged openstack/os-vif stable/2024.2: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/952815 | |
| 23:13:47 | gmaan | yeah, I have seen 3 digit only few times. | |
| #openstack-nova - 2025-08-15 | |||
| 00:25:37 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 02:44:05 | opendevreview | Merged openstack/os-vif stable/2024.2: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/952778 | |
| 04:48:16 | opendevreview | Merged openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 06:11:55 | opendevreview | Merged openstack/nova stable/2024.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/955305 | |
| 07:22:38 | gibi | melwitt: dansmith: replied in the vtpm series https://review.opendev.org/c/openstack/nova/+/941483/21#message-f8bc46bbe27eeb78f1ec79040932d9e3fa2db062 | |
| 08:01:26 | opendevreview | Merged openstack/nova master: Drop unused method https://review.opendev.org/c/openstack/nova/+/927680 | |
| 08:09:20 | opendevreview | Jonas Schäfer proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 08:25:42 | Guest23994 | sean-k-mooney can we agree on the rfe for waiting for ovn-controller at live migration? https://bugs.launchpad.net/neutron/+bug/2069718 | |
| 08:26:32 | Guest23994 | The submitted patches are only as POC, of cause some adjustments are needed. But I wanted first to discuss if the approach is fine for you. | |
| 08:43:59 | sthoffmann[m] | Sorry, messed up my nick. | |
| 08:45:23 | opendevreview | Markus Hentsch proposed openstack/nova master: Adapt to Cinder and Glance changes for image encryption https://review.opendev.org/c/openstack/nova/+/926326 | |
| 09:16:20 | opendevreview | Lajos Katona proposed openstack/os-vif stable/2024.1: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/957490 | |
| 09:16:21 | opendevreview | Lajos Katona proposed openstack/os-vif stable/2024.1: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/957491 | |
| 09:45:11 | jssfr | sean-k-mooney, melwitt, I updated the vTPM patch. I find the version check implementation a bit clunky, but it's the variant which I found causes least impact in unittesting. A quick feedback on whether that's the right direction or you'd prefer something else would be appreciated: https://review.opendev.org/c/openstack/nova/+/955657/3/nova/virt/libvirt/driver.py#590 | |