Friday, 2018-11-09

*** jrist has quit IRC02:30
*** hongbin has joined #openstack-requirements02:42
*** jrist has joined #openstack-requirements02:43
openstackgerritMerged openstack/requirements master: update constraint for oslo.service to new release 1.33.0  https://review.openstack.org/61637103:03
prometheanfireefried: progress :D03:05
openstackgerrit98k proposed openstack/requirements master: Add python 3.6 unit test job  https://review.openstack.org/61676104:19
*** openstackstatus has quit IRC04:59
*** hongbin has quit IRC05:07
*** openstack has joined #openstack-requirements07:09
*** ChanServ sets mode: +o openstack07:09
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-openstackclient to new release 3.17.0  https://review.openstack.org/61683707:18
*** e0ne has quit IRC07:31
*** jpich has joined #openstack-requirements08:58
*** e0ne has joined #openstack-requirements09:39
*** e0ne has quit IRC10:56
*** e0ne has joined #openstack-requirements11:44
*** dtantsur is now known as dtantsur|brb11:59
*** vpickard_ is now known as vpickard12:00
*** dtantsur|brb is now known as dtantsur13:09
efriedprometheanfire: Cool, looks like the nova patch has merged now too. Thanks again for the help.13:21
openstackgerritDaisuke Fujita proposed openstack/requirements master: Add tornado to global requirements  https://review.openstack.org/61692213:34
*** jpich has quit IRC13:45
*** jpich has joined #openstack-requirements13:53
prometheanfireefried: thanks for keeping on it :D14:31
openstackgerritMatthew Thode proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/61678014:34
dirkprometheanfire: can I revert https://review.openstack.org/#/c/541752/ ?14:36
dirkprometheanfire: currently renderspec doesn't allow merging patches anymore, e.g. https://review.openstack.org/#/c/580918/14:37
dirkand I'm not sure that removing the requirements-check would be the good thing to do14:37
prometheanfiredirk: looking, I'd rather not because it's not a co-installed package from our previous conversations14:42
prometheanfiredirk: also, looks like the last patch passed requirements-check?14:42
dirkprometheanfire: ah, why?14:54
dirkHmm, maybe I already disabled it14:54
prometheanfireit showed as passing, not that it didn't run (or ran non-voting)14:55
dirkAh, because they removed it15:12
dirkYou're looking at the one which fails unit test, right?15:13
dirkThat's not a solution15:13
dirkSo how can we have testing for an official project without its requirements in g-r?15:13
prometheanfirethe latest patch15:13
prometheanfirerequirements is for things that need co-installability tests15:14
prometheanfirethat and constraints (which don't require the test)15:14
dirkWe do want constraints15:14
prometheanfireand that doesn't require the test :D15:19
*** bnemec is now known as beekneemech15:53
*** e0ne has quit IRC16:35
mordredprometheanfire: requirements-check job doens't seem to honor depends-on ...16:35
mordredprometheanfire: nevermind! I actually just suck16:36
mordredprometheanfire: re: https://review.openstack.org/#/c/616612/ and twisted - I wasn't really planning on supporting twisted things or using the twisted extra ... do we need to include it in the constraints for completeness I suppose?16:39
prometheanfirelol16:44
prometheanfiremordred: openstack_requirements (pip install it) provides generate-constraints16:45
prometheanfirewhat's generally recommened for adding libs is to add it to global-reqs then run generate-constraints16:45
prometheanfireline 412 here is the command the bot runs for us nightly16:46
prometheanfirehttps://review.openstack.org/#/c/615388/2/doc/source/dependency-management.rst16:46
prometheanfirereviews there would be nice :P16:46
mordredprometheanfire: oh - that's really helpful docs16:52
prometheanfireya, getting reviews this week hasn't been the most fun though :P16:52
mordredtell me about it16:54
mordredprometheanfire: would you be open to a patch that accepts non-absolute paths to python commands?16:55
mordredI ask, because my version of the command looks like this:16:55
mordredtox -egenerate -- -p /home/mordred/src/github.com/pyenv/pyenv/shims/python -b blacklist.txt -p /home/mordred/src/github.com/pyenv/pyenv/shims/python2.7 -p /home/mordred/src/github.com/pyenv/pyenv/shims/python3.6 -r global-requirements.txt --version-map 3.6:3.4 --version-map 3.6:3.5 > upper-constraints.txt16:55
mordredoh - also, I don't seem to have +2 on project-team-guide16:56
prometheanfiremordred: ya, that works (just specifying python2.7 instead of full path)16:57
*** dtantsur is now known as dtantsur|afk17:00
prometheanfiremordred: and feel free to post an update to that patch if you feel like, I'm traveling today17:02
*** jpich has quit IRC17:10
mordredprometheanfire: sweet17:34
*** e0ne has joined #openstack-requirements18:09
*** zul has quit IRC18:12
*** e0ne has quit IRC18:33
*** e0ne has joined #openstack-requirements18:36
*** e0ne has quit IRC18:58
*** zigo has quit IRC19:25
*** zigo has joined #openstack-requirements19:27
*** e0ne has joined #openstack-requirements20:01
*** e0ne has quit IRC20:04
*** efried is now known as fried_rice20:09
*** vpickard is now known as vpickard_21:52

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