Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2026-09-07
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
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
14:00:37 sean-k-mooney stephenfin: for reason rdo was still using 65.5.1
14:01:03 sean-k-mooney but we fixed that in may so all good
14:01:22 stephenfin cool
14:01:41 stephenfin I mean, if you're talking strictly about packaging rdo could always carry a revert patch too
14:01:53 sean-k-mooney stephenfin: no i was aware of that as well (moving off 20.04) but that was not the only place usign old things to build
14:02:17 sean-k-mooney in general i woudl not advise they ever do that
14:02:19 sean-k-mooney but yes
14:02:37 sean-k-mooney rdo shoudl not carrry patches to the code
14:02:49 stephenfin The distro setuptools version no longer matters for installs with pip since builds are isolated by default, so it would only be packaging (or the aforementioned "my Python is too old") scenarios that would break
14:03:07 stephenfin agree, but the escape hatch is there
14:03:31 stephenfin speaking of escape hatches, looks like setuptools' docs aren't happy https://setuptools.pypa.io/en/latest/userguide/license_migration.html
14:03:47 sean-k-mooney yep, although i belive we have to disabel build isolation in some case for downsteam reasons
14:03:53 stephenfin I could fix it but they ignore my issues so 🤷
14:04:05 sean-k-mooney specifcly to use a fips complient python-cyrotography
14:04:10 stephenfin not bitter...
14:04:47 stephenfin sean-k-mooney: you need a fips-complint cryptopgraphy during build??
14:04:51 stephenfin *compliant
14:05:16 sean-k-mooney yes and in the final artifact
14:05:35 sean-k-mooney anyway htat out of scope fo upstream
14:06:30 sean-k-mooney but downstream python3-cyrptograph must come form the rpms for complance reasons as it need to be built with a specific fips certifed version fo openssl ectra
14:09:09 opendevreview ribaudr proposed openstack/nova master: This adds an alias for Hibiscus https://review.opendev.org/c/openstack/nova/+/1004424
14:24:15 opendevreview ribaudr proposed openstack/nova master: Add service version for Hibiscus https://review.opendev.org/c/openstack/nova/+/1004430
15:29:21 opendevreview ribaudr proposed openstack/nova-specs master: Move Hibiscus implemented specs https://review.opendev.org/c/openstack/nova-specs/+/1004442
15:42:16 Uggla Reminder: upstream meeting in ~20mn
15:45:59 gibi Uggla: I added a topic to the open discussion agenda
15:46:17 Uggla gibi, ok!
16:00:47 Uggla #startmeeting nova
16:00:47 opendevmeet Meeting started Mon Sep 7 16:00:47 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:47 opendevmeet Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:47 opendevmeet The meeting name has been set to 'nova'

Earlier   Later