*** vpickard_ has joined #openstack-requirements | 00:01 | |
*** vpickard has quit IRC | 00:02 | |
*** cjloader has quit IRC | 00:44 | |
*** edmondsw has joined #openstack-requirements | 01:02 | |
*** cjloader has joined #openstack-requirements | 01:06 | |
*** edmondsw has quit IRC | 01:07 | |
*** cjloader has joined #openstack-requirements | 01:37 | |
*** hongbin has joined #openstack-requirements | 02:02 | |
*** cjloader has joined #openstack-requirements | 02:12 | |
*** lbragstad[m] has quit IRC | 02:37 | |
*** smcginnis[m] has quit IRC | 02:37 | |
*** jhesketh has quit IRC | 02:37 | |
*** lbragstad[m] has joined #openstack-requirements | 02:37 | |
*** smcginnis[m] has joined #openstack-requirements | 02:37 | |
*** jhesketh has joined #openstack-requirements | 02:37 | |
*** lbragstad[m] has quit IRC | 02:41 | |
*** smcginnis[m] has quit IRC | 02:41 | |
*** masayukig1 has quit IRC | 02:41 | |
*** vpickard_ has quit IRC | 02:41 | |
*** vpickard_ has joined #openstack-requirements | 02:42 | |
*** mtreinish has quit IRC | 02:44 | |
*** oanson has quit IRC | 02:44 | |
*** spsurya has quit IRC | 02:44 | |
*** spsurya has joined #openstack-requirements | 02:45 | |
*** mtreinish has joined #openstack-requirements | 02:45 | |
*** oanson has joined #openstack-requirements | 02:45 | |
*** tonyb has quit IRC | 02:46 | |
*** andreykurilin has quit IRC | 02:46 | |
*** openstackgerrit has quit IRC | 02:46 | |
*** tonyb has joined #openstack-requirements | 02:47 | |
*** andreykurilin has joined #openstack-requirements | 02:47 | |
*** openstackgerrit has joined #openstack-requirements | 02:48 | |
*** dtroyer has quit IRC | 02:50 | |
*** adriant has quit IRC | 02:50 | |
*** jroll has quit IRC | 02:50 | |
*** jrist has quit IRC | 02:50 | |
*** otherwiseguy has quit IRC | 02:50 | |
*** hongbin has quit IRC | 02:50 | |
*** snapiri has quit IRC | 02:50 | |
*** jmccrory has quit IRC | 02:50 | |
*** strigazi has quit IRC | 02:50 | |
*** corvus has quit IRC | 02:50 | |
*** jts72 has quit IRC | 02:50 | |
*** number80 has quit IRC | 02:50 | |
*** dtroyer has joined #openstack-requirements | 02:50 | |
*** adriant has joined #openstack-requirements | 02:50 | |
*** jroll has joined #openstack-requirements | 02:50 | |
*** jrist has joined #openstack-requirements | 02:50 | |
*** otherwiseguy has joined #openstack-requirements | 02:50 | |
*** hongbin has joined #openstack-requirements | 02:50 | |
*** snapiri has joined #openstack-requirements | 02:50 | |
*** jmccrory has joined #openstack-requirements | 02:50 | |
*** edmondsw has joined #openstack-requirements | 02:51 | |
*** strigazi has joined #openstack-requirements | 02:51 | |
*** corvus has joined #openstack-requirements | 02:51 | |
*** jts72 has joined #openstack-requirements | 02:51 | |
*** number80 has joined #openstack-requirements | 02:51 | |
*** edmondsw has quit IRC | 02:55 | |
*** masayukig1 has joined #openstack-requirements | 03:14 | |
*** hongbin has quit IRC | 03:50 | |
*** lbragstad[m] has joined #openstack-requirements | 03:52 | |
*** smcginnis[m] has joined #openstack-requirements | 03:52 | |
*** udesale has joined #openstack-requirements | 04:26 | |
*** cjloader has joined #openstack-requirements | 05:14 | |
*** CrayZee has joined #openstack-requirements | 05:31 | |
*** amoralej|off is now known as amoralej | 07:20 | |
*** hwoarang has joined #openstack-requirements | 07:44 | |
*** hwoarang has quit IRC | 07:47 | |
*** hwoarang has joined #openstack-requirements | 07:48 | |
*** ralonsoh has joined #openstack-requirements | 08:00 | |
*** dtantsur|afk is now known as dtantsur | 08:04 | |
openstackgerrit | Dmitry Tantsur proposed openstack/requirements master: Add metalsmith to projects.txt https://review.openstack.org/570770 | 08:27 |
---|---|---|
openstackgerrit | Lukáš Doktor proposed openstack/requirements master: Freeze cmd2 library on python2 https://review.openstack.org/570811 | 10:55 |
*** udesale_ has joined #openstack-requirements | 11:33 | |
*** udesale has quit IRC | 11:34 | |
*** udesale_ has quit IRC | 11:37 | |
*** amoralej is now known as amoralej|lunch | 11:55 | |
*** edmondsw_ has joined #openstack-requirements | 12:11 | |
*** udesale has joined #openstack-requirements | 12:11 | |
*** udesale has quit IRC | 12:17 | |
*** udesale has joined #openstack-requirements | 12:34 | |
*** vpickard_ is now known as vpickard | 12:53 | |
*** vpickard is now known as vpickard_ | 12:59 | |
*** vpickard_ is now known as vpickard | 12:59 | |
*** amoralej|lunch is now known as amoralej | 13:18 | |
*** dtantsur is now known as dtantsur|brb | 13:31 | |
*** lbragstad has joined #openstack-requirements | 13:32 | |
*** edmondsw_ is now known as edmondsw | 13:42 | |
*** jroll has quit IRC | 14:01 | |
*** jroll has joined #openstack-requirements | 14:02 | |
*** cjloader has joined #openstack-requirements | 14:06 | |
*** bnemec has joined #openstack-requirements | 14:20 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for instack-undercloud to new release 7.4.12 https://review.openstack.org/570943 | 14:26 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for tripleo-common to new release 7.6.13 https://review.openstack.org/570944 | 14:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/pike: update constraint for os-net-config to new release 7.3.6 https://review.openstack.org/570945 | 14:32 |
*** dtantsur|brb is now known as dtantsur | 15:00 | |
*** CrayZee has quit IRC | 15:02 | |
*** vpickard has quit IRC | 15:09 | |
*** vpickard has joined #openstack-requirements | 15:09 | |
*** udesale_ has joined #openstack-requirements | 15:54 | |
*** udesale has quit IRC | 15:57 | |
*** udesale__ has joined #openstack-requirements | 16:07 | |
*** udesale_ has quit IRC | 16:10 | |
*** udesale__ has quit IRC | 16:14 | |
*** eandersson has quit IRC | 16:53 | |
*** ralonsoh has quit IRC | 17:02 | |
*** dtantsur is now known as dtantsur|afk | 18:28 | |
*** amoralej is now known as amoralej|off | 18:38 | |
*** lbragstad has quit IRC | 18:39 | |
*** snapiri- has joined #openstack-requirements | 18:44 | |
*** jmccrory_ has joined #openstack-requirements | 18:51 | |
*** snapiri has quit IRC | 18:53 | |
*** jmccrory has quit IRC | 18:53 | |
*** jmccrory_ is now known as jmccrory | 18:53 | |
*** lbragstad has joined #openstack-requirements | 19:00 | |
openstackgerrit | Matthew Thode proposed openstack/requirements master: Freeze cmd2 library on python2 https://review.openstack.org/570811 | 19:40 |
*** cjloader has quit IRC | 19:57 | |
*** cjloader_ has joined #openstack-requirements | 19:57 | |
*** cjloader_ has quit IRC | 19:57 | |
*** cjloader has joined #openstack-requirements | 20:01 | |
fungi | prometheanfire: so what's the tactic being taken with the cmd2 situation going forward? environment markers? | 20:41 |
fungi | nevermind, i see https://review.openstack.org/570811 is headed that direction | 20:42 |
fungi | i basically suggested something similar in https://github.com/python-cmd2/cmd2/issues/421 | 20:43 |
prometheanfire | :D | 20:43 |
*** cjloader has quit IRC | 20:54 | |
*** cjloader has joined #openstack-requirements | 21:02 | |
*** edmondsw has quit IRC | 21:13 | |
prometheanfire | reqs install unconstrained, neat | 21:29 |
dhellmann | prometheanfire: ? | 21:33 |
prometheanfire | dhellmann: http://logs.openstack.org/11/570811/3/check/openstack-tox-py27/1f3b8f4/job-output.txt.gz | 21:34 |
prometheanfire | not seeing why it's trying to install that version | 21:34 |
dhellmann | our py27 environment doesn't use the constraints list? | 21:35 |
prometheanfire | for validating formatting and stuff, I don't think so | 21:35 |
dhellmann | well, that's is then | 21:35 |
dhellmann | stestr uses cliff uses cmd2 doesn't work under py27 | 21:35 |
prometheanfire | http://logs.openstack.org/11/570811/3/check/openstack-tox-py27/1f3b8f4/job-output.txt.gz#_2018-05-29_19_45_06_718027 | 21:35 |
prometheanfire | yep | 21:36 |
dhellmann | so we need to cap cmd2 or use the constraints or something | 21:36 |
dhellmann | using the constraints is probably simplest? | 21:36 |
dhellmann | although it feels a bit odd for the unit tests in that repo, since it doesn't need to be coinstallable | 21:36 |
prometheanfire | yep, I'm trying to think of side effects | 21:36 |
* dhellmann nods | 21:36 | |
prometheanfire | like, what inherits the testenv env (constraints tests do, but I think the last listed constraint wins, so... | 21:38 |
prometheanfire | also, 'tox -e py27' work locally | 21:38 |
dhellmann | oh, I'd just put it in the deps list in each env | 21:38 |
prometheanfire | so, wtf | 21:38 |
dhellmann | there's an issue with the mirror | 21:39 |
dhellmann | they removed the bad cmd2 release from pypi, but didn't remove the files, so the mirror didn't update | 21:39 |
dhellmann | so 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 directly | 21:40 |
dhellmann | a little redundancy among friends never hurt | 21:40 |
prometheanfire | sure | 21:40 |
prometheanfire | tox -e validate works locally too | 21:42 |
dhellmann | does that try to install cmd2? | 21:42 |
dhellmann | well, 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 |
prometheanfire | yes | 21:47 |
prometheanfire | ya, I don't think we fix the mirroring issue | 21:47 |
dhellmann | maybe we want an exclusion for 9.0.0 now, too? | 21:48 |
dhellmann | according to https://github.com/python-cmd2/cmd2/issues/421 they deleted it | 21:49 |
dhellmann | there'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.py | 21:49 |
openstackgerrit | Matthew Thode proposed openstack/requirements master: add stestr directory to gitignore https://review.openstack.org/571032 | 21:50 |
*** cjloader has quit IRC | 22:27 | |
fungi | prometheanfire: 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.7 | 23:18 |
*** cjloader has joined #openstack-requirements | 23:19 | |
fungi | as far as i know there's nothing else we can do on the mirror side of things to solve this particular conundrum | 23:20 |
fungi | if 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 work | 23:21 |
fungi | even constraints is a bit of a hack to rely on solving this in the long term | 23:22 |
fungi | since it only really fixes it for people installing along with our constraints file, and not in the more general case | 23:22 |
*** cjloader has quit IRC | 23:24 | |
fungi | hrm, no i'm misreading the pip output there | 23:24 |
fungi | i guess it's cliff and not stestr? | 23:25 |
fungi | oh! it's https://git.openstack.org/cgit/openstack/cliff/tree/requirements.txt#n5 | 23:26 |
fungi | yeah | 23:26 |
fungi | so we likely need the environment markers there | 23:27 |
fungi | and any similar fix to cliff is going to fail requirements-check making this a chicken-and-egg fix | 23:28 |
fungi | let me know if you need one or the other bypassing the ci system | 23:29 |
fungi | probably 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=^cmd2 | 23:31 |
prometheanfire | fungi: it sounds like we need to force something | 23:32 |
prometheanfire | it would be interesting to see 'co-dependant' deps in zuul/gerrit | 23:32 |
fungi | we've resisted implementing tat as a feature so as to discourage it as a regular development (anti)pattern | 23:33 |
fungi | it's okay for exceptional situations to require exceptional manual actions, in my opinion | 23:33 |
fungi | makes them more intentional and avoids normalizing them into the culture | 23:33 |
prometheanfire | wfm | 23:34 |
prometheanfire | just one of those idle thoughts | 23:34 |
fungi | we 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 |
prometheanfire | fungi: I'm fine with either order for the forced merge | 23:35 |
fungi | are 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 +2 | 23:36 |
prometheanfire | just +2+W'd it | 23:36 |
fungi | cool, i'll elevate my privs and get it merged, with some associated commentary | 23:37 |
prometheanfire | sure | 23:37 |
openstackgerrit | Merged openstack/requirements master: Freeze cmd2 library on python2 https://review.openstack.org/570811 | 23:39 |
prometheanfire | thanks | 23:40 |
prometheanfire | so, next step is cliff, which should just match reqs then release cliff | 23:41 |
*** openstackstatus has joined #openstack-requirements | 23:43 | |
*** ChanServ sets mode: +v openstackstatus | 23:44 | |
fungi | right | 23:52 |
fungi | at least, that's what i'm hoping! | 23:53 |
fungi | hooray for release automation is all i have to say about that | 23:53 |
prometheanfire | :D | 23:53 |
prometheanfire | fungi: I did just make a review for it, so we'll see :D | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!