| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-10-31 | |||
| 14:34:08 | nicolairuckel | I see | |
| 14:34:28 | sean-k-mooney | what this is telling me is your patch should not need to chown or chmod the file | |
| 14:50:30 | nicolairuckel | That's something. | |
| 15:27:46 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Allow to perform parallel live migrations https://review.opendev.org/c/openstack/nova/+/955784 | |
| 15:55:36 | bauzas | oh doh, the TC session is also very interesting | |
| 16:01:03 | sean-k-mooney | bauzas:oh the crypto stuff | |
| 16:01:18 | sean-k-mooney | bauzas: not very impornat | |
| 16:01:34 | sean-k-mooney | we will need to adress post quantum crypto eventually | |
| 16:01:39 | sean-k-mooney | but not in 2026.1 | |
| 16:02:35 | sean-k-mooney | my take is we finish the eventlet removal work first then we can think about how to adapt the minimal part of nova that will be impacted | |
| 16:03:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 16:03:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 16:03:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 16:03:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 16:03:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode https://review.opendev.org/c/openstack/nova/+/965466 | |
| 16:03:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 16:03:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional https://review.opendev.org/c/openstack/nova/+/965821 | |
| 16:29:58 | opendevreview | Merged openstack/nova master: [CI][nova-next]test with placement ac optimizations https://review.opendev.org/c/openstack/nova/+/937275 | |
| 16:30:11 | opendevreview | Merged openstack/nova master: [func]Test with optimize_for_wide_provider_trees https://review.opendev.org/c/openstack/nova/+/963036 | |
| 17:29:30 | dansmith | Uggla: is the kid noise coming from you? | |
| 17:30:00 | Uggla | dansmith, I apologize, forget to mute... | |
| 17:30:23 | Uggla | dansmith thanks for letting me know | |
| 17:33:59 | bauzas | Uggla: still in the TC meeting, but I see nova done for the PTG, I guess I won't need to join once the session ends at 5:40pm UTC ? | |
| 17:34:19 | Uggla | I next still cyborg. | |
| 17:34:27 | Uggla | s/I/in/ | |
| 17:34:52 | Uggla | I mean, I have set next to end. But we are still discussing cyborg | |
| 17:35:11 | Uggla | I guess we are not far from the end. | |
| 17:38:48 | bauzas | ack, I'll try to join as soon as the TC session stops | |
| 18:03:14 | bauzas | Uggla: well, false promise I made :) Happy Halloween everyone :) | |
| 18:13:46 | Uggla | oh yes, Happy Halloween ! | |
| 18:59:00 | cardoe | Uggla: I hear you're trying to get bauzas to drink the Rust Kool-aid. :-D | |
| 19:04:58 | Uggla | cardoe, yes I'm a rust big fan, so trying to communicate the love. | |
| 21:13:59 | opendevreview | melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 21:14:00 | opendevreview | melanie witt proposed openstack/nova master: Add handling for vTPM secret permission error https://review.opendev.org/c/openstack/nova/+/963648 | |
| 21:14:00 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 21:14:01 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 21:14:01 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 21:14:02 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 21:14:02 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 21:14:03 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 21:14:04 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 21:14:04 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 21:14:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 21:14:06 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 21:14:08 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 21:14:08 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| #openstack-nova - 2025-11-03 | |||
| 06:17:15 | opendevreview | ChungWon Lee proposed openstack/nova master: Reduce node build fail value by time https://review.opendev.org/c/openstack/nova/+/939929 | |
| 06:26:22 | opendevreview | ChungWon Lee proposed openstack/nova master: Reduce node build fail value by time https://review.opendev.org/c/openstack/nova/+/939929 | |
| 09:44:57 | opendevreview | ChungWon Lee proposed openstack/nova master: Reduce node build fail value by time https://review.opendev.org/c/openstack/nova/+/939929 | |
| 09:53:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 09:53:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional https://review.opendev.org/c/openstack/nova/+/965821 | |
| 09:53:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 09:53:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 09:53:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode https://review.opendev.org/c/openstack/nova/+/965466 | |
| 09:53:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 09:56:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 09:56:59 | sean-k-mooney | Uggla: oh ya thats today have you updated the wiki? | |
| 09:57:22 | Uggla | sean-k-mooney, not yet gonna update it. | |
| 09:57:28 | sean-k-mooney | https://meetings.opendev.org/#Nova_Team_Meeting | |
| 09:57:34 | sean-k-mooney | it looks correct | |
| 09:57:50 | gibi | sambork_: I moved https://review.opendev.org/c/openstack/nova/+/955915 out from my series and put it top of master as it belongs to the vnc series you volunteered to. Feel free to build top of it | |
| 09:58:02 | sean-k-mooney | Uggla: oh i didnt mean the adgenda | |
| 09:58:07 | sean-k-mooney | but that would be good to update as well | |
| 09:58:39 | sambork_ | gibi, ack thanks! | |
| 09:59:41 | sean-k-mooney | Uggla: i was asking becaue the meetign page has ical files so i could just update it in my calandar | |
| 09:59:57 | jkulik | sean-k-mooney: (since I got from the PTG that pinging people is _the way_) if you find some time, could you take another look at https://review.opendev.org/c/openstack/nova/+/959696 ? | |
| 10:00:27 | sean-k-mooney | sure | |
| 10:00:53 | sean-k-mooney | oh i remember looking at this before | |
| 10:03:23 | sean-k-mooney | jkulik: you mention there is a second change ye plan to propose? is that also pushed for reveiw or still somethign ye are working on | |
| 10:03:54 | jkulik | I haven't pushed it, yet. I've got the code downstream already and I think I described it in the bug-report. | |
| 10:04:44 | sean-k-mooney | ```Another reason is, that nova-api queries all server-groups from the DB - fetching all the data and creating all the objects - and afterwards applies the limits/offset. This is highly inefficient if the number of server-groups is larger than the pagination limit.``` | |
| 10:04:51 | sean-k-mooney | it that part | |
| 10:05:02 | jkulik | yes | |
| 10:05:11 | sean-k-mooney | ack | |
| 10:06:14 | jkulik | I didn't want to overwhelm upstream with too many parallel patches, so I thought pushing one patch at a time would be better. | |
| 10:06:26 | Uggla | sean-k-mooney, yep fyi the ics here should be ok to import https://meetings.opendev.org/#Nova_Team_Meeting, and now the wiki page https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting should be ok too regarding team meetings. | |
| 10:08:24 | sean-k-mooney | jkulik: my only real feedback is https://review.opendev.org/c/openstack/nova/+/959696/2/nova/tests/unit/api/openstack/compute/test_server_groups.py#393 is not a unit test it sa functional test but its not discimilar to the other "unit" tests in the file so i think its ok | |
| 10:08:52 | sean-k-mooney | its defiently testing more then a single fucntion in isolation | |
| 10:09:36 | gibi | jkulik: I have a request inline to improve condition and I would like to ask you to provide a releasenotes | |
| 10:09:41 | gibi | otherwise it looks OK to me | |
| 10:10:07 | jkulik | thank you both | |
| 10:10:38 | sean-k-mooney | gibi: ah using is none thats fair | |
| 10:11:27 | sean-k-mooney | an empty set woudl resutl in member bing an empty list otherwiws when then intersection is doen | |
| 10:13:01 | sean-k-mooney | jkulik: adding a release note also makes sense but i woudl suggest updating the release not in your second patch so we have one release note for the bug assuming we merge both this cycle and backport both together | |
| 10:13:36 | jkulik | ok. | |
| 10:16:45 | sean-k-mooney | jkulik: is that the only nova review you have open currently. i feel like there was another but looking at your gerrit list i dont see it | |
| 10:17:42 | sean-k-mooney | i may be miss remembering as i knwo there were a few patches that were dicussed at the ptg that i intened to loop back to this week | |
| 10:21:24 | jkulik | I don't have another open PR myself, but was wondering over in #openstack-eventlet-removal how to best handle olso.vmware's use of a Timeout in https://review.opendev.org/c/openstack/oslo.vmware/+/942109 | |
| 10:21:28 | opendevreview | Pablo Colson proposed openstack/nova stable/2025.1: wip https://review.opendev.org/c/openstack/nova/+/965902 | |
| 10:21:57 | jkulik | sean-k-mooney: maybe you're remembering the NVRAM topic, because I joined in on the discussion? it was nicolairuckel's though. | |
| 10:46:39 | opendevreview | Johannes Kulik proposed openstack/nova master: api: Pre-query not deleted members in server groups https://review.opendev.org/c/openstack/nova/+/959696 | |
| 11:09:25 | sean-k-mooney | jkulik: i think it was either that or some of noonedeadpunk topic where we said we shoudl just merge the existing patchs for iothread and live migration concurrency | |
| 11:58:19 | opendevreview | Johannes Beisiegel proposed openstack/nova master: fix cinderclient _SESSION http_retries https://review.opendev.org/c/openstack/nova/+/965914 | |
| 12:42:03 | opendevreview | Pablo Colson proposed openstack/nova stable/2025.1: wip https://review.opendev.org/c/openstack/nova/+/965902 | |
| 13:07:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]nova-alternative-configuration tests eventlet https://review.opendev.org/c/openstack/nova/+/965922 | |
| 13:07:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move monkey_patch call to each cmd from init https://review.opendev.org/c/openstack/nova/+/965923 | |
| 13:07:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Default native threading for sch, api and metadata https://review.opendev.org/c/openstack/nova/+/965924 | |
| 13:11:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Default native threading for sch, api and metadata https://review.opendev.org/c/openstack/nova/+/965924 | |
| 13:13:00 | gibi | sean-k-mooney: dansmith: ^^ as agreed on the PTG, lets switch the default early in G ^^ | |
| 13:13:25 | sean-k-mooney | oh for the threading model ack | |