| *** dangtrinhnt has joined #openstack-requirements | 01:32 | |
| *** hongbin has joined #openstack-requirements | 01:56 | |
| *** hongbin has quit IRC | 02:11 | |
| *** toabctl has quit IRC | 02:19 | |
| *** toabctl has joined #openstack-requirements | 02:23 | |
| *** hongbin has joined #openstack-requirements | 02:29 | |
| *** udesale has joined #openstack-requirements | 04:55 | |
| *** udesale has quit IRC | 04:58 | |
| *** udesale has joined #openstack-requirements | 04:58 | |
| *** udesale has quit IRC | 04:58 | |
| *** udesale has joined #openstack-requirements | 05:05 | |
| *** hongbin has quit IRC | 05:09 | |
| *** udesale has quit IRC | 05:40 | |
| *** udesale has joined #openstack-requirements | 05:53 | |
| openstackgerrit | Tony Breeds proposed openstack/requirements master: update constraint for oslo.service to new release 1.32.1 https://review.openstack.org/615676 | 06:00 |
|---|---|---|
| openstackgerrit | Merged openstack/requirements master: update constraint for oslo.db to new release 4.42.0 https://review.openstack.org/615678 | 06:08 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.openstack.org/615731 | 06:12 |
| openstackgerrit | Merged openstack/requirements master: update constraint for oslo.messaging to new release 9.2.0 https://review.openstack.org/615668 | 06:15 |
| openstackgerrit | Merged openstack/requirements master: update constraint for oslo.policy to new release 1.41.0 https://review.openstack.org/615670 | 06:15 |
| openstackgerrit | Merged openstack/requirements master: update constraint for osprofiler to new release 2.5.1 https://review.openstack.org/615673 | 06:15 |
| openstackgerrit | Merged openstack/requirements master: update constraint for sphinx-feature-classification to new release 0.3.1 https://review.openstack.org/615674 | 06:15 |
| openstackgerrit | Merged openstack/requirements master: update constraint for futurist to new release 1.8.0 https://review.openstack.org/615675 | 06:18 |
| openstackgerrit | Merged openstack/requirements master: update constraint for oslo.upgradecheck to new release 0.1.1 https://review.openstack.org/615679 | 06:18 |
| *** e0ne has joined #openstack-requirements | 06:24 | |
| openstackgerrit | Merged openstack/requirements master: update constraint for pbr to new release 5.1.1 https://review.openstack.org/615672 | 06:31 |
| openstackgerrit | Merged openstack/requirements master: update constraint for oslo.config to new release 6.7.0 https://review.openstack.org/615669 | 06:31 |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-freezerclient to new release 2.0.0 https://review.openstack.org/615652 | 06:31 |
| *** e0ne has quit IRC | 06:44 | |
| *** e0ne has joined #openstack-requirements | 06:48 | |
| *** udesale has quit IRC | 06:54 | |
| *** e0ne has quit IRC | 07:07 | |
| openstackgerrit | Merged openstack/requirements master: Add monasca-analytics to projects.txt https://review.openstack.org/615555 | 07:21 |
| *** udesale has joined #openstack-requirements | 07:35 | |
| *** ccamacho has joined #openstack-requirements | 08:03 | |
| *** jpich has joined #openstack-requirements | 08:57 | |
| *** e0ne has joined #openstack-requirements | 11:11 | |
| *** dtantsur|afk is now known as dtantsur | 11:33 | |
| *** e0ne has quit IRC | 12:29 | |
| *** e0ne has joined #openstack-requirements | 12:43 | |
| *** zul has joined #openstack-requirements | 13:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for neutron-lib to new release 1.20.0 https://review.openstack.org/615894 | 14:45 |
| *** e0ne has quit IRC | 15:06 | |
| openstackgerrit | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.openstack.org/615731 | 15:47 |
| prometheanfire | dhellmann: you know much about the xfails stuff? it seems to me that it checks the dependency===1.2.3 is co-defined with the caller of the dependency. What I think I'd like recorded is the bad version of the caller (since it's typically capping something) and the version of the dep known to fail. | 15:57 |
| dhellmann | I don't know what xfails is | 15:58 |
| dhellmann | is that a job/tool in the requirements repo? | 15:58 |
| prometheanfire | ya, but it's not been used in a while | 16:01 |
| prometheanfire | think tony made it | 16:01 |
| dhellmann | yeah, I don't know anything about that :-/ | 16:02 |
| prometheanfire | basically, I'm tired of holding back opentracing and now salt :P | 16:02 |
| prometheanfire | I could add exclusions to g-r but that doesn't seem right | 16:02 |
| prometheanfire | upstream is slow to move to resolve their caps | 16:03 |
| prometheanfire | at least openshift was somewhat quick to update their allowable kubernetes version | 16:03 |
| dhellmann | so opentracing and salt have caps on some of their dependencies that are lower than the versions we use? | 16:05 |
| prometheanfire | not quite | 16:06 |
| prometheanfire | jaegerclient has a cap that doesn't allow opentracing 2.0.0 https://github.com/jaegertracing/jaeger-client-python/issues/199 https://github.com/jaegertracing/jaeger-client-python/pull/206 | 16:07 |
| prometheanfire | salt is an odd one, it fails validation even though it shouldn't :| | 16:07 |
| prometheanfire | http://logs.openstack.org/31/615731/1/check/requirements-tox-py27-check-uc/3712d13/job-output.txt.gz#_2018-11-06_06_35_48_249574 | 16:09 |
| prometheanfire | https://github.com/saltstack/salt/blob/2018.3.3/requirements/base.txt#L8 | 16:09 |
| dhellmann | tornado 5.1.1 is > the max of 5.0 there | 16:10 |
| prometheanfire | but the version defined in their reqs file is 6.0 as the cap | 16:10 |
| prometheanfire | for py2.7 | 16:11 |
| dhellmann | is that job running under python 3? | 16:16 |
| dhellmann | the venv name implies it is not | 16:16 |
| prometheanfire | the tox job itself is defined as py27 | 16:17 |
| dhellmann | and the tox config looks like it sets basepython properly | 16:18 |
| dhellmann | so maybe it's a bug in the check script | 16:18 |
| prometheanfire | maybe, will have to check, just something I've noticed | 16:19 |
| *** ccamacho has quit IRC | 16:27 | |
| *** udesale has quit IRC | 16:32 | |
| openstackgerrit | Ben Nemec proposed openstack/requirements master: Exclude oslo.cache 1.31.1 https://review.openstack.org/615935 | 16:35 |
| *** andymccr has quit IRC | 16:36 | |
| openstackgerrit | Matthew Thode proposed openstack/requirements master: update tornado https://review.openstack.org/615940 | 16:43 |
| openstackgerrit | Ben Nemec proposed openstack/requirements master: Exclude oslo.cache 1.31.1 https://review.openstack.org/615935 | 16:45 |
| prometheanfire | tonyb: ping, bugging you about constraints publishing again :D | 16:50 |
| prometheanfire | I think we need to set recreate=True for the uc checks (since it doesn't seem to update if something is already installed | 17:12 |
| prometheanfire | that or change pip install to include -I | 17:12 |
| prometheanfire | that or change pip install to include -U | 17:12 |
| prometheanfire | which it has, odd | 17:12 |
| prometheanfire | also, we should remove basepython | 17:14 |
| prometheanfire | However, if ignore_basepython_conflict is set, the value is ignored and we force the basepython implied from the factor name. | 17:14 |
| *** ccamacho has joined #openstack-requirements | 17:22 | |
| prometheanfire | dhellmann: so.. change the value of basepytohn within testenv and the output changes :D | 17:30 |
| *** e0ne has joined #openstack-requirements | 17:32 | |
| *** ccamacho has quit IRC | 17:33 | |
| *** jpich has quit IRC | 17:34 | |
| prometheanfire | https://github.com/tox-dev/tox/issues/477 I think | 17:36 |
| prometheanfire | tonyb: basepython stuff again | 17:36 |
| prometheanfire | probably need to set envdir separate for each python | 17:37 |
| dhellmann | the dir should be different by default unless we're explicitly sharing them | 17:37 |
| prometheanfire | ok | 17:40 |
| dhellmann | it defaults to .tox/$envname right? | 17:40 |
| *** e0ne has quit IRC | 18:03 | |
| *** dtantsur is now known as dtantsur|afk | 18:06 | |
| *** e0ne has joined #openstack-requirements | 18:06 | |
| *** e0ne has quit IRC | 18:24 | |
| prometheanfire | true | 18:37 |
| prometheanfire | I think maybe check_conflicts just takes the last value in the list to check or something | 18:55 |
| prometheanfire | cat .tox/py27-check-uc/lib/python2.7/site-packages/salt-2018.3.3.dist-info/METADATA | 19:42 |
| prometheanfire | has tornado twice | 19:42 |
| prometheanfire | so that's no good | 19:42 |
| tonyb | prometheanfire: that isn't a problem for us but it probably is for nova that are using the envdir stuff. I'll go poke around | 19:45 |
| prometheanfire | ya | 19:46 |
| prometheanfire | tonyb: it looks like it's salt's fault (what I'm looking at at least | 19:46 |
| prometheanfire | Requirement already satisfied: tornado<6.0,>=4.2.1 in ./.tox/py27-check-uc/lib/python2.7/site-packages (from salt) (5.1.1) | 19:46 |
| prometheanfire | salt 2018.3.3 has requirement tornado<5.0,>=4.2.1, but you'll have tornado 5.1.1 which is incompatible. | 19:46 |
| prometheanfire | pip install salt with python2.7 with tornado already updated to 5.1.1 should fail | 19:46 |
| prometheanfire | https://github.com/saltstack/salt/blob/2018.3.3/requirements/base.txt#L8-L9 doesn't seem to be obeyed | 19:46 |
| tonyb | prometheanfire: Ok | 19:48 |
| prometheanfire | they have their own custom requirements.txt parser | 19:50 |
| prometheanfire | setup.py is 1k lines long :D | 19:56 |
| prometheanfire | can we get a second person looking at https://review.openstack.org/615935 ? | 21:41 |
| *** efried has joined #openstack-requirements | 21:42 | |
| efried | mordred: Is there going to be a bot or other effort to percolate this https://review.openstack.org/#/c/615441/ to projects importing monotonic? | 21:43 |
| mordred | I didn't do it | 21:43 |
| efried | I think it's not possible to update requirements on such a project (thereby triggering the requirements-check job) without doing that first. | 21:43 |
| efried | I'm about to do it for oslo.service, just wanted to make sure I didn't stomp on something a bot was gonna do. | 21:44 |
| mordred | efried: no bot updates in work ... I think doing it by hand is fine | 21:45 |
| efried | ack, review coming atcha :P | 21:45 |
| mordred | efried: although now that you mention it - I probablyshoudl follow up and do that more broadly | 21:45 |
| efried | "blah blah blah, I'm a bad person, blah blah blah" | 21:46 |
| efried | <kisses> | 21:46 |
| *** dangtrinhnt has quit IRC | 22:01 | |
| *** dangtrinhnt has joined #openstack-requirements | 22:02 | |
| openstackgerrit | Merged openstack/requirements master: Updated from generate-constraints https://review.openstack.org/615731 | 23:11 |
| *** vpickard is now known as vpickard_ | 23:25 | |
| efried | bnemec: Sorry, I'm pretty confused about https://review.openstack.org/#/c/616013/ -- how does l-c work with a py version restriction like this? | 23:39 |
| bnemec | efried: I think it should look like: monotonic>=0.6;python_version<'3.3' | 23:41 |
| bnemec | Which would mean that on python versions where it needs to be installed, it requires at least 0.6. | 23:41 |
| efried | bnemec: okay, I think that makes sense. | 23:42 |
| * efried tries | 23:42 | |
| efried | bnemec: was going to say: so we can't restrict the requirements.txt version to <py3 because that effectively removes the lower bound for py3, which conflicts with l-c; but we can't drop the requirement from l-c because then we would be unrestricted for py2... | 23:44 |
| efried | I hope that's not the case... | 23:44 |
| bnemec | I guess we'll find out when zuul reports in. :-) | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!