Thursday, 2018-08-30

openstackgerritTony Breeds proposed openstack/requirements master: Ensure we install libssl-dev on dpkg systems  https://review.openstack.org/58844100:30
openstackgerritTony Breeds proposed openstack/requirements master: add mit-krb5 to bindep for gentoo  https://review.openstack.org/59449600:31
tonybprometheanfire: Okay you do it00:36
openstackgerritMerged openstack/requirements master: Blacklist pysaml2 4.6.0  https://review.openstack.org/59717901:05
*** hongbin has joined #openstack-requirements01:54
prometheanfiretonyb: I will, just been busy the last two days04:49
tonybprometheanfire: okay04:49
*** hongbin has quit IRC04:56
*** ccamacho has joined #openstack-requirements05:38
*** udesale has joined #openstack-requirements05:53
*** snapiri has joined #openstack-requirements05:59
*** openstackgerrit has quit IRC06:07
*** udesale has quit IRC06:07
*** udesale has joined #openstack-requirements06:08
*** olivierb_ has joined #openstack-requirements07:47
*** hwoarang has joined #openstack-requirements07:49
*** jpich has joined #openstack-requirements07:56
*** dtantsur|afk is now known as dtantsur08:49
*** openstackgerrit has joined #openstack-requirements09:16
openstackgerritMerged openstack/requirements master: Ensure we install libssl-dev on dpkg systems  https://review.openstack.org/58844109:16
*** udesale has quit IRC09:26
*** ccamacho has quit IRC09:39
*** ccamacho has joined #openstack-requirements09:39
openstackgerritMerged openstack/requirements stable/rocky: update constraint for paunch to new release 3.2.0  https://review.openstack.org/59637210:15
*** udesale has joined #openstack-requirements10:33
*** udesale has quit IRC10:44
*** udesale has joined #openstack-requirements10:44
*** hwoarang has quit IRC11:20
*** hwoarang has joined #openstack-requirements11:20
*** rnm has joined #openstack-requirements11:34
*** rnm is now known as rmart0411:36
*** rmart04 has quit IRC11:40
*** rnm has joined #openstack-requirements11:40
*** rnm has quit IRC11:42
*** rnm has joined #openstack-requirements11:42
*** rnm is now known as rmart0411:43
*** hwoarang has quit IRC11:46
*** hwoarang has joined #openstack-requirements11:46
*** gouthamr has quit IRC11:49
*** dmellado has quit IRC11:50
*** udesale has quit IRC12:08
*** udesale has joined #openstack-requirements12:11
*** oanson has joined #openstack-requirements12:30
*** udesale has quit IRC12:40
*** udesale has joined #openstack-requirements12:53
*** toabctl has joined #openstack-requirements13:44
*** udesale has quit IRC13:49
*** udesale has joined #openstack-requirements13:50
*** udesale has quit IRC15:04
*** olivier__ has joined #openstack-requirements15:21
*** olivierb_ has quit IRC15:22
*** olivier__ has quit IRC15:46
*** olivierb_ has joined #openstack-requirements15:48
*** jpich has quit IRC16:10
*** shachar has joined #openstack-requirements16:12
*** snapiri has quit IRC16:14
*** ccamacho has quit IRC16:19
prometheanfirelbragstad: it doesn't hurt anything, just looks odd https://github.com/openstack/keystone/blob/14.0.0/requirements.txt#L1216:28
*** bnemec has quit IRC16:35
*** rmart04 has quit IRC16:41
*** gouthamr has joined #openstack-requirements16:49
*** ccamacho has joined #openstack-requirements16:58
*** dtantsur is now known as dtantsur|afk17:04
*** bnemec has joined #openstack-requirements17:17
*** ccamacho has quit IRC17:24
*** dmellado has joined #openstack-requirements17:44
*** toabctl has quit IRC18:51
*** toabctl has joined #openstack-requirements19:07
*** olivier__ has joined #openstack-requirements19:19
*** olivierb_ has quit IRC19:20
*** gagehugo has joined #openstack-requirements19:50
lbragstado/ we have reason to believe the requirements for stable/ocata in keystoneauth are causing gate failures19:52
lbragstadfor example - https://review.openstack.org/#/c/598212/19:52
lbragstadspecifically 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_90952619:53
*** olivier__ has quit IRC19:54
dhellmannlbragstad : 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
lbragstadinteresting19:58
dhellmannoh, hmm19:58
dhellmannI wonder if that's literally trying to install openstack-requirements and that has requests pinned19:58
dhellmannprometheanfire , tonyb : ^^19:58
lbragstadyeah - that's what it looked like to me19:58
lbragstadbut i'm also relatively illiterate with respect the to intricacies of how our requirements system works19:59
prometheanfiredhellmann: that's what it looks like19:59
prometheanfirewonder how that's getting pulled in20:00
dhellmannlbragstad : try backporting this change and see if it works on that branch: https://review.openstack.org/#/c/568984/20:00
dhellmannprometheanfire , lbragstad : the tox_install.sh script installs openstack-requirements20:01
dhellmanndunno why20:01
dhellmannline 2220:01
dhellmannso you could also try just removing that line20:01
prometheanfirelooks right20:02
lbragstadi assume https://review.openstack.org/#/c/568984/2/tox.ini,unified@12 will need to be updated to stable/ocata instead of stable/queens20:05
dhellmannyeah20:05
lbragstadi 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.txt20:08
lbragstadhttps://review.openstack.org/#/c/529164/ ?20:11
dhellmannlbragstad : that part may not be needed? you may be able to get away with just the tox.ini changes and removing tox_install.sh20:13
dhellmannI guess it's not so much a backport as crafting a patch to do the important parts of that for this branch20:14
lbragstadthe tox.ini changes reference doc/requirements.txt - i just omit those then?20:14
dhellmannthe important bit is removing the special install command and adding the constraint argument to deps20:14
dhellmannso you can keep whatever requirements files you actually have20:14
openstackgerritYossi Boaron proposed openstack/requirements master: Update Openshift upper constraints  https://review.openstack.org/59556920:21
*** mugsie has quit IRC20:25
lbragstaddhellmann gagehugo https://review.openstack.org/59834620:28
lbragstadthat apparently works for me locally20:28
dhellmannthat looks like it is probably right; let's see what zuul has to say20:28
lbragstadbut that is proposed to stable/ocata and afaik stable/pike still has tox_install.sh20:29
lbragstadhttps://github.com/openstack/keystoneauth/tree/stable/pike/tools20:29
gagehugolbragstad should that be s/queens/ocata on upper-constraints?20:29
gagehugohttps://review.openstack.org/#/c/598346/1/tox.ini@1320:29
dhellmannlbragstad : do the tox_install.sh scripts in the branches match?20:29
dhellmannthat requirement that was messing you up was being force-installed from inside the script20:30
lbragstadgagehugo done20:31
gagehugonow if zuul is happy, then :)20:32
lbragstaddhellmann 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.sh20:32
tonybdhellmann: Oh :/ I s'pose it could be I guess a new release might fix it20:32
dhellmannyeah, the constrained version is the other potential difference20:32
*** mugsie has joined #openstack-requirements20:39
lbragstadfailed because lxml wasn't installed https://review.openstack.org/#/c/598346/220:43
gagehugomight have to add those to test-requirements20:44
*** rmart04 has joined #openstack-requirements20:47
lbragstadgagehugo done - pushed a new version of the patch to stable/ocata for ksa20:56
lbragstadif that works then we can do the same for stable/pike since it looks like that's broken, too20:56
lbragstadand ksm20:56
*** rmart04 has quit IRC21:16
*** dmellado has quit IRC21:31
*** dmellado has joined #openstack-requirements21:31
lbragstadgagehugo so close - https://review.openstack.org/#/c/598346/621:38
lbragstadbut 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_55914721:38
lbragstador - ARA is expecting a version of ansible that isn't available?21:39
dhellmannthere was a new ARA release today22:15
lbragstadshould that affect stable branches?22:15
dhellmannlbragstad : no idea. I saw a comment on one review from bnemec about a fix but I can't find that any more22:36
bnemecdhellmann: lbragstad: Yeah, there's an infra patch up to use newer ansible that should fix that.22:37
* bnemec digs up link22:37
bnemechttps://review.openstack.org/591527/22:37
*** adriant has quit IRC23:23
*** adriant has joined #openstack-requirements23:23
*** ianw has joined #openstack-requirements23:25
ianwtonyb: so i think everything is now working re build & release jobs?23:25
ianwhttp://logs.openstack.org/periodic/git.openstack.org/openstack/requirements/master/release-wheel-mirror/84c1ba3/ara-report/23:26
ianwhas actually released23:27
tonybianw: Great.  theer is a bindep change from prometheanfire that I was going to fast merge to check the failed.txt23:36
ianwok, you should get the failed.txt on just the gate check jobs though?23:41
tonybianw: True.  I can just recheck it23:43
tonybianw: https://review.openstack.org/#/c/594496/2 in the check pipeline now23:45
tonybianw: It only impacts gentoo so I was thinking it'd be safe to fast approve23:45
ianwyeah, 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 binaries23:49
ianwit sounds like *a lot* of work though23:49
prometheanfirelol23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!