| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-09-04 | |||
| 20:07:02 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 20:07:03 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Only load same-host instances when checking share usage https://review.opendev.org/c/openstack/nova/+/1004057 | |
| 20:39:42 | gouthamr | s | |
| 21:10:35 | dansmith | melwitt: looks okay to me | |
| 21:16:25 | gouthamr | dansmith: do you mind me folding follow up comments into a single commit post these merging | |
| 21:16:39 | gouthamr | like a low-risk follow up that'll land next week? | |
| 21:16:48 | dansmith | which comments? | |
| 21:17:22 | gouthamr | you're right about "is_mounted()".. it's a mount path that we're check | |
| 21:17:25 | gouthamr | checking* | |
| 21:17:35 | dansmith | oh yeah that's why I said "not critical to fix right now" | |
| 21:18:07 | gouthamr | ++ ack, ty will watch for melwitt's comments as well | |
| 21:39:29 | melwitt | will be looking | |
| 21:42:14 | gmaan | dansmith: thanks for review on task tracking fix, I replied to your commment https://review.opendev.org/c/openstack/nova/+/1003386/comments/25784b40_6a5b7aad | |
| 21:42:27 | gmaan | if we are ok to do RPC versioning bupm, | |
| 21:43:22 | gmaan | * RPC version bump, then i can add a new RPC cast to end the task which will not impact the dest rollback for old compute mixed env | |
| 21:43:55 | gmaan | I was thinking to handle that generacally in part-3 | |
| #openstack-nova - 2026-09-05 | |||
| 01:04:32 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Fix the test_migrate_disk_and_power_off_crash_finish_revert_migration https://review.opendev.org/c/openstack/nova/+/1003910 | |
| 01:16:20 | opendevreview | Ghanshyam Maan proposed openstack/nova master: [func test]Catch hanging task at graceful shutdown https://review.opendev.org/c/openstack/nova/+/1003266 | |
| 01:43:01 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | |
| 01:44:51 | gmaan | gibi: ^^ i saw there were lot of threadpool factory logs in functional threading jobs, fixing it ^^ | |
| 02:11:01 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Reduce the unnecessary executors shutdown logs in test jobs https://review.opendev.org/c/openstack/nova/+/1004160 | |
| 05:46:34 | opendevreview | Merged openstack/nova master: Use a per-instance cephx identity for CephFS shares https://review.opendev.org/c/openstack/nova/+/1001900 | |
| 22:37:16 | opendevreview | Merged openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | |
| #openstack-nova - 2026-09-06 | |||
| 02:29:29 | opendevreview | Merged openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 19:45:49 | opendevreview | Ivan Anfimov proposed openstack/os-vif master: Fix typo https://review.opendev.org/c/openstack/os-vif/+/1004326 | |
| 19:46:37 | opendevreview | Ivan Anfimov proposed openstack/os-vif master: Fix typo https://review.opendev.org/c/openstack/os-vif/+/1004326 | |
| 20:23:29 | opendevreview | Merged openstack/nova master: Reconcile stale Manila share access rules https://review.opendev.org/c/openstack/nova/+/1001916 | |
| 21:02:47 | opendevreview | Merged openstack/nova master: Only load same-host instances when checking share usage https://review.opendev.org/c/openstack/nova/+/1004057 | |
| 23:51:20 | opendevreview | Ivan Anfimov proposed openstack/os-vif master: Fix typos https://review.opendev.org/c/openstack/os-vif/+/1004326 | |
| #openstack-nova - 2026-09-07 | |||
| 07:12:27 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace StaticallyDelayingWrapper with futurist DelayingExecutor https://review.opendev.org/c/openstack/nova/+/997410 | |
| 08:17:38 | opendevreview | HyungJoon Kim proposed openstack/nova master: Add regression test for bug #2166652 https://review.opendev.org/c/openstack/nova/+/1004388 | |
| 08:48:36 | gibi | stephenfin: Uggla: do we need to bump the min version of os-vif in nova to 5.2.1? https://github.com/openstack/nova/blob/master/requirements.txt#L54 | |
| 10:06:17 | stephenfin | gibi: Unless we need some particular feature that was added recently, I don't think we have to. The packaging issues was only present in 5.2.0 which won't get dragged in by default | |
| 10:07:46 | stephenfin | 3.1.0 is nearly 4 years old https://pypi.org/project/os-vif/3.1.0/ There's a high possibility it doesn't actually work, but we could likely say the same about a lot of our minimums? | |
| 10:08:28 | stephenfin | So maybe we could bump that, or maybe we should just go and bump to the latest major version (X.0.0) for all our deps? idk | |
| 11:10:13 | opendevreview | HyungJoon Kim proposed openstack/nova master: Sort claimed PCI devices by host PCI address https://review.opendev.org/c/openstack/nova/+/1004411 | |
| 11:33:35 | opendevreview | Merged openstack/os-vif master: Fix typos https://review.opendev.org/c/openstack/os-vif/+/1004326 | |
| 12:55:11 | Uggla | gouthamr, dansmith, melwitt well done for the virtiofs cold migration! 👍 | |
| 13:17:58 | opendevreview | Emma Foley proposed openstack/placement master: [pyproject] Update setuptools config https://review.opendev.org/c/openstack/placement/+/1004418 | |
| 13:18:02 | sean-k-mooney | Uggla: an fyi we may need to fix one packaging issue in placement before we cut rc1 ^ | |
| 13:18:14 | sean-k-mooney | or do an RC2 if we have already done that | |
| 13:18:44 | sean-k-mooney | emma basiclaly is portign the same fix that stephenfin did for nova https://github.com/openstack/nova/commit/3881260b1cb179cd0f8bb2a37282325a5f5fb24a | |
| 13:22:25 | sean-k-mooney | i also fixed this in cybrog watcher as well | |
| 13:22:48 | sean-k-mooney | we shoudl proably check all the pyproject.tomls for this | |
| 13:23:39 | sean-k-mooney | looks like os-vif might be impacted as well https://github.com/openstack/os-vif/blob/master/pyproject.toml#L40-L46 | |
| 13:23:49 | sean-k-mooney | stephenfin: ^ we shoudl fix that too ya? | |
| 13:24:09 | stephenfin | ideally yes | |
| 13:25:06 | stephenfin | Though as I noted in the nova commit message you linked, setuptools currently provides a fallback. It just spews a lot of warnings, and there's a good likelihood the PyPA folks will remove it at some point (they love deleting code more than I do) | |
| 13:25:34 | sean-k-mooney | it was apprently breaking something downstream when we were buidlign placement | |
| 13:25:44 | sean-k-mooney | but for the libs that less of a problem i think | |
| 13:26:10 | sean-k-mooney | so i think we cna fix it in os-vif and the other repos but its less impactful there | |
| 13:26:28 | sean-k-mooney | os-vif is also a bit weired in that we have 3 seperate packages | |
| 13:26:36 | sean-k-mooney | the fined shoudl still work fine | |
| 13:26:47 | sean-k-mooney | but its a littel diffent then placment or nova | |
| 13:27:47 | gibi | stephenfin: OK, so we don't need 5.2.1 os-vif as minimum just need to ignore 5.2.0 which was yanked I guess so it won't get pulled in | |
| 13:28:13 | stephenfin | yeah, there's no reason for 5.2.0 to get pulled in | |
| 13:28:29 | stephenfin | we could yank if we wanted but 🤷 | |
| 13:28:37 | gibi | cool then I think we don't need to risk RC1 to change the minimums now | |
| 13:28:52 | gibi | even though I would like to have better minimums | |
| 13:28:58 | sean-k-mooney | i dont think we added any hard dep on the new version | |
| 13:29:02 | gibi | I just never had the time | |
| 13:29:03 | sean-k-mooney | it fixes some bugs | |
| 13:29:35 | gibi | as of the placement pyproject.toml change, it fails if placement is used as a git submodule | |
| 13:29:37 | sean-k-mooney | but i dont think we are depending on a new fucntionaltiy although neutorn will be enabling a new feature by default | |
| 13:29:41 | gibi | probably to wide matching happens | |
| 13:29:44 | stephenfin | sean-k-mooney: My comment was less about e.g. 5.1.x -> 5.2.x than it was about 3.1.x -> 5.2.x | |
| 13:30:10 | sean-k-mooney | right i dont hink we striclly need to go form 3.1.x to 5.x | |
| 13:30:17 | sean-k-mooney | we shoudl eventully do that | |
| 13:30:35 | sean-k-mooney | as next cycle neuton are removign supprot for disabling a feature that will require the new version in nova | |
| 13:31:12 | sean-k-mooney | today you can configre neuton to work with the older os-vif on the nova side | |
| 13:31:31 | sean-k-mooney | they are makign the new feature the default this cycle | |
| 13:31:46 | sean-k-mooney | so it woudl be good to reaise our min sooner rather then later | |
| 13:49:50 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Use constraints option https://review.opendev.org/c/openstack/placement/+/996893 | |
| 13:49:51 | opendevreview | Stephen Finucane proposed openstack/placement master: Migrate requirements to pyproject.toml https://review.opendev.org/c/openstack/placement/+/1004423 | |
| 13:49:51 | opendevreview | Stephen Finucane proposed openstack/placement master: Replace license classifier https://review.opendev.org/c/openstack/placement/+/1004422 | |
| 13:50:45 | opendevreview | Stephen Finucane proposed openstack/placement master: [pyproject] Update setuptools config https://review.opendev.org/c/openstack/placement/+/1004418 | |
| 13:50:46 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Use constraints option https://review.opendev.org/c/openstack/placement/+/996893 | |
| 13:50:46 | opendevreview | Stephen Finucane proposed openstack/placement master: Replace license classifier https://review.opendev.org/c/openstack/placement/+/1004422 | |
| 13:50:47 | opendevreview | Stephen Finucane proposed openstack/placement master: Migrate requirements to pyproject.toml https://review.opendev.org/c/openstack/placement/+/1004423 | |
| 13:55:22 | sean-k-mooney | stephenfin: i assume we will do most of ^ early next cycle | |
| 13:56:02 | stephenfin | the license classifer might be worth doing now, since that's another thing PyPA might pull at a moments notice, but we could backport it later also. The rest can wait | |
| 13:56:09 | sean-k-mooney | that the one i think we shoudl hold | |
| 13:56:12 | sean-k-mooney | well not hold | |
| 13:56:23 | sean-k-mooney | but it was nto suprpoted on older distos | |
| 13:56:33 | sean-k-mooney | right we had to bump to ubuntu 24.04 | |
| 13:56:41 | sean-k-mooney | to have the string form supproted in our package tools | |
| 13:56:55 | sean-k-mooney | i dotn knwo if cents 10 stream ships a new enough setuptools? | |
| 13:57:17 | sean-k-mooney | we just swaped to usign it for pypi in our source to image builds so that hsoudl be fine | |
| 13:57:21 | sean-k-mooney | i just dont knwo about rdo | |
| 13:57:31 | sean-k-mooney | not that that really activly maintianed right now | |
| 13:58:31 | sean-k-mooney | so ya it may jsut be worth checkign the versoin requrieemnt for that before we proced with it | |
| 13:58:42 | sean-k-mooney | that said we have moved a numebr of things to the string already | |
| 13:58:57 | sean-k-mooney | so that ship may have already sailed | |
| 13:59:01 | stephenfin | sean-k-mooney: not quite. The issue was it requires setuptools >= 77.0.0 https://setuptools.pypa.io/en/latest/userguide/license_migration.html | |
| 13:59:12 | stephenfin | and that only supports Python >= 3.9 | |
| 13:59:40 | sean-k-mooney | right but it was capped in rdo, i knwo we had to go fix that btu i dont recall if it was merged | |
| 13:59:51 | sean-k-mooney | i think it was but not entirly sure | |
| 13:59:54 | stephenfin | the change you're thinking of was that we had to move off Ubuntu 20.04 since that ships with Python 3.8 by default | |
| 14:00:07 | stephenfin | move the packaging job off that, more specifically | |
| 14:00:18 | sean-k-mooney | ah it was https://review.rdoproject.org/r/c/deps/python-setuptools/+/58844 | |