| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-01-15 | |||
| 13:55:18 | ralonsoh | sean-k-mooney, not in U/S, for sure in D/S | |
| 13:55:56 | sean-k-mooney | ack | |
| 15:23:49 | jkulik | kj/cle | |
| 15:24:04 | jkulik | sorry. mistyped | |
| 16:03:43 | gmaan | gibi: ack | |
| 16:24:17 | gmaan | gibi: lgtm, but unit test failure is related, you need to fix one unit test for threading mode | |
| 16:26:59 | gmaan | I am wondering, how that test was passing on PS 14 ? there also max_worker were changed from 1000 to 5 | |
| 16:27:49 | gmaan | this is PS 14 result where it should have failed https://zuul.opendev.org/t/openstack/build/19d41e51d45a43aaa11ecf11bea7e6c7/log/job-output.txt#19666 | |
| 17:56:53 | opendevreview | sean mooney proposed openstack/nova master: FairLockGuard: Support cross-thread sharing and nesting https://review.opendev.org/c/openstack/nova/+/973438 | |
| 20:11:00 | opendevreview | Merged openstack/nova master: Migrate upper functional job to Python 3.13 https://review.opendev.org/c/openstack/nova/+/970011 | |
| #openstack-nova - 2026-01-16 | |||
| 03:05:37 | melwitt | in case anyone else runs into this, launchpad is currently down and is being worked on to fix: https://status.canonical.com | |
| 08:22:20 | amorin_ | hello team, we used to tune api-paste.ini in nova, neutron etc. to enable some custom middleware in our platform. I will have to perform the same on placement, but AFAICS placement is not using paste-deploy so that wont be possible. Do you have a rough idea on the best way to achieve similar thing in placement? | |
| 08:29:33 | jkulik | amorin: iirc, you'd have to change the code. afaics, all middleware get loaded here: https://github.com/openstack/placement/blob/5e61583504c4911ac862e1d0f2d7eb79e37066d0/placement/deploy.py#L89-L117 | |
| 08:31:05 | amorin | oh nice! | |
| 08:31:26 | amorin | thank you for the hint | |
| 08:32:46 | gibi | amorin: agree with jkulik, unfortunately placement has no configurable way to add a middleware. I think it is a valid feature request if you want to change it | |
| 08:36:03 | gibi | gmaan: I explained why PS14 was not affected but 15 is affected by that unit test failure. I will respin and fix it today https://review.opendev.org/c/openstack/nova/+/966016/15#message-5e9654d20672cf2d1cbacf6d015cfecd9de5333e | |
| 08:44:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016 | |
| 08:44:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | |
| 08:44:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 08:47:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016 | |
| 08:47:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | |
| 08:47:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 09:26:33 | amorin | thank you gibi | |
| 11:07:20 | sean-k-mooney | gibi: so on paste deploy in general its not really maintianed anymore so there has been dicssion about what we will eventually replace ti with. teh way api-paste.ini works is slightly problemtaic in that no other repalcement really supprot that kind of configuablity so one of the propasl was to remove midelware customeisation entirly or implemen a subset of it in oslo.wsgi, | |
| 11:07:22 | sean-k-mooney | that the is the new oslo project stephenfin was proposing to hlep project move. | |
| 11:07:58 | sean-k-mooney | quite a few project like placement and watcher never supproted middleware customisation so adding sopport for that today kind fo feels like addign supprot for eventlet to a project | |
| 11:08:23 | sean-k-mooney | in its current form it woudl be adding technial debt that we may not be able to maintian in the long term | |
| 11:09:23 | sean-k-mooney | so it could be a posibel feature but its not one i woudl rush to implement unit we have a long term plan for what to do with the paste/past-deploy stack | |
| 11:10:58 | sean-k-mooney | one of the suggestion i hasd was ot replace it with an oslo.config list of midelware isntead in the service .conf file or similar but i dont know if there is any concreate propals in genarl beyond the mailing list dicussion that stpehn started last year | |
| 11:15:58 | gibi | sean-k-mooney: note that I did not advocated directly for paste.ini but just for configurable middleware. I'm fine selecting a tool that is futureproof for that. | |
| 11:17:09 | sean-k-mooney | ack. its not exacty hard to implent a parser for paste.ini to supprot that as well in general | |
| 11:17:46 | sean-k-mooney | just pointing out that there is technial debt in that area so commmiting to supproting past.ini in a project is less tirvial then it would be if it was activly maintianed | |
| 11:19:12 | gibi | ack, I still think that configurable middleware is a valid feature request for placement, while I agree that we should not implement it via paste.ini as in nova due to valid maintainability concerns you raised | |
| 11:25:22 | sean-k-mooney | the oauth midelwayre that had the cve is not enabled in nova sdefault piple lien is it? | |
| 11:25:41 | sean-k-mooney | i dont see it in https://github.com/openstack/nova/blob/master/etc/nova/api-paste.ini although i dont really know what i should be lookign for | |
| 11:27:44 | sean-k-mooney | placement presumable does not use ExternalAuth2Protocol either? | |
| 11:28:06 | sean-k-mooney | well external_oauth2_token | |
| 11:30:45 | sean-k-mooney | https://bugs.launchpad.net/keystonemiddleware/+bug/2129018/comments/35 | |
| 11:31:05 | sean-k-mooney | gibi: https://bugs.launchpad.net/keystonemiddleware/+bug/2129018 is the tacker for it | |
| 11:31:18 | sean-k-mooney | it went public this week | |
| 11:32:49 | sean-k-mooney | based on zigo's comment would have to configure nova to accpate oauth2 tokens in the first place to enable the vulnerbale middlware | |
| 11:32:52 | sean-k-mooney | [pipeline:main] | |
| 11:32:54 | sean-k-mooney | pipeline = ext_oauth2_token | |
| 11:32:56 | sean-k-mooney | [filter:ext_oauth2_token] | |
| 11:32:58 | sean-k-mooney | paste.filter_factory = keystonemiddleware.external_oauth2_token:filter_factory | |
| 11:33:24 | sean-k-mooney | if that is correct the placment and watcher would nto be affected sicnce they are using hard coded midellware adn we dont config that o my knowlaage | |
| 11:38:06 | sean-k-mooney | tracker and keystone seam to be the only thigns that reference it https://codesearch.opendev.org/?q=external_oauth2_token&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 11:38:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | |
| 11:38:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | |
| 11:44:12 | gibi | sean-k-mooney: yeah that is also how I understood the last comment in that bug | |
| 12:26:20 | opendevreview | sean mooney proposed openstack/nova master: [WIP] Add PCI device groups for atomic allocation https://review.opendev.org/c/openstack/nova/+/973604 | |
| 13:15:26 | opendevreview | Merged openstack/os-vif master: Stabilize functional test https://review.opendev.org/c/openstack/os-vif/+/973455 | |
| 13:17:49 | opendevreview | Bodo Petermann proposed openstack/os-vif master: Fixed bridge name when per_port_bridge is used https://review.opendev.org/c/openstack/os-vif/+/966410 | |
| 13:27:41 | zigo | sean-k-mooney: I'd need to have someone confirm, that's my understand from the oauth2 doc at https://docs.openstack.org/keystonemiddleware/latest/middlewarearchitecture.html#configuration-for-external-authorization only. | |
| 14:11:31 | opendevreview | Johannes Kulik proposed openstack/nova master: WIP: Attaching a volume returns HTTP 202 https://review.opendev.org/c/openstack/nova/+/971068 | |
| 14:15:34 | opendevreview | Masanori Kuroha proposed openstack/nova master: Copy applied provider config https://review.opendev.org/c/openstack/nova/+/948304 | |
| 14:35:55 | opendevreview | Lajos Katona proposed openstack/nova master: blueprint: iothreads-for-instances https://review.opendev.org/c/openstack/nova/+/939254 | |
| 15:01:42 | opendevreview | Johannes Kulik proposed openstack/nova master: Attaching a volume returns HTTP 202 https://review.opendev.org/c/openstack/nova/+/971068 | |
| 16:32:50 | opendevreview | Merged openstack/placement master: Migrate upper functional job to Python 3.13 https://review.opendev.org/c/openstack/placement/+/970052 | |
| 18:24:46 | opendevreview | sean mooney proposed openstack/nova master: [WIP] Add PCI device groups for atomic allocation https://review.opendev.org/c/openstack/nova/+/973604 | |
| 18:36:11 | opendevreview | sean mooney proposed openstack/nova master: Support os-vif TAP pre-creation for OVS/OVN ports https://review.opendev.org/c/openstack/nova/+/973149 | |
| 18:46:55 | gmaan | gibi: +w, actually I checked master version of that test and did not realized that commit changed the 1000 value to 5 in test also. all good. | |
| 18:50:37 | sean-k-mooney | gmaan: you might take a look at https://review.opendev.org/c/openstack/nova/+/973438 if you have time. it should fix the test instablity in threading mode | |
| 18:51:11 | sean-k-mooney | it can wait till next week but you reviewed the orgianl so you likely have more context then most | |
| 18:51:24 | gmaan | sean-k-mooney: ack, not sure I can check today but will take a look later | |
| 18:51:30 | gmaan | sure | |
| 18:52:25 | sean-k-mooney | no worries its getting late on a firday so im goign to be heading off soon anyway | |
| 18:53:11 | gmaan | yup, have a nice weekend | |
| 21:03:30 | opendevreview | Merged openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016 | |
| #openstack-nova - 2026-01-18 | |||
| 10:50:57 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Extend functional test coverage of UEFI boot guests https://review.opendev.org/c/openstack/nova/+/969263 | |
| 10:52:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | |
| 10:52:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | |
| 10:52:35 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load ssm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | |
| 11:11:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmware auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | |
| #openstack-nova - 2026-01-19 | |||
| 07:04:56 | opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | |
| 07:50:54 | do3meli | hi, can someone please help to get workflows triggered for https://review.opendev.org/c/openstack/nova/+/972276 ? | |
| 08:01:04 | stephenfin | gibi: o/ any chance you can look at https://review.opendev.org/c/openstack/nova/+/971460 again? I replied to your question | |
| 08:35:03 | opendevreview | Merged openstack/nova-specs master: Create specs directory for 2026.2 Hibiscus https://review.opendev.org/c/openstack/nova-specs/+/973373 | |
| 08:43:45 | gibi | stephenfin: done. And as I see we did it in parallel with bauzas :) so you are now double approved | |
| 08:43:50 | bauzas | lol | |
| 08:44:40 | bauzas | tkajinam: I'll look at gibi's comments today, you'll get a merge conflict on the top of your series due to the fact we just approved another unrelated patch fyi | |
| 08:50:16 | gibi | bauzas: is the conflict due to the parallel refactor series? If so we can decide on the landing order to limit the number of conflicts | |
| 08:53:21 | bauzas | gibi: I explained it in the top-level comment for 971460, my opinion is that we can quickly merge stephenfin's patch because tkajinam's series is still actively under review | |
| 08:54:18 | bauzas | gibi: I just saw you made good comments on the firmware autodetection patches, mostly the one about the default XML setting we should or shouldn't set | |
| 08:54:22 | gibi | ahh OK so it is not the mem encryption context refactor | |
| 08:55:02 | bauzas | nope | |
| 08:55:06 | gibi | cool then | |
| 08:55:38 | gibi | stephenfin's patch is landing and that is just a single patch so no further decision needed | |
| 08:59:37 | opendevreview | Johannes Beisiegel proposed openstack/nova master: Adds packing_host_pci_numa_cells_allocation_strategy option https://review.opendev.org/c/openstack/nova/+/952451 | |
| 09:01:10 | opendevreview | Johannes Beisiegel proposed openstack/nova master: Adds packing_host_pci_numa_cells_allocation_strategy option https://review.opendev.org/c/openstack/nova/+/952451 | |
| 09:11:24 | tkajinam | bauzas, no problem about rebase. I'll do it once that patch is merged | |
| 10:17:56 | opendevreview | Merged openstack/nova stable/2025.2: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/972276 | |
| 10:54:28 | gibi | stephenfin: when you have a minute I updated https://review.opendev.org/c/openstack/nova/+/969936 | |
| 10:55:51 | stephenfin | done | |
| 10:56:03 | gibi | thanks | |
| 11:12:54 | opendevreview | Merged openstack/nova master: libvirt: Remove import hacks https://review.opendev.org/c/openstack/nova/+/971460 | |
| 11:23:27 | opendevreview | Merged openstack/nova master: mem-enc: create generic check for mem encryption support by host https://review.opendev.org/c/openstack/nova/+/967969 | |
| 12:47:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Extend functional test coverage of UEFI boot guests https://review.opendev.org/c/openstack/nova/+/969263 | |
| 12:47:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | |