Tuesday, 2013-12-03

*** rongze has quit IRC00:00
*** kgriffs_afk is now known as kgriffs00:04
*** oz_akan_ has joined #openstack-marconi00:09
*** kgriffs is now known as kgriffs_afk00:14
*** oz_akan_ has quit IRC00:14
*** cpallares has quit IRC00:21
*** amitgandhi has joined #openstack-marconi00:22
*** amitgandhi has quit IRC00:28
*** oz_akan_ has joined #openstack-marconi00:34
*** oz_akan_ has quit IRC00:39
*** amitgandhi has joined #openstack-marconi00:45
*** oz_akan_ has joined #openstack-marconi01:11
*** nosnos has joined #openstack-marconi01:26
*** rongze has joined #openstack-marconi01:27
*** rongze has quit IRC01:31
*** oz_akan_ has quit IRC01:52
*** flwang has joined #openstack-marconi01:55
*** flwang has left #openstack-marconi01:56
*** flwang has joined #openstack-marconi01:56
*** reed has quit IRC02:00
*** rongze has joined #openstack-marconi02:06
*** oz_akan_ has joined #openstack-marconi02:54
*** nosnos has quit IRC02:58
*** oz_akan_ has quit IRC02:58
*** nosnos has joined #openstack-marconi02:59
*** flwang has quit IRC03:08
*** oz_akan_ has joined #openstack-marconi03:09
*** oz_akan_ has quit IRC03:14
*** amitgandhi has quit IRC03:16
*** amitgandhi has joined #openstack-marconi03:16
*** amitgandhi has quit IRC03:21
*** oz_akan_ has joined #openstack-marconi04:10
*** oz_akan_ has quit IRC04:14
*** flwang has joined #openstack-marconi04:17
*** russell_h has quit IRC04:24
*** russell_h has joined #openstack-marconi04:27
*** rongze has quit IRC04:31
*** jamieh has joined #openstack-marconi04:33
*** oz_akan_ has joined #openstack-marconi05:11
*** oz_akan_ has quit IRC05:15
*** jamieh has quit IRC05:20
*** russell_h has quit IRC05:42
*** russell_h has joined #openstack-marconi05:42
*** ykaplan has joined #openstack-marconi05:50
*** oz_akan_ has joined #openstack-marconi06:13
*** oz_akan_ has quit IRC06:17
*** ykaplan has quit IRC07:23
*** flaper87|afk is now known as flaper8708:03
*** oz_akan_ has joined #openstack-marconi08:15
*** oz_akan_ has quit IRC08:19
*** ykaplan has joined #openstack-marconi09:07
*** yassine has joined #openstack-marconi09:10
*** oz_akan_ has joined #openstack-marconi09:15
*** oz_akan_ has quit IRC09:20
*** flwang has quit IRC09:28
*** flwang has joined #openstack-marconi10:14
*** oz_akan_ has joined #openstack-marconi10:16
*** oz_akan_ has quit IRC10:21
*** ykaplan has quit IRC10:28
*** rongze has joined #openstack-marconi10:28
*** ykaplan has joined #openstack-marconi10:30
*** rongze has quit IRC10:46
*** rongze has joined #openstack-marconi11:03
openstackgerritFei Long Wang proposed a change to openstack/marconi: Fixes wrong hrefs when getting set of messages  https://review.openstack.org/5809211:09
*** oz_akan_ has joined #openstack-marconi11:17
*** oz_akan_ has quit IRC11:21
*** ykaplan has quit IRC11:25
*** ykaplan has joined #openstack-marconi11:33
*** gordonsim has joined #openstack-marconi11:35
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add `simple` example for the client library  https://review.openstack.org/5755411:36
*** rongze has quit IRC11:39
openstackgerritA change was merged to openstack/python-marconiclient: Add `simple` example for the client library  https://review.openstack.org/5755411:39
*** fifieldt has quit IRC11:40
*** rongze_ has joined #openstack-marconi11:56
openstackgerritFei Long Wang proposed a change to openstack/marconi: Fixes wrong hrefs when getting set of messages  https://review.openstack.org/5809212:06
*** oz_akan_ has joined #openstack-marconi12:18
*** wirehead_ has quit IRC12:19
*** oz_akan_ has quit IRC12:22
*** wirehead_ has joined #openstack-marconi12:30
flwanganybody there?13:11
*** jergerber has joined #openstack-marconi13:11
*** jamieh has joined #openstack-marconi13:12
flaper87flwang: o/13:14
flwangI'm curious is there any plan to enable py3 in Marconi given it's a new project and the cost will be low13:14
*** ayoung has quit IRC13:22
*** oz_akan_ has joined #openstack-marconi13:25
*** oz_akan_ has quit IRC13:28
flaper87flwang: there is. We have a non-voting gate13:29
flaper87and I was planning to start migrating it. Do you want to work on that?13:29
*** oz_akan_ has joined #openstack-marconi13:29
flwangi see, i mean do we want to enable it as a gate13:29
flaper87flwang: we want to make it voting as soon as it start's being green :D13:29
flwangI'd like to do some investigation13:30
flaper87flwang: go ahead and make our day13:31
flaper87:D13:31
flwangflaper87: sure, btw, would you mind revisiting my patch ? https://review.openstack.org/#/c/58092/13:33
*** tedross has joined #openstack-marconi13:34
flaper87flwang: small nit, otherwise it LGTM13:37
*** jamieh has quit IRC13:40
*** mpanetta has joined #openstack-marconi13:41
flwangflaper87: will fix in next PS13:42
openstackgerritFei Long Wang proposed a change to openstack/marconi: Fixes wrong hrefs when getting set of messages  https://review.openstack.org/5809214:13
openstackgerritFei Long Wang proposed a change to openstack/marconi: Fixes wrong hrefs when getting set of messages  https://review.openstack.org/5809214:20
*** nosnos has quit IRC14:20
*** rongze_ has quit IRC14:27
*** thomasem has joined #openstack-marconi14:36
*** rongze has joined #openstack-marconi14:38
*** amitgandhi has joined #openstack-marconi14:38
*** alcabrera has joined #openstack-marconi14:40
alcabreraGood morning!14:41
mpanettaalcabrera: morning14:41
alcabrerampanetta: hey! :)14:43
mpanettahow goes alcabrera?14:44
flaper87alcabrera: GOOD MORNING14:47
flaper87:D14:47
*** jcru has joined #openstack-marconi14:47
alcabreraflaper87: :D14:47
alcabrerampanetta: it's going pretty well. A good, slow morning before I get to reviews/code/fixes/blogs/everything.14:48
mpanettaSlow is nice14:48
mpanettaWhen does kgriffs_afk usually come around?14:49
alcabrerahe'll be here in 10 minutes, I'm guessing, since we have a  notifications meeting. As for usually... seems to vary between 30 minutes from now and an hour from now.14:50
mpanettaAh cool thanks.14:52
alcabreraflaper87: how do you feel about doing 5 super tiny reviews for annoying marconi bugs/small optimizations? ;)14:53
* alcabrera is ready to provide some links14:54
*** amitgand_ has joined #openstack-marconi14:54
flaper87alcabrera: lets do it!14:55
alcabreraAlright, here we go!14:55
alcabrera1. https://review.openstack.org/#/c/59507/ (fix: memcached backend in oslo.cache returns None for set)14:56
megan_where for the notifications meeting o/14:56
alcabrera2. https://review.openstack.org/#/c/59511/ (opt: mongodb hints)14:56
alcabrera3. https://review.openstack.org/#/c/59513/ (rm: proxy cruft)14:56
alcabrera4. https://review.openstack.org/#/c/59539/ (mongodb tests consume storage like crazy)14:56
flaper87alcabrera: 1. is it downstream only? I don't remember updating the oslo review14:56
*** kgriffs_afk is now known as kgriffs14:56
alcabreraflaper87: yeah, it's only downstream. I commented on the oslo.cache review awhile ago with this issue.14:57
alcabrera5. https://review.openstack.org/#/c/59541/ (fix: shard test failing)14:57
alcabrerathat's all of them.14:57
alcabreramegan_w: hey! :D14:57
alcabrerakgriffs: o/14:57
flaper87alcabrera: yeah, I haven't addressed it yet because we're still changing things in the cache API14:57
flaper87T_T14:57
kgriffsyo yo14:57
flaper87kgriffs: btw, I commented on that cache review. Great comments man!14:57
flaper87I'd like your feedback there before starting to work on another version14:57
kgriffsflaper87: kk14:57
flaper87s/another/a new/14:58
*** amitgandhi has quit IRC14:58
kgriffsI am going to try to start commenting on other oslo patches in general, but... so many ambitions, so little time.14:58
kgriffs:p14:59
alcabrerathere's just too much to do, kgriffs! ;)14:59
kgriffsalcabrera: yep!14:59
* kgriffs needs to invent a pop-tart that slows down time when you eat it14:59
flaper87alcabrera: fix: mongodb tests consume storage like crazy15:00
flaper87LOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOL15:00
mpanettaPlease!15:00
flaper87that happened to me as well15:00
flaper87:D15:00
alcabrerayeeeepp, flaper87!15:00
alcabreraIt was terrible. D:15:00
* kgriffs mkdir temporal_pop_tart; cd temporal_pop_tart; git init .15:01
kgriffsok15:01
kgriffsmeeting time?15:01
alcabreraready15:01
megan_wready15:01
* mpanetta is starting to worry that someone may fall through the roof here...15:01
kgriffsamitgand_: ready for mtg?15:02
flaper87alcabrera: all +2 but (4)15:02
kgriffsalcabrera: are you in the office?15:02
*** dragondm has joined #openstack-marconi15:02
* alcabrera ssh poptart_box; cd temporal_pop_tart; touch recipe.txt; git ci -am "add recipe skeleton"; exit15:02
flwango/15:02
*** ildikov has joined #openstack-marconi15:02
thomasemo/15:03
alcabrerakgriffs: yup15:03
dragondmo/15:03
kgriffsalcabrera: is amitgand_ around?15:03
kgriffsdragondm, thomasem, flaper87: yo, thanks for coming!15:03
flaper87\o/15:03
dragondmno problem.15:03
alcabrerakgriffs: yup, I just ping'd him15:03
kgriffskk15:03
flaper87100euros, thanks!15:03
kgriffswe'll start in 2 mins15:03
alcabreraflaper87: thanks for the quick reviews!15:04
* kgriffs hands out pop-tarts15:04
amitgand_kgriffs: hi15:04
kgriffsamitgand_: thanks for coming!15:04
*** amitgand_ is now known as amitgandhi15:04
kgriffsI wanted to get your ideas since you did a hackday project around notifications15:04
kgriffsok15:04
kgriffslet's get this party started.15:04
* kgriffs puts on party hat15:04
kgriffs#startmeeting marconi-notifications-brainstorm15:05
openstackMeeting started Tue Dec  3 15:05:09 2013 UTC and is due to finish in 60 minutes.  The chair is kgriffs. Information about MeetBot at http://wiki.debian.org/MeetBot.15:05
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:05
openstackThe meeting name has been set to 'marconi_notifications_brainstorm'15:05
kgriffs#link https://etherpad.openstack.org/p/marconi-notifications-brainstorm15:05
kgriffs#topic goals15:05
amitgandhi#link https://github.com/amitgandhinz/cloud_notifications#api15:05
flaper87o/15:05
kgriffsOK, first off I wanted to get a rough consensus around what use cases we are trying to enable15:06
kgriffsthoughts?15:06
megan_wapp development15:06
megan_walerting15:06
amitgandhireporting15:07
flaper87and notification15:07
thomasemdefinitely reporting15:07
alcabreranotifications, in the sense of, when X happens, users [Y] get a notifications in one or more of [Z] forms.15:07
* amitgandhi moves to etherpad15:07
flaper87(events)15:07
flaper87alcabrera: out of my head15:07
flaper87thanks15:07
alcabreraflaper87: :)15:07
flwangalarm15:07
kgriffsok, let's get some concrete examples for those15:08
kgriffsalerting?15:08
megan_w1.  send me email when i use more than 100GB of CDN usage15:08
kgriffsmegan_w: is that an alarm?15:08
megan_wmaybe i'm confusing alarm and alerting15:09
megan_wsorry15:09
alcabreraa disk drive is reaching 90% capacity -> ops manager gets an sms and an email15:09
thomasemSend me a report at the end of each day detailing the operation failures15:09
thomasemBut, I guess that's more from a deployers perspective.15:09
*** ayoung has joined #openstack-marconi15:09
flaper87lets all go to the etherpad15:09
alcabrera+115:09
* mpanetta watches15:10
flaper87and then we can come back here15:10
*** gordc has joined #openstack-marconi15:10
thomasemOkay, cool15:10
mpanettaI'm lurking, I hope that is ok. :)15:10
alcabrerampanetta: lurk away!15:10
kgriffsflaper87:  +115:10
kgriffsjust go and type in examples15:10
flwangan alarm is like the alarm using in Ceilometer15:12
gordonsimI'm not entirely clear on what exactly notifications means here. On the one hand there is a general pub-sub type mechanism to allow subscribers to be notified when messages of interest are published. On the other is a service built on top of something like that, which is essentially a precanned listener that handles the notification of a message by generating some 'external' notification (e.g. sends an SMS/Emai15:14
gordonsimis this more of the second?15:15
gordonsimor is the first in scope as well?15:15
flaper87gordonsim: it's pretty much both, TBH!15:16
flaper87The final goal is to allow people to do the second15:16
flaper87gordonsim: want to join? https://etherpad.openstack.org/p/marconi-notifications-brainstorm15:16
megan_wi suppose there is something non-immediate about a report, right?  that's what makes it differnt from alerting on some of these use cases..?15:17
gordonsimflaper87: yes, I'm there15:17
flaper87gordonsim: ah sorry, I don't see your nick15:17
flaper87:D15:17
alcabreramegan_w: I'm with you on that idea. The alarm would be triggered on some condition, whereas the report probably has a periodic nature.15:17
megan_wright15:17
alcabreraalarm behaves very much like alert15:18
kgriffsmegan_w: noted15:18
flwangalcabrera: yep, but I prefer to use alarm to be consistent with the term using in Ceilometer15:19
flwangand generally, alarm will trigger a binding action15:20
alcabreraflwang: noted - the earlier we agree on common terminology, the better off we'll be in the long run. :)15:20
flwangalcabrera: +115:21
kgriffs#info Need way to avoid 2 email workers from resending the same message15:27
kgriffsOK folks, looks like we are winding down on use cases and terminology15:28
kgriffslet's wrap that up so we can move on to the next topic15:28
flwangkgriffs: +1 :D15:28
mpanettaSo templates would be for presentation only?  Why not use them to define the types of notifications as well?  Or does that not make sense?15:33
alcabrerampanetta: hmm... I think template == output, so it could be the envelope for a given notification as presented to the target device/user. Trigger should be the input, in my mind.15:35
*** apmelton1 has joined #openstack-marconi15:35
mpanettaAh ok15:35
kgriffsmpanetta: yeah, each sink could have an associated template and trigger/pattern thingy15:36
mpanettaSo the templates would be a way to define what you are subscribing to?15:36
alcabrerampanetta: nah, that would be part of the pattern/trigger. The pattern/trigger might say, "when X arrives on channel [Y], notify [Z] using this particular Template"15:38
mpanettaOk, tht makes sense.15:38
kgriffs#topic elephants15:38
kgriffs(see etherpad)15:38
mpanettaSo colorful heh.  Never done this etherpad thing before.15:39
alcabrerampanetta: this is one of the prettiest ones I've seen. 13 users. :D15:40
kgriffsok, great thoughts so far!15:41
kgriffswe have 20 minutes left15:41
kgriffsMoving on15:41
thomasemcool15:41
kgriffs#topic features15:42
mpanettaEverything is an 'event' :P15:42
kgriffsAny other features people want to add/talk about?15:42
kgriffs(see etherpad under "Features: Stuff Users See")15:42
megan_wsort of15:43
megan_wi assume we leave the actual SMS, email, etc up to someone else?15:43
kgriffsmegan_w: +115:44
kgriffsI think the marconi project should provide 2-3 reference sinks.15:44
kgriffsemail15:45
kgriffswebhook15:45
kgriffsone more?15:45
amitgandhisms15:45
megan_wyeah15:45
amitgandhiand i think we need to do APN15:46
mpanettaAPN?15:46
amitgandhipush notifications15:46
mpanettaAH ok15:46
amitgandhiit gives us the mobile market15:46
amitgandhiamazon SNS introduced it a few months back also15:46
megan_wtotally agreed.  people expect push here15:47
*** whenry has joined #openstack-marconi15:48
kgriffsI like APN15:49
kgriffsI want to get people thinking mobile apps15:49
kgriffslet's do that one15:49
kgriffspeople can insert their APN creds into the reference sink to try it out15:50
alcabreraFor the record, since I wasn't aware of APN...15:51
alcabrera#info  Apple Push Notification Service (APNS)15:51
thomasemThanks, was about to ask. =]15:51
kgriffsGoogle has something similar for Android15:52
kgriffs(I forget the name)15:52
kgriffsOK, we are short on time15:52
flaper87APN = http://en.wikipedia.org/wiki/Access_Point_Name15:52
kgriffsOne more topic I wanted to touch on, then we will wrap up this session15:52
alcabrera#info GCM = Google Cloud Message, Android's analogue to APNS15:53
kgriffsflaper87: The "s" in "APNs" is significant, it turns out. :p15:53
flaper87:P15:53
mpanettaTMTLA15:53
kgriffs#topic how to publish messages to a sink15:54
kgriffsOK, so thoughts on how events will get published?15:55
kgriffsworker pool?15:55
kgriffspipleline stage within marconi itself?15:56
kgriffssomething else?15:56
alcabreraI'm +1 for worker pool, so that the workers themselves can be scaled as needed15:56
* flaper87 has to step out for a bit!15:56
amitgandhi+1 for WP15:56
* flaper87 has a call15:56
alcabreraflaper87: D:15:56
flwangI prefer pipeline15:56
alcabreraflaper87: jk, see you soon.15:56
kgriffsflaper87: need your vote on the above real quick15:56
flaper87if you guys want to discuss that, go ahead! I'll catch up. I don't want to break the flow15:56
kgriffs(or when you get back)15:56
flaper87sure15:56
flaper87mmh, WP, it'll allow us to run them concurrently and / or distributed15:57
flaper87actually, concurrently :D15:57
alcabreraI like the idea of a transformation pipeline, since it has a functional flavor to it. However, if a pipeline was the connecting mechanism, I'd want that to delegate out to the workers.15:57
flaper87we don't want to send yet another message to send a notification15:57
*** reed has joined #openstack-marconi15:57
alcabreraflaper87: +115:58
flwangpipeline allow us to add numbers of transformer15:58
flwangand the transformer can be customied by user15:58
kgriffsso, with the pipeline approach we could just run lots of wsgi workers. non-blocking I/O would be crucial.15:58
kgriffsalso, as flwang pointed out, you can build up a simple workflow (think filter pipeline in photoshop)15:59
kgriffsdownside is you lose some efficiency since you can't scale queues and notifications independently15:59
kgriffsoz_akan_: thoughts?15:59
dragondm^ we are working on this for Ceilometer..15:59
alcabreraMy primary concern with the pipeline approach is the matter of independent worker scaling - would we just launch as many marconi instances as needed to meet the demand for all types of sinks (sms, apns, hooks, etc.)?16:00
amitgandhii like that worker pools can be scaled horizontally and are more resilient to workers crashing16:00
oz_akan_hi, I wasn't following the thread, let me have a look16:00
kgriffswe might also consider some kind of hybrid idea16:00
kgriffsfor example, you could set up a distributed pipeline by stacking queues end-to-end16:00
mpanettaOh how unixy of you16:01
kgriffsleave marconi-queues to do it's think. marconi-notifications sets up the queues, manages subscriptions and workers16:01
kgriffsmpanetta: why thank you! I try. :D16:01
mpanetta:)16:02
mpanettamarconi == | :)16:02
*** ayoung has quit IRC16:02
kgriffsapp | marconi | worker | sms16:02
alcabrerayup - functional transforms all the way through16:03
kgriffsapp | queue1 | worker_a | queue2 | worker_b | sms16:03
kgriffssomething like that.16:03
mpanettaSounds good to me16:03
kgriffsI think we just invented workflow-as-a-service16:03
* kgriffs winks at Mirantis16:04
alcabreralol16:04
mpanettalol16:04
alcabreraso that's another good elephant16:04
alcabreraactually16:04
alcabrerayeah, what's the deal with Mirantis + workers?16:04
alcabreraIs that something we can use?16:04
flwangkgriffs: it's Mistral workflow-as-a-service16:04
alcabreraflwang: thanks for the clarification!16:04
kgriffsyep16:04
alcabrera#link https://wiki.openstack.org/wiki/Mistral16:05
kgriffsIs there a concrete use case for doing multiple transformations or whatever?16:05
amitgandhioooph cloud cron!16:05
amitgandhi*oooh16:05
kgriffsJust thinking we may be getting ahead of ourselves16:05
kgriffsamitgandhi: ooh, that would be nice16:05
alcabreraanyway16:06
amitgandhiits a mistral use case16:06
* alcabrera gets back to topic16:06
kgriffswe could set up a "cron" job that hits a web hook for the event data, then pushes it to a topic16:06
alcabrera"how to publish messages to a sink"16:06
kgriffsok folks, looks like we are out of time16:06
thomasemhmm? kgriffs Ceilometer is already and will be doing pipeline work for its own stuff. Is this in an effort to expose a pipeline to clients?16:06
kgriffsI really appreciate everyones thoughts. We had a very productive brainstorming session!16:06
thomasemIndeed16:07
amitgandhiyeh this was fun16:07
kgriffsthomasem: I definitely don't want to reinvent workflow/distributed pipeline16:07
amitgandhinext steps?16:07
mpanettaMistral looks very interesting...16:07
thomasemHaha, yeah. That'd be a bit wild.16:07
thomasem:P16:07
kgriffsOK, so I will attempt to summarize the etherpad into a pretty wiki16:07
kgriffsAnd link that to the bp16:08
kgriffs#link https://blueprints.launchpad.net/marconi/+spec/notifications16:08
flwangcool16:08
thomasem=] Cheers everyone!16:08
alcabreraawesome, good session. :D16:08
mpanettakgriffs: So, about that merge... ;)16:09
kgriffsthen maybe next monday we can discuss sub-blueprints and get volunteers to start laying down code16:09
kgriffs#endmeeting16:09
openstackMeeting ended Tue Dec  3 16:09:19 2013 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:09
openstackMinutes:        http://eavesdrop.openstack.org/meetings/marconi_notifications_brainstorm/2013/marconi_notifications_brainstorm.2013-12-03-15.05.html16:09
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/marconi_notifications_brainstorm/2013/marconi_notifications_brainstorm.2013-12-03-15.05.txt16:09
openstackLog:            http://eavesdrop.openstack.org/meetings/marconi_notifications_brainstorm/2013/marconi_notifications_brainstorm.2013-12-03-15.05.log.html16:09
alcabrerahmm...16:09
alcabrerakgriffs: should we reference this minute over at the wiki standard location?16:09
alcabreras/minute/meeting16:09
kgriffsalcabrera: sure!16:10
kgriffsalcabrera: want to take care of that?16:10
alcabrerakgriffs: alright, I'll link things up. I was already on it! ;)16:10
kgriffsI will also link to it from the wiki16:10
kgriffsalcabrera: rock on16:10
* kgriffs gives alcabrera a fresh box of pap tarts16:10
flwangalcabrera: kgriffs: could you pls revisit my patch https://review.openstack.org/#/c/58092/ ?16:11
alcabreraflwang: will do!16:11
* alcabrera noms on the pop tarts16:11
kgriffsflwang: will do16:12
flwangthank you, guys16:12
mpanettaThere is a python lib that does task flow management?  Interesting...16:12
flwangso tired to catch you guys every night :P16:12
flwangmpanetta: yep16:12
flwangit's named as taskflow16:13
flwangdriving by Yahoo!16:13
mpanettaflwang: Yes, they mention it at the Mistra web page.16:13
flwangmpanetta: https://wiki.openstack.org/wiki/TaskFlow16:13
mpanettaVery interesting16:13
flwangwhich has been used in Cinder16:14
flwangand Glance is investigating it16:14
* mpanetta has much research to do...16:14
mpanettakgriffs: Do you have a moment?16:14
alcabrerameeting minutes all linked up16:14
alcabreratime to review the things16:15
alcabreraflwang: heh, turns out I was reviewing your patch just before this meeting started! I had the tab open and I was looking over the test. :)16:16
flwangI'm standing by to address any coming comments :)16:17
*** ayoung has joined #openstack-marconi16:17
kgriffsmpanetta: sure16:20
kgriffsbtw, I still have a patch or two to rebase (sharding) and then I will bug alcabrera and flaper87 to get them reviewed ASAP16:21
alcabreraflwang: +2'd16:21
flwangalcabrera: cool, thanks. flaper87: I need your bless16:21
alcabrerakgriffs: I'll be ready to review when you've rebased. I have 4 bug fixes and 1 optimization (< 10 LOC each on average) for you to check out.16:21
kgriffsflwang: approved, but added a some comments (no show stoppers)16:22
flwangkgriffs: thanks for the comments and the approval, it's my first baby in Marconi16:24
kgriffsflwang: w00t16:24
* kgriffs gives flwang a high-five16:24
alcabreraflwang: nice! I'm looking to seeing you make more awesome happen in marconi. :)16:24
flwangkgriffs: i have approved many patches, but i'm a little bit excited to get my first approval from Marconi :D16:24
alcabreralooking forward to16:24
flwangalcabrera: sure, thanks dude16:25
openstackgerritA change was merged to openstack/marconi: Fixes wrong hrefs when getting set of messages  https://review.openstack.org/5809216:25
flwangkgriffs: i mean in Glance :)16:25
flwangthank you, guys16:26
alcabreraw00t16:27
mpanettakgriffs: Cool.  I would like to have something today if possible.  I have tests I need to run.16:34
mpanettaI'm feeling a bit testy. :P16:34
*** yassine has quit IRC16:49
* kgriffs steps on eggshells around mpanetta16:49
mpanettaHmm, bad pun I guess :P16:50
kgriffsalcabrera: btw, thanks for helping me overwrite all the free space on my drive. If someone steals my laptop, they won't be able to recover any deleted files now!16:50
* kgriffs needs that unit test fix!16:50
mpanettahaha16:50
alcabrerakgriffs: np, no problem at all. ;)16:50
mpanettaIt isn't a bug, it is a feature!16:51
kgriffsmpanetta: you said it!16:51
alcabreraIt *really* showed me the limitations of my SSD Linux partition (~50GB). :P16:51
alcabreraI've never filled up space more quickly than that.16:51
mpanettaI take it the test created a lot of messages...16:51
kgriffsalcabrera: let's get this merged asap! https://review.openstack.org/#/c/59539/1/tests/unit/queues/storage/test_impl_mongodb.py16:52
alcabreraI'll address that feedback now. :)16:53
kgriffsthanks!16:55
kgriffsI'm going to get tired typing "rm". :p16:55
openstackgerritA change was merged to openstack/marconi: fix: cache contract for memcached backend  https://review.openstack.org/5950716:56
* kgriffs writes a bash script entitled "clean"16:57
mpanettahah16:59
kgriffsmpanetta: btw, I am rebasing the sharding patches right now. will be ready for review soon!16:59
* kgriffs wants parallel testing for when MARCONI_TEST_MONGODB=117:00
alcabrerakgriffs: yeeeesss ^17:01
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: fix: mongodb tests consume storage like crazy  https://review.openstack.org/5953917:03
alcabrerakgriffs: ^^ (made it work with the mixin. Turns out it was as simple as flipping the order of inheritance.)17:03
alcabreraflaper87: ^^17:03
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: fix: shard test failing  https://review.openstack.org/5954117:04
mpanettakgriffs: Thank you :)17:04
flaper87alcabrera: another thought in-line.17:08
flaper87that implementation breaks _purge_database mro scanning.17:09
alcabreraflaper87: good point - I like the idea about making the mixin inherit from testing.TestBase.17:09
alcabreraExcept at that point, I'd skip out on calling it a mixin and call it something else. :)17:09
flaper87alcabrera: sorry about that. I shouldn't have said mixin in first place17:09
alcabreraupdating the patch. :D17:09
alcabrerano worries17:10
flaper87I did think about a mixin, I just realized that it's not that good after all17:10
alcabreraI'm calling it: MongodbTestBase17:10
flaper87alcabrera: +1, wasn't there one already? You may want to put it in marconi/tests as opposed to tests/17:11
alcabreraI don't think we have one of those. :/17:11
alcabreraGiven that it's mongodb-specific, I'm inclined to keep it in tests.unit.queues.storage.impl_mongodb17:11
alcabrerait's going to override _prepare_conf, too, it seems.17:13
*** gordc has left #openstack-marconi17:13
alcabrerahrmmm...17:15
alcabrerasubtyping gets so weird sometimes. :/17:15
*** mpanetta is now known as mpanetta_lunch17:15
*** reed has quit IRC17:16
flaper87alcabrera: ah, yeah, you're right. it's mongo stuff and should go under impl_mongo17:19
flaper87-ETOOMANYTHINGS17:19
alcabreraflaper87: I think the easiest solution is just to have every class reimplement _purge_databases. It gets pretty nasty if we start getting all hierarchical, given that...17:24
alcabrera_purge_databases is defined first in testing.TestBase, which...17:24
alcabrerais then the parent of ControllerBase, which is then the parent of...17:24
alcabreraXControllerTest17:24
flaper87alcabrera: mmh, wait, if you make that base class inherit from testing.testBase and then the others inherit from MongodbBase, what issues are there?17:25
alcabreraMRO continues to strike, because the _purge_databases that is used is the first one declared in the parents list, e.g, class Y(A, B)17:25
alcabreravs. Y(B, A)17:25
*** kgriffs is now known as kgriffs_afk17:25
flaper87yeah, but I mean, just have Y(MongodbBase)17:26
alcabrerahmmm...17:26
flaper87ah17:26
flaper87no nevermind, I got your point17:26
flaper87in that case, ignore me!17:26
flaper87I agree with you17:26
alcabrerabut MongodbXControllerTest needs XControllerTest :P17:26
alcabrerait's messy, heh. :x17:27
flaper87I forgot about the Controllers classes17:27
alcabrerayup - it gets all diamondified17:27
*** ykaplan has quit IRC17:30
alcabreraI'm reverting to manual method override.17:30
alcabreraI think we can come back to this later and attempt a more elegant solution.17:30
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: fix: mongodb tests consume storage like crazy  https://review.openstack.org/5953917:35
alcabreraflaper87: now with a 100% more explicit commit message. ;)17:35
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: fix: shard test failing  https://review.openstack.org/5954117:35
*** ayoung has quit IRC17:39
*** reed has joined #openstack-marconi17:41
alcabreraflaper87: thanks!17:49
flaper87alcabrera: ;)17:49
flaper87alcabrera: kgriffs_afk this one should be pretty easy to review: https://review.openstack.org/#/c/59566/17:49
*** ayoung has joined #openstack-marconi17:51
*** kgriffs_afk is now known as kgriffs17:52
kgriffsstupid intertubes17:52
kgriffsback online at a thai restaurant17:53
kgriffs(lunch)17:53
alcabrerammmm, thai...17:53
alcabrerawhich reminds me - I should go heat up my lunch.17:53
alcabreraI'll brb17:53
*** alcabrera is now known as alcabrera|afk17:53
*** alcabrera|afk is now known as alcabrera17:55
* kgriffs kgriffs is rebasing ALL THE THINGS!17:56
alcabrerakgriffs: go, kgriffs, go! :D17:56
*** rongze has quit IRC17:58
flaper87kgriffs: go go go!18:09
* flaper87 will have authentic Italian pizza for dinner!18:09
openstackgerritKurt Griffiths proposed a change to openstack/marconi: chore: Pass cache into drivers  https://review.openstack.org/5444518:10
openstackgerritKurt Griffiths proposed a change to openstack/marconi: feat(sharding): Cache shard mappings  https://review.openstack.org/5494518:10
alcabrerakgriffs: awesome!18:12
alcabreraflaper87: delicious!18:12
alcabreraI'll review those cache patches very soon.18:12
kgriffsthose two are ready for review18:12
alcabreracache patch... cache patch... say that fast 4 or 5 times!18:12
kgriffsLOL18:12
kgriffsI think this channel wins for most lolz/day18:13
flaper87LOOOOOOOOOOOOL18:14
flaper87huahuahua18:14
kgriffsI think the foundation should track that metric18:14
flaper87alcabrera: that must be tweeted18:14
kgriffsvital stat18:14
kgriffs;)18:14
flaper87kgriffs: hahaha18:14
flaper87I've always said MArconi channel is the coolest, funiest and happiest channel throughout openstack18:15
flaper87kk, brb18:15
flaper87dinner18:15
kgriffsI'd love to see a LOL graphite18:15
mpanetta_lunchMmm Thai is right.18:16
alcabrera:P18:16
*** mpanetta_lunch is now known as mpanetta18:16
kgriffsthen we could be like, "hey guys, looks like our LOLz are trending down this past month. we need to buckle down and get those numbers back up"18:16
mpanettakgriffs: (LOL)18:16
kgriffsWe'll have to hire ourselves a certified Lolz Master to keep us on track.18:17
alcabrerakgriffs: haha18:17
alcabreraROFL-driven development18:18
kgriffsLOOOOOOOOL18:18
alcabrera"have no doubts - team morale is always on the up"18:18
thomasemContagious cheer in this IRC channel.18:19
thomasemI must admit.18:19
mpanettaWeee!18:19
alcabrera:)18:19
* kgriffs is starting to wonder if he's had too much caffeine for the day18:20
* thomasem is betting on it.18:20
thomasem=P18:20
mpanettaUnless the merge is done, no you have not had enough :P18:20
thomasemLol18:21
alcabreraah, yes - reviews.18:21
alcabreraI'll do that now.18:21
kgriffsmpanetta: guess I better go get another coke18:21
kgriffs;)18:21
alcabrerakgriffs: review for a review? Those 5 patches are one +2 away from making marconi happier for everyone. ;)18:21
kgriffsah yes18:21
kgriffsI will do those right after mine are merged, or in between rebasing again, whichever comes first18:22
alcabrerakk18:22
kgriffsflaper87: I can haz teh reviewz?18:23
alcabreraI keep expecting docstrings to contain cat-english since it comes up on this channel from time to time.18:24
alcabrera"Pass teh cachez in for lower latencies."18:24
kgriffsalcabrera: don't tempt me!18:24
alcabreraThings like that.18:24
alcabrera:P18:24
kgriffsROFL18:24
kgriffs"...for teh lower latenzees"18:25
alcabreralol18:26
* kgriffs goes back to reviewens teh codez18:27
alcabrerakgriffs: both patches look good to me. +218:30
mpanettaDoo eeet!18:30
*** mpanetta has quit IRC18:31
kgriffsflaper87: get back here and do some reviews!18:32
kgriffsAnd bring some pizza with you!18:33
alcabreraflaper87 just needs to give his blessing after that delicious italian pizza's all gone. :P18:33
* kgriffs wants pizza18:33
*** openstackgerrit has quit IRC18:33
*** openstackgerrit has joined #openstack-marconi18:33
kgriffsok guys, I'mm making my way over to the office. be back in a few18:33
alcabrerakgriffs: kk18:35
*** gordonsim has left #openstack-marconi18:37
*** kgriffs is now known as kgriffs_afk18:43
*** rongze has joined #openstack-marconi18:49
*** sandywalsh_ has joined #openstack-marconi18:53
*** rongze has quit IRC18:54
flaper87back18:56
alcabreraflaper87: w00t18:57
alcabreraI'm about to head to a meeting.18:57
alcabrerakgriffs_afk: wanted you to check the cache patches whenever you got a chance. :)18:57
flaper87alcabrera: kk, any urgent patch to review?18:57
alcabreraI hope your pizza was *awesome*, by the way.18:57
alcabrerajust the cache patchews.18:57
alcabrera*patches18:57
flaper87it was great. Spicy!18:58
flaper87:D18:58
alcabrerayum!18:58
alcabreraI'll probably be back in an hour. If I miss you in the process, I hope the rest of your night goes well, flaper87. :d18:58
alcabrera:D18:58
flaper87Usage: yum [options] COMMAND18:58
alcabreralol :P18:59
flaper87alcabrera: thanks buddy, you too!18:59
flaper87take care18:59
*** alcabrera is now known as alcabrera|afk18:59
*** amitgand_ has joined #openstack-marconi19:02
*** mpanetta has joined #openstack-marconi19:03
*** amitgandhi has quit IRC19:05
*** kgriffs_afk is now known as kgriffs19:11
*** mpanetta has quit IRC19:13
*** mpanetta has joined #openstack-marconi19:13
*** reed has quit IRC19:25
*** reed has joined #openstack-marconi19:28
*** rongze has joined #openstack-marconi19:51
*** rongze has quit IRC19:56
*** jcru has quit IRC20:01
*** jcru has joined #openstack-marconi20:12
*** alcabrera|afk is now known as alcabrera20:22
openstackgerritKurt Griffiths proposed a change to openstack/marconi: fix(mongo): Delete claimed message fails  https://review.openstack.org/5500120:46
*** amitgandhi has joined #openstack-marconi20:48
alcabrerakgriffs: +2'd20:48
alcabrerakgriffs: I won't be around tomorrow. I've reviewed quite nearly every patch in the queue. Anything else you'd like me to keep in mind while I'm away?20:49
alcabreraponder, consider, retweet, etc.? :P20:49
kgriffsheh20:49
kgriffsnah, I think we're cool. If have a few minutes tomorrow morning to check on my patches to see if they need a final approval, that would be great.20:50
alcabreraI don't think I'll be able to, since I'll be at the UGA vet getting my cat thoroughly checked up, but...20:51
alcabrerayou guys (flaper87) can ninja approve whatever I've +2'd20:51
alcabrera^^ consensus grant20:51
alcabreras/consensus/consent20:51
flaper87alcabrera: AWESOME! :D20:51
alcabreraI've reviewed those through and through and I'm not expecting much to change. :)20:51
alcabreraanyway~20:52
*** amitgand_ has quit IRC20:52
alcabreraI'm out for the day.20:52
alcabreraStay lively, guys. :D20:52
mpanettasee ya20:52
alcabrerampanetta: o/20:52
*** alcabrera has quit IRC20:53
*** rongze has joined #openstack-marconi20:53
*** rongze has quit IRC20:58
*** jcru has quit IRC21:00
*** jcru has joined #openstack-marconi21:09
*** sandywalsh_ has quit IRC21:10
*** sandywalsh_ has joined #openstack-marconi21:27
*** rongze has joined #openstack-marconi21:55
*** rongze has quit IRC22:00
*** reed has quit IRC22:06
*** reed has joined #openstack-marconi22:07
*** reed has quit IRC22:10
*** reed has joined #openstack-marconi22:10
*** ayoung has quit IRC22:14
*** tedross has quit IRC22:21
*** thomasem has quit IRC22:27
*** fifieldt_ has joined #openstack-marconi22:52
*** rongze has joined #openstack-marconi22:57
*** rongze has quit IRC23:02
*** fifieldt_ has quit IRC23:05
*** fifieldt_ has joined #openstack-marconi23:07
*** ametts has quit IRC23:11
*** mpanetta has quit IRC23:17
*** mpanetta has joined #openstack-marconi23:30
*** amitgandhi has quit IRC23:32
*** jamieh has joined #openstack-marconi23:35
*** oz_akan_ has quit IRC23:37
*** oz_akan_ has joined #openstack-marconi23:38
*** oz_akan_ has quit IRC23:42
*** jamieh has quit IRC23:48
*** openstackgerrit has quit IRC23:56
*** openstackgerrit has joined #openstack-marconi23:56
*** rongze has joined #openstack-marconi23:59

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