| opendevreview | Merged openstack/requirements master: update constraint for python-openstackclient to new release 10.0.0 https://review.opendev.org/c/openstack/requirements/+/988943 | 08:27 |
|---|---|---|
| opendevreview | Merged openstack/requirements master: update constraint for oslo.serialization to new release 5.10.0 https://review.opendev.org/c/openstack/requirements/+/988936 | 08:37 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.concurrency to new release 7.5.0 https://review.opendev.org/c/openstack/requirements/+/988944 | 09:20 |
| opendevreview | Merged openstack/requirements master: update constraint for stevedore to new release 5.8.0 https://review.opendev.org/c/openstack/requirements/+/988938 | 09:20 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.policy to new release 5.1.0 https://review.opendev.org/c/openstack/requirements/+/988942 | 09:20 |
| opendevreview | Merged openstack/requirements master: update constraint for openstacksdk to new release 4.13.0 https://review.opendev.org/c/openstack/requirements/+/988947 | 09:32 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.metrics to new release 0.16.0 https://review.opendev.org/c/openstack/requirements/+/988935 | 09:32 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-cyborgclient to new release 2.8.0 https://review.opendev.org/c/openstack/requirements/+/989084 | 09:39 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-freezerclient to new release 6.3.0 https://review.opendev.org/c/openstack/requirements/+/989086 | 09:40 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-glanceclient to new release 4.12.0 https://review.opendev.org/c/openstack/requirements/+/989087 | 09:40 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for neutron-lib to new release 4.0.0 https://review.opendev.org/c/openstack/requirements/+/989088 | 09:40 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-manilaclient to new release 6.1.0 https://review.opendev.org/c/openstack/requirements/+/989090 | 09:47 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for os-brick to new release 7.1.0 https://review.opendev.org/c/openstack/requirements/+/989091 | 09:50 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-barbicanclient to new release 7.4.0 https://review.opendev.org/c/openstack/requirements/+/989093 | 10:05 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.context to new release 6.4.0 https://review.opendev.org/c/openstack/requirements/+/988940 | 10:18 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.log to new release 8.2.0 https://review.opendev.org/c/openstack/requirements/+/988931 | 10:18 |
| dtantsur | Hi folks! Did we break something in requirements, or is tox broken, or is it a wrong moon phase? | 10:50 |
| dtantsur | The conflict is caused by: | 10:50 |
| dtantsur | The user requested oslo.context>=2.22.0 | 10:50 |
| dtantsur | The user requested (constraint) oslo-context===6.3.0,===6.4.0 | 10:50 |
| opendevreview | Merged openstack/requirements master: update constraint for python-heatclient to new release 5.2.0 https://review.opendev.org/c/openstack/requirements/+/988927 | 10:52 |
| frickler | dtantsur: https://review.opendev.org/c/openstack/requirements/+/988940 merged just 30 minutes ago, where do you see that? I don't know how you could get two conflicting "===" constraints anywhere | 11:03 |
| dtantsur | frickler: any attempt to build a tox environment locally | 11:04 |
| dtantsur | I'm also puzzled how it is even possible | 11:04 |
| frickler | well, working fine for me in a couple of repos I tested locally, so might be something wrong with your local setup? | 11:09 |
| dtantsur | What can become wrong in my setup by merging https://review.opendev.org/c/openstack/requirements/+/988940 though? | 11:09 |
| frickler | well I can't look into your setup, but likely you need to find out where the oslo-context===6.3.0 constraint comes from which is now outdated | 11:22 |
| dtantsur | The wrong in my setup is using tools that break with unhelpful messages, i.e. tox :-/ | 12:00 |
| dtantsur | The broken version has "-c https://releases.openstack.org/constraints/upper/master -c/home/dtantsur/Projects/ironic/.tox/pep8/constraints.txt" in its invocation. A worktree of the same Ironic repo does not have the former, and it works. | 12:03 |
| dtantsur | Reverting https://review.opendev.org/c/openstack/ironic/+/985905 works, so I guess I need to upset cardoe... | 12:08 |
| cardoe | That doesn’t make sense. | 12:13 |
| cardoe | It’s just causing -c to be built up into the command line instead of passing it manually as a flag. | 12:13 |
| dtantsur | cardoe: I agree, but what else can we do? | 12:15 |
| cardoe | Figure out what’s special on your system. | 12:15 |
| dtantsur | cardoe: nothing I can think of. | 12:15 |
| cardoe | Tox has backends for building up the executed command. | 12:16 |
| cardoe | The old way hardcoded into the dependencies passing -c because the deps are treated as a list of args due to pip taking a list of packages or -r and a requirements file. | 12:16 |
| cardoe | My guess is your local tox isn’t using the pip backend | 12:17 |
| dtantsur | It doesn't make much sense either, do it? | 12:18 |
| dtantsur | * does it | 12:18 |
| dtantsur | Even less sense: why this particular checkout broke at the moment oslo-context got into constraints... | 12:18 |
| dtantsur | Well, upgrading tox to the very latest solved something *shrug* | 12:20 |
| cardoe | What version did you have? | 12:20 |
| dtantsur | cardoe: 4.35.0 old one, 4.54.0 now | 12:20 |
| cardoe | When I tested this when stephenfin proposed it the command run by the pip backend of tox was identical | 12:21 |
| cardoe | He wrote the implementation in tox. | 12:21 |
| dtantsur | The command looks the same, it just succeeds :) | 12:21 |
| cardoe | My best guess is that somehow on your machine it used a different backend. | 12:21 |
| stephenfin | dtantsur: are you using tox-uv? | 12:21 |
| dtantsur | stephenfin: I'm using nothing fancy. Tox used to come from Fedora repos. A plain Ironic checkout that has existed for years. | 12:22 |
| dtantsur | (now using tox from `pip install --user`) | 12:22 |
| * dtantsur now assumes a wrong moon phase | 12:22 | |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.middleware to new release 8.1.0 https://review.opendev.org/c/openstack/requirements/+/988941 | 12:22 |
| opendevreview | Merged openstack/requirements master: update constraint for keystonemiddleware to new release 13.0.0 https://review.opendev.org/c/openstack/requirements/+/988412 | 12:22 |
| dtantsur | Well, GREAT, this ^^ merged and now I'm broken again, now with oslo.middleware | 12:23 |
| stephenfin | Can you run `tox -e py311 --notest -r | fpaste`? | 12:23 |
| stephenfin | Or share the output of tox run somehow | 12:23 |
| dtantsur | stephenfin: now it passes again Oo This is how the last failure looked: https://paste.opendev.org/show/bCwEv5W8TeNaXKKvKg2K/ | 12:27 |
| cardoe | dtantsur: broken even with new tox? | 12:28 |
| dtantsur | cardoe: it got broken with new tox, then fixed again. I cannot find any logic behind what is going on. | 12:28 |
| cardoe | My only other guess is fedora has an older virtualenv. | 12:28 |
| cardoe | I’m not against a revert. Just want to figure out how to fix this. | 12:29 |
| dtantsur | could be.. does not quite explain the randomness | 12:29 |
| stephenfin | I use tox from Fedora though. Always have | 12:29 |
| cardoe | So the only thing I see is that -c is a suffix instead of a prefix to pip | 12:30 |
| dtantsur | Yep, I removed the pip installed tox, and the Fedora version works too now | 12:30 |
| cardoe | I swear when I tested this the order of args was even the same. | 12:30 |
| dtantsur | Well, I no longer reproduce the problem, sooo... request solved, thanks everyone? | 12:33 |
| dtantsur | I wish we got any understanding of it, but that's life | 12:33 |
| cardoe | That looks like pip struggling with the . Vs the - ? | 12:33 |
| dtantsur | Could be. It's not a new situation though | 12:34 |
| cardoe | Well it’s not normalizing the name in one part of the output. | 12:36 |
| cardoe | Those metadata files it’s reading should store the normalized name. | 12:37 |
| stephenfin | cardoe: dtantsur: So I think there's a bug in tox alright, but it's easy to address. I'll continue the discussion in #openstack-ironic. | 13:11 |
| *** ykarel_ is now known as ykarel | 13:13 | |
| opendevreview | Radomir Dopieralski proposed openstack/requirements master: Update Django to 5.2.12 https://review.opendev.org/c/openstack/requirements/+/979862 | 13:16 |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements stable/2025.1: update constraint for oslo.messaging to new release 16.1.1 https://review.opendev.org/c/openstack/requirements/+/989133 | 13:43 |
| opendevreview | Merged openstack/requirements master: update constraint for python-glanceclient to new release 4.12.0 https://review.opendev.org/c/openstack/requirements/+/989087 | 14:04 |
| opendevreview | Merged openstack/requirements master: update constraint for python-freezerclient to new release 6.3.0 https://review.opendev.org/c/openstack/requirements/+/989086 | 14:04 |
| opendevreview | Merged openstack/requirements master: update constraint for os-brick to new release 7.1.0 https://review.opendev.org/c/openstack/requirements/+/989091 | 14:29 |
| opendevreview | Merged openstack/requirements master: update constraint for neutron-lib to new release 4.0.0 https://review.opendev.org/c/openstack/requirements/+/989088 | 14:29 |
| opendevreview | Merged openstack/requirements master: update constraint for python-cyborgclient to new release 2.8.0 https://review.opendev.org/c/openstack/requirements/+/989084 | 14:31 |
| opendevreview | Merged openstack/requirements master: update constraint for python-manilaclient to new release 6.1.0 https://review.opendev.org/c/openstack/requirements/+/989090 | 14:57 |
| opendevreview | Merged openstack/requirements master: update constraint for taskflow to new release 6.3.0 https://review.opendev.org/c/openstack/requirements/+/988924 | 16:31 |
| opendevreview | Merged openstack/requirements master: update constraint for oslo.config to new release 10.4.0 https://review.opendev.org/c/openstack/requirements/+/988945 | 16:31 |
| opendevreview | Merged openstack/requirements master: Remove nose https://review.opendev.org/c/openstack/requirements/+/988848 | 18:18 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove logic around Python 3-only branches https://review.opendev.org/c/openstack/requirements/+/989213 | 19:37 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Insist on minimal pyproject.toml file https://review.opendev.org/c/openstack/requirements/+/989214 | 19:37 |
| opendevreview | Merged openstack/requirements master: update constraint for python-barbicanclient to new release 7.4.0 https://review.opendev.org/c/openstack/requirements/+/989093 | 22:15 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!