| *** ykarel_ is now known as ykarel | 08:53 | |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Silence Python warnings https://review.opendev.org/c/openstack/pbr/+/954708 | 10:25 |
|---|---|---|
| *** clarkb is now known as Guest21734 | 10:51 | |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Silence Python warnings https://review.opendev.org/c/openstack/pbr/+/954708 | 11:02 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Use unicode literals https://review.opendev.org/c/openstack/pbr/+/954714 | 11:02 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Remove (now) unnecessary u prefixes https://review.opendev.org/c/openstack/pbr/+/954715 | 11:02 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Silence Python warnings https://review.opendev.org/c/openstack/pbr/+/954708 | 12:43 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Use unicode literals https://review.opendev.org/c/openstack/pbr/+/954714 | 12:43 |
| opendevreview | Stephen Finucane proposed openstack/pbr master: Remove (now) unnecessary u prefixes https://review.opendev.org/c/openstack/pbr/+/954715 | 12:43 |
| opendevreview | Merged openstack/pbr master: Centralise Python 2/3 compat code https://review.opendev.org/c/openstack/pbr/+/954514 | 13:25 |
| opendevreview | Merged openstack/pbr master: Tweak loading distribution version https://review.opendev.org/c/openstack/pbr/+/954041 | 13:36 |
| opendevreview | Merged openstack/pbr master: Use importlib.metadata where possible https://review.opendev.org/c/openstack/pbr/+/954042 | 13:36 |
| fungi | noonedeadpunk: as for the underlying cause of the failures, it looks like pillow isn't publishing wheels for python 3.12 yet, so it's being built from sdist when installed in the propose-translation-update jobs and is missing at least libjpeg-dev headers to enable that | 14:29 |
| fungi | https://zuul.opendev.org/t/openstack/build/195e55079d82445492f961997583032f for example | 14:34 |
| fungi | looks like https://opendev.org/openstack/horizon/src/branch/master/requirements.txt#L24 includes pillow as a horizon install_requires but https://opendev.org/openstack/horizon/src/branch/master/bindep.txt lacks the toolchain/headers necessary for installing it on noble | 14:39 |
| noonedeadpunk | fungi: yeah... I wonder if it's worth adding a different defenition for these broken projects separately | 14:39 |
| fungi | i mean, fundamentally it's indicating that horizon can't currently be installed for python 3.12 | 14:40 |
| noonedeadpunk | but also - are project requirements really valid for doc and getting strings for zanata update? | 14:40 |
| noonedeadpunk | as this sounds like unrelated things | 14:40 |
| noonedeadpunk | but Horizon runs noble jobs? | 14:40 |
| fungi | i wonder how | 14:41 |
| noonedeadpunk | But also I wonder what pillow are we talkign about? | 14:41 |
| noonedeadpunk | As this one does have even 3.13: https://pypi.org/project/pillow/ | 14:42 |
| noonedeadpunk | https://github.com/python-pillow/Pillow/blob/main/pyproject.toml#L29-L30 | 14:42 |
| fungi | Collecting Pillow>=9.1.0 (from -r requirements.txt (line 24)) | 14:42 |
| fungi | Pillow-9.5.0.tar.gz | 14:42 |
| noonedeadpunk | u-c has pillow===11.3.0 | 14:43 |
| noonedeadpunk | https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L309 | 14:43 |
| fungi | says it's doing `pip install -c /home/zuul/src/opendev.org/openstack/horizon/upper-constraints.txt -r requirements.txt -r test-requirements.txt` | 14:44 |
| noonedeadpunk | so it feels like it's held back somewhere for purpose of not got broken for 18.04 | 14:44 |
| noonedeadpunk | wait | 14:44 |
| noonedeadpunk | so we have lower constraints there? | 14:44 |
| noonedeadpunk | I don't see that in repo | 14:44 |
| noonedeadpunk | ok | 14:44 |
| noonedeadpunk | so I guess we need to find then where it comes from.... | 14:45 |
| fungi | https://zuul.opendev.org/t/openstack/build/195e55079d82445492f961997583032f/console#3/0/15/ubuntu-noble | 14:45 |
| fungi | "src":"/home/zuul/src/opendev.org/openstack/requirements/upper-constraints.txt" | 14:46 |
| noonedeadpunk | ah | 14:46 |
| fungi | so it's copying it | 14:46 |
| noonedeadpunk | well, I'm still very confused about versions.... | 14:46 |
| fungi | yeah, i'm still trying to find what's forcing it to that verison | 14:47 |
| fungi | version | 14:47 |
| noonedeadpunk | but we should be seeing conflict with constraints | 14:47 |
| fungi | unless an earlier task is switching the reqs repo branch checkout | 14:48 |
| noonedeadpunk | or later task does some kind of sed... | 14:49 |
| fungi | looks like we have pillow===9.5.0 in constraints for stable/2024.1 | 14:52 |
| fungi | and stable/2024.2 | 14:52 |
| fungi | it increased in stable/2025.1 | 14:53 |
| *** Guest21734 is now known as clarkb | 14:53 | |
| clarkb | there is probably an override checkout somewhere due to running on bionic previously | 14:54 |
| fungi | yep! | 14:54 |
| fungi | i just found it | 14:54 |
| fungi | https://zuul.opendev.org/t/openstack/build/195e55079d82445492f961997583032f/log/zuul-info/inventory.yaml#164-173 | 14:54 |
| fungi | checkout: stable/2024.2 | 14:54 |
| clarkb | I suspect simply removing the override may make things happy again in that case | 14:55 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Drop requirements branch override for translations https://review.opendev.org/c/openstack/project-config/+/954747 | 14:58 |
| fungi | noonedeadpunk: clarkb: ^ | 14:58 |
| noonedeadpunk | ah, doh | 14:58 |
| noonedeadpunk | I overlooked it obviously... | 14:58 |
| noonedeadpunk | but it's not only occurance | 14:59 |
| fungi | yeah, release notes jobs still do it, but they run on ubuntu-jammy for now | 14:59 |
| noonedeadpunk | I somehow feel there's a task duplication there... | 15:00 |
| noonedeadpunk | ah. nno | 15:00 |
| noonedeadpunk | but I commented | 15:00 |
| fungi | thanks! | 15:03 |
| clarkb | noonedeadpunk: fungi: side note (except for horizon maybe) there is almost zero reason to install any deps at all. You just need the python babel toolchain iirc | 15:04 |
| clarkb | horizon is different because it uses django which uses its own toolingand I'm not sure how bespoke that is. But the babel stuff is gnutext in python and very standardized | 15:05 |
| clarkb | I don't know why we didn't do that at the time. Maybe it was just easier to put babel in requirements.txt and not think about it from there | 15:05 |
| fungi | noonedeadpunk: oh, good catch, that was upstream-translation-updates not propose-translation-updates | 15:07 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Drop requirements branch override for translations https://review.opendev.org/c/openstack/project-config/+/954747 | 15:08 |
| fungi | and since they're both on ubuntu-noble now neither should need to use older constraints files | 15:10 |
| fungi | openstack doesn't do translation updates for older branches anyway afaik | 15:10 |
| clarkb | * gnu gettext | 15:12 |
| opendevreview | James E. Blair proposed openstack/project-config master: Remove bionic/focal arm64 wheel jobs https://review.opendev.org/c/openstack/project-config/+/954758 | 16:09 |
| opendevreview | Merged openstack/project-config master: Remove bionic/focal arm64 wheel jobs https://review.opendev.org/c/openstack/project-config/+/954758 | 16:25 |
| opendevreview | James E. Blair proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 16:27 |
| opendevreview | James E. Blair proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs, also xenial https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 16:35 |
| opendevreview | James E. Blair proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 16:50 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 17:11 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 17:16 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Drop openstack-tox-py3[89]-arm64 from project-templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954762 | 17:16 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Drop openstack-tox-py3[89]-arm64 from project-templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954762 | 17:20 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 17:20 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Make dummy ubuntu-xenial nodeset https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954763 | 17:20 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954764 | 17:26 |
| clarkb | gmaan: give me a minute I'm trying to come up with a stack of changes that addresses the opendev concerns then we can work on properly cleaning up the openstack side of things | 17:27 |
| gmaan | clarkb: sure | 17:27 |
| clarkb | gmaan: that said I think https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954763/ fixes the most immediate issue if you want to review that one | 17:27 |
| gmaan | +1 | 17:28 |
| gmaan | clarkb: I will rebase on top of your, basically to try removing the older template and add note for us not to add more stable branch jobs for arm64 whenever there is new stable branch cut | 17:31 |
| fungi | unfortunate that there's references in old neutron stable branches | 17:31 |
| fungi | i suppose another option could be to keep referenced jobs but gut their configuration and give them an empty nodeset | 17:32 |
| fungi | maybe what we need is an always-failing nodeless abstract job which echoes a clear message into the log | 17:33 |
| fungi | and then make it the parent of any jobs that use nodesets we want to eventually clean up | 17:34 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Remove bionic- and focal- arm64 wheel build jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 17:36 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Drop openstack-tox-py3[89]-arm64 from project-templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954762 | 17:36 |
| clarkb | gmaan: ^ ok I think 954761 is good as is but then 954762 may need iterative development | 17:40 |
| fungi | apparently openstack/requirements includes references too | 17:41 |
| clarkb | ya I'm digging into that next | 17:45 |
| opendevreview | Clark Boylan proposed openstack/openstack-zuul-jobs master: Remove bionic-arm64 wheel build jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954761 | 18:26 |
| opendevreview | Merged openstack/openstack-zuul-jobs master: Make dummy ubuntu-xenial nodeset https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954763 | 18:40 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954781 | 18:47 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954781 | 18:48 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954781 | 18:56 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954781 | 19:11 |
| opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Remove arm64 unit tests job from stable branches https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/954781 | 19:31 |
| opendevreview | Merged openstack/project-config master: Drop requirements branch override for translations https://review.opendev.org/c/openstack/project-config/+/954747 | 23:36 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!