| *** clsacramento has quit IRC | 00:12 | |
| *** sigmavirus has quit IRC | 00:20 | |
| *** dims_ has quit IRC | 00:20 | |
| *** _sigmavirus24 has joined #openstack-requirements | 00:21 | |
| *** dims has joined #openstack-requirements | 00:31 | |
| *** armax has quit IRC | 03:09 | |
| *** armax has joined #openstack-requirements | 03:19 | |
| *** armax has quit IRC | 03:58 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/388555 | 07:19 |
|---|---|---|
| *** jpich has joined #openstack-requirements | 09:18 | |
| *** dootniz is now known as kragniz | 09:59 | |
| *** rtheis has joined #openstack-requirements | 10:00 | |
| stevemar | o/ | 11:58 |
| stevemar | hello requirements folks! | 12:00 |
| *** breton has joined #openstack-requirements | 12:00 | |
| stevemar | we have an issue in keystonemiddleware, it seems kombu 4.0.0 was released recently and broke a bunch of our stuff: http://logs.openstack.org/98/392098/1/check/gate-keystonemiddleware-python27-ubuntu-trusty/cb33283/testr_results.html.gz | 12:00 |
| stevemar | i'm sure oslo.messaging is also affected | 12:01 |
| stevemar | but when looking at upper-constraints, it should have never landed 4.0.0, it's capped at 3.0.37! https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L165 | 12:01 |
| stevemar | so two questions: 1) why isn't the u-c cap working? and 2) i plan to submit a change to g-r to stop the install of 4.0.0, should i use <4.0.0 or !=4.0.0 (presumably the changes that broke us won't be fixed in 4.1.0) | 12:02 |
| jroll | stevemar: looks like the pip install there isn't using u-c :/ | 12:04 |
| jroll | that would go here: https://github.com/openstack/keystonemiddleware/blob/master/tox.ini#L8 | 12:04 |
| stevemar | jroll: hmm, i thought all the jobs now were using u-c so we didn't have to define it in tox.ini? | 12:05 |
| stevemar | jroll: hmm.. something like this change: https://github.com/openstack/keystone/commit/85ae2454c9eea8fa134df74527cbd1f2e910fe05 | 12:07 |
| stevemar | jroll: guess i still need that specific line: install_command = pip install -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages} | 12:07 |
| jroll | stevemar: that commit is about certain jobs that didn't *support* u-c at all, you still need the install_command line | 12:07 |
| jroll | ya | 12:07 |
| jroll | stevemar: that removed the venv-specific overrides for things that didn't support u-c | 12:08 |
| stevemar | jroll: yeah, i suppose i could drop that in. still leaves a what-are-we-going-to-do-about-kombu-4.0.0 question | 12:08 |
| jroll | stevemar: yeah, I looked at it briefly yesterday, looks like it's only in their dev req's | 12:09 |
| jroll | hrm, https://github.com/celery/kombu/issues/641 | 12:10 |
| stevemar | jroll: was there a suggestion about only capping requirements for services, not libraries? | 12:10 |
| jroll | stevemar: not that I've seen | 12:11 |
| stevemar | jroll: taking a quick look and it seems mostly services (nova/glance) have the install command that uses u-c | 12:11 |
| stevemar | eh | 12:11 |
| jroll | interesting, we use it in ironic libraries | 12:11 |
| *** _sigmavirus24 is now known as sigmavirus | 12:12 | |
| *** sigmavirus has joined #openstack-requirements | 12:12 | |
| stevemar | http://codesearch.openstack.org/?q=install_command%20%3D%20pip%20install%20-c%7Benv%3AUPPER_CONSTRAINTS_FILE%3Ahttps%3A%2F%2Fgit.openstack.org%2Fcgit%2Fopenstack%2Frequirements%2Fplain%2Fupper-constraints.txt%7D%20%7Bopts%7D%20%7Bpackages%7D&i=nope&files=&repos= | 12:12 |
| jroll | it looks like kombu folks don't support kombu 4.0 installed without amqp 2.0 | 12:12 |
| jroll | stevemar: oh, it's a bit different for libs, now I remember | 12:14 |
| jroll | http://codesearch.openstack.org/?q=%7Btoxinidir%7D%2Ftools%2Ftox_install.sh%20%7Benv%3AUPPER_CONSTRAINTS_FILE&i=nope&files=&repos= | 12:14 |
| stevemar | jroll: yeah, i vaguely remember that too | 12:14 |
| jroll | so the problem is that libs are in u-c themselves, so they need a script to change u-c, such that they can install from source | 12:14 |
| jroll | stevemar: e.g. https://github.com/openstack/python-ironicclient/blob/master/tools/tox_install.sh | 12:15 |
| stevemar | oh the weird "tox_install" thing that everyone copied around | 12:15 |
| stevemar | jroll: looks like oslo.messaging is aware of it: https://review.openstack.org/#/q/project:openstack/oslo.messaging | 12:15 |
| jroll | ah, yep | 12:16 |
| stevemar | i'll wait and see how the requirements team wants the oslo team to handle it (and then do the same) | 12:17 |
| stevemar | we don't get nearly the volume of changes in keystonemiddleware | 12:17 |
| jroll | ++ | 12:17 |
| jroll | I suspect we might end up pinning it, just so we don't have churn for u-c changes every time they release kombu | 12:17 |
| stevemar | yeah | 12:18 |
| stevemar | we have ~10 open changes in keystonemiddleware, we can be broken for a bit :P | 12:18 |
| jroll | heh | 12:18 |
| openstackgerrit | Gevorg Davoian proposed openstack/requirements: Set upper constraint on kombu to be < 4.0.0 https://review.openstack.org/392135 | 12:38 |
| stevemar | jroll: ^ :) | 12:42 |
| jroll | ha, there it is | 12:43 |
| openstackgerrit | Gevorg Davoian proposed openstack/requirements: Set upper constraint on kombu to be < 4.0.0 https://review.openstack.org/392135 | 12:49 |
| openstackgerrit | Merged openstack/requirements: Bump vnware-nsxlib to 0.2.0 https://review.openstack.org/391114 | 13:24 |
| *** armax has joined #openstack-requirements | 15:23 | |
| *** armax has quit IRC | 15:28 | |
| *** tdurakov has quit IRC | 16:09 | |
| *** tdurakov has joined #openstack-requirements | 16:11 | |
| *** armax has joined #openstack-requirements | 16:32 | |
| *** armax has quit IRC | 16:37 | |
| *** armax has joined #openstack-requirements | 16:43 | |
| *** jpich has quit IRC | 17:01 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for os-brick to new release 1.7.0 https://review.openstack.org/392244 | 17:18 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/requirements: Uncap amqp 2.0+ in global requirements https://review.openstack.org/392245 | 17:18 |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for keystoneauth1 to new release 2.15.0 https://review.openstack.org/392246 | 17:18 |
| sigmavirus | prometheanfire: should https://review.openstack.org/392135 update the u-c file too? | 17:19 |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for django_openstack_auth to new release 3.0.0 https://review.openstack.org/392247 | 17:19 |
| prometheanfire | yes | 17:20 |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for os_vif to new release 1.3.0 https://review.openstack.org/392248 | 17:20 |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for python-ceilometerclient to new release 2.7.0 https://review.openstack.org/392249 | 17:22 |
| prometheanfire | sigmavirus: though kombu is already at kombu===3.0.37 | 17:22 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/requirements: Uncap amqp 2.0+ in global requirements https://review.openstack.org/392245 | 17:30 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/requirements: Uncap amqp 2.0+ in global requirements https://review.openstack.org/392245 | 17:31 |
| *** armax has quit IRC | 17:49 | |
| *** armax has joined #openstack-requirements | 18:47 | |
| *** armax has quit IRC | 18:47 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for XStatic-Angular to new release 1.5.8.0 https://review.openstack.org/392275 | 18:57 |
| openstackgerrit | OpenStack Release Bot proposed openstack/requirements: update constraint for XStatic-D3 to new release 3.5.17.0 https://review.openstack.org/392276 | 18:57 |
| sigmavirus | prometheanfire: then how is this a problem? | 19:03 |
| sigmavirus | meh whatever | 19:03 |
| prometheanfire | sigmavirus: exactly | 19:03 |
| *** strigazi is now known as strigazi_AFK | 19:09 | |
| sigmavirus | prometheanfire: sounds like you should figure out why constraints are broken ;) | 19:12 |
| prometheanfire | uc updates? | 19:12 |
| sigmavirus | no there are a fair number of jobs that are not obeying constraints it seems | 19:17 |
| prometheanfire | sigmavirus: your kombu update is in https://review.openstack.org/388555 | 19:18 |
| prometheanfire | ya, that's harder to police :| | 19:18 |
| prometheanfire | bring it up tomorrow morning | 19:18 |
| *** armax has joined #openstack-requirements | 19:22 | |
| *** clsacramento has joined #openstack-requirements | 19:34 | |
| *** rtheis has quit IRC | 20:29 | |
| *** r1chardj0n3s_afk is now known as r1chardj0n3s | 20:43 | |
| tonyb | toabctl: Like dhellmann says we need PTL/CPL sign off | 23:20 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!