| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-14 | |||
| 13:04:41 | sean-k-mooney | we could clear the cache when we detach an interface | |
| 13:05:00 | sean-k-mooney | we could refuse to update the capablities if the device is not in the aviable sate | |
| 13:07:03 | bbezak | I'm wondering if we could look to existed.extra_info for capabilities | |
| 13:07:07 | bbezak | but maybe that is too narrow | |
| 13:07:08 | bbezak | thinking | |
| 13:07:10 | bbezak | :) | |
| 13:07:17 | bbezak | here https://github.com/openstack/nova/blob/1ad11b13884baeaa6ed9f8f5818f4d176f4d3134/nova/pci/manager.py#L271-L289 | |
| 13:07:56 | sean-k-mooney | you mean merge the values form the db with those form libvirt and have teh db ones take precidence while its in claimmed or allocated | |
| 13:11:28 | gibi | Uggla: here is my eventlet removal summary https://gibizer.github.io/posts/Eventlet-Removal-Flamingo-PTG/ feel free to link to it in you PTG summary mail | |
| 13:11:31 | bbezak | sth like that | |
| 13:11:51 | bbezak | I'm not taking account other pci devices that may don | |
| 13:12:09 | bbezak | I'm not taking into account other pci devices that may don't like that | |
| 13:12:13 | bbezak | I guess | |
| 13:27:58 | sean-k-mooney | bbezak: sorry on a call | |
| 13:28:10 | bbezak | no rush! | |
| 13:29:24 | sean-k-mooney | bbezak: we likely need to experiment with what is the correct approch. | |
| 13:45:16 | bbezak | yeah | |
| 14:07:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| 14:11:08 | opendevreview | sean mooney proposed openstack/nova master: Remove workaround for ovn live migration https://review.opendev.org/c/openstack/nova/+/946950 | |
| 14:28:28 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 15:00:47 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.2: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932522 | |
| 22:59:22 | opendevreview | Merged openstack/nova stable/2024.2: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932522 | |
| #openstack-nova - 2025-04-15 | |||
| 03:49:02 | melwitt | gibi, sean-k-mooney, Uggla: just a fyi that I pushed a new PS on the volume disconnect connection_info patch to not compare connection_info and instead handle disconnect on the destination itself. please lmk if you spot any problems in the idea https://review.opendev.org/c/openstack/nova/+/946600 | |
| 05:43:07 | opendevreview | Merged openstack/nova stable/2025.1: wrap wsgi_app.init_application with latch_error_on_raise https://review.opendev.org/c/openstack/nova/+/945439 | |
| 05:43:13 | opendevreview | Merged openstack/nova stable/2024.1: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/946974 | |
| 05:43:22 | opendevreview | Merged openstack/nova stable/2024.1: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944010 | |
| 05:43:32 | opendevreview | Merged openstack/nova stable/2024.1: Fix deepcopy usage for BlockDeviceMapping in get_root_info https://review.opendev.org/c/openstack/nova/+/926696 | |
| 05:43:44 | opendevreview | Merged openstack/nova stable/2024.1: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/945816 | |
| 05:54:15 | opendevreview | Yaguang Tang proposed openstack/nova stable/2023.2: Fix deepcopy usage for BlockDeviceMapping in get_root_info https://review.opendev.org/c/openstack/nova/+/947185 | |
| 05:59:55 | opendevreview | Yaguang Tang proposed openstack/nova stable/2023.2: Fix deepcopy usage for BlockDeviceMapping in get_root_info https://review.opendev.org/c/openstack/nova/+/926697 | |
| 06:05:29 | opendevreview | Yaguang Tang proposed openstack/nova stable/2023.2: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/945903 | |
| 08:07:37 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2023.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944011 | |
| 08:09:43 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2023.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944011 | |
| 09:19:36 | masahito | sean-k-mooney: hi, if you have time, please review the no request_spec bug fix https://review.opendev.org/c/openstack/nova/+/939658 | |
| 11:00:24 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | |
| 11:04:45 | opendevreview | Elod Illes proposed openstack/nova stable/2024.1: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932532 | |
| 11:10:23 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | |
| 11:11:22 | opendevreview | Merged openstack/nova stable/2023.2: [Stable-Only]Make CastAsCall configurable https://review.opendev.org/c/openstack/nova/+/934434 | |
| 11:11:31 | opendevreview | Merged openstack/nova stable/2023.2: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934305 | |
| 11:28:22 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add remove-os-volumes_boot-api spec https://review.opendev.org/c/openstack/nova-specs/+/947210 | |
| 11:45:40 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet sleep with time.sleep https://review.opendev.org/c/openstack/nova/+/947212 | |
| 11:48:57 | opendevreview | Stefan K proposed openstack/nova-specs master: Add Cloud Hypervisor support spec https://review.opendev.org/c/openstack/nova-specs/+/945549 | |
| 11:50:25 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.2: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/947214 | |
| 11:50:46 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | |
| 11:53:50 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | |
| 12:28:42 | opendevreview | Merged openstack/nova stable/2023.2: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934306 | |
| 12:36:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove superfluous monkey patching form func test https://review.opendev.org/c/openstack/nova/+/947237 | |
| 13:05:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet based WSGI server entry points https://review.opendev.org/c/openstack/nova/+/947240 | |
| 13:30:35 | sean-k-mooney | gibi: oh we did? deprecate it? | |
| 13:30:40 | gibi | yepp in Rocky | |
| 13:30:45 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/549510/3/nova/service.py | |
| 13:30:52 | gibi | so I'm ripping it out with pleasure | |
| 13:30:55 | sean-k-mooney | ok i didnt think we actully had a warning | |
| 13:30:59 | sean-k-mooney | works for me | |
| 13:32:13 | gibi | more cleanups will come top of that patch just need a bit of massage in the functional test as the only user of WSGIServer and WSGIService is the nova-api-metadata fixture in the function env. I will re-implement that and then we can remove a bunch of stuff from nova.service | |
| 13:33:13 | sean-k-mooney | cool | |
| 13:33:34 | sean-k-mooney | im find with small incremental patchs for this so keep them comming | |
| 13:33:58 | sean-k-mooney | by the way we might alos have some docs we can clean up for this | |
| 13:34:17 | gibi | good point about the docs... | |
| 13:34:21 | gibi | I need to look | |
| 13:34:33 | sean-k-mooney | https://github.com/openstack/nova/blob/1ad11b13884baeaa6ed9f8f5818f4d176f4d3134/doc/source/cli/nova-api.rst#L2 | |
| 13:34:42 | sean-k-mooney | so we have cli docs for the entry points | |
| 13:35:13 | sean-k-mooney | but i think we havce other ferences in the admin or install guide | |
| 13:35:29 | gibi | thanks | |
| 13:35:50 | sean-k-mooney | like https://github.com/openstack/nova/blob/1ad11b13884baeaa6ed9f8f5818f4d176f4d3134/doc/source/admin/support-compute.rst?plain=1#L54 | |
| 13:36:51 | sean-k-mooney | if you search for nova-api and nova-api-metadta you will see all the relevent sections | |
| 13:37:15 | sean-k-mooney | the docs cleanup cna be in a follow up patch if you dont want to add them in the removal patch | |
| 13:37:49 | gibi | I think I will add them in the removal patch as the doc refers to the entry point I deleted | |
| 13:38:02 | sean-k-mooney | some refrences ot nova-api are actully refering to the db so we cant just nuke all fo them without checking each one unfortunetly | |
| 13:38:12 | gibi | yeah | |
| 13:38:51 | gibi | also the guru mediation doc needs to be reprhased GMR for nova-api is still a thing | |
| 13:39:30 | sean-k-mooney | yep it is but we should likely default to documenting using the file based approch to triggering it | |
| 13:39:40 | sean-k-mooney | you can do sig_usr2 with uwsgi | |
| 13:39:50 | sean-k-mooney | but the file based approch is more universal | |
| 13:39:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove WSGIService and WSGIServer classes https://review.opendev.org/c/openstack/nova/+/947245 | |
| 13:41:06 | gibi | yeah file based is the way | |
| 13:41:18 | gibi | I learned that the hard way during nova-operator / must-gather work :) | |
| 13:42:10 | sean-k-mooney | well it nice ot share those hard earned lessosn while we are updating the docs anyway | |
| 13:44:20 | gibi | yepp | |
| 13:44:24 | gibi | totally agree | |
| 14:30:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove WSGIService and WSGIServer classes https://review.opendev.org/c/openstack/nova/+/947245 | |
| 14:37:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove WSGIService and WSGIServer classes https://review.opendev.org/c/openstack/nova/+/947245 | |
| 15:01:14 | Uggla | Nova meeting in ~1h. | |
| 15:03:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove nova.service.process_launcher https://review.opendev.org/c/openstack/nova/+/947260 | |
| 15:12:13 | elodilles | :) | |
| 15:40:33 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: Fix deepcopy usage for BlockDeviceMapping in get_root_info https://review.opendev.org/c/openstack/nova/+/926697 | |
| 15:40:34 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/945903 | |
| 15:41:18 | Uggla | Nova meeting in ~20mn. | |
| 15:58:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [hacking] N373 do not use eventlet primitives https://review.opendev.org/c/openstack/nova/+/947265 | |
| 16:00:27 | Uggla | #startmeeting nova | |
| 16:00:27 | opendevmeet | Meeting started Tue Apr 15 16:00:27 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:27 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:27 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:47 | Uggla | Hello everyone | |
| 16:00:52 | gibi | o/ | |
| 16:01:16 | masahito | o/ | |
| 16:01:20 | Uggla | Note: I have a contraint a the top of the hour. So let's try to keep this meeting no more than 1h. | |
| 16:01:59 | elodilles | o/ | |
| 16:02:04 | Uggla | awaiting a moment for people to join. | |
| 16:03:16 | Uggla | #topic Bugs (stuck/critical) | |