Tuesday, 2018-04-17

*** jroll has joined #openstack-requirements00:00
*** cjloader has quit IRC00:07
*** masayukig has quit IRC00:09
*** dmueller[m] has quit IRC00:09
*** jroll has quit IRC00:16
*** jroll has joined #openstack-requirements00:17
*** andreas_s has joined #openstack-requirements01:00
*** cjloader has joined #openstack-requirements01:02
prometheanfiredhellmann: that unicode error is for a period01:03
prometheanfire0x2e01:03
*** andreas_s has quit IRC01:05
*** cjloader has quit IRC01:06
*** hongbin_ has joined #openstack-requirements01:12
openstackgerritMatthew Thode proposed openstack/requirements master: fix unicode errors  https://review.openstack.org/56176101:12
prometheanfiredhellmann: ^maybe?01:12
prometheanfiretrying -egenerate now01:13
*** pabelanger_ has joined #openstack-requirements02:02
*** otherwiseguy_ has joined #openstack-requirements02:02
*** cjloader has joined #openstack-requirements02:02
*** pabelanger has quit IRC02:03
*** pabelang- has joined #openstack-requirements02:03
*** otherwiseguy has quit IRC02:04
*** otherwiseguy has joined #openstack-requirements02:04
*** pabelang- is now known as pabelanger02:05
*** otherwiseguy_ has quit IRC02:06
*** pabelanger_ has quit IRC02:06
*** cjloader has quit IRC02:07
*** cjloader has joined #openstack-requirements02:08
*** cjloader has quit IRC02:13
*** adriant has joined #openstack-requirements02:17
adriantHello, anyone potentially around who can answer some silly questions about global-requirements?02:18
prometheanfireadriant: maybe02:21
prometheanfire:D02:21
adriantprometheanfire: well, mainly I'm curious how it works when one project wants to use versions greater than in the global-requirements. Basically, are all official openstack projects held back by the slowest member?02:22
prometheanfireyep02:22
adriantand how would things work for projects that given the end of py27 want to become exclusively py3?02:22
prometheanfireatm, I don't think that's allowed (via the TC)02:23
adrianthttps://pythonclock.org/02:23
adriant^ well they don't have long to decide02:23
prometheanfireyep, we are aware02:23
adriant:P02:23
prometheanfireiirc the TC still states py27 is supported02:23
prometheanfirenot sure they have allowed py3 only stuff02:23
prometheanfirewhat lib is it btw?02:24
adriantgiven that most deployments use venvs or docker though, is there still a huge need for global requirements to be held back by the slowest member?02:24
adriantprometheanfire: I'm applying one of our projects for official status and curious what that entails from the requirements side02:25
adriantprometheanfire: most of my requirements are fine, expect maybe django 2.0 which Horizon doesn't support yet (but will soon), and wanting to use https://dramatiq.io/02:26
prometheanfireadriant: yep, distros are still a thing02:26
prometheanfireadding a lib is fine as long as it follows our guide (licence is good, testing is good, py versions supported are good, code is good, nothing like it in use is good)02:27
adriantprometheanfire: well I convinced the above library to change their license from AGPL to LGPL so I know that's now acceptable :P02:29
adriantbut it's a py3 only library so I doubt it could get added until something was decided re py27 support02:29
prometheanfireah02:31
prometheanfiretonyb: around?02:31
tonybkinda sorta on calls02:33
tonyb'sup?02:33
prometheanfiretonyb: we don't ally py3 only libs in reqs do we?02:33
tonybprometheanfire: ? rephrase?02:34
adriantallow*02:34
adrianttonyb: can py3 only libraries be added to global-requirements02:35
prometheanfireI don't think so02:35
prometheanfireit may be more of a tc level thing to decide, not sure02:35
prometheanfiremaybe not02:35
tonybadriant: what does consumer look like on py2?02:36
adrianttonyb: what do you mean?02:36
adriantThe library was only ever py3, it's new02:36
adriantit's not a version, outright new library02:36
adrianttonyb: https://dramatiq.io02:37
adriantso no conflicts or sudden breakages to get hit by02:37
tonybadriant: Sorry which OpenStack service wants to use that library and what will happen on that service if it's running on py202:37
adrianttonyb: no existing service, I want to use it for a service I'm applying with for official status02:38
adriantand a service I want to ideally also make py3 only02:38
adriantfor reference: https://review.openstack.org/#/c/55364302:38
tonybadriant: okay I'll review once I get off this call, it's a little more complex than I can multi-task02:40
adrianttonyb: np! Just wanted to start asking about it since I'd rather not hit issues after potentially getting official status (if it happens)02:41
prometheanfiretonyb: thanks02:41
adriantand with py27 being dead soon I don't see the worth in keeping my service compatible with py2702:41
adriantespecially when a really useful library I want to use is py3 only (a good excuse to drop py27 support for me)02:42
adrianttonyb, prometheanfire: thanks!02:42
*** cjloader has joined #openstack-requirements02:58
*** cjloader has quit IRC03:06
prometheanfiretonyb: also, when you have time https://review.openstack.org/56176103:07
prometheanfiredhellmann: that 'fix' worked locally03:07
*** cjloader has joined #openstack-requirements03:16
*** cjloader has quit IRC03:21
*** andreas_s has joined #openstack-requirements03:45
*** andreas_s has quit IRC03:50
*** cjloader has joined #openstack-requirements03:57
*** otherwiseguy has quit IRC04:03
*** cjloader has quit IRC04:03
*** otherwiseguy_ has joined #openstack-requirements04:03
*** andymccr has quit IRC04:05
*** andymccr has joined #openstack-requirements04:05
*** udesale has joined #openstack-requirements04:11
*** hongbin_ has quit IRC04:21
dhellmannprometheanfire , tonyb , adriant : please take the question about python-3-only services to the mailing list. I think that's something we're going to need to address very soon.04:48
adriantdhellmann: agreed. Will send an email out to openstack-dev (and SIGs)?04:48
*** cjloader has joined #openstack-requirements04:57
*** cjloader has quit IRC05:07
openstackgerritShachar Snapiri proposed openstack/requirements master: Update skydive version to 0.4.5  https://review.openstack.org/56030505:12
openstackgerritShachar Snapiri proposed openstack/requirements master: Update skydive-client lc version to 0.4.4  https://review.openstack.org/56178505:15
tonybdhellmann, Yup. it's clearly not a requirements decsion on how we handle py3 only services05:29
*** masayukig has joined #openstack-requirements05:29
*** dmueller[m] has joined #openstack-requirements05:35
prometheanfiretonyb: http://logs.openstack.org/periodic/git.openstack.org/openstack/requirements/master/propose-updates/fe35da6/job-output.txt.gz#_2018-04-16_06_07_52_80338505:39
prometheanfiretonyb: that's the problem05:39
prometheanfirenot sure what triggered it05:39
tonybYeah I'd like to know that, it started on 2018-04-04 06:22:14.246180 (http://logs.openstack.org/periodic/git.openstack.org/openstack/requirements/master/propose-updates/d12d717/job-output.txt.gz#_2018-04-04_06_22_14_246180)05:45
tonybprometheanfire: no idea why that's different05:45
tonybprometheanfire: ooooh you're patching the symptom not the cause05:47
prometheanfiremaybe?05:47
tonybprometheanfire: the real problem is that libvirt-python no longer builds on xenial05:47
prometheanfirereally I'm not sure we do much data sanitation :(05:47
prometheanfireoh, wonderful :|05:47
tonybSo it'd be much better to make sure we failed with *that* error no the ascii thing but we need to fix the libvirt thing05:48
eanderssonHow do we deal with python 2.7 only requirements and lower-constraints?05:53
eanderssone.g. https://review.openstack.org/#/c/560541/2/requirements.txt05:53
eanderssonIs that the only way? ^05:54
prometheanfirehttps://www.python.org/dev/peps/pep-0496/05:54
prometheanfireenv specifiers05:55
eanderssonReferring to this error > Package u'qpid-python' is used in requirements.txt but not in lower-constraints.txt05:57
eanderssonbecause lower-constrains is run using python 3, so it does not know about qpid-python05:58
*** cjloader has joined #openstack-requirements06:02
prometheanfireeandersson: can you link to the error/line?06:03
eanderssonThe error is06:04
eandersson> > Package u'qpid-python' is used in requirements.txt but not in lower-constraints.txt06:04
eanderssonLet me find the complete logs from my local workstation06:04
eanderssonhttp://paste.openstack.org/show/719337/06:06
prometheanfirewhy is it in your requirements.txt?06:06
prometheanfireif you aren't using it, just remove it :D06:06
eanderssonhaha yea I vote for removing as well06:07
eanderssonbut I don't know why it's there in the first place06:07
eanderssonI'll confirm with the Senlin people I guess =06:07
eandersson=]06:07
prometheanfirenor I06:07
prometheanfiregit blame :D06:08
*** cjloader has quit IRC06:12
*** andreas_s has joined #openstack-requirements06:17
eanderssonhaha I tried, but was updated by the bot last time, and didn't care enough to dig deeper :D06:28
*** florianf has joined #openstack-requirements06:48
*** cjloader has joined #openstack-requirements06:56
*** cjloader has quit IRC07:01
*** hoangcx_ has joined #openstack-requirements07:09
*** hoangcx_ has quit IRC07:10
*** hoangcx_ has joined #openstack-requirements07:10
*** hoangcx_ has quit IRC07:11
*** ralonsoh has joined #openstack-requirements07:18
*** ralonsoh has quit IRC07:19
*** cjloader has joined #openstack-requirements07:57
*** jpich has joined #openstack-requirements08:04
*** cjloader has quit IRC08:07
openstackgerritShachar Snapiri proposed openstack/requirements master: Update skydive-client lc version to 0.4.5  https://review.openstack.org/56178508:10
*** udesale_ has joined #openstack-requirements08:32
*** udesale has quit IRC08:34
*** ralonsoh has joined #openstack-requirements08:54
*** edmondsw has joined #openstack-requirements08:58
*** edmondsw has quit IRC09:02
*** cjloader has joined #openstack-requirements09:02
*** cjloader has quit IRC09:07
*** udesale__ has joined #openstack-requirements09:32
*** udesale_ has quit IRC09:35
*** cjloader has joined #openstack-requirements10:00
*** cjloader has quit IRC10:12
*** cjloader has joined #openstack-requirements11:02
*** cjloader has quit IRC11:07
*** udesale__ has quit IRC11:34
*** cjloader has joined #openstack-requirements12:02
*** cjloader has quit IRC12:06
openstackgerritStephen Finucane proposed openstack/requirements master: Add psycopg2-binary to global-requirements  https://review.openstack.org/56192412:23
*** edmondsw has joined #openstack-requirements12:37
*** cjloader has joined #openstack-requirements13:02
*** cjloader has quit IRC13:07
*** andreykurilin has joined #openstack-requirements13:20
dhellmannadriant : openstack-dev should be enough13:31
dhellmanneandersson, prometheanfire : only oslo.messaging should need to have a dependency on qpid-python13:32
*** udesale has joined #openstack-requirements13:51
*** andreas_s has quit IRC13:52
*** cjloader has joined #openstack-requirements14:01
*** udesale has quit IRC14:02
*** cjloader has quit IRC14:06
*** cjloader has joined #openstack-requirements14:11
*** johnsom has quit IRC14:11
*** johnsom has joined #openstack-requirements14:12
*** ralonsoh has quit IRC14:13
*** cjloader has quit IRC14:18
*** kiennt26_ has joined #openstack-requirements14:34
*** cjloader has joined #openstack-requirements14:47
*** cjloader has quit IRC14:48
*** cjloader has joined #openstack-requirements14:48
prometheanfiredhellmann: is https://bugs.launchpad.net/openstack-requirements/+bug/1719009 done?15:11
openstackLaunchpad bug 1719009 in OpenStack Global Requirements "per project requirements" [High,New]15:11
dhellmannprometheanfire : I believe it is.15:30
prometheanfiredhellmann: feel freel to assign to self and close15:31
dhellmanndone15:31
dhellmannthanks for the reminder15:32
prometheanfire:D15:32
prometheanfireevery tuesday I have a task to go over bugs :D15:32
dhellmanngood system15:33
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.messaging to new release 6.2.0  https://review.openstack.org/56198215:36
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.rootwrap to new release 5.14.0  https://review.openstack.org/56198315:37
openstackgerritIsaku Yamahata proposed openstack/requirements master: update pecan to 1.3.2  https://review.openstack.org/55589115:38
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.concurrency to new release 3.27.0  https://review.openstack.org/56198415:39
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.service to new release 1.31.1  https://review.openstack.org/56198515:41
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.config to new release 6.1.0  https://review.openstack.org/56198615:41
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.versionedobjects to new release 1.33.1  https://review.openstack.org/56198715:42
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.db to new release 4.36.0  https://review.openstack.org/56198815:42
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.reports to new release 1.28.0  https://review.openstack.org/56198915:42
openstackgerritIsaku Yamahata proposed openstack/requirements master: update pecan to 1.3.2  https://review.openstack.org/55589115:42
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.vmware to new release 2.28.0  https://review.openstack.org/56199015:43
*** kiennt26_ has quit IRC15:43
*** cdent has joined #openstack-requirements16:15
dhellmanncdent : so about the process for updating minimum dependencies...16:15
* cdent nods16:15
dhellmannif the upper constraint is already set to the value you need, you can just make a patch in-tree to change the requirements file(s)16:16
dhellmannif the constraint is not high enough, I think we do need to raise that first16:16
dhellmannso I guess that means a patch in requirements that can be linked to the in-tree patch via a depends-on16:16
dhellmannprometheanfire : does that sound right to you? ^^16:16
prometheanfireI'm not sure what cdent wants16:17
cdentprometheanfire: I'm trying to determine if this https://review.openstack.org/#/c/561687/ is the canonical way to raise a minimum test-requirement16:17
prometheanfireif wanting to bump the min.... if uc is already at or above the desired new min, just bump lower-constrants16:18
cdentas I need to do something similar for gabbi and heat16:18
prometheanfireah16:18
cdentso I've released a new gabbi16:18
cdent(that has the bug fix that heat needs)16:18
cdentso presumably both lower and upper need to go up?16:18
prometheanfireok, so is the new minimum the new release?16:18
dhellmannif we update the constraint first, that should show us that it works with the test suites16:18
dhellmannthen it's safe to update the min in a specific project16:18
prometheanfirewe try and separate the uc and lc bumps into separate reviews16:19
dhellmannand of course if it doesn't work, we can add an exclusion and update the constraint to the new release when that's ready16:19
prometheanfiredhellmann++16:19
dhellmannprometheanfire : well, they're always going to be separate reviews now, since the lower constraints change is in the project tree :-)16:19
prometheanfire?16:20
cdentdhellmann: I"ve got a dangling reference: above you said "if we update the constraint first" but there are two files with that in their naem?16:20
prometheanfiredhellmann: oh, per project, right16:20
prometheanfirecdent: ok, the following is what I think you need to do16:21
prometheanfireupdate uc in the requirements repo16:21
prometheanfirethen make a dependant patch in heat to update lc16:21
dhellmannI think the test jobs will enforce the order we're talking about, because the u-c value won't be compatible with the new range in heat if the u-c hasn't been updated first16:21
prometheanfiredhellmann: yep16:21
dhellmannwe should write that down in the project team guide16:21
* dhellmann goes to check if the new dependencies section was approved16:21
dhellmannpartly: https://review.openstack.org/#/c/561058/116:22
cdentso let me confirm I've got this right:16:23
cdentin requirments: update upper-constraints.txt, in heat update test-requirements.txt and depends-on the upper-constraints change. Later some kind of automation will see that heat now has a lower bound on gabbi and will update lower-constraints?16:24
dhellmannnot quite16:24
dhellmannin the heat patch, you need to update test-requirements.txt and lower-constraints.txt within the heat tree16:24
prometheanfirein heat update test-requirements.txt AND lower-constraints.txt16:24
prometheanfiresame time :D16:25
dhellmannI don't know what we're doing with the lower-constraints.txt file in the requirements repo16:25
*** jpich has quit IRC16:25
prometheanfiredhellmann: right now it's vestigial16:25
dhellmannright16:26
cdentoh16:26
cdentokay, that makes somewhat more sense16:26
dhellmannI need to drop offline to change locations. one of us should write a patch to the project team guide to describe this. who wants to do that?16:27
* cdent steps back16:27
cdentI don't feel like I get it enough to do so16:27
prometheanfiredhellmann: yep, lunching16:27
dhellmannok, I'll work on that after lunch16:27
dhellmannbbiab16:27
cdentbut please add me to the review and I will happily review16:27
prometheanfiredhellmann: commented in the ptg review16:28
prometheanfirecdent: https://review.openstack.org/#/c/56105816:28
cdenthmmm, was going to reference this chat from elsewhere, but looks like the irc logs are failing to update16:29
cdentare ow it has16:30
cdenttyping and logging are hard (also turns out my n key is stuck)16:31
dhellmanncdent , prometheanfire : https://review.openstack.org/56200617:03
dhellmannwe could do more to explain the "why" of those steps but that at least gives the "how"17:03
prometheanfiredhellmann: someone is trying to disolve gentoo as an org :|17:05
prometheanfireguess I'll be busy for a while17:05
dhellmann!17:06
prometheanfireya, kinda annoying17:06
dhellmannhow does that even work? I mean, if they don't want to participate, shouldn't they just quit? Why "dissolve" it?17:10
prometheanfirebecause they wanted to basically be placed in leadership positions17:11
prometheanfireyou know of drobbins?17:12
prometheanfirehe's not the one to contact NM's AG, but he did want to be placed (not elected) to council (our tech leadership)17:12
openstackgerritChris Dent proposed openstack/requirements master: Update gabbi to 1.42.0 for sake of heat  https://review.openstack.org/56202918:51
*** cdent has quit IRC19:04
*** florianf has quit IRC19:10
*** cjloader has quit IRC19:51
*** cjloader_ has joined #openstack-requirements19:51
*** cjloader_ has quit IRC19:52
*** cjloader has joined #openstack-requirements20:02
*** cjloader has quit IRC20:20
*** cjloader has joined #openstack-requirements20:21
dhellmanntonyb , prometheanfire : what's the state of the art for setting up tox with libraries? I'm running into trouble with taskflow and am trying https://review.openstack.org/562040 as a solution for this error: http://logs.openstack.org/41/561341/2/check/build-openstack-sphinx-docs/3247c74/job-output.txt.gz#_2018-04-16_18_31_18_40861620:22
dhellmannI would like to stop using it entirely, but taskflow is in the constraint list so I guess I can't?20:23
dhellmannmaybe I only can't because of the fact that taskflow uses "extras" for test dependencies?20:23
dhellmannprometheanfire : sorry, I missed your earlier reply about gentoo. I don't know anyone involved, I'm just confused about why someone would push to dissolve an open source project like that.20:24
dhellmannit sounds like drobbins isn't playing well with others20:24
prometheanfiredhellmann: I think you are doing that right20:24
dhellmannah, so we still need the script for libs?20:24
prometheanfiredhellmann: I was just venting, it's been stressful being in that role20:24
dhellmannI can imagine20:25
prometheanfireI kinda want to step down after our elections in july, going to be busy with life changes20:26
dhellmannI hope the situation can be worked out20:26
prometheanfireme too, he's done this before too20:26
prometheanfirehttps://archives.gentoo.org/gentoo-nfp/message/8f64b8360969dd09bc376d25638be67e20:26
prometheanfire200820:26
prometheanfirefor some easy reading20:27
prometheanfiredhellmann: about tox_install.sh, I think it's still needed, but tony would probably have the final say20:29
prometheanfiredhellmann: it's possible the siblings stuff fixes it maybe?20:29
dhellmannmaybe? taskflow doesn't use neutron or horizon, so it shouldn't need any of that20:32
dhellmannI think the issue is that taskflow appears in the upper constraints list, so we have to edit that list to run the tests properly against master20:32
dhellmannso unless we stop using extras so we can remove taskflow itself from the list of deps in tox.ini, it seems we need the script20:33
dhellmannI'll see if my work-around fixes the problem20:33
prometheanfireoh, right20:33
prometheanfireya, I think it's still needed for that20:33
*** bnemec has joined #openstack-requirements20:34
*** cjloader has quit IRC20:57
*** cjloader has joined #openstack-requirements21:08
*** cjloader has joined #openstack-requirements21:08
*** edmondsw has quit IRC21:24
openstackgerritDoug Hellmann proposed openstack/requirements master: exclude eventlet 0.21.0  https://review.openstack.org/56100921:32
prometheanfirenew version?21:43
*** cjloader has quit IRC21:53
openstackgerritDoug Hellmann proposed openstack/requirements master: simplify error handling in generate command  https://review.openstack.org/56206522:18
openstackgerritMerged openstack/requirements master: Update gabbi to 1.42.0 for sake of heat  https://review.openstack.org/56202922:25
tonybdhellmann: http://git.openstack.org/cgit/openstack-infra/shade/tree/tox.ini I think is the correct thing these days.23:19
tonybdhellmann: and then you can remove all the additional scripts23:20
dhellmanntonyb : yeah, I came up with https://review.openstack.org/562049 but need to debug the failures23:35
dhellmannprometheanfire : that eventlet exclusion patch never merged because it was based on another patch that was rebased before it merged23:36
dhellmannso it got stuck with enough votes to merge but based on a thing that never would23:36
tonybdhellmann: Looks good.  Seems like the requirements-check failures are simple enough (based on only looking at the logs)23:38
prometheanfireah23:40
dhellmanntonyb : yeah, I just need to spend the time on it tomorrow23:41

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