Tuesday, 2016-08-16

openstackgerritTang Chen proposed openstack/requirements: Update osprofiler requirement  https://review.openstack.org/35568801:39
*** abashmak has joined #openstack-requirements01:51
abashmakhello, anybody in here?01:52
*** abashmak has left #openstack-requirements01:57
prometheanfireand they are gone01:58
openstackgerritMatthew Thode proposed openstack/requirements: Updated from generate-constraints  https://review.openstack.org/35334903:26
stevemarprometheanfire: still around?03:38
prometheanfirestevemar: yarp03:38
prometheanfirethey aren't back03:39
stevemarprometheanfire: is approving a u-c bump a single core thing? https://review.openstack.org/#/c/355616/03:39
patchbotstevemar: patch 355616 - requirements - update constraint for osc-lib to new release 1.0.103:39
stevemarprometheanfire: regardless, would be great to get eyes on that03:39
prometheanfirestevemar: yes03:39
stevemarprometheanfire: yeah, abashmak left as quick as he came :)03:39
prometheanfirethough I think we may change that to two03:39
prometheanfirewe talked about it last meeting03:40
stevemarprometheanfire: "wheeeee"03:40
stevemarhehe03:40
stevemari actually lol'ed03:41
stevemarprometheanfire: thank you, it'll help unstick the OSC gate03:41
prometheanfire:D03:41
prometheanfireI need to come up with some ascii art to leave in every comment I approve03:41
prometheanfirethough probably won't show up right in the web client03:42
stevemarprometheanfire: start leaving shrugs03:43
prometheanfirestevemar: how is it holding you back btw?03:43
prometheanfiresorry, diferent review03:43
prometheanfirenot osc-lib03:43
* stevemar approved, i guess ¯\_(ツ)_/¯03:43
prometheanfireosprofiler wants to update global-requirements (min version03:43
stevemarin osc we need osc-lib 1.0.1 to be used in our gate job, need a last minute fix03:44
stevemar(i guess you already guessed that since you said 'sorry, different review' :) )03:45
prometheanfireya, np on that one03:45
prometheanfireosprofiler 1.4.0 has released, and a new initializer class has been added. It is useful to users when initializint the notifier in other projects. So update the global requirement.03:46
prometheanfirethat's not a reason to update gr03:46
prometheanfireuc, sure and if a project requires it, sure03:46
stevemarprometheanfire: yeah, should only bump gr when ready, that change (which they are pushing in keystone) is a pretty big change03:52
prometheanfirefair enough03:53
prometheanfireya03:53
prometheanfirefigured out why the oslo.db update failed03:53
prometheanfireupdate master with a mitaka package will do that03:53
stevemarhehe03:54
prometheanfirestevemar: you know if you can change the target branch in a review?04:03
prometheanfireI mean submitting to the same change-id04:03
stevemarprometheanfire: i normally click the cherry-pick button in gerrit04:04
prometheanfireya, this is to overwrite that bad one04:04
stevemarbut that'll copy it, not really change the target04:04
prometheanfirewe'll see04:04
stevemarcherry-pick and abandon is a bad idea :)04:05
prometheanfirenope04:05
prometheanfirehuh, didn't update...04:06
prometheanfiremaybe found a bug?04:06
* stevemar shrugs at prometheanfire04:11
stevemarbed time for me, kid just woke up, time to put him back to sleep04:11
prometheanfireya, notabug04:12
prometheanfirehttps://review.openstack.org/#/q/I67d7a5000bfe0c98717d3e29d23edc9c6117e765,n,z shows both reviews04:12
prometheanfirestevemar: nn04:12
openstackgerritThomas Bechtold proposed openstack/requirements: Add dogtag-pki used by Barbican  https://review.openstack.org/35231005:01
*** jpena|off has quit IRC06:02
*** jpena|off has joined #openstack-requirements06:04
*** jpena|off has quit IRC06:10
*** jpena|off has joined #openstack-requirements06:11
openstackgerritMerged openstack/requirements: update constraint for osc-lib to new release 1.0.1  https://review.openstack.org/35561606:55
openstackgerritMerged openstack/requirements: update constraint for cliff to new release 2.2.0  https://review.openstack.org/35561006:59
openstackgerritOpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints  https://review.openstack.org/35334907:04
*** jpena|off is now known as jpena07:07
*** jpena has quit IRC09:46
*** amoralej has quit IRC09:46
*** jpena has joined #openstack-requirements09:47
*** sdague has joined #openstack-requirements10:23
*** jpena is now known as jpena|lunch11:57
*** sigmavirus|away is now known as sigmavirus12:05
*** jpena|lunch is now known as jpena12:56
*** jpena is now known as jpena|brb13:33
*** jpena|brb is now known as jpena13:58
*** abashmak has joined #openstack-requirements15:11
abashmakGood morning15:11
prometheanfirecan someone ack https://review.openstack.org/35571115:11
prometheanfireabashmak: ?15:11
abashmakI have a question. I'm trying to add a global requirement, instructions say to run "generate-constraints"15:12
abashmakI can't find the tool anywhere15:12
prometheanfirehttps://github.com/openstack/requirements/blob/master/openstack_requirements/cmds/generate.py I think15:13
prometheanfirebut if you are trying to add an entry to global-requirements the just add it, the gate should do the rest15:13
abashmakso the gate will modify uppper-constraintes.txt?15:14
abashmakOr should I do it?15:14
prometheanfireyou should do that too15:15
prometheanfirefor the initial commit15:15
abashmakI see, thank you prometheanfire!15:16
prometheanfire:D15:16
prometheanfireabashmak: also, stay in the channel next time :P15:16
abashmakwill do :)15:17
openstackgerritAlexander Bashmakov proposed openstack/requirements: Adding pybrctl to global-requirements.  https://review.openstack.org/35600615:19
prometheanfireabashmak: oh y15:22
abashmakyes?15:22
prometheanfireabashmak: oh ya, add the answers from https://github.com/openstack/requirements/blob/master/README.rst#for-new-requirements to your commit message15:22
abashmakprometheanfire: how can I tell if the library is Python3 compatible?15:25
prometheanfireabashmak: do they say they are on pypi?15:25
prometheanfireotherwise, testing15:25
prometheanfireor if the code is simple, looking15:26
abashmakthey are on https://pypi.python.org/pypi/pybrctl/0.1.115:26
sigmavirusoh god pybrctl's source looks gross and they have no tests15:27
sigmavirusthey only claim to support py2.715:27
prometheanfiresigmavirus: ya, just noticed15:27
prometheanfirethe code is short, so it might15:28
prometheanfireor might be converted easilly15:28
sigmavirusyeah I'm doing a visual scan15:28
prometheanfirebut at the moment it's a no based on no tests and no py315:28
abashmakIf not, I'll go back to making shell calls to 'brctl', no biggie15:29
prometheanfireabashmak: sorry, just not looking good15:30
abashmakthat's fine, thanks for the prompt feedback. Should I abandon the patch?15:31
prometheanfireabashmak: also, iirc you can create bridges in ip15:31
sigmavirusabashmak: also the library has a lot of methods that raise a NotImplementedError and so it looks like it's barely finished or maintained15:31
prometheanfireabashmak: unless you want to work with upstream to fix all the things :P15:31
prometheanfireabashmak: ip link add name bridge_name type bridge15:32
prometheanfireyep :D15:32
prometheanfirehttps://wiki.archlinux.org/index.php/Network_bridge#With_iproute215:32
abashmakprometheanfire: thank you for the link, I'll look into using that instead of brctl15:33
prometheanfireabashmak: thanks :D15:33
* prometheanfire remembers converting to ip from brctl a year or so ago15:34
*** david-lyle_ has joined #openstack-requirements17:09
*** david-lyle has quit IRC17:13
*** david-lyle_ is now known as david-lyle17:13
*** sdague has quit IRC21:04

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