Tuesday, 2013-11-19

*** oz_akan_ has joined #openstack-marconi00:01
*** oz_akan_ has quit IRC00:06
*** oz_akan_ has joined #openstack-marconi00:16
*** oz_akan_ has quit IRC00:21
*** jergerber has quit IRC00:51
*** flwang has joined #openstack-marconi00:53
*** jcru has quit IRC01:00
*** nosnos has joined #openstack-marconi01:21
*** amitgandhi has joined #openstack-marconi01:22
*** amitgandhi has quit IRC01:29
*** flwang has quit IRC01:40
*** oz_akan_ has joined #openstack-marconi02:03
*** whenry has joined #openstack-marconi02:08
*** flwang has joined #openstack-marconi02:35
*** reed has quit IRC02:39
*** oz_akan_ has quit IRC02:49
*** oz_akan_ has joined #openstack-marconi02:50
*** amitgandhi has joined #openstack-marconi03:26
*** amitgandhi has quit IRC03:31
*** ayoung has quit IRC03:36
*** amitgandhi has joined #openstack-marconi04:26
*** amitgandhi has quit IRC04:31
*** nosnos has quit IRC05:23
*** nosnos has joined #openstack-marconi05:23
*** amitgandhi has joined #openstack-marconi05:27
*** amitgandhi has quit IRC05:31
*** cpallares has quit IRC05:48
*** oz_akan_ has quit IRC06:00
*** amitgandhi has joined #openstack-marconi06:28
*** amitgandhi has quit IRC06:32
*** fifieldt has quit IRC06:50
*** nosnos_ has joined #openstack-marconi07:05
*** nosnos has quit IRC07:05
*** oz_akan_ has joined #openstack-marconi07:13
*** oz_akan_ has quit IRC07:18
*** amitgandhi has joined #openstack-marconi07:28
*** amitgandhi has quit IRC07:32
*** amitgandhi has joined #openstack-marconi08:29
*** amitgandhi has quit IRC08:34
*** flaper87|afk is now known as flaper8708:37
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Bootstrap Messages support  https://review.openstack.org/5238908:43
*** yassine has joined #openstack-marconi08:56
openstackgerritSascha Peilicke proposed a change to openstack/python-marconiclient: Support building wheels (PEP-427)  https://review.openstack.org/5712508:58
openstackgerritSascha Peilicke proposed a change to openstack/marconi: Support building wheels (PEP-427)  https://review.openstack.org/5715009:26
*** flwang has quit IRC09:29
*** amitgandhi has joined #openstack-marconi09:29
*** nosnos_ has quit IRC09:33
*** amitgandhi has quit IRC09:34
*** nosnos has joined #openstack-marconi09:34
*** asalkeld has joined #openstack-marconi09:55
*** amitgandhi has joined #openstack-marconi10:30
*** amitgandhi has quit IRC10:35
*** asalkeld has quit IRC10:59
*** asalkeld has joined #openstack-marconi10:59
openstackgerritA change was merged to openstack/marconi: Start 2014.1 development  https://review.openstack.org/5321911:19
*** amitgandhi has joined #openstack-marconi11:31
*** amitgandhi has quit IRC11:35
*** flwang has joined #openstack-marconi12:20
*** amitgandhi has joined #openstack-marconi12:32
*** tedross has joined #openstack-marconi12:35
*** amitgandhi has quit IRC12:36
*** vkmc has joined #openstack-marconi12:42
*** ykaplan has joined #openstack-marconi12:43
flaper87ykaplan: Glad to see you around! :)12:59
ykaplanflaper87, glad to be here :)12:59
ykaplanflaper87, was just working on the tests13:00
*** whenry has quit IRC13:02
flaper87ykaplan: awesome!!!! Looking forward to review that patch!13:05
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Remove unused packages  https://review.openstack.org/5722313:19
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Remove unused packages  https://review.openstack.org/5722313:20
*** mpanetta has joined #openstack-marconi13:23
mpanettamornin all13:28
*** ykaplan has quit IRC13:29
*** amitgandhi has joined #openstack-marconi13:33
*** nosnos has quit IRC13:35
*** amitgandhi has quit IRC13:38
*** ykaplan has joined #openstack-marconi13:42
*** aniuskad has joined #openstack-marconi13:43
flaper87mpanetta: morning13:48
mpanettaMorning13:48
mpanettaWhat time is it for you flaper87?13:48
flaper87mpanetta: 14:5013:48
flaper87WTF? 14:50 already?13:49
flaper87oh man!13:49
flaper87T_T13:49
mpanettaI take it you have had a busy day? :)13:49
mpanettaTime flies when you are having fun right?13:49
flaper87mpanetta: yeah! :(13:49
flaper87I'm happy to see time flying because that means I still want to do what I'm doing13:50
flaper87:D13:50
flaper87but man, -ETOOMANYTHINGS13:50
*** jcru has joined #openstack-marconi13:51
mpanettahaha gotta hate that error ;)13:51
*** jergerber has joined #openstack-marconi14:16
*** aniuskad has quit IRC14:17
*** alcabrera has joined #openstack-marconi14:21
alcabreraGood morning! :D14:22
*** amitgandhi has joined #openstack-marconi14:24
mpanettaMorning alcabrera!14:25
alcabrerampanetta: hey! Things go well over here. I'm a bit on the sleepy side today. How about you? :)14:29
*** amitgandhi has quit IRC14:29
mpanettaI feel much better then I did yesterday :)14:29
*** amitgandhi has joined #openstack-marconi14:30
*** aniuskad has joined #openstack-marconi14:31
alcabreraawesome, glad to hear it. :D14:34
flaper87alcabrera: MORNING!14:36
alcabreraflaper87: hey!!! :D14:37
flaper87man, next time I decide to get rid of oslo.config in client libraries, please, really, for the sake of friendship, STOP ME!14:38
alcabreralol14:39
alcabrerawas it *that* bad?14:39
alcabreraI mean, trying to remove it?14:40
alcabreraflaper87: I noticed that rbyant requested its removal in the review, and that you ended up not doing so. What happened?14:43
flaper87alcabrera: I'm removing the whole oslo.config dependency, which means that config option will dissapear14:43
flaper87It's almost done14:44
alcabreraah. Hope it goes well. :)14:45
alcabreraMy own attempt at refactoring the sharding patch did not go so well. :P14:46
alcabreraI spent an hour yesterday running with a really cool functools.partial idea, and then realized at the last minute that it wasn't going to work. >.>14:46
alcabreramostly, with the help of 'tox -e pep8', e.g., 'self' is not defined (at class scope).14:46
alcabrerawhich makes sense, but I didn't think far ahead enough to see that. D:14:47
flaper87:(14:49
flaper87alcabrera: btw, re that patch!14:49
flaper87I was thinking that we could maybe go through it together14:49
flaper87today14:49
alcabrerasweet. Sounds good to me.14:49
alcabreraEarlier in the day works better for me, so - the next two hours I'm mostly free. :)14:50
alcabrerajust let me know when you get a spare moment and we'll take out that patch!14:50
*** malini_afk is now known as malini14:51
alcabreraalso, quick side note - "wheel support" is *awesome*! I'm glad to see a patch for that! Also shows me how to do it the pbr-way. :)14:51
*** haomaiwang has quit IRC15:06
*** haomaiwang has joined #openstack-marconi15:07
alcabreraflaper87: I put together a list of pending sharding patches to help out with our review process: https://etherpad.openstack.org/p/sharding-patches15:07
*** ayoung has joined #openstack-marconi15:09
flaper87alcabrera: Yeah, the wheel support if very cool!15:11
flaper87alcabrera: awesome, give me 5 more mins and we'll go through it15:11
alcabreraflaper87: kk15:11
*** haomaiwang has quit IRC15:12
flaper87I hate computers http://paste.openstack.org/show/53602/15:23
* alcabrera clicks15:23
flaper87I hate computers http://paste.openstack.org/show/53602/15:24
alcabrera*.pyc15:24
alcabrerathey've gotten me more times than I can count15:24
flaper87already deleted them15:25
flaper87:(15:25
alcabrera:/15:25
flaper87I have literally no clue15:26
mpanettaEww15:26
mpanettaThat is a nasty one15:26
alcabrerahmmm...15:27
alcabreraThose make me think of import-time side effects.15:27
alcabreraI tend to look at __init__.pys when things get that weird/15:27
flaper87somehow, python thinks they are the same module15:27
alcabreraflaper87: hmmm... this looks suspicious: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/__init__.py15:30
alcabreraspecifically: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/__init__.py#L2115:30
flaper87why ?15:30
flaper87you know what's funny?15:30
alcabrerathat exposes the name 'errors' from marconiclient.errors at the same level as marconiclcient.transport.errors15:30
flaper87that line there, loads the marconiclient.transport.errors15:31
flaper87instead of marconiclient.errors15:31
flaper87:D15:31
flaper87but, if I do that from the shell, it'll load the marconiclient.errors module15:31
alcabrerawait... what? @_@15:31
alcabreraweeeeird15:31
flaper87hahahahaha, time to show our WTFFace15:32
alcabrerayuuuppp15:32
alcabreraidea: when doing an intra-project import in an __init__, to ++safety, how about 'from myproj import mymodule as _mymodule'?15:33
alcabreraflaper87: verified that L21 is causing the issue.15:35
alcabreraI CLI'd from marconiclient root in my venv, repro'd the steps you shared in http://paste.openstack.org/show/53602/, modified L21 to ('from ... as _errors'), and then the error in http://paste.openstack.org/show/53602/ disappears.15:36
alcabreraIt's an import-time side-effect causing the silliness.15:36
flaper87alcabrera: [GCC 4.8.1 20130603 (Red Hat 4.8.1-1)] <- good boy15:41
flaper87:D15:41
alcabreralol15:42
flaper87alcabrera: that worked! Thanks!15:42
alcabreranp. :)15:42
flaper87I remember fixing this but ...15:42
flaper87AHHHHHHHHHHHHHHHHHHHHHHHHHHH15:42
flaper87the s/exceptions/errors/ patch15:42
flaper87I blaim that patch15:42
alcabrera:P15:42
alcabreraflaper87: I'm going to submit a patch for this error. (_errors). :D15:43
flaper87alcabrera: I already fixed it in the patch I'm about to submit but feel free to do that15:43
flaper87I'll base it on yours15:43
openstackgerritAlejandro Cabrera proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors  https://review.openstack.org/5726215:44
alcabreraThe first client-side patch I've submitted in ages.15:45
alcabreraflaper87: hehe, we'll double fix it and make sure it never comes back. ;)15:45
flaper87alcabrera: +1 thanks for the patch15:46
alcabreraglad to help! :D15:46
flaper87I won't even wait for jenkins15:46
flaper87stupid error15:46
flaper87T_T15:46
flaper87I think I'll ninja-approve that patch15:46
alcabreratox'd locally - passed pep8, pypy, py27, and py3315:46
alcabreraworks for me15:47
*** russell_h_ has joined #openstack-marconi15:51
openstackgerritA change was merged to openstack/python-marconiclient: Support building wheels (PEP-427)  https://review.openstack.org/5712515:52
alcabreraw00t15:52
alcabrerajenkins has been a little slow today. :P15:52
*** russell_h has quit IRC15:52
*** russell_h_ is now known as russell_h15:52
openstackgerritA change was merged to openstack/marconi: Support building wheels (PEP-427)  https://review.openstack.org/5715015:53
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Remove the dependency on oslo.config  https://review.openstack.org/5726515:53
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors  https://review.openstack.org/5726215:53
flaper87alcabrera: man, I did something stupid15:53
flaper87T_T15:53
flaper87alcabrera: can you resubmit your patch ?15:54
flaper87I cherrypicked instead of rebasing15:54
flaper87T_T15:54
flaper87sorry about that15:54
alcabrera:(15:55
* alcabrera resubmits15:55
openstackgerritAlejandro Cabrera proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors  https://review.openstack.org/5726815:55
alcabreraflaper87: that should do it. :)15:56
flaper87oh you submitted a new one, you could've re-sent the same patch. I'll approve this15:56
flaper87alcabrera: can you abandon this: https://review.openstack.org/#/c/57262/215:57
alcabreragit review grumped: "failed to push some refs to"15:57
alcabrerayup15:57
alcabreraflaper87: done. :D15:57
alcabrera"Abandoned for great redundancy"15:57
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Remove the dependency on oslo.config  https://review.openstack.org/5726515:58
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Bootstrap Messages support  https://review.openstack.org/5238915:58
flaper87cool, now we're talking15:58
flaper87alcabrera: I'm ready15:58
alcabreraflaper87: g+-style?16:00
flaper87alcabrera: yup16:00
alcabreraI'll be ready in 2m.16:01
alcabreraVPN will now disconnect me. >.>16:01
alcabrerabrb16:01
flaper87kk16:01
*** acabrera has joined #openstack-marconi16:02
*** reed has joined #openstack-marconi16:04
*** alcabrera has quit IRC16:05
*** haomaiwang has joined #openstack-marconi16:18
*** haomaiwang has quit IRC16:23
*** ykaplan has quit IRC16:23
*** ykaplan has joined #openstack-marconi16:24
*** cpallares has joined #openstack-marconi16:26
*** ykaplan has quit IRC16:29
*** aniuskad has quit IRC16:32
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: connect sharding manager to control drivers  https://review.openstack.org/5460516:33
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: connect sharding manager to control drivers  https://review.openstack.org/5460516:35
*** russell_h has quit IRC16:36
*** russell_h has joined #openstack-marconi16:36
*** oz_akan_ has joined #openstack-marconi16:39
openstackgerritA change was merged to openstack/python-marconiclient: fix: import-time side effect on errors  https://review.openstack.org/5726816:51
*** oz_akan_ has quit IRC16:57
*** oz_akan_ has joined #openstack-marconi16:57
*** jcru has quit IRC16:58
*** jcru has joined #openstack-marconi16:59
*** flwang has quit IRC17:13
*** pycabrera has joined #openstack-marconi17:13
*** pycabrera is now known as alcabrera17:13
alcabreraflaper87: thaaaanks! :D17:13
flaper87alcabrera: thank you :D That was a really good pair-reviewing session and we got the sharding patch sorted out! w0000t17:14
alcabreraagreed. :)17:15
alcabreramore next time - 'til then, I'll be fixing up the issues.17:15
*** acabrera has quit IRC17:16
flaper87alcabrera: if you've 5mins between those rebases, pls, I'd love your feedback here: https://review.openstack.org/#/c/57265/17:21
alcabreraflaper87: I'll check it out now. :)17:21
ekarlsoqq/win 3517:21
flaper87passwd ?17:22
flaper87alcabrera: thanks :D17:22
flaper87brb17:26
*** flaper87 is now known as flaper87|afk17:27
alcabreraflaper87|afk: +2'd w/ suggestions17:27
*** alcabrera is now known as alcabrera|afk17:27
*** amitgandhi has quit IRC17:29
*** reed has quit IRC17:30
*** kgriffs_afk is now known as kgriffs17:47
openstackgerritZhihao Yuan proposed a change to openstack/marconi: feat(shard): queue listing from multiple sources  https://review.openstack.org/5312717:49
*** amitgandhi has joined #openstack-marconi17:50
openstackgerritZhihao Yuan proposed a change to openstack/marconi: feat(health): make health API storage-aware  https://review.openstack.org/5345718:02
*** flaper87|afk is now known as flaper8718:18
*** reed has joined #openstack-marconi18:31
*** kgriffs is now known as kgriffs_afk18:50
*** kgriffs_afk is now known as kgriffs18:55
*** alcabrera|afk is now known as alcabrera19:01
flaper87alcabrera: can I get your bless here? https://review.openstack.org/#/c/52389/19:27
* alcabrera clicks19:27
flaper87kgriffs: could you please take a look at these patches? https://review.openstack.org/#/q/status:open+project:openstack/python-marconiclient,n,z :D19:27
alcabreraflaper87: running a quick sanity check. :)19:29
alcabreraflaper87: +219:30
*** oz_akan_ has quit IRC19:34
*** oz_akan_ has joined #openstack-marconi19:34
*** oz_akan_ has quit IRC19:38
*** jcru_ has joined #openstack-marconi19:41
*** jcru has quit IRC19:45
*** jcru_ has quit IRC19:49
*** alcabrera is now known as alcabrera|afk19:55
*** yassine has quit IRC19:59
*** alcabrera|afk is now known as alcabrera20:14
*** kgriffs is now known as kgriffs_afk20:16
*** malini is now known as malini_afk20:29
*** jcru has joined #openstack-marconi20:29
*** ametts has quit IRC20:42
*** oz_akan_ has joined #openstack-marconi20:50
*** oz_akan_ has quit IRC20:51
*** oz_akan_ has joined #openstack-marconi20:52
*** kgriffs_afk is now known as kgriffs20:56
*** kgriffs is now known as kgriffs_afk21:06
alcabreraGood night, everyone. :)21:17
*** alcabrera has quit IRC21:17
*** kgriffs_afk is now known as kgriffs21:17
*** amitgandhi has quit IRC21:40
*** amitgandhi has joined #openstack-marconi22:01
*** tedross has quit IRC22:13
*** amit has joined #openstack-marconi22:18
*** amit is now known as Guest6710822:19
*** Guest67108 has quit IRC22:20
*** amitgandhi has quit IRC22:21
*** amitgandhi has joined #openstack-marconi22:22
*** mpanetta has quit IRC22:24
*** ayoung has quit IRC22:28
*** amitgandhi has quit IRC22:59
*** jcru has quit IRC22:59
*** flaper87 is now known as flaper87|afk23:00
*** jergerber has quit IRC23:14
*** kgriffs is now known as kgriffs_afk23:26

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