| *** tkajinam is now known as Guest299 | 04:46 | |
| *** soniya29 is now known as soniya29|lunch | 07:30 | |
| *** soniya29|lunch is now known as soniya29 | 08:14 | |
| ralonsoh | fungi, sorry, I don't know what I'm doing wrong here | 08:35 |
|---|---|---|
| ralonsoh | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_115/869196/7/check/neutron-tox-py311/115fa08/job-output.txt | 08:35 |
| ralonsoh | I've tried creating my own job | 08:36 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/869196/7/zuul.d/base.yaml | 08:36 |
| ralonsoh | should I manually install python3.11? I think so | 08:39 |
| frickler | ralonsoh: that's a tox regression, octavia was running into the same issue. newest tox tries to find interpreters for all environments, not only the one being executed. workaround is to drop "basepython=3.x" from tox.ini everywhere | 08:43 |
| frickler | see https://github.com/tox-dev/tox/issues/2811 | 08:43 |
| ralonsoh | frickler, ahhhh I saw the same in cinderlib | 08:43 |
| ralonsoh | so good to know this, thanks! | 08:43 |
| frickler | see also https://review.opendev.org/c/openstack/octavia/+/868765 | 08:44 |
| ralonsoh | I'll remove the basepython definition, I have a testing patch for this | 08:45 |
| ralonsoh | thanks a lot | 08:45 |
| opendevreview | Ade Lee proposed openstack/project-config master: Add FIPS job for ubuntu https://review.opendev.org/c/openstack/project-config/+/867112 | 10:59 |
| *** rlandy|out is now known as rlandy | 11:09 | |
| *** dviroel|ourt is now known as dviroel | 11:45 | |
| ade_lee | fungi, clarkb - hey - please take a look at the latest fips patches -- I think I've captured what you mentioned yesterday | 13:19 |
| ade_lee | fungi, clarkb https://review.opendev.org/c/openstack/project-config/+/867112 and https://review.opendev.org/c/zuul/zuul-jobs/+/866881 | 13:21 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Update tox.ini to work with tox 4 https://review.opendev.org/c/openstack/pbr/+/869190 | 13:41 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Tie recursion calls to Dist object, not module https://review.opendev.org/c/openstack/pbr/+/869082 | 13:41 |
| *** dasm|off is now known as dasm | 13:50 | |
| ade_lee | fungi, clarkb any feedback? | 15:41 |
| *** dviroel is now known as dviroel|lunch | 16:01 | |
| fungi | ade_lee: sorry, it's on my list but i'm buried in first-quarter paperwork and i think clarkb may still be under the weather | 16:05 |
| *** soniya29 is now known as soniya29|out | 16:08 | |
| ade_lee | fungi, ack thanks - whenever you can get to it -- your suggestions have been super helpful so far. | 16:10 |
| ade_lee | its at least passing ci now | 16:10 |
| fungi | yeah, sorry i've been stretched a bit thin. the tox v4 situation landing on top of winter holidays sapped much of my already limited available time | 16:11 |
| ade_lee | fungi, quick q -- I introduced a new job in project-config -- do I need that to merge before I can try using it? | 16:12 |
| fungi | yes | 16:12 |
| ade_lee | ok :) | 16:12 |
| fungi | since it's in openstack/project-config it won't be usable until it merges | 16:12 |
| fungi | because openstack/project-config is designated as a trusted config repository for zuul, there are security-related implications with using unreviewed configuration | 16:13 |
| ade_lee | understood - well, whenever you all can get to it, I'll make any needed changes | 16:14 |
| opendevreview | Julia Kreger proposed openstack/pbr master: DNM: Make envlist py3-ey https://review.opendev.org/c/openstack/pbr/+/869389 | 16:26 |
| *** JasonF is now known as JayF | 16:28 | |
| *** dviroel|lunch is now known as dviroel | 17:10 | |
| opendevreview | Julia Kreger proposed openstack/pbr master: Update tox.ini to work with tox 4 https://review.opendev.org/c/openstack/pbr/+/869190 | 17:29 |
| clarkb | ya sorry this morning is the first day I've even managed to sit in front of a proper keyboard and I need to catch up on paperwork myself | 17:49 |
| gmann | clarkb: fungi: frickler: please check this, it has good amount of positive response https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/867849/3 | 18:45 |
| gmann | and its base patch to fix tox4 things https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/869260/1 | 18:46 |
| clarkb | note that pin won't work for any repos that use tox requirements | 18:49 |
| fungi | clarkb: sort of. if those requirements are already installed (which seems to be the case for most of them) then tox will skip creating a new tox installation. also you can add tox<4 into the tox.requires list to force it to not upgrade itself to v4 | 18:50 |
| fungi | if tox.requires includes things which are not already preinstalled and doesn't have an explicit tox<4 in it, then yes it will upgrade itself | 18:51 |
| clarkb | fungi: right but that requires editing tox anyway which isn't much different than making tox v4 work | 18:51 |
| clarkb | I guess the issue is tox v4 keeps updating with new breaking behavior | 18:52 |
| fungi | agreed. i pointed that out (some projects will need stable branch tox.ini changes to keep tox from upgrading itself) but that seemed to be considered the lesser of two evils | 18:52 |
| JayF | Yeah; if tox 4.0 was stable pushing support for it down would be sensible. Since it's not; pinning is the lesser evil I think :/ | 18:53 |
| gmann | yeah, we never know what all we might need to keep fixing for future latest tox on stable | 18:56 |
| clarkb | ya I mean thats why I've strongly recommended everyone start thinking about nox. Since the problems are likely to persist even on master | 18:58 |
| clarkb | I think the changes in https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/867849/3/zuul.d/project-templates.yaml don't work exactly the way we think they do | 18:58 |
| clarkb | a project-template modifies a project not jobs. What that means is the entire project using one or more of those templates will have tox pinned to <4 | 18:59 |
| clarkb | let me leave a comment inline and see if corvus can confirm | 18:59 |
| fungi | the edits in zuul.d/jobs.yaml should work though | 19:00 |
| fungi | also jobs running on xenial don't need to pin tox, they won't install v4 anyway | 19:01 |
| opendevreview | Merged openstack/openstack-zuul-jobs master: Fix tox4 error https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/869260 | 19:08 |
| clarkb | to address the vars in project-templates thing I think you can set the vars at the job variant level within the template. Its a lot more verbose but that would achieve what you are going for there I think | 19:11 |
| clarkb | yup corvus confirms that my understanding of the behavior there is correct | 19:29 |
| gmann | ok, let me update that | 19:32 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Update tox.ini to work with tox 4 https://review.opendev.org/c/openstack/pbr/+/869190 | 20:27 |
| opendevreview | Ghanshyam Mann proposed openstack/openstack-zuul-jobs master: Pin tox<4 for stable branches (<=stable/zed) testing https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/867849 | 20:28 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Remove numpy dependencies https://review.opendev.org/c/openstack/pbr/+/869402 | 20:31 |
| clarkb | gmann: fwiw I expect what you've got in that latest patchset to work. | 21:16 |
| *** dviroel is now known as dviroel|afk | 21:19 | |
| *** rlandy is now known as rlandy|out | 22:13 | |
| *** dasm is now known as dasm|off | 22:33 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!