[14:31:44] mooooorning [14:32:29] mornin [15:16:54] sorry connection problems. drdee, erosen, how'd the demo go [15:17:02] milimetric: pretty well [15:17:07] everything worked as expected [15:17:13] cool, wanna hang out and talk next steps? [15:17:35] yeah [15:17:37] let's do it [15:20:41] yooooo [15:44:02] New patchset: Milimetric; "fixes the namespace edits problem" [analytics/wikimetrics] (master) - https://gerrit.wikimedia.org/r/70827 [15:44:39] Change merged: Milimetric; [analytics/wikimetrics] (master) - https://gerrit.wikimedia.org/r/70827 [16:15:50] erosen: I'm gonna do lunch so I'll see you at standup [16:16:06] sounds good [16:30:02] oof i need to do that too [16:30:08] back in a bit [16:41:02] tnegrin: did you put a public ssh key on office wiki? if so, linky please! [16:41:05] i'll put it on the RT ticket [16:41:16] ah no -- give me a few minutes [16:42:46] ok, cool, we're going to have to wait for 3 days anyway [16:42:47] ops policy [16:43:01] and it might be difficult to show you around until you have shell access [16:43:07] you can join for when I show adam today, if you like [16:43:12] but you won't be able to sign in yourself [16:50:18] ok - it turns out that that meeting conflicts with other meetings that I need to attend [16:52:21] ok [17:00:20] erosen hii [17:00:21] standup [17:12:24] erosen: let me know when you're done with zero stuff and we can hangout [17:12:46] milimetric: k, will do [17:12:59] I'm just deploying them now [17:13:27] milimetric: actually do you have a second to switch to limn mode? [17:14:35] sure [17:14:44] definitely [17:15:02] two (hopefully) simple questions [17:15:20] how do I force the time ticks to be months? [17:27:09] New patchset: Erik Zachte; "New script for breakdown of uploaders per month 'new/repeat uploaders' plus 'edited earlier on other wikis or not'" [analytics/wikistats] (master) - https://gerrit.wikimedia.org/r/70850 [17:29:26] Change merged: Erik Zachte; [analytics/wikistats] (master) - https://gerrit.wikimedia.org/r/70850 [17:35:25] drdee: tnegrin: sent e-mail with updated dashboards [17:36:34] looking at them now! [17:37:48] want to join the hangout? [17:37:59] https://plus.google.com/hangouts/_/2da993a9acec7936399e9d78d13bf7ec0c0afdbc [17:55:24] tnegrin: could you open up the Review UMAPI mocks event and add a hangout? [17:55:30] it's a link off the event details page [17:55:33] y [17:55:48] ty :) [17:57:02] https://plus.google.com/hangouts/_/408fbd078a629a90392dea3b6d03737fe542318f [17:57:15] link to hangout -- also in updated invite [17:57:21] cool, thx [19:03:25] tnegrin, drdee, Evan and I are in standup talking next steps [19:03:27] just fyi [19:03:59] cool -- I need to go see my doctor right now [19:24:27] Drat. I missed tnegrin by a few minutes :-( [20:01:08] drdee, ottomata, erosen, standup hangout for the post mortem? [20:01:12] ahngout? [20:01:13] oh standup [20:01:27] toby isn't around and he didn't add a hangout to the meeting [20:03:40] hmm i can't find adrian either [20:26:49] back [20:26:52] hi [20:26:59] got internet connection stable [20:27:08] tnegrin: ^^ [20:27:30] hey stefan! [20:27:34] read e-mails, I'm up-to-date with 738, I know Erik mentioned it was sync-ing to stat1 instead of stat1002 [20:27:43] good to have you back [20:27:47] tnegrin: hi Toby [20:28:02] thank you [20:28:06] hey -- in a meeting now, chat in a bit? [20:28:12] sure [20:31:15] New patchset: Milimetric; "TODOs cleanup" [analytics/wikimetrics] (master) - https://gerrit.wikimedia.org/r/70914 [20:31:27] Change merged: Milimetric; [analytics/wikimetrics] (master) - https://gerrit.wikimedia.org/r/70914 [20:33:14] hey average! [20:33:46] average! we missed youuuuu [20:33:59] erosen, added another item in asana and assigned it to myself: "full round of flake8 style cleanup" [20:34:04] nice [20:34:09] hey average is back, yay :) [20:34:44] so i'll try to stay away from celery stuff for now, when you're done i'll style check that too [20:35:06] also, I'm just going to push until 696 is ready [20:35:09] git review is insane [20:35:54] hi drdee ottomata erosen milimetric [20:36:18] I missed you too, but couldn't get in touch, now I can :) [21:36:21] k erosen, I've pushed flake8 changes [21:36:35] nice [21:36:35] how did it go? [21:36:38] I've been bad at this but I'll try to make sure I run it before I commit [21:36:41] eh, just grunt work [21:36:46] did you figure out how to tell it to ignore specific lines? [21:36:49] about 45 minutes worth [21:37:03] no, i just ignore all the __init__ and fixed all the import * in specific files [21:37:19] cool [21:37:27] I believe PyCharm optimizes imports so I'll run that on it before we release maybe [21:37:39] how does it do that? [21:37:53] but flake8 is now completely silent when run on / so I'm happy :) [21:38:14] ideally gerrit would catch flake8 problems for us. Now *that* would make it worth its salt [21:38:23] anyway, nite for now, gotta go level a rain barrel [21:38:41] 0% rain turned into a tornado warning with 100% rain :) [21:43:28] woah [21:43:30] nice [21:43:42] assuming the tornado isn't too bad [21:52:11] What gerrit instructions on wikimedia.org should I be following for the initial push of varnishkafka? [22:09:17] Snaps: I'm not a very experienced gerrit person, but the main thing to think about is whether you use git review [22:09:41] Snaps: here is the intro tutorial (intended for a pretty broad audience) that has the git review instructions in it: http://www.mediawiki.org/wiki/Gerrit/Tutorial [22:10:06] Snaps: this part: http://www.mediawiki.org/wiki/Gerrit/Tutorial#Prepare_to_work_with_gerrit [22:10:27] but maybe you were asking something more specific ... [22:11:55] Snaps: 1 sec [22:12:33] I found a bunch of tutorials, and they didnt really match eachother.. Thats one of them [22:12:58] but if thats the best one, I'll follow that :) [22:13:26] ottomata; around? [22:13:44] can you add a .gitreview file to operations/varnishkafka.git repo? [22:14:14] i can't push it because i am no ops man [22:14:50] paravoid created a new repo [22:14:52] Snaps will fix this tomorrow, you can't push right now [22:15:01] its operations/software/varnish/varnishkafka now [22:15:21] drdee: is a .gitreview file is strictly required? [22:15:54] yes [22:16:48] ori-l: really? I feel like I've done gitreview without a .gitreview file by manually adding gerrit as a remote [22:17:12] oh, you mean to push changes for review? yes, it's not a technical requirement, just a social one [22:17:22] i think i can add it if you want [22:18:22] ori-l: sorry to pry, but how does the presence of the gitreview file effect the review process (assuming you use git review in both cases)? [22:18:45] ori-l: please do, to the new operations/software/varnish/varnishkafka repo [22:22:10] Thanks :) [22:23:13] erosen: you can't use 'git-review' (the actual tool) without it [22:23:26] tho you can 'git push gerrit HEAD:refs/for/master' [22:23:29] Snaps: np [22:24:20] ori-l: intesting. I'll take your word for it, but for some reason I thought i was able to do that [22:30:30] erosen: try it; i could be wrong [22:30:48] ori-l: will do next time I submit something for review [22:30:57] Its pushed now, thanks for your help guys [22:31:41] New patchset: Stefan.petrea; "Squashed commit of the following:" [analytics/wikistats] (master) - https://gerrit.wikimedia.org/r/70929 [22:34:34] New patchset: Stefan.petrea; "New mobile goes to stat1002 instead of stat1001" [analytics/wikistats] (master) - https://gerrit.wikimedia.org/r/70929 [22:39:07] sent out e-mail on 738 [22:39:21] patchset above ^^ [22:39:38] this should wrap it up. Hope Erik has a look at it tommorow [22:40:01] switching back to dclass [23:17:38] milimetric: you around?