Earlier  
Posted Nick Remark
#openstack-nova - 2025-10-31
14:01:28 bauzas grizzly-FriB1 is empty
14:01:47 dansmith looks like it is to me
14:02:26 Uggla bauzas sounds good to me
14:02:50 bauzas oh doh, my maths
14:03:01 bauzas this is 2pm UTC now
14:03:07 Uggla I have not booked grizzly-FriB5 because I have the secret expectation we will manage to finish early.
14:03:45 bauzas Uggla: I'll need to skip the last hour then for us due to the tc meeting
14:04:18 Uggla ok
14:19:57 gibi sean-k-mooney: we have some tempest results with threaded compute. It is not good but not terrible https://review.opendev.org/c/openstack/nova/+/965467/1#message-518b76afc6b252d2f42c48d65f69f6570d9ecb8e
14:20:08 gibi e.g. we have passing complex tests
14:20:09 gibi 2025-10-31 13:07:09.576696 | controller | {2} tempest.scenario.test_instances_with_cinder_volumes.TestInstancesWithCinderVolumes.test_instances_with_cinder_volumes_on_all_compute_nodes [348.575935s] ... ok
14:30:34 nicolairuckel sean-k-mooney, does https://gitlab.com/libvirt/libvirt/-/issues/828#note_2858916854 mean it works with a more recent Ubuntu version?
14:33:09 sean-k-mooney nicolairuckel: i have not tried your patch yet
14:33:17 sean-k-mooney but that confirm it shoudl not need to be 644
14:33:24 sean-k-mooney 600 shoudl be enough
14:33:43 sean-k-mooney nicolairuckel: ill pull in your patch a bit later today when i get time
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: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092
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: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: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:57 opendevreview Balazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional https://review.opendev.org/c/openstack/nova/+/965821
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: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: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502
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: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: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: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: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: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: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629
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: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: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:08 opendevreview melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477
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
#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: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091
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: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: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: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

Earlier   Later