| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-09-04 | |||
| 19:21:46 | gouthamr | from my (probably biased) vantage point, the reconciliation of rules because we're trying not to wedge instances on cold migration is good hardening.. cold migration will be used no doubt if we shipped this even without the reconciliation - but, we're doing this right and i wouldn't do it differently.. | |
| 19:22:59 | gouthamr | I know of a few <redacted> clouds that would be happy to have this feature land in H.. but they're all really waiting for a day their can hot-attach cephfs and call it a day - it's killing their cloud right now when person 100001 is coming to them to enable this, and they have to tell them they'd have to roll out their own tenant-scoped ceph cluster | |
| 19:23:16 | dansmith | gouthamr: we also know the reconciliation will not be backportable if it becomes more of a problem than you think | |
| 19:23:24 | gouthamr | yes | |
| 19:23:36 | melwitt | yeah, that's the problem, reconcile would not be backportable | |
| 19:23:53 | dansmith | IME instances wedged in the middle of migrations is not that uncommon so I'm nervous about having no path for cleanup for something users can trigger | |
| 19:24:49 | gouthamr | ack; i'll flush this out and test it and have something for your eyes in a bit.. the psutil change was the simplest :) but, the share mapping object update, i'll deep-test | |
| 19:25:09 | melwitt | that is my experience also. maybe the biggest complaint about nova is stuff stuck in a state requiring manual database intervention to recover | |
| 19:25:16 | dansmith | yup | |
| 19:25:36 | dansmith | gouthamr: okay then I'll take a break for a bit so I can be available for a bit later | |
| 19:25:49 | gouthamr | ack, thank you dansmith! | |
| 19:25:55 | gouthamr | same, melwitt and sean-k-mooney! | |
| 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: Replace license classifier https://review.opendev.org/c/openstack/placement/+/1004422 | |
| 13:49:51 | opendevreview | Stephen Finucane proposed openstack/placement master: Migrate requirements to pyproject.toml https://review.opendev.org/c/openstack/placement/+/1004423 | |
| 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: Replace license classifier https://review.opendev.org/c/openstack/placement/+/1004422 | |
| 13:50:46 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Use constraints option https://review.opendev.org/c/openstack/placement/+/996893 | |
| 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 | |