| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-21 | |||
| 16:50:52 | bauzas | maintain* even | |
| 16:50:59 | bauzas | but this doesn't sound easy | |
| 16:51:03 | artom | So the value of the job is what - guarantee that will still work with the oldest feasible versions of things? | |
| 16:51:13 | artom | (lower-constraints I mean) | |
| 16:51:25 | stephenfin | sean-k-mooney: We will always have pinning in the form of upper-constraints though | |
| 16:51:26 | sean-k-mooney | yes basically to make sure when we backport we dont raise the requiremetns as a result | |
| 16:51:41 | clarkb | the correct way to maintain setuptools in your requirements is via the pyproject file | |
| 16:51:44 | sean-k-mooney | stephenfin: right but for stable we are not allowed to raise miniums | |
| 16:51:47 | clarkb | not requirements | |
| 16:52:14 | stephenfin | and? we can't raise maximums either | |
| 16:52:17 | clarkb | the new pyproject file allows you to specify build time deps (possibly even tools other than setuptools) | |
| 16:52:28 | clarkb | I don't know what would be involved to make all of that work with pbr though | |
| 16:53:50 | bauzas | clarkb: how other projects consider this issue ? do they want to pin setuptools as well ? | |
| 16:54:01 | bauzas | or do they just fix the decorator issue ? | |
| 16:54:12 | bauzas | (and backport without specific care) | |
| 16:54:22 | bauzas | I guess we're not alone in the dark | |
| 16:54:31 | clarkb | msot of the fixes I've seen have been to fix the dependencies | |
| 16:54:41 | bauzas | and about backports ? | |
| 16:54:44 | clarkb | openstack governance had to switch pydot2 to pydot for example | |
| 16:55:03 | clarkb | zuul-jobs dropped support for old python3.5 which allowed it to use a newer version of a lib | |
| 16:55:27 | clarkb | I don't know how backports or stable issues have been handled | |
| 16:56:34 | bauzas | gibi: I guess you have to write something and bubble it to the community for cross-project thinking | |
| 16:57:08 | gibi | bauzas: I can write up a summary from today on the ML | |
| 16:57:09 | bauzas | that could become an epic saga, but it's worth starting it | |
| 16:57:28 | bauzas | gibi: appreciated, here lemme summarize for the audience | |
| 16:57:46 | bauzas | option 2 seems to be the better, but we struggle finding a good way to write it | |
| 16:58:22 | artom | Stupid question, but could we not just install virtualenv==<version we want> first, and *then* tox? | |
| 16:58:23 | bauzas | option 1 seems to be the alternative, option 3 seems not accepted | |
| 16:58:27 | artom | Or would tox just upgrade? | |
| 16:58:49 | gibi | artom: that could be also something to try | |
| 16:58:56 | bauzas | artom: virtualenv pulls the latest setuptools IIRC | |
| 16:59:04 | bauzas | but that's a flag | |
| 16:59:15 | gibi | bauzas: old virtualenv will pull old setuptools I guess | |
| 16:59:22 | bauzas | again, the problem is how to trigger this flag thru tox | |
| 16:59:42 | bauzas | gibi: I remember having played with it before and you're right | |
| 17:00:01 | bauzas | but you can explicitely specific to pull the latest setuptools | |
| 17:00:07 | bauzas | either way, we're at time | |
| 17:00:24 | bauzas | gibi: thanks for writing the summary and raising it to the community | |
| 17:00:34 | bauzas | folks, wrapping up | |
| 17:00:36 | bauzas | thanks | |
| 17:00:40 | bauzas | #endmeeting | |
| 17:00:41 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-09-21-16.00.log.html | |
| 17:00:41 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-09-21-16.00.txt | |
| 17:00:41 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-09-21-16.00.html | |
| 17:00:41 | opendevmeet | Meeting ended Tue Sep 21 17:00:40 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:00:44 | gibi | bauzas: thanks | |
| 17:01:08 | elodilles | thanks bauzas o/ | |
| 17:02:25 | stephenfin | I've already tagged people, but there are some necessary fixes for the DB migration tests here that we should probably merge before I forget about them https://review.opendev.org/c/openstack/nova/+/810291 | |
| 17:02:44 | stephenfin | The alternative is to rediscover this stuff once someone adds a DB migration in Yoga or later | |
| 17:02:57 | stephenfin | (https://review.opendev.org/c/openstack/nova/+/805738 proves that they work) | |
| 17:03:22 | bauzas | stephenfin: I just made use of the review-priority label for your change :) | |
| 17:03:32 | gibi | stephenfin: thanks, I will try to get to them | |
| 17:04:51 | opendevreview | Stephen Finucane proposed openstack/nova master: tools: Ignore bot-generated branch creation patches https://review.opendev.org/c/openstack/nova/+/810285 | |
| 17:05:06 | stephenfin | lyarwood: sorry for the dumb typo in that fixed. Bash is the worst. Fixed ^ | |
| 17:05:33 | opendevreview | Balazs Gibizer proposed openstack/placement stable/xena: [WIP] try to pin setuptools via tox.ini https://review.opendev.org/c/openstack/placement/+/810293 | |
| 17:07:13 | gibi | trying [tox]/requires ^^ | |
| 17:12:33 | bauzas | gibi: me too | |
| 17:12:35 | bauzas | \o | |
| 17:53:36 | gibi | it seems [tox]requires does not help us pinning setuptools | |
| 18:46:48 | fungi | just to clarify, very old virtualenv pulls old setuptools, but a while back virtualenv switched to pulling whatever the most recent setuptools available is unless you can tell it not to, so just pinning to a slightly older virtualenv doesn't help as it will still insist on grabbing the latest available setuptools | |
| 18:47:42 | fungi | also there's the fact that this isn't purely a ci side problem, you can't really know or control what version of setuptools a user will have | |
| 18:48:21 | fungi | because it's considered part of the build environment not part of the runtime environment, so python packaging doesn't handle it the same way with the same expectations | |
| 18:51:03 | fungi | the idea historically was that there were mechanisms to insist on a minimum acceptable version of build dependendencies and raise errors if they weren't new enough, but not really any reliable way to indicate maximum acceptable versions | |
| 18:52:08 | fungi | of course, that was also predicated on build dependencies maintaining backward compatibility, and setuptools removing a feature is what's put us in this situation | |
| 19:00:47 | clarkb | upstream pypa seems to say everyone should use pyproject.toml to address this | |
| 19:00:55 | clarkb | which is why it is safe for them to drop functionality like that | |
| #openstack-nova - 2021-09-22 | |||
| 07:15:52 | bauzas | good morning Nova | |
| 08:15:40 | lyarwood | Morning | |
| 08:26:19 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 08:32:55 | bauzas | lyarwood: your eyes would be appreciated for https://review.opendev.org/c/openstack/placement/+/810193 | |
| 08:33:34 | bauzas | and morning btw. :) | |
| 08:45:33 | lyarwood | bauzas: ack done | |
| 08:45:49 | bauzas | lyarwood: ack thanks | |
| 09:07:14 | opendevreview | Merged openstack/placement stable/xena: Bump min decorator to 4.0.0 https://review.opendev.org/c/openstack/placement/+/810193 | |
| 09:15:52 | opendevreview | MaAoyu proposed openstack/nova master: swap multiattach volume, report an error VolumeNotFound https://review.opendev.org/c/openstack/nova/+/807025 | |
| 09:30:35 | bauzas | sean-k-mooney: when you're up, you can +1 this ;) https://review.opendev.org/c/openstack/releases/+/810372 | |
| 09:30:59 | bauzas | not sure the email is the right one | |
| 09:35:12 | sean-k-mooney[m] | done, that is my work email, work@seanmooney.info which i use for my git commits is an alias of my personal one currently so its better for release stuff to go to my redhat one. i’ll see it quicker. | |
| 09:38:23 | sean-k-mooney[m] | i should see if i can get gmail to forward work@seanmooney.info to my work one automatically | |
| 09:39:09 | sean-k-mooney[m] | i very rarely use it for anything | |
| 09:51:57 | bauzas | sean-k-mooney: no worries, you can update the patch if you want to modify your email | |
| 09:52:39 | bauzas | ah, saw your +1 | |
| 09:54:55 | sean-k-mooney[m] | yep its fine as it is :) | |
| 12:03:28 | songwenping | bauzas: hi, i cannot find where to cleanup vgpu resource in the update_available_resources, and on my env, the vgpu resources really arenot cleaned if i remove the vgpu type from n-cpu.conf. | |
| 12:12:34 | gibi | bauzas: summarized our yesterday's discussion about setuptools on the ML http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024987.html | |
| 12:15:34 | bauzas | songwenping: do you have existing allocations ? | |
| 12:18:17 | songwenping | no, nothing in the allocations table. | |
| 12:19:21 | bauzas | songwenping: mmm, OK, I'll try to verify that in my env | |
| 12:20:18 | songwenping | cool, thanks. | |
| 12:29:48 | opendevreview | Merged openstack/placement master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/placement/+/809366 | |
| 12:29:49 | opendevreview | Merged openstack/placement stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/placement/+/809363 | |
| 12:45:12 | opendevreview | norman shen proposed openstack/nova master: Recreate mdev devices according to placement https://review.opendev.org/c/openstack/nova/+/810220 | |
| 15:56:59 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: [stable-only][WIP] Pin virtualenv and setuptools https://review.opendev.org/c/openstack/nova/+/810461 | |
| 16:26:42 | opendevreview | Merged openstack/placement stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/placement/+/809364 | |
| 19:18:27 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with more free memory first https://review.opendev.org/c/openstack/nova/+/805649 | |
| #openstack-nova - 2021-09-23 | |||
| 05:06:27 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 05:06:28 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 05:06:28 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 05:06:29 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 05:06:29 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |