Tuesday, 2018-05-29

*** vpickard_ has joined #openstack-requirements00:01
*** vpickard has quit IRC00:02
*** cjloader has quit IRC00:44
*** edmondsw has joined #openstack-requirements01:02
*** cjloader has joined #openstack-requirements01:06
*** edmondsw has quit IRC01:07
*** cjloader has joined #openstack-requirements01:37
*** hongbin has joined #openstack-requirements02:02
*** cjloader has joined #openstack-requirements02:12
*** lbragstad[m] has quit IRC02:37
*** smcginnis[m] has quit IRC02:37
*** jhesketh has quit IRC02:37
*** lbragstad[m] has joined #openstack-requirements02:37
*** smcginnis[m] has joined #openstack-requirements02:37
*** jhesketh has joined #openstack-requirements02:37
*** lbragstad[m] has quit IRC02:41
*** smcginnis[m] has quit IRC02:41
*** masayukig1 has quit IRC02:41
*** vpickard_ has quit IRC02:41
*** vpickard_ has joined #openstack-requirements02:42
*** mtreinish has quit IRC02:44
*** oanson has quit IRC02:44
*** spsurya has quit IRC02:44
*** spsurya has joined #openstack-requirements02:45
*** mtreinish has joined #openstack-requirements02:45
*** oanson has joined #openstack-requirements02:45
*** tonyb has quit IRC02:46
*** andreykurilin has quit IRC02:46
*** openstackgerrit has quit IRC02:46
*** tonyb has joined #openstack-requirements02:47
*** andreykurilin has joined #openstack-requirements02:47
*** openstackgerrit has joined #openstack-requirements02:48
*** dtroyer has quit IRC02:50
*** adriant has quit IRC02:50
*** jroll has quit IRC02:50
*** jrist has quit IRC02:50
*** otherwiseguy has quit IRC02:50
*** hongbin has quit IRC02:50
*** snapiri has quit IRC02:50
*** jmccrory has quit IRC02:50
*** strigazi has quit IRC02:50
*** corvus has quit IRC02:50
*** jts72 has quit IRC02:50
*** number80 has quit IRC02:50
*** dtroyer has joined #openstack-requirements02:50
*** adriant has joined #openstack-requirements02:50
*** jroll has joined #openstack-requirements02:50
*** jrist has joined #openstack-requirements02:50
*** otherwiseguy has joined #openstack-requirements02:50
*** hongbin has joined #openstack-requirements02:50
*** snapiri has joined #openstack-requirements02:50
*** jmccrory has joined #openstack-requirements02:50
*** edmondsw has joined #openstack-requirements02:51
*** strigazi has joined #openstack-requirements02:51
*** corvus has joined #openstack-requirements02:51
*** jts72 has joined #openstack-requirements02:51
*** number80 has joined #openstack-requirements02:51
*** edmondsw has quit IRC02:55
*** masayukig1 has joined #openstack-requirements03:14
*** hongbin has quit IRC03:50
*** lbragstad[m] has joined #openstack-requirements03:52
*** smcginnis[m] has joined #openstack-requirements03:52
*** udesale has joined #openstack-requirements04:26
*** cjloader has joined #openstack-requirements05:14
*** CrayZee has joined #openstack-requirements05:31
*** amoralej|off is now known as amoralej07:20
*** hwoarang has joined #openstack-requirements07:44
*** hwoarang has quit IRC07:47
*** hwoarang has joined #openstack-requirements07:48
*** ralonsoh has joined #openstack-requirements08:00
*** dtantsur|afk is now known as dtantsur08:04
openstackgerritDmitry Tantsur proposed openstack/requirements master: Add metalsmith to projects.txt  https://review.openstack.org/57077008:27
openstackgerritLukáš Doktor proposed openstack/requirements master: Freeze cmd2 library on python2  https://review.openstack.org/57081110:55
*** udesale_ has joined #openstack-requirements11:33
*** udesale has quit IRC11:34
*** udesale_ has quit IRC11:37
*** amoralej is now known as amoralej|lunch11:55
*** edmondsw_ has joined #openstack-requirements12:11
*** udesale has joined #openstack-requirements12:11
*** udesale has quit IRC12:17
*** udesale has joined #openstack-requirements12:34
*** vpickard_ is now known as vpickard12:53
*** vpickard is now known as vpickard_12:59
*** vpickard_ is now known as vpickard12:59
*** amoralej|lunch is now known as amoralej13:18
*** dtantsur is now known as dtantsur|brb13:31
*** lbragstad has joined #openstack-requirements13:32
*** edmondsw_ is now known as edmondsw13:42
*** jroll has quit IRC14:01
*** jroll has joined #openstack-requirements14:02
*** cjloader has joined #openstack-requirements14:06
*** bnemec has joined #openstack-requirements14:20
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for instack-undercloud to new release 7.4.12  https://review.openstack.org/57094314:26
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for tripleo-common to new release 7.6.13  https://review.openstack.org/57094414:29
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for os-net-config to new release 7.3.6  https://review.openstack.org/57094514:32
*** dtantsur|brb is now known as dtantsur15:00
*** CrayZee has quit IRC15:02
*** vpickard has quit IRC15:09
*** vpickard has joined #openstack-requirements15:09
*** udesale_ has joined #openstack-requirements15:54
*** udesale has quit IRC15:57
*** udesale__ has joined #openstack-requirements16:07
*** udesale_ has quit IRC16:10
*** udesale__ has quit IRC16:14
*** eandersson has quit IRC16:53
*** ralonsoh has quit IRC17:02
*** dtantsur is now known as dtantsur|afk18:28
*** amoralej is now known as amoralej|off18:38
*** lbragstad has quit IRC18:39
*** snapiri- has joined #openstack-requirements18:44
*** jmccrory_ has joined #openstack-requirements18:51
*** snapiri has quit IRC18:53
*** jmccrory has quit IRC18:53
*** jmccrory_ is now known as jmccrory18:53
*** lbragstad has joined #openstack-requirements19:00
openstackgerritMatthew Thode proposed openstack/requirements master: Freeze cmd2 library on python2  https://review.openstack.org/57081119:40
*** cjloader has quit IRC19:57
*** cjloader_ has joined #openstack-requirements19:57
*** cjloader_ has quit IRC19:57
*** cjloader has joined #openstack-requirements20:01
fungiprometheanfire: so what's the tactic being taken with the cmd2 situation going forward? environment markers?20:41
funginevermind, i see https://review.openstack.org/570811 is headed that direction20:42
fungii basically suggested something similar in https://github.com/python-cmd2/cmd2/issues/42120:43
prometheanfire:D20:43
*** cjloader has quit IRC20:54
*** cjloader has joined #openstack-requirements21:02
*** edmondsw has quit IRC21:13
prometheanfirereqs install unconstrained, neat21:29
dhellmannprometheanfire: ?21:33
prometheanfiredhellmann: http://logs.openstack.org/11/570811/3/check/openstack-tox-py27/1f3b8f4/job-output.txt.gz21:34
prometheanfirenot seeing why it's trying to install that version21:34
dhellmannour py27 environment doesn't use the constraints list?21:35
prometheanfirefor validating formatting and stuff, I don't think so21:35
dhellmannwell, that's is then21:35
dhellmannstestr uses cliff uses cmd2 doesn't work under py2721:35
prometheanfirehttp://logs.openstack.org/11/570811/3/check/openstack-tox-py27/1f3b8f4/job-output.txt.gz#_2018-05-29_19_45_06_71802721:35
prometheanfireyep21:36
dhellmannso we need to cap cmd2 or use the constraints or something21:36
dhellmannusing the constraints is probably simplest?21:36
dhellmannalthough it feels a bit odd for the unit tests in that repo, since it doesn't need to be coinstallable21:36
prometheanfireyep, I'm trying to think of side effects21:36
* dhellmann nods21:36
prometheanfirelike, what inherits the testenv env (constraints tests do, but I think the last listed constraint wins, so...21:38
prometheanfirealso, 'tox -e py27' work locally21:38
dhellmannoh, I'd just put it in the deps list in each env21:38
prometheanfireso, wtf21:38
dhellmannthere's an issue with the mirror21:39
dhellmannthey removed the bad cmd2 release from pypi, but didn't remove the files, so the mirror didn't update21:39
dhellmannso rather than trying to apply the constraints globally in tox.ini, I would only add the constraints for the jobs that are failing by modifying those environments directly21:40
dhellmanna little redundancy among friends never hurt21:40
prometheanfiresure21:40
prometheanfiretox -e validate works locally too21:42
dhellmanndoes that try to install cmd2?21:42
dhellmannwell, it doesn't really matter. Unless you're using the same mirror the CI system is, you aren't going to be able to reproduce the problems locally.21:43
prometheanfireyes21:47
prometheanfireya, I don't think we fix the mirroring issue21:47
dhellmannmaybe we want an exclusion for 9.0.0 now, too?21:48
dhellmannaccording to https://github.com/python-cmd2/cmd2/issues/421 they deleted it21:49
dhellmannthere's an 0.9.1 though, and I guess that's likely to have similar issues, since they're trying to enforce the fix in setup.py21:49
openstackgerritMatthew Thode proposed openstack/requirements master: add stestr directory to gitignore  https://review.openstack.org/57103221:50
*** cjloader has quit IRC22:27
fungiprometheanfire: dhellmann: i don't think there's any 0.9.0 causing problems at this point. it's that pip grabs the 0.9.1 sdist and then decides after trying to install it that it's not supported under python 2.723:18
*** cjloader has joined #openstack-requirements23:19
fungias far as i know there's nothing else we can do on the mirror side of things to solve this particular conundrum23:20
fungiif stestr (which is where we're currently tripping over it) but more generally all projects declaring an open-ended dependency on cmd2 implement the same environment markers then it ought to work23:21
fungieven constraints is a bit of a hack to rely on solving this in the long term23:22
fungisince it only really fixes it for people installing along with our constraints file, and not in the more general case23:22
*** cjloader has quit IRC23:24
fungihrm, no i'm misreading the pip output there23:24
fungii guess it's cliff and not stestr?23:25
fungioh! it's https://git.openstack.org/cgit/openstack/cliff/tree/requirements.txt#n523:26
fungiyeah23:26
fungiso we likely need the environment markers there23:27
fungiand any similar fix to cliff is going to fail requirements-check making this a chicken-and-egg fix23:28
fungilet me know if you need one or the other bypassing the ci system23:29
fungiprobably bypassing ci validation on 570811 would make the most sense, and then manually sync the environment markers syntax to any repos showing up in http://codesearch.openstack.org/?q=^cmd223:31
prometheanfirefungi: it sounds like we need to force something23:32
prometheanfireit would be interesting to see 'co-dependant' deps in zuul/gerrit23:32
fungiwe've resisted implementing tat as a feature so as to discourage it as a regular development (anti)pattern23:33
fungiit's okay for exceptional situations to require exceptional manual actions, in my opinion23:33
fungimakes them more intentional and avoids normalizing them into the culture23:33
prometheanfirewfm23:34
prometheanfirejust one of those idle thoughts23:34
fungiwe almost only ever _need_ them in two situations: 1. dealing with unresolvable outside dependency changes, and 2. to fix a problem we created when we bypassed the ci system for another change ;)23:34
prometheanfirefungi: I'm fine with either order for the forced merge23:35
fungiare you good with 570811? as a (transitive) core reviewer for requirements, i'm happy to approve and merge manually so long as it has your +223:36
prometheanfirejust +2+W'd it23:36
fungicool, i'll elevate my privs and get it merged, with some associated commentary23:37
prometheanfiresure23:37
openstackgerritMerged openstack/requirements master: Freeze cmd2 library on python2  https://review.openstack.org/57081123:39
prometheanfirethanks23:40
prometheanfireso, next step is cliff, which should just match reqs then release cliff23:41
*** openstackstatus has joined #openstack-requirements23:43
*** ChanServ sets mode: +v openstackstatus23:44
fungiright23:52
fungiat least, that's what i'm hoping!23:53
fungihooray for release automation is all i have to say about that23:53
prometheanfire:D23:53
prometheanfirefungi: I did just make a review for it, so we'll see :D23:54

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