*** oz_akan_ has joined #openstack-marconi | 00:01 | |
*** oz_akan_ has quit IRC | 00:06 | |
*** oz_akan_ has joined #openstack-marconi | 00:16 | |
*** oz_akan_ has quit IRC | 00:21 | |
*** jergerber has quit IRC | 00:51 | |
*** flwang has joined #openstack-marconi | 00:53 | |
*** jcru has quit IRC | 01:00 | |
*** nosnos has joined #openstack-marconi | 01:21 | |
*** amitgandhi has joined #openstack-marconi | 01:22 | |
*** amitgandhi has quit IRC | 01:29 | |
*** flwang has quit IRC | 01:40 | |
*** oz_akan_ has joined #openstack-marconi | 02:03 | |
*** whenry has joined #openstack-marconi | 02:08 | |
*** flwang has joined #openstack-marconi | 02:35 | |
*** reed has quit IRC | 02:39 | |
*** oz_akan_ has quit IRC | 02:49 | |
*** oz_akan_ has joined #openstack-marconi | 02:50 | |
*** amitgandhi has joined #openstack-marconi | 03:26 | |
*** amitgandhi has quit IRC | 03:31 | |
*** ayoung has quit IRC | 03:36 | |
*** amitgandhi has joined #openstack-marconi | 04:26 | |
*** amitgandhi has quit IRC | 04:31 | |
*** nosnos has quit IRC | 05:23 | |
*** nosnos has joined #openstack-marconi | 05:23 | |
*** amitgandhi has joined #openstack-marconi | 05:27 | |
*** amitgandhi has quit IRC | 05:31 | |
*** cpallares has quit IRC | 05:48 | |
*** oz_akan_ has quit IRC | 06:00 | |
*** amitgandhi has joined #openstack-marconi | 06:28 | |
*** amitgandhi has quit IRC | 06:32 | |
*** fifieldt has quit IRC | 06:50 | |
*** nosnos_ has joined #openstack-marconi | 07:05 | |
*** nosnos has quit IRC | 07:05 | |
*** oz_akan_ has joined #openstack-marconi | 07:13 | |
*** oz_akan_ has quit IRC | 07:18 | |
*** amitgandhi has joined #openstack-marconi | 07:28 | |
*** amitgandhi has quit IRC | 07:32 | |
*** amitgandhi has joined #openstack-marconi | 08:29 | |
*** amitgandhi has quit IRC | 08:34 | |
*** flaper87|afk is now known as flaper87 | 08:37 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Bootstrap Messages support https://review.openstack.org/52389 | 08:43 |
---|---|---|
*** yassine has joined #openstack-marconi | 08:56 | |
openstackgerrit | Sascha Peilicke proposed a change to openstack/python-marconiclient: Support building wheels (PEP-427) https://review.openstack.org/57125 | 08:58 |
openstackgerrit | Sascha Peilicke proposed a change to openstack/marconi: Support building wheels (PEP-427) https://review.openstack.org/57150 | 09:26 |
*** flwang has quit IRC | 09:29 | |
*** amitgandhi has joined #openstack-marconi | 09:29 | |
*** nosnos_ has quit IRC | 09:33 | |
*** amitgandhi has quit IRC | 09:34 | |
*** nosnos has joined #openstack-marconi | 09:34 | |
*** asalkeld has joined #openstack-marconi | 09:55 | |
*** amitgandhi has joined #openstack-marconi | 10:30 | |
*** amitgandhi has quit IRC | 10:35 | |
*** asalkeld has quit IRC | 10:59 | |
*** asalkeld has joined #openstack-marconi | 10:59 | |
openstackgerrit | A change was merged to openstack/marconi: Start 2014.1 development https://review.openstack.org/53219 | 11:19 |
*** amitgandhi has joined #openstack-marconi | 11:31 | |
*** amitgandhi has quit IRC | 11:35 | |
*** flwang has joined #openstack-marconi | 12:20 | |
*** amitgandhi has joined #openstack-marconi | 12:32 | |
*** tedross has joined #openstack-marconi | 12:35 | |
*** amitgandhi has quit IRC | 12:36 | |
*** vkmc has joined #openstack-marconi | 12:42 | |
*** ykaplan has joined #openstack-marconi | 12:43 | |
flaper87 | ykaplan: Glad to see you around! :) | 12:59 |
ykaplan | flaper87, glad to be here :) | 12:59 |
ykaplan | flaper87, was just working on the tests | 13:00 |
*** whenry has quit IRC | 13:02 | |
flaper87 | ykaplan: awesome!!!! Looking forward to review that patch! | 13:05 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Remove unused packages https://review.openstack.org/57223 | 13:19 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Remove unused packages https://review.openstack.org/57223 | 13:20 |
*** mpanetta has joined #openstack-marconi | 13:23 | |
mpanetta | mornin all | 13:28 |
*** ykaplan has quit IRC | 13:29 | |
*** amitgandhi has joined #openstack-marconi | 13:33 | |
*** nosnos has quit IRC | 13:35 | |
*** amitgandhi has quit IRC | 13:38 | |
*** ykaplan has joined #openstack-marconi | 13:42 | |
*** aniuskad has joined #openstack-marconi | 13:43 | |
flaper87 | mpanetta: morning | 13:48 |
mpanetta | Morning | 13:48 |
mpanetta | What time is it for you flaper87? | 13:48 |
flaper87 | mpanetta: 14:50 | 13:48 |
flaper87 | WTF? 14:50 already? | 13:49 |
flaper87 | oh man! | 13:49 |
flaper87 | T_T | 13:49 |
mpanetta | I take it you have had a busy day? :) | 13:49 |
mpanetta | Time flies when you are having fun right? | 13:49 |
flaper87 | mpanetta: yeah! :( | 13:49 |
flaper87 | I'm happy to see time flying because that means I still want to do what I'm doing | 13:50 |
flaper87 | :D | 13:50 |
flaper87 | but man, -ETOOMANYTHINGS | 13:50 |
*** jcru has joined #openstack-marconi | 13:51 | |
mpanetta | haha gotta hate that error ;) | 13:51 |
*** jergerber has joined #openstack-marconi | 14:16 | |
*** aniuskad has quit IRC | 14:17 | |
*** alcabrera has joined #openstack-marconi | 14:21 | |
alcabrera | Good morning! :D | 14:22 |
*** amitgandhi has joined #openstack-marconi | 14:24 | |
mpanetta | Morning alcabrera! | 14:25 |
alcabrera | mpanetta: hey! Things go well over here. I'm a bit on the sleepy side today. How about you? :) | 14:29 |
*** amitgandhi has quit IRC | 14:29 | |
mpanetta | I feel much better then I did yesterday :) | 14:29 |
*** amitgandhi has joined #openstack-marconi | 14:30 | |
*** aniuskad has joined #openstack-marconi | 14:31 | |
alcabrera | awesome, glad to hear it. :D | 14:34 |
flaper87 | alcabrera: MORNING! | 14:36 |
alcabrera | flaper87: hey!!! :D | 14:37 |
flaper87 | man, next time I decide to get rid of oslo.config in client libraries, please, really, for the sake of friendship, STOP ME! | 14:38 |
alcabrera | lol | 14:39 |
alcabrera | was it *that* bad? | 14:39 |
alcabrera | I mean, trying to remove it? | 14:40 |
alcabrera | flaper87: I noticed that rbyant requested its removal in the review, and that you ended up not doing so. What happened? | 14:43 |
flaper87 | alcabrera: I'm removing the whole oslo.config dependency, which means that config option will dissapear | 14:43 |
flaper87 | It's almost done | 14:44 |
alcabrera | ah. Hope it goes well. :) | 14:45 |
alcabrera | My own attempt at refactoring the sharding patch did not go so well. :P | 14:46 |
alcabrera | I 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 |
alcabrera | mostly, with the help of 'tox -e pep8', e.g., 'self' is not defined (at class scope). | 14:46 |
alcabrera | which makes sense, but I didn't think far ahead enough to see that. D: | 14:47 |
flaper87 | :( | 14:49 |
flaper87 | alcabrera: btw, re that patch! | 14:49 |
flaper87 | I was thinking that we could maybe go through it together | 14:49 |
flaper87 | today | 14:49 |
alcabrera | sweet. Sounds good to me. | 14:49 |
alcabrera | Earlier in the day works better for me, so - the next two hours I'm mostly free. :) | 14:50 |
alcabrera | just let me know when you get a spare moment and we'll take out that patch! | 14:50 |
*** malini_afk is now known as malini | 14:51 | |
alcabrera | also, 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 IRC | 15:06 | |
*** haomaiwang has joined #openstack-marconi | 15:07 | |
alcabrera | flaper87: I put together a list of pending sharding patches to help out with our review process: https://etherpad.openstack.org/p/sharding-patches | 15:07 |
*** ayoung has joined #openstack-marconi | 15:09 | |
flaper87 | alcabrera: Yeah, the wheel support if very cool! | 15:11 |
flaper87 | alcabrera: awesome, give me 5 more mins and we'll go through it | 15:11 |
alcabrera | flaper87: kk | 15:11 |
*** haomaiwang has quit IRC | 15:12 | |
flaper87 | I hate computers http://paste.openstack.org/show/53602/ | 15:23 |
* alcabrera clicks | 15:23 | |
flaper87 | I hate computers http://paste.openstack.org/show/53602/ | 15:24 |
alcabrera | *.pyc | 15:24 |
alcabrera | they've gotten me more times than I can count | 15:24 |
flaper87 | already deleted them | 15:25 |
flaper87 | :( | 15:25 |
alcabrera | :/ | 15:25 |
flaper87 | I have literally no clue | 15:26 |
mpanetta | Eww | 15:26 |
mpanetta | That is a nasty one | 15:26 |
alcabrera | hmmm... | 15:27 |
alcabrera | Those make me think of import-time side effects. | 15:27 |
alcabrera | I tend to look at __init__.pys when things get that weird/ | 15:27 |
flaper87 | somehow, python thinks they are the same module | 15:27 |
alcabrera | flaper87: hmmm... this looks suspicious: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/__init__.py | 15:30 |
alcabrera | specifically: https://github.com/openstack/python-marconiclient/blob/master/marconiclient/transport/__init__.py#L21 | 15:30 |
flaper87 | why ? | 15:30 |
flaper87 | you know what's funny? | 15:30 |
alcabrera | that exposes the name 'errors' from marconiclient.errors at the same level as marconiclcient.transport.errors | 15:30 |
flaper87 | that line there, loads the marconiclient.transport.errors | 15:31 |
flaper87 | instead of marconiclient.errors | 15:31 |
flaper87 | :D | 15:31 |
flaper87 | but, if I do that from the shell, it'll load the marconiclient.errors module | 15:31 |
alcabrera | wait... what? @_@ | 15:31 |
alcabrera | weeeeird | 15:31 |
flaper87 | hahahahaha, time to show our WTFFace | 15:32 |
alcabrera | yuuuppp | 15:32 |
alcabrera | idea: when doing an intra-project import in an __init__, to ++safety, how about 'from myproj import mymodule as _mymodule'? | 15:33 |
alcabrera | flaper87: verified that L21 is causing the issue. | 15:35 |
alcabrera | I 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 |
alcabrera | It's an import-time side-effect causing the silliness. | 15:36 |
flaper87 | alcabrera: [GCC 4.8.1 20130603 (Red Hat 4.8.1-1)] <- good boy | 15:41 |
flaper87 | :D | 15:41 |
alcabrera | lol | 15:42 |
flaper87 | alcabrera: that worked! Thanks! | 15:42 |
alcabrera | np. :) | 15:42 |
flaper87 | I remember fixing this but ... | 15:42 |
flaper87 | AHHHHHHHHHHHHHHHHHHHHHHHHHHH | 15:42 |
flaper87 | the s/exceptions/errors/ patch | 15:42 |
flaper87 | I blaim that patch | 15:42 |
alcabrera | :P | 15:42 |
alcabrera | flaper87: I'm going to submit a patch for this error. (_errors). :D | 15:43 |
flaper87 | alcabrera: I already fixed it in the patch I'm about to submit but feel free to do that | 15:43 |
flaper87 | I'll base it on yours | 15:43 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors https://review.openstack.org/57262 | 15:44 |
alcabrera | The first client-side patch I've submitted in ages. | 15:45 |
alcabrera | flaper87: hehe, we'll double fix it and make sure it never comes back. ;) | 15:45 |
flaper87 | alcabrera: +1 thanks for the patch | 15:46 |
alcabrera | glad to help! :D | 15:46 |
flaper87 | I won't even wait for jenkins | 15:46 |
flaper87 | stupid error | 15:46 |
flaper87 | T_T | 15:46 |
flaper87 | I think I'll ninja-approve that patch | 15:46 |
alcabrera | tox'd locally - passed pep8, pypy, py27, and py33 | 15:46 |
alcabrera | works for me | 15:47 |
*** russell_h_ has joined #openstack-marconi | 15:51 | |
openstackgerrit | A change was merged to openstack/python-marconiclient: Support building wheels (PEP-427) https://review.openstack.org/57125 | 15:52 |
alcabrera | w00t | 15:52 |
alcabrera | jenkins has been a little slow today. :P | 15:52 |
*** russell_h has quit IRC | 15:52 | |
*** russell_h_ is now known as russell_h | 15:52 | |
openstackgerrit | A change was merged to openstack/marconi: Support building wheels (PEP-427) https://review.openstack.org/57150 | 15:53 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Remove the dependency on oslo.config https://review.openstack.org/57265 | 15:53 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors https://review.openstack.org/57262 | 15:53 |
flaper87 | alcabrera: man, I did something stupid | 15:53 |
flaper87 | T_T | 15:53 |
flaper87 | alcabrera: can you resubmit your patch ? | 15:54 |
flaper87 | I cherrypicked instead of rebasing | 15:54 |
flaper87 | T_T | 15:54 |
flaper87 | sorry about that | 15:54 |
alcabrera | :( | 15:55 |
* alcabrera resubmits | 15:55 | |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/python-marconiclient: fix: import-time side effect on errors https://review.openstack.org/57268 | 15:55 |
alcabrera | flaper87: that should do it. :) | 15:56 |
flaper87 | oh you submitted a new one, you could've re-sent the same patch. I'll approve this | 15:56 |
flaper87 | alcabrera: can you abandon this: https://review.openstack.org/#/c/57262/2 | 15:57 |
alcabrera | git review grumped: "failed to push some refs to" | 15:57 |
alcabrera | yup | 15:57 |
alcabrera | flaper87: done. :D | 15:57 |
alcabrera | "Abandoned for great redundancy" | 15:57 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Remove the dependency on oslo.config https://review.openstack.org/57265 | 15:58 |
openstackgerrit | Flavio Percoco proposed a change to openstack/python-marconiclient: Bootstrap Messages support https://review.openstack.org/52389 | 15:58 |
flaper87 | cool, now we're talking | 15:58 |
flaper87 | alcabrera: I'm ready | 15:58 |
alcabrera | flaper87: g+-style? | 16:00 |
flaper87 | alcabrera: yup | 16:00 |
alcabrera | I'll be ready in 2m. | 16:01 |
alcabrera | VPN will now disconnect me. >.> | 16:01 |
alcabrera | brb | 16:01 |
flaper87 | kk | 16:01 |
*** acabrera has joined #openstack-marconi | 16:02 | |
*** reed has joined #openstack-marconi | 16:04 | |
*** alcabrera has quit IRC | 16:05 | |
*** haomaiwang has joined #openstack-marconi | 16:18 | |
*** haomaiwang has quit IRC | 16:23 | |
*** ykaplan has quit IRC | 16:23 | |
*** ykaplan has joined #openstack-marconi | 16:24 | |
*** cpallares has joined #openstack-marconi | 16:26 | |
*** ykaplan has quit IRC | 16:29 | |
*** aniuskad has quit IRC | 16:32 | |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: connect sharding manager to control drivers https://review.openstack.org/54605 | 16:33 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: connect sharding manager to control drivers https://review.openstack.org/54605 | 16:35 |
*** russell_h has quit IRC | 16:36 | |
*** russell_h has joined #openstack-marconi | 16:36 | |
*** oz_akan_ has joined #openstack-marconi | 16:39 | |
openstackgerrit | A change was merged to openstack/python-marconiclient: fix: import-time side effect on errors https://review.openstack.org/57268 | 16:51 |
*** oz_akan_ has quit IRC | 16:57 | |
*** oz_akan_ has joined #openstack-marconi | 16:57 | |
*** jcru has quit IRC | 16:58 | |
*** jcru has joined #openstack-marconi | 16:59 | |
*** flwang has quit IRC | 17:13 | |
*** pycabrera has joined #openstack-marconi | 17:13 | |
*** pycabrera is now known as alcabrera | 17:13 | |
alcabrera | flaper87: thaaaanks! :D | 17:13 |
flaper87 | alcabrera: thank you :D That was a really good pair-reviewing session and we got the sharding patch sorted out! w0000t | 17:14 |
alcabrera | agreed. :) | 17:15 |
alcabrera | more next time - 'til then, I'll be fixing up the issues. | 17:15 |
*** acabrera has quit IRC | 17:16 | |
flaper87 | alcabrera: if you've 5mins between those rebases, pls, I'd love your feedback here: https://review.openstack.org/#/c/57265/ | 17:21 |
alcabrera | flaper87: I'll check it out now. :) | 17:21 |
ekarlso | qq/win 35 | 17:21 |
flaper87 | passwd ? | 17:22 |
flaper87 | alcabrera: thanks :D | 17:22 |
flaper87 | brb | 17:26 |
*** flaper87 is now known as flaper87|afk | 17:27 | |
alcabrera | flaper87|afk: +2'd w/ suggestions | 17:27 |
*** alcabrera is now known as alcabrera|afk | 17:27 | |
*** amitgandhi has quit IRC | 17:29 | |
*** reed has quit IRC | 17:30 | |
*** kgriffs_afk is now known as kgriffs | 17:47 | |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: feat(shard): queue listing from multiple sources https://review.openstack.org/53127 | 17:49 |
*** amitgandhi has joined #openstack-marconi | 17:50 | |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: feat(health): make health API storage-aware https://review.openstack.org/53457 | 18:02 |
*** flaper87|afk is now known as flaper87 | 18:18 | |
*** reed has joined #openstack-marconi | 18:31 | |
*** kgriffs is now known as kgriffs_afk | 18:50 | |
*** kgriffs_afk is now known as kgriffs | 18:55 | |
*** alcabrera|afk is now known as alcabrera | 19:01 | |
flaper87 | alcabrera: can I get your bless here? https://review.openstack.org/#/c/52389/ | 19:27 |
* alcabrera clicks | 19:27 | |
flaper87 | kgriffs: could you please take a look at these patches? https://review.openstack.org/#/q/status:open+project:openstack/python-marconiclient,n,z :D | 19:27 |
alcabrera | flaper87: running a quick sanity check. :) | 19:29 |
alcabrera | flaper87: +2 | 19:30 |
*** oz_akan_ has quit IRC | 19:34 | |
*** oz_akan_ has joined #openstack-marconi | 19:34 | |
*** oz_akan_ has quit IRC | 19:38 | |
*** jcru_ has joined #openstack-marconi | 19:41 | |
*** jcru has quit IRC | 19:45 | |
*** jcru_ has quit IRC | 19:49 | |
*** alcabrera is now known as alcabrera|afk | 19:55 | |
*** yassine has quit IRC | 19:59 | |
*** alcabrera|afk is now known as alcabrera | 20:14 | |
*** kgriffs is now known as kgriffs_afk | 20:16 | |
*** malini is now known as malini_afk | 20:29 | |
*** jcru has joined #openstack-marconi | 20:29 | |
*** ametts has quit IRC | 20:42 | |
*** oz_akan_ has joined #openstack-marconi | 20:50 | |
*** oz_akan_ has quit IRC | 20:51 | |
*** oz_akan_ has joined #openstack-marconi | 20:52 | |
*** kgriffs_afk is now known as kgriffs | 20:56 | |
*** kgriffs is now known as kgriffs_afk | 21:06 | |
alcabrera | Good night, everyone. :) | 21:17 |
*** alcabrera has quit IRC | 21:17 | |
*** kgriffs_afk is now known as kgriffs | 21:17 | |
*** amitgandhi has quit IRC | 21:40 | |
*** amitgandhi has joined #openstack-marconi | 22:01 | |
*** tedross has quit IRC | 22:13 | |
*** amit has joined #openstack-marconi | 22:18 | |
*** amit is now known as Guest67108 | 22:19 | |
*** Guest67108 has quit IRC | 22:20 | |
*** amitgandhi has quit IRC | 22:21 | |
*** amitgandhi has joined #openstack-marconi | 22:22 | |
*** mpanetta has quit IRC | 22:24 | |
*** ayoung has quit IRC | 22:28 | |
*** amitgandhi has quit IRC | 22:59 | |
*** jcru has quit IRC | 22:59 | |
*** flaper87 is now known as flaper87|afk | 23:00 | |
*** jergerber has quit IRC | 23:14 | |
*** kgriffs is now known as kgriffs_afk | 23:26 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!