| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-05-27 | |||
| 16:56:21 | Uggla | To my mind that sounds a valid one. At least regarding the comment from Brian. | |
| 16:58:29 | gibi | db error in the neutron code, this is a neutron bug | |
| 16:58:49 | gibi | at least neutron should not return 500 | |
| 16:59:14 | sean-k-mooney | so we had a vm create request with --network | |
| 16:59:28 | sean-k-mooney | it existsed at the api but by the time we got to the compute it as deleteed | |
| 16:59:48 | sean-k-mooney | i guesss it coudl happen when we do the quota check as well | |
| 17:00:01 | gmaan | and nova do capture NetworkNotFound exception if it is not found and requested https://github.com/openstack/nova/blob/221a3e89e8988bc664298106ee691a4e41ca71f9/nova/api/openstack/compute/servers.py#L842 | |
| 17:00:02 | sean-k-mooney | in anycase ya we shoudl get back a 400 not a 500 | |
| 17:00:43 | gmaan | or somewhere neutron exception is lost and not raised to controller ? | |
| 17:00:47 | sean-k-mooney | the expected behivor here shold be that nova puts the vm in error | |
| 17:01:03 | sean-k-mooney | and when the vm is deleted we would clean up any prot that were created then | |
| 17:01:09 | sean-k-mooney | based on teh delete on termeinate behaivor | |
| 17:01:20 | gmaan | yeah, bug say ignore the deleted network port and continue VM creation which does not seems right | |
| 17:01:42 | sean-k-mooney | its not form a nova point of view because we could not honner the orginal request | |
| 17:02:33 | sean-k-mooney | so depending on where this failed it soudl be burried in cell 0, rejected at the quoata check with a 400 or in error in the cell db if it got all the way to the compute | |
| 17:04:05 | sean-k-mooney | reading it more careflyy im tempeted to mark it inviald in nova | |
| 17:04:35 | sean-k-mooney | in there case its the quota check that is failing whne we list all prots for a tenatnt | |
| 17:05:05 | Uggla | sean-k-mooney, if you are unsure we can keep it open for next week. | |
| 17:05:26 | sean-k-mooney | althogh if we are missing the try accpate on teh network not found then thats a valid bug i guess | |
| 17:07:18 | Uggla | sale for valid ? | |
| 17:07:55 | gmaan | I have not tracked the exact call of validate_networks() but nova handle NetworkNotFound in _validate_requested_network_ids | |
| 17:07:55 | gmaan | https://github.com/openstack/nova/blob/221a3e89e8988bc664298106ee691a4e41ca71f9/nova/network/neutron.py#L988 | |
| 17:09:06 | gmaan | does bug mention 500 is returned? or 400 ? | |
| 17:09:14 | sean-k-mooney | gmaan: i think neturon is returnign a 500 | |
| 17:09:17 | sean-k-mooney | https://paste.openstack.org/show/bt3BWnyrBbQspiFx8KPZ/ | |
| 17:09:39 | gmaan | ohk | |
| 17:09:51 | sean-k-mooney | i think we need more info to be sure | |
| 17:10:13 | sean-k-mooney | the do have a NetworkNotFound excption https://paste.openstack.org/show/bJL9b5aY99XpZlLZdzgy/ | |
| 17:10:15 | gmaan | I think nova hanlding the error correctly if neuton return NetworkNotFound | |
| 17:10:35 | gmaan | and controller also handle NetworkNotFound so 400 will be return | |
| 17:11:21 | sean-k-mooney | can you comment to that effect on the bug | |
| 17:11:47 | gmaan | sure, will do after TC meeting | |
| 17:11:58 | sean-k-mooney | the first step to fixing this if we were to fix it in nova woudl be a repoducer test anyway | |
| 17:12:03 | sean-k-mooney | iesally a functional one | |
| 17:13:45 | Uggla | sean-k-mooney, does that mean we can set it as valid ? | |
| 17:15:43 | Uggla | I need to close the meeting, so I propose to look at that bug offline later or next week. | |
| 17:15:47 | Uggla | Thanks all and thanks for the extended time for bug triage. | |
| 17:15:54 | gmaan | Uggla: I will check exact trace of validate_network and see if nova missing NetworkNotFound handling anywhere | |
| 17:15:54 | Uggla | #endmeeting | |
| 17:15:54 | opendevmeet | Meeting ended Tue May 27 17:15:54 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:15:54 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.html | |
| 17:15:54 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.txt | |
| 17:15:54 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.log.html | |
| 17:15:57 | gmaan | and comment in bug | |
| 17:16:02 | gmaan | aftr TC meeting | |
| 17:16:37 | Uggla | thanks gmaan | |
| 17:16:52 | Uggla | Sorry I close the meeting a bit too quickly | |
| 17:17:47 | Uggla | Anyway time for me to afk, see you tomorrow. | |
| 17:17:50 | gmaan | its fine, actually 17 UTC, TC meeting start so difficult to respond here on time :) | |
| 17:19:19 | Uggla | no worries, I think I would close the meeting without bug triage because we were almost on top of the hour. | |
| 17:21:35 | sean-k-mooney | i set it to incomplete | |
| 17:21:39 | sean-k-mooney | we can come back next week | |
| 17:22:00 | Uggla | ok sean-k-mooney | |
| 17:22:03 | sean-k-mooney | but i suspect if the neutorn issue is fixed nova will do the right thing | |
| 21:04:30 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | |
| 21:04:30 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 21:09:05 | opendevreview | Callum Dickinson proposed openstack/nova master: Add more flavor metadata to libvirt guest XML https://review.opendev.org/c/openstack/nova/+/942974 | |
| 21:09:49 | Callum027 | sean-k-mooney: I know it's not strictly necessary but I've rebased the branches to make sure they're fine to merge with the newer master branch | |
| 21:10:05 | Callum027 | (the image meta blueprint ones, and the bugfix for image ID in libvirt metadata) | |
| 21:10:32 | Callum027 | Also added some more detail to the release notes as discussed in the meeting | |
| #openstack-nova - 2025-05-28 | |||
| 05:12:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused validator for uri element https://review.opendev.org/c/openstack/nova/+/951084 | |
| 09:05:39 | opendevreview | DongHun, Kim proposed openstack/nova master: Fix PCI device mapping leak after evacuate for GPU VMs https://review.opendev.org/c/openstack/nova/+/951099 | |
| 09:09:34 | opendevreview | DongHun, Kim proposed openstack/nova master: Fix PCI device mapping leak after evacuate for GPU VMs https://review.opendev.org/c/openstack/nova/+/951099 | |
| 10:26:33 | sean-k-mooney | Callum027: thanks ill see if i can review them this morning before i get distracted by other things | |
| 10:27:19 | sean-k-mooney | Callum027: i would suggest putting https://review.opendev.org/c/openstack/nova/+/942974 in the same seriese as the rest given its indicating it will be in merge conflcit with it | |
| 10:28:57 | sean-k-mooney | perhaps put the image ones on top or put this on top of them either way works | |
| 12:04:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/951007 | |
| 12:04:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | |
| 12:24:51 | opendevreview | DongHun, Kim proposed openstack/nova master: Fix PCI device mapping leak after evacuate for GPU VMs https://review.opendev.org/c/openstack/nova/+/951099 | |
| 12:46:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/951007 | |
| 12:46:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | |
| 12:46:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/951129 | |
| 12:50:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/951129 | |
| 12:50:31 | gibi | dansmith: ^^ FYI | |
| 13:22:27 | dansmith | gibi: nice | |
| 13:48:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Validated that PCI alias has proper ids https://review.opendev.org/c/openstack/nova/+/951129 | |
| 13:48:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Validate [pci]alias at service startup https://review.opendev.org/c/openstack/nova/+/951143 | |
| 14:49:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Validate [pci]alias at service startup https://review.opendev.org/c/openstack/nova/+/951143 | |
| 14:49:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/951151 | |
| 14:50:22 | gibi | sean-k-mooney: ^^ with caching included :) | |
| 15:44:37 | sean-k-mooney | gibi: very nice. il take a look | |
| 21:34:43 | Callum027 | sean-k-mooney: Ah, right, of course, good idea, I'll do that shortly (and hopefully Gerrit starts working for me soon because I'm getting load failures) | |
| 23:19:01 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | |
| 23:19:02 | opendevreview | Callum Dickinson proposed openstack/nova master: Add more flavor metadata to libvirt guest XML https://review.opendev.org/c/openstack/nova/+/942974 | |
| 23:19:02 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| #openstack-nova - 2025-05-29 | |||
| 00:15:52 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 00:23:12 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service role spec https://review.opendev.org/c/openstack/nova-specs/+/951218 | |
| 00:23:42 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Re-propose API policy service role spec https://review.opendev.org/c/openstack/nova-specs/+/951218 | |
| 00:27:09 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 00:34:33 | gmaan | sean-k-mooney: ^^ proposed the manager role spec (fixing the review comment from code/last cycle review) and also re-proposing the service role spec, please review. proposed both spec separately | |
| 00:37:00 | gmaan | I added the new policies proposal also to filter out the host-thing from manager permission | |
| 02:43:10 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 03:51:48 | opendevreview | sean mooney proposed openstack/nova-specs master: Add cpu-teiring with cpu_shares spec https://review.opendev.org/c/openstack/nova-specs/+/951222 | |
| 05:05:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace license classifier https://review.opendev.org/c/openstack/nova/+/951226 | |
| 05:20:04 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 06:47:31 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 09:09:51 | roger_wang | I wonder if anyone could review this patch of Improve Error Handling for Placement API Responses (https://review.opendev.org/c/openstack/nova/+/943530). much appreciated. | |
| 13:09:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace license classifier https://review.opendev.org/c/openstack/nova/+/951226 | |
| 16:11:06 | opendevreview | Merged openstack/nova-specs master: Re-propose vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/947542 | |
| 16:42:46 | gmaan | sean-k-mooney: thanks for reviews on RBAC specs, I would like to highlight this and if you feel strong to change it? https://review.opendev.org/c/openstack/nova-specs/+/951218/2/specs/2025.2/approved/policy-service-role-default.rst#49 | |