| *** irclogbot_3 has joined #openstack-requirements | 00:17 | |
| smcginnis | dhellmann: Appears to be a change in behavior in the lower-constraints jobs. I know they had been working at least up until recently. | 00:18 |
|---|---|---|
| smcginnis | Don't think the zuul update would have affected that. Maybe a tox upgrade? | 00:18 |
| *** hongbin has joined #openstack-requirements | 02:22 | |
| *** udesale has joined #openstack-requirements | 03:21 | |
| *** hongbin has quit IRC | 04:12 | |
| *** TheJulia has quit IRC | 04:31 | |
| *** coreycb has quit IRC | 04:31 | |
| *** TheJulia has joined #openstack-requirements | 04:31 | |
| *** dustinc has quit IRC | 04:31 | |
| *** spsurya has quit IRC | 04:31 | |
| *** adrianreza has quit IRC | 04:32 | |
| *** kmalloc has quit IRC | 04:32 | |
| *** johnsom has quit IRC | 04:32 | |
| *** coreycb has joined #openstack-requirements | 04:34 | |
| *** spsurya has joined #openstack-requirements | 04:34 | |
| *** adrianreza has joined #openstack-requirements | 04:34 | |
| *** johnsom has joined #openstack-requirements | 04:35 | |
| *** dustinc has joined #openstack-requirements | 04:39 | |
| *** kmalloc has joined #openstack-requirements | 04:39 | |
| *** udesale has quit IRC | 04:47 | |
| *** udesale has joined #openstack-requirements | 04:47 | |
| *** dustinc has quit IRC | 05:50 | |
| *** openstackgerrit has joined #openstack-requirements | 06:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.openstack.org/641568 | 06:14 |
| *** dims has quit IRC | 06:39 | |
| *** dims has joined #openstack-requirements | 06:41 | |
| *** tosky has joined #openstack-requirements | 07:53 | |
| *** hberaud has quit IRC | 08:20 | |
| *** hberaud has joined #openstack-requirements | 08:22 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| *** jpich has joined #openstack-requirements | 08:43 | |
| *** jpich has quit IRC | 10:13 | |
| *** jpich has joined #openstack-requirements | 10:14 | |
| *** dtantsur is now known as dtantsur|brb | 10:28 | |
| *** hberaud is now known as hberaud|school-r | 10:56 | |
| *** johnsom has quit IRC | 10:59 | |
| *** adrianreza has quit IRC | 10:59 | |
| *** johnsom has joined #openstack-requirements | 10:59 | |
| *** spsurya has quit IRC | 10:59 | |
| *** adrianreza has joined #openstack-requirements | 11:00 | |
| *** spsurya has joined #openstack-requirements | 11:00 | |
| *** dtantsur|brb is now known as dtantsur | 12:05 | |
| *** hberaud|school-r is now known as hberaud | 12:24 | |
| *** udesale has quit IRC | 12:47 | |
| dhellmann | edmondsw , smcginnis , prometheanfire : the lower constraints files were built by installing your primary dependencies at their lower bound and not constraining anything else, so if you need to update a package you could try just removing that 1 constraint and see what you get | 12:48 |
| dhellmann | efried : oh, that's unfortunate :-/ | 12:48 |
| *** udesale has joined #openstack-requirements | 12:48 | |
| dhellmann | edmondsw , smcginnis , prometheanfire : it looks like that error is coming from the isort package, so maybe check the constraint on that one, too | 12:50 |
| edmondsw | dhellmann yeah, I noticed the isort thing, but that (or the thing that pulls it in, pkg_resources) doesn't appear to have ever been in openstack/requirements and I wasn't sure what I should set as lc for those | 12:59 |
| edmondsw | I decided to just try removing appdirs from lc | 13:00 |
| edmondsw | right or wrong, that seems to have resolved it | 13:00 |
| dhellmann | ok | 13:00 |
| *** irclogbot_3 has quit IRC | 13:01 | |
| *** altlogbot_0 has quit IRC | 13:01 | |
| dhellmann | I would be surprised if pkg_resources has a dependency, since it's shipped as part of setuptools. I wonder if they're vendoring isort. | 13:01 |
| dhellmann | either way, I'm glad it's working | 13:01 |
| *** altlogbot_1 has joined #openstack-requirements | 13:02 | |
| *** irclogbot_0 has joined #openstack-requirements | 13:02 | |
| edmondsw | dhellmann oh, I skimmed that too quickly. pkg_resources doesn't depend on isort, it's just what pulled it in | 13:03 |
| edmondsw | http://logs.openstack.org/45/645245/1/check/openstack-tox-lower-constraints/dbd4f19/job-output.txt.gz#_2019-03-21_17_38_26_801207 | 13:03 |
| edmondsw | I'm not sure how to tell why it pulled it in... | 13:03 |
| edmondsw | (line above the one I linked) | 13:03 |
| dhellmann | pbr is resolving an entry point via pkg_resources, so it's either importing something that uses isort or isort defines an entry point in that namespace | 13:04 |
| edmondsw | I don't really understand how all that works | 13:20 |
| smcginnis | dhellmann: Morning. Did you see the other discussion yesterday that the lower-constraints jobs aren't actually working? | 13:20 |
| edmondsw | I think this probably traces back to a neutron entry point, but I don't see one for isort specifically | 13:20 |
| edmondsw | smcginnis you mean for nova, or more generally? | 13:21 |
| smcginnis | edmondsw: It was pointed out in nova, but apparently a wider spread issue. | 13:21 |
| smcginnis | But conflicting that we are getting your failure with the lower constraints, so I'm not sure what the whole picture is here. | 13:21 |
| edmondsw | I'm the one that pointed it out in nova, but sounds like that has taken on a life of it's own | 13:22 |
| edmondsw | I thought the problem in nova was a problem specific to nova... well, not to say someone else couldn't be doing the same things wrong... | 13:22 |
| smcginnis | edmondsw: From what I understand, your failure is using the lower constraints and it fails due to a package being too low. But the later conversation in nova was that there are at least some jobs that are not using the lower constraints at all in the lower-constraint job. | 13:22 |
| edmondsw | conversation started here: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-03-21.log.html#t2019-03-21T20:25:37 | 13:23 |
| smcginnis | One specific instance, this l-c is set to 1.1.1 but 1.4.0 was what gets installed according to the pip freeze: https://opendev.org/openstack/nova/src/branch/master/lower-constraints.txt#L175 | 13:24 |
| edmondsw | and they realized things were broken here: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-03-21.log.html#t2019-03-21T20:46:14 | 13:24 |
| edmondsw | which is what I think you're referring to | 13:24 |
| smcginnis | I think that led to this later: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-03-22.log.html#t2019-03-22T00:03:34 | 13:25 |
| edmondsw | right, so that's a nova error, right? They shouldn't list the requirements file in their install_command | 13:28 |
| edmondsw | precisely to avoid things like this | 13:28 |
| smcginnis | Maybe, but I think that's what the majority of projects have set up in their tox.ini files, so either this was always not working correctly, or something changed with pip or other package behavior. | 13:29 |
| smcginnis | Hmm, I do see Cinder has "install_command = pip install {opts} {packages}", so maybe it is a smaller issue. | 13:30 |
| smcginnis | Yeah, that seems to be the case. | 13:31 |
| efried | smcginnis: The issue is that the nova l-c job is inheriting install_command from [testenv] and that install_command is hardcoded to pull in upper-constraints. sean-k-mooney is fixing. | 13:56 |
| efried | iow the Cinder one should be fine as long as it doesn't mention upper-constraints | 13:57 |
| efried | inheriting install_command isn't the problem | 13:57 |
| *** efried is now known as fried_rice | 13:58 | |
| *** bnemec is now known as beekneemech | 13:59 | |
| *** cgoncalves has quit IRC | 15:17 | |
| *** cgoncalves has joined #openstack-requirements | 15:18 | |
| *** altlogbot_1 has quit IRC | 15:21 | |
| *** altlogbot_3 has joined #openstack-requirements | 15:26 | |
| *** irclogbot_0 has quit IRC | 15:30 | |
| *** irclogbot_1 has joined #openstack-requirements | 15:32 | |
| *** irclogbot_1 has quit IRC | 15:36 | |
| *** irclogbot_1 has joined #openstack-requirements | 15:38 | |
| *** dustinc has joined #openstack-requirements | 16:08 | |
| *** smrcascao has quit IRC | 16:09 | |
| *** fried_rice is now known as fried_rolls | 16:30 | |
| *** e0ne has joined #openstack-requirements | 16:48 | |
| *** jpich has quit IRC | 16:54 | |
| *** e0ne has quit IRC | 16:59 | |
| *** udesale has quit IRC | 17:00 | |
| *** dtantsur is now known as dtantsur|afk | 17:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for virtualbmc to new release 1.5.0 https://review.openstack.org/645820 | 17:49 |
| *** gmann is now known as gmann_afk | 17:52 | |
| *** gmann_afk is now known as gmann | 18:11 | |
| *** fried_rolls is now known as fried_rice | 19:07 | |
| *** e0ne has joined #openstack-requirements | 21:54 | |
| *** e0ne has quit IRC | 22:36 | |
| *** tosky has quit IRC | 23:01 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!