openstackgerrit | Tony Breeds proposed openstack/requirements master: Ensure we install libssl-dev on dpkg systems https://review.openstack.org/588441 | 00:30 |
---|---|---|
openstackgerrit | Tony Breeds proposed openstack/requirements master: add mit-krb5 to bindep for gentoo https://review.openstack.org/594496 | 00:31 |
tonyb | prometheanfire: Okay you do it | 00:36 |
openstackgerrit | Merged openstack/requirements master: Blacklist pysaml2 4.6.0 https://review.openstack.org/597179 | 01:05 |
*** hongbin has joined #openstack-requirements | 01:54 | |
prometheanfire | tonyb: I will, just been busy the last two days | 04:49 |
tonyb | prometheanfire: okay | 04:49 |
*** hongbin has quit IRC | 04:56 | |
*** ccamacho has joined #openstack-requirements | 05:38 | |
*** udesale has joined #openstack-requirements | 05:53 | |
*** snapiri has joined #openstack-requirements | 05:59 | |
*** openstackgerrit has quit IRC | 06:07 | |
*** udesale has quit IRC | 06:07 | |
*** udesale has joined #openstack-requirements | 06:08 | |
*** olivierb_ has joined #openstack-requirements | 07:47 | |
*** hwoarang has joined #openstack-requirements | 07:49 | |
*** jpich has joined #openstack-requirements | 07:56 | |
*** dtantsur|afk is now known as dtantsur | 08:49 | |
*** openstackgerrit has joined #openstack-requirements | 09:16 | |
openstackgerrit | Merged openstack/requirements master: Ensure we install libssl-dev on dpkg systems https://review.openstack.org/588441 | 09:16 |
*** udesale has quit IRC | 09:26 | |
*** ccamacho has quit IRC | 09:39 | |
*** ccamacho has joined #openstack-requirements | 09:39 | |
openstackgerrit | Merged openstack/requirements stable/rocky: update constraint for paunch to new release 3.2.0 https://review.openstack.org/596372 | 10:15 |
*** udesale has joined #openstack-requirements | 10:33 | |
*** udesale has quit IRC | 10:44 | |
*** udesale has joined #openstack-requirements | 10:44 | |
*** hwoarang has quit IRC | 11:20 | |
*** hwoarang has joined #openstack-requirements | 11:20 | |
*** rnm has joined #openstack-requirements | 11:34 | |
*** rnm is now known as rmart04 | 11:36 | |
*** rmart04 has quit IRC | 11:40 | |
*** rnm has joined #openstack-requirements | 11:40 | |
*** rnm has quit IRC | 11:42 | |
*** rnm has joined #openstack-requirements | 11:42 | |
*** rnm is now known as rmart04 | 11:43 | |
*** hwoarang has quit IRC | 11:46 | |
*** hwoarang has joined #openstack-requirements | 11:46 | |
*** gouthamr has quit IRC | 11:49 | |
*** dmellado has quit IRC | 11:50 | |
*** udesale has quit IRC | 12:08 | |
*** udesale has joined #openstack-requirements | 12:11 | |
*** oanson has joined #openstack-requirements | 12:30 | |
*** udesale has quit IRC | 12:40 | |
*** udesale has joined #openstack-requirements | 12:53 | |
*** toabctl has joined #openstack-requirements | 13:44 | |
*** udesale has quit IRC | 13:49 | |
*** udesale has joined #openstack-requirements | 13:50 | |
*** udesale has quit IRC | 15:04 | |
*** olivier__ has joined #openstack-requirements | 15:21 | |
*** olivierb_ has quit IRC | 15:22 | |
*** olivier__ has quit IRC | 15:46 | |
*** olivierb_ has joined #openstack-requirements | 15:48 | |
*** jpich has quit IRC | 16:10 | |
*** shachar has joined #openstack-requirements | 16:12 | |
*** snapiri has quit IRC | 16:14 | |
*** ccamacho has quit IRC | 16:19 | |
prometheanfire | lbragstad: it doesn't hurt anything, just looks odd https://github.com/openstack/keystone/blob/14.0.0/requirements.txt#L12 | 16:28 |
*** bnemec has quit IRC | 16:35 | |
*** rmart04 has quit IRC | 16:41 | |
*** gouthamr has joined #openstack-requirements | 16:49 | |
*** ccamacho has joined #openstack-requirements | 16:58 | |
*** dtantsur is now known as dtantsur|afk | 17:04 | |
*** bnemec has joined #openstack-requirements | 17:17 | |
*** ccamacho has quit IRC | 17:24 | |
*** dmellado has joined #openstack-requirements | 17:44 | |
*** toabctl has quit IRC | 18:51 | |
*** toabctl has joined #openstack-requirements | 19:07 | |
*** olivier__ has joined #openstack-requirements | 19:19 | |
*** olivierb_ has quit IRC | 19:20 | |
*** gagehugo has joined #openstack-requirements | 19:50 | |
lbragstad | o/ we have reason to believe the requirements for stable/ocata in keystoneauth are causing gate failures | 19:52 |
lbragstad | for example - https://review.openstack.org/#/c/598212/ | 19:52 |
lbragstad | specifically the python requests library being out of sync with what openstack-requirements is expecting http://logs.openstack.org/12/598212/1/check/build-openstack-sphinx-docs/dda9886/job-output.txt.gz#_2018-08-30_18_32_44_909526 | 19:53 |
*** olivier__ has quit IRC | 19:54 | |
dhellmann | lbragstad : that looks like a side-effect of having tox_install.sh in that branch still. In some other cases we've had to backport the change to remove that, although those did have a different error message. | 19:57 |
lbragstad | interesting | 19:58 |
dhellmann | oh, hmm | 19:58 |
dhellmann | I wonder if that's literally trying to install openstack-requirements and that has requests pinned | 19:58 |
dhellmann | prometheanfire , tonyb : ^^ | 19:58 |
lbragstad | yeah - that's what it looked like to me | 19:58 |
lbragstad | but i'm also relatively illiterate with respect the to intricacies of how our requirements system works | 19:59 |
prometheanfire | dhellmann: that's what it looks like | 19:59 |
prometheanfire | wonder how that's getting pulled in | 20:00 |
dhellmann | lbragstad : try backporting this change and see if it works on that branch: https://review.openstack.org/#/c/568984/ | 20:00 |
dhellmann | prometheanfire , lbragstad : the tox_install.sh script installs openstack-requirements | 20:01 |
dhellmann | dunno why | 20:01 |
dhellmann | line 22 | 20:01 |
dhellmann | so you could also try just removing that line | 20:01 |
prometheanfire | looks right | 20:02 |
lbragstad | i assume https://review.openstack.org/#/c/568984/2/tox.ini,unified@12 will need to be updated to stable/ocata instead of stable/queens | 20:05 |
dhellmann | yeah | 20:05 |
lbragstad | i think https://review.openstack.org/#/c/568984/2/test-requirements.txt is going to need to be dependent on a change to introduce doc/requirements.txt | 20:08 |
lbragstad | https://review.openstack.org/#/c/529164/ ? | 20:11 |
dhellmann | lbragstad : that part may not be needed? you may be able to get away with just the tox.ini changes and removing tox_install.sh | 20:13 |
dhellmann | I guess it's not so much a backport as crafting a patch to do the important parts of that for this branch | 20:14 |
lbragstad | the tox.ini changes reference doc/requirements.txt - i just omit those then? | 20:14 |
dhellmann | the important bit is removing the special install command and adding the constraint argument to deps | 20:14 |
dhellmann | so you can keep whatever requirements files you actually have | 20:14 |
openstackgerrit | Yossi Boaron proposed openstack/requirements master: Update Openshift upper constraints https://review.openstack.org/595569 | 20:21 |
*** mugsie has quit IRC | 20:25 | |
lbragstad | dhellmann gagehugo https://review.openstack.org/598346 | 20:28 |
lbragstad | that apparently works for me locally | 20:28 |
dhellmann | that looks like it is probably right; let's see what zuul has to say | 20:28 |
lbragstad | but that is proposed to stable/ocata and afaik stable/pike still has tox_install.sh | 20:29 |
lbragstad | https://github.com/openstack/keystoneauth/tree/stable/pike/tools | 20:29 |
gagehugo | lbragstad should that be s/queens/ocata on upper-constraints? | 20:29 |
gagehugo | https://review.openstack.org/#/c/598346/1/tox.ini@13 | 20:29 |
dhellmann | lbragstad : do the tox_install.sh scripts in the branches match? | 20:29 |
dhellmann | that requirement that was messing you up was being force-installed from inside the script | 20:30 |
lbragstad | gagehugo done | 20:31 |
gagehugo | now if zuul is happy, then :) | 20:32 |
lbragstad | dhellmann yeah - they appear to match https://github.com/openstack/keystoneauth/blob/stable/ocata/tools/tox_install.sh vs https://github.com/openstack/keystoneauth/blob/stable/pike/tools/tox_install.sh | 20:32 |
tonyb | dhellmann: Oh :/ I s'pose it could be I guess a new release might fix it | 20:32 |
dhellmann | yeah, the constrained version is the other potential difference | 20:32 |
*** mugsie has joined #openstack-requirements | 20:39 | |
lbragstad | failed because lxml wasn't installed https://review.openstack.org/#/c/598346/2 | 20:43 |
gagehugo | might have to add those to test-requirements | 20:44 |
*** rmart04 has joined #openstack-requirements | 20:47 | |
lbragstad | gagehugo done - pushed a new version of the patch to stable/ocata for ksa | 20:56 |
lbragstad | if that works then we can do the same for stable/pike since it looks like that's broken, too | 20:56 |
lbragstad | and ksm | 20:56 |
*** rmart04 has quit IRC | 21:16 | |
*** dmellado has quit IRC | 21:31 | |
*** dmellado has joined #openstack-requirements | 21:31 | |
lbragstad | gagehugo so close - https://review.openstack.org/#/c/598346/6 | 21:38 |
lbragstad | but it actually looks like it's failing on something ansible related now? http://logs.openstack.org/46/598346/6/check/legacy-tempest-dsvm-neutron-src/b1624d3/job-output.txt.gz#_2018-08-30_21_29_58_559147 | 21:38 |
lbragstad | or - ARA is expecting a version of ansible that isn't available? | 21:39 |
dhellmann | there was a new ARA release today | 22:15 |
lbragstad | should that affect stable branches? | 22:15 |
dhellmann | lbragstad : no idea. I saw a comment on one review from bnemec about a fix but I can't find that any more | 22:36 |
bnemec | dhellmann: lbragstad: Yeah, there's an infra patch up to use newer ansible that should fix that. | 22:37 |
* bnemec digs up link | 22:37 | |
bnemec | https://review.openstack.org/591527/ | 22:37 |
*** adriant has quit IRC | 23:23 | |
*** adriant has joined #openstack-requirements | 23:23 | |
*** ianw has joined #openstack-requirements | 23:25 | |
ianw | tonyb: so i think everything is now working re build & release jobs? | 23:25 |
ianw | http://logs.openstack.org/periodic/git.openstack.org/openstack/requirements/master/release-wheel-mirror/84c1ba3/ara-report/ | 23:26 |
ianw | has actually released | 23:27 |
tonyb | ianw: Great. theer is a bindep change from prometheanfire that I was going to fast merge to check the failed.txt | 23:36 |
ianw | ok, you should get the failed.txt on just the gate check jobs though? | 23:41 |
tonyb | ianw: True. I can just recheck it | 23:43 |
tonyb | ianw: https://review.openstack.org/#/c/594496/2 in the check pipeline now | 23:45 |
tonyb | ianw: It only impacts gentoo so I was thinking it'd be safe to fast approve | 23:45 |
ianw | yeah, good to see what's failing. i was thinking it would be good to have an even finer-grained bindep.txt; somehow mapping python requirements to binaries | 23:49 |
ianw | it sounds like *a lot* of work though | 23:49 |
prometheanfire | lol | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!