[00:08:04] (03PS1) 10Jdlrobson: Fix QUnit in Chrome [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91537 [00:17:07] kaldari, MaxSem: how did the LD go? [00:17:31] seems like it's working [00:17:44] is logmsgbot dead? [00:17:55] heh [00:20:33] far out, thanks kaldari and MaxSem : [00:20:34] :) [00:21:34] awjr: only tested on test.wiki so far, syncing to other wikis now [00:23:05] kk [00:24:19] awjr: it's synced everywhere now, and seems to be working [00:25:13] \o/ [00:29:13] awjr: hmm, actually it doesn't look like everything is actually updated [00:29:40] need to wait for RL cache to refresh perhaps? [00:29:45] but it is firing event logging events [00:30:14] are you seeing that in console or in the datastream? [00:30:23] console [00:30:47] it looks like the cherrypick didn't actually pick up the change for some reason [00:30:54] gimme a minute... [00:31:50] maybe a touch and rescyn? [00:31:54] *resync [00:31:57] the config change went out too? [00:34:05] awjr: I think you copied "Test agile hybrid template" to set up Flow in mingle. In them and in test___scrum_template, the "How does this velocity chart work?" link to the wiki is a 404. [00:34:57] that is lame [00:35:05] ah, found a working one at https://wikimedia.mingle.thoughtworks.com/projects/mobile/wiki/How_does_this_velocity_chart_work%3F [00:35:06] unless they are expecting you to create the wiki page yourself, spagewmf? [00:35:32] (03CR) 10CSteipp: "(1 comment)" [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/86820 (owner: 10Yurik) [00:36:06] MaxSem: you still around? [00:36:12] yup [00:36:17] how do you feel about merging https://gerrit.wikimedia.org/r/#/c/91058/2 ? [00:36:24] kaldari: get it worked out? [00:36:50] awjr, can do if kaldari deploys [00:37:20] ah yeah maybe he has his hands too full now; it can wait til tomorrow - it's already been broken for basically ever [00:41:21] kaldari, need help with this stuff? [00:41:30] I might in a sec [00:44:08] MaxSem, awjr: sorry about that. I just restored the config stuff in mobile.php for now, to make sure it keeps working while I figure out why the cherrypick didn't work [00:45:21] MaxSem: so I thought I cherrypicked the change, but it didn't seem to get picked up. Here's one of my cherrypick commits: https://gerrit.wikimedia.org/r/#/c/91534/ [00:45:40] does that look right? [00:46:39] kaldari, I usually create MF branches corresponding to WMF ones and cherry-pick into them [00:47:01] ah [00:47:11] I've no idea if cherrypicking into submodules work [00:47:39] yeah, I guess it doesn't :P [00:47:50] because submodules are just pointers to tree state and you need your own tree for cherrypicks [00:47:55] I'll do it from the extension repo [00:59:48] MaxSem: I tried creating a deployment branch for the extension and updating .gitreview (like it says at https://wikitech.wikimedia.org/wiki/How_to_deploy_code#Creating_an_extension_deployment_branch), but whenever I try to do git push origin wmf/1.22wmf22 it gives me Authentication failure :( [01:00:42] kaldari, did you touch the existing wmf21 and wmf22 branches? [01:01:41] eariler, yes, but not just now [01:01:44] kaldari: what does git config —get remote.origin.url show? [01:02:04] (that's two dashes) [01:02:30] https://gerrit.wikimedia.org/r/p/mediawiki/extensions/MobileFrontend.git [01:02:52] kaldari: i think you cloned a read-only copy [01:02:56] the proto should be ssh [01:03:21] kaldari: actually i take that back [01:03:27] that's the origin url for the submodule? [01:03:27] dammit, I'm too sleepy to be efficient now [01:03:36] (03CR) 10Dr0ptp4kt: [C: 04-1] "(4 comments)" [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/90519 (owner: 10Yurik) [01:03:53] that's the origin from within MobileFrontend in my MediaWiki deployment branch [01:04:09] ok [01:04:16] apparently I don't know how to do this :( [01:04:27] what does 'git branch' show you? [01:04:36] wmf/1.22wmf22 [01:04:52] and you created the branch with —track to track the remote? [01:04:58] yes [01:05:30] (03PS1) 10MaxSem: Define EventLogging schemas in MobileFrontend [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91545 [01:05:34] I've never actually done a cherry-pick of an extension before, so maybe I'm doing something wrong [01:05:45] kaldari, ^^^ [01:05:54] do we need to do wmf21? [01:06:10] bleh, yes [01:06:27] (03CR) 10Kaldari: [C: 032] Define EventLogging schemas in MobileFrontend [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91545 (owner: 10MaxSem) [01:06:31] (03PS1) 10MaxSem: Define EventLogging schemas in MobileFrontend [extensions/MobileFrontend] (wmf21) - 10https://gerrit.wikimedia.org/r/91546 [01:06:43] (03CR) 10MaxSem: [C: 032 V: 032] Define EventLogging schemas in MobileFrontend [extensions/MobileFrontend] (wmf21) - 10https://gerrit.wikimedia.org/r/91546 (owner: 10MaxSem) [01:06:44] MaxSem: yeah, both unfortunately [01:07:07] (03Merged) 10jenkins-bot: Define EventLogging schemas in MobileFrontend [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91545 (owner: 10MaxSem) [01:07:35] MaxSem: Sorry for my FAILs. Maybe next time you can walk me through it so I can figure out what I'm doing wrong [01:07:44] i've never actually tried to create a remote deployment branch from the command line before; you can (or at least used to be able to) do it from gerrit [01:07:59] kaldari, no problem - we're almost done [01:08:08] it's also possible you need special permissions to do it for the project [01:08:37] kaldari: take a look at https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/MobileFrontend,branches [01:08:50] that's the easiest way to manage the remote branches [01:09:33] ah [01:09:56] kaldari, I've created the submodule commits - can you deploy them? [01:10:57] I don't risk deploying myself at 5:10 am [01:11:18] MaxSem: sure... [01:14:14] ok, it's on test.wiki now [01:14:18] checking it [01:14:57] still works :) [01:15:02] deploying elsewhere [01:20:28] arg, it still wasn't picked up on tin [01:20:37] kaldari@tin:/a/common/php-1.22wmf22$ git submodule update --init --recursive extensions/MobileFrontend/ [01:20:37] Submodule 'extensions/MobileFrontend' () registered for path 'extensions/MobileFrontend' [01:20:38] fatal: reference is not a tree: f8e8b5b8695d61f46ea03f4b3d9e3f116313ebe5 [01:20:39] Unable to checkout 'f8e8b5b8695d61f46ea03f4b3d9e3f116313ebe5' in submodule path 'extensions/MobileFrontend' [01:21:23] kaldari: did you git fetch and rebase? [01:21:24] why init and recursive? [01:21:34] awjr: yes [01:22:03] er try again kaldari? [01:22:10] MaxSem: doesn't work regardless [01:22:22] did you do the git log and make sure the commit was there? [01:22:41] awjr: yeah, looks like it [01:22:46] :| [01:22:48] it's not seeing the commit [01:23:00] ah - did the commit get merged? [01:23:03] for that specific branch [01:23:04] ? [01:23:12] i've totally done that before - merge one branch but forget to merge the other [01:23:33] yeah the —init —recursive thing shouldnt be needed for MobileFrontend but it's in the documentation as some extensions need it [01:23:52] kaldari, the problem is that my commits were just merged [01:24:07] https://gerrit.wikimedia.org/r/#/c/91548/ [01:24:24] never trust jerkins when deploying, always set V+2 and submit, or it's going to take a looong time [01:24:33] ahha [01:24:42] so if you try again, should work kaldari [01:24:43] ok, lemme try it agian... [01:25:18] no, still not seeing the commit [01:25:23] yeah the deployment branches are the one place it's actually acceptable to hit 'submit' rather than just +2 [01:25:27] kaldari, I've updated code on tin [01:25:32] or so i've been told [01:26:25] MaxSem: how'd you get it to work? [01:26:56] pull && submodule update, nothing special [01:27:16] your mistake was that you attempted to deploy before stuff got merged [01:27:19] maybe you did it before my 2nd try [01:27:29] yeah, I guess so [01:27:52] alright, if this doesn't work, let's scrap it and deal with it tomorrow [01:28:00] it's way too late for everyone - especially MaxSem [01:28:27] kaldari, are you pushing or I should? [01:28:49] I can push it [01:28:52] hopefully :) [01:32:11] ok, those are both pushed out [01:32:26] and everything seems to be running still [01:32:39] now I'll do the config change again, which is the easy part [01:36:34] OK, that's done too [01:36:52] testing [01:37:17] still firing events [01:37:32] OK, I think everything is good now [01:37:36] huzzah! [01:37:39] wee [01:37:59] * MaxSem hits the pillow [01:38:01] thanks kaldari and MaxSem [01:38:03] night night MaxSem [01:38:09] MaxSem: thanks for rescuing the deployment. I hope I can figure out how to do that correctly in the future [01:38:52] kaldari: real quick - were youa ble to verify events properly coming into the data stream? [01:39:01] i dont have access to the data so i can't dbl check [01:39:02] yeah [01:39:05] far out [01:39:07] nice work [01:39:23] awjr: oh I thought you were going to tell me how to do that :) [01:39:39] ! [01:39:53] I only know how to check the events are firing [01:40:26] ok [01:40:31] i dont know myself [01:40:42] would I run a query on stat1? [01:40:57] kaldari: send an email out to the team, let everyone know the current state of things and ask for folks to verify everything is OK [01:41:04] kaldari: yeah i believe the data goes to stat1 but i dont know details [01:41:06] awjr: good idea [01:42:06] oh kaldari [01:42:11] yeah [01:42:13] i think i may have found an email on the swubject [01:42:30] oh nope, it's for campaign tracking; different thing [01:43:10] alright, i have no idea. events firing is good but only half the battle. [01:43:28] so, shoot out the email and if you can keep an eye out for responses [01:43:29] true [01:43:50] i have to jet but i'll be back in an an hour and a half and i can check back then [02:32:47] blerch [07:04:29] tfinc: around? [08:33:51] playing with varnish on mobile01 . beta [10:49:22] (03CR) 10Mark Bergsma: [C: 031] "Adding cookies is only problematic for caching if the cache needs to Vary on this cookie. This doesn't appear to be the case here." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91401 (owner: 10Kaldari) [15:52:14] (03CR) 10Krinkle: "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91537 (owner: 10Jdlrobson) [15:53:50] (03CR) 10Krinkle: "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91537 (owner: 10Jdlrobson) [16:08:07] (03PS1) 10Jdlrobson: Regression: Fix login logo [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91615 [16:23:35] hrmmmmmmm [16:23:37] http://wiki.verkata.com/license/ [16:23:43] > This site, WikiPedia Mobile, uses Verkata's Content Transcoder Technology to reformat content so that it can be viewed on mobile devices. Content is optimized for the phones of the mobile operator Helio. [16:24:21] it's apparently a guy in SF even [16:52:48] jeremyb, we're moving away from HTML reformatting [16:55:21] ahaha, http://wiki.verkata.com/en/wiki/2013_Harbin_smog <-- edit links still there [16:59:46] https://en.wikipedia.org/w/index.php?title=Wikipedia:Mirrors_and_forks/Vwxyz&diff=578571054&oldid=548239089 [17:00:22] MaxSem: special:random is served in place, doesn't redirect [17:00:23] :P [17:00:34] sometimes looks like vector, sometimes not [17:04:17] (03CR) 10Awjrichards: [C: 032] Regression: Fix login logo [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91615 (owner: 10Jdlrobson) [17:04:18] brion: gah, looks like i've been missing emails from mobile-feedback-l because mailman decided to automatically unsubscribe me :| [17:04:30] dang [17:04:51] (03Merged) 10jenkins-bot: Regression: Fix login logo [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91615 (owner: 10Jdlrobson) [17:05:14] brion: need to get re-added [17:05:30] brion: also, i had apps/android/java-mwapi created. Doing a new js api [17:06:32] brion: any complaints from the commons release so far? [17:06:49] not that i've noticed [17:07:23] ah we should go ahead and create apps/android/wikipedia and apps/ios/wikipedia so they're ready to start committing :D [17:08:08] brion: heh, yeah [17:08:24] brion: also, I want to enforce some jenkins things for the new repos [17:08:32] brion: prepped the one for java-mwapi, will commit shortly [17:08:40] awesome [17:09:02] brion: mostly around tests, javadoc and such [17:11:05] (03PS1) 10Yuvipanda: Initial commit with appropriate project structure [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91628 [17:11:08] brion: ^ [17:11:33] brion: 1. TDD, so I first write tests, then stub them out to get them to compile, then write the code, 2. Checkstyle checks for plenty of things [17:11:44] ok reqs to end of https://www.mediawiki.org/wiki/Git/New_repositories/Requests [17:11:54] nice [17:12:24] brion: thoughts on mandatory JavaDoc? [17:12:29] I have been doing it and I quite like it [17:12:34] i say yes [17:12:37] docs are good [17:12:41] similar to forcing git commit messages [17:12:42] it's good to stay in the habit :D [17:12:44] yeah, agreed brion [17:12:55] brion: so I am going to make checkstyle voting on jenkins [17:13:06] niiiice [17:13:41] brion: it'll also stick to the mw guidelines where possible ' if ( than if(' for example [17:13:51] brion: but those also exist in the Sun java style guidelines, so it's ok [17:13:58] killed the 80 char limits tho [17:13:59] :P [17:14:04] heh [17:14:08] we ain't using no teletypes [17:14:11] we can always tweak it later if we feel like [17:14:12] indeed [17:14:32] brion: want to +2? :D [17:14:38] i wish java had map and array list literal primitives :) [17:14:41] looking it over right now [17:14:47] brion: heh, ok [17:15:32] (03CR) 10Brion VIBBER: [C: 032 V: 032] "woo" [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91628 (owner: 10Yuvipanda) [17:17:05] now lemme make sure i can build it :D [17:17:12] hehe :D [17:18:28] whoops gotta reinstall mvn [17:18:40] brion: hehe [17:19:08] "BUILD SUCCESS" [17:19:51] brion: try intellij? [17:19:59] i'm wondering if we should ship the IML files [17:23:16] ok that was weird [17:23:25] intellij wasn't showing the directory in the import dialog for like a minute [17:23:34] i see [17:23:43] some sort of inotify-in-os-x equvialent bug? [17:23:50] likely yeah [17:23:54] weeeeird [17:23:58] ok it opened, let's try build [17:24:47] YuviPanda: trying to figure out if i can make a run config for the tests... [17:25:04] brion: yeah, there's a junit runner thing that worekd for me [17:27:30] YuviPanda: ok i seem to be able to make a junit runner with *either* the ApiTests or the ApiConstructionTests but not both? [17:27:38] YuviPanda: brion mhurd : how are our releases looking ? [17:27:42] yeah sharing the .iml might be easier, if it doesn't include per-user crap :D [17:27:55] tfinc: i haven't heard any pitchforks yet so that's a good sign :D [17:27:56] brion: it sadly does, and changes for tiny things (like re-arranging your damn window) [17:28:03] damn [17:28:29] tfinc: what brion said [17:28:45] assuming that holds lets do a market push at the same time as yesterday then [17:28:57] tfinc: I looked for reviews (none yet) but apparently i have been unsubscribed from mobile-feedback-l for a while automatically(!?!) and hence have missed a few crash reports [17:29:00] tfinc: sounds good [17:29:13] tfinc: can you re-subscribe me? [17:29:21] hmmi may have to fix my sub to that as well [17:29:41] YuviPanda: why can't you just subscribe ? its just a mailman list [17:29:48] tfinc: nope, it's a private list [17:30:01] you added me the first time, IIRC [17:33:25] brion: https://bugzilla.wikimedia.org/show_bug.cgi?id=56110 [17:33:39] brion: can you try running checkstyle? [17:33:50] brion: brew install checkstyle, then checkstyle -c checkstyle.xml -r src [17:34:35] Successfully subscribed: ypandian@wikimedia.org [17:34:36] YuviPanda: [17:34:46] tfinc: hmm, okay [17:35:21] YuviPanda: it whinges about a couple things in tests [17:36:48] brion: in the tests directory? [17:36:51] brion: i hadn't fixed those yet [17:36:56] yeah [17:36:58] ok :D [17:37:17] brion: but I geuss this is reasonable enogh [17:37:24] brion: don't want to have unreasoanble coding standards :D [17:37:28] ha [17:37:33] brion: also there are checkstyle plugins for IntelliJ too [17:38:06] nice [17:47:27] brion: i forgot how much fun untangling retain cycles can be... uuuhhhhhh [17:51:08] (03PS1) 10Robmoen: Modify epoch test to restore original url on teardown [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91635 [17:54:15] awjr: jdlrobson I was trying to reply with https://wikitech.wikimedia.org/wiki/EventLogging for the thread about how to verify if events are coming in, but it just bounced :| [17:54:45] huh, i got it YuviPanda but it looksl ike you sent from your gmail account which might be why it bounced [17:55:10] awjr: hmm, cross posting between mobile-tech and analytics-l, I think. it went through to analytics [17:55:22] awjr: essentially, you can do 'zsub vanadium.eqiad.wmnet:8600 | grep ' and watch [17:55:55] marktraceur: have i broken anything so far today? :) [17:56:03] YuviPanda: from somewhere on the cluster (eg fenari/tin)? [17:56:03] Not to my knowledge [17:56:05] I can check [17:56:12] marktraceur: hehe [17:56:18] awjr: yeah, I remember running it from stat1 [17:56:25] YuviPanda: can you add that :) [17:56:28] awjr: since that's all I have access to, but rest of cluster should work too [17:56:36] mhurd: Confirmed not banned at commons. :) [17:56:43] it would've taken us a while to figure that out from this documentation [17:56:44] awjr: it's kinda obvious, because of the .wmnet thing, no? [17:56:45] marktraceur: haha yay! [17:56:53] awjr: ah, yeah. [17:57:17] YuviPanda: a "here's you double check daata's coming in" section would be useful for quick needs [17:57:22] thanks :) [17:58:06] MaxSem: add education reqs to https://office.wikimedia.org/wiki/Job_descriptions/Software_Developer_Backend_(Mobile) [17:59:30] tfinc, doned [17:59:37] wow, i was looking for an image of a 'hat tip' and there's a whole category on commons called 'hat tipping' [18:03:10] awjr: check https://wikitech.wikimedia.org/wiki/EventLogging now [18:05:52] awjr: slide 18 still needs love [18:06:33] it will be done :) [18:07:06] tfinc: headsup I might not last all of the quarterly meeting, it goes on until 4:30 AM. Will try to stick on for as much as possible tho [18:07:34] thanks YuviPanda [18:07:47] (03PS1) 10Kaldari: Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 [18:07:58] we'll break it up into two parts. feel free drop off on the second as it'll be very data heavy [18:08:12] (03CR) 10jenkins-bot: [V: 04-1] Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [18:08:53] https://gerrit.wikimedia.org/r/#/c/91401/ didn't get merged :( [18:08:59] tfinc: ok [18:09:39] has deployment branch been cut yet Reedy? [18:09:42] Yup [18:09:44] k [18:09:51] tfinc: gail wants to climb tonight [18:10:08] kaldari: if we ened to get that out asap we can lightning deploy it [18:10:14] mhurd: yup, she mailed me [18:10:16] looks like we got ops' blessing [18:10:38] awjr: yeah, we need to have that deployed before Monday [18:11:40] ok kaldari [18:12:11] kaldari: wait - deployed everywhere, or just in sync with the deployment train? [18:12:38] deployed everywhere would be best [18:12:54] Guess I'll ping greg about another lighting deploy [18:13:22] kaldari: ok. i need to focus on quarterly review stuff until it's over this afternoon. can you work with MaxSem to get it merged and you guys figure out getting it deployed during LD today? of course be sure to schedule with greg-g as necessary [18:13:33] sure [18:13:36] sure [18:14:12] * greg-g waves [18:14:27] Something seems to be wrong with Jenkins. It can't merge this change even though it's completely rebased: https://gerrit.wikimedia.org/r/#/c/91640/ [18:15:12] I hope I didn't screw up the entire MobileFrontend repo yesterday [18:16:08] greg-g: Do you know if Jenkins is having any problems today? [18:17:17] (03PS2) 10Kaldari: Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 [18:20:32] oh well, seems to be fine now [18:20:35] kaldari: haven't looked specifically [18:20:39] ah, just slow? [18:21:28] greg-g: it gave a "Build failed" on a completely rebased patch. [18:22:04] greg-g: I deleted 1 whitespace character, resubmitted, and it worked. [18:22:21] without doing another rebase [18:22:32] multichill|2, have you seen my response at https://bugzilla.wikimedia.org/show_bug.cgi?id=50857 ? [18:22:49] awjr: About your EventLogging question [18:22:56] YuviPanda: as long as your around for slides 19 - 31 to include anything that i forget then we'll be fine [18:23:12] tfinc: can you share the presentaiton iwth me? [18:23:14] *with [18:23:47] YuviPanda: check mobile tech [18:23:51] Was the concern that events were not getting recorded at all, or not replicated to tables on stat1 log db? [18:23:56] kenanwang mailed it out [18:24:16] tfinc: ah, got it [18:24:30] kaldari: huh, sorry, not sure how to debug that one [18:24:51] greg-g: no problem, at least it worked the 2nd time :) [18:25:07] tfinc: slide 18 done. [18:26:00] kaldari: if at first.. [18:29:40] (03CR) 10Jdlrobson: "has this change been run through a designer?" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [18:30:00] (03CR) 10Jdlrobson: "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [18:30:58] kenanwang: https://bugzilla.wikimedia.org/show_bug.cgi?id=53902 this needs your input asap (obviously after quarterly review) [18:31:46] awjr: haha .. ok [18:31:55] awjr: way to go with cheesy [18:32:12] my mom always said 'go with what you know' [18:32:23] kaity: can I run a design tweak by you? [18:41:06] kaldari, do you want to LD today? [18:41:22] maybe :) [18:41:45] cuz if it's today we need to prepare now due to the meeting [18:42:11] StevenW: the concern was that we weren't sure whether or not events were getting recorded and if they were getting recorded correctly. i felt it important to dbl check since we made a small bug significant change [18:42:37] since we just had the branch cut, would anyone object to a deploy from master, or would we want a cherry pick? [18:42:51] who added "* moretime from VE team to make skin, RL and workon JavaScript Ulibraries in core " to https://etherpad.wikimedia.org/p/MobileAsks ? [18:42:56] i need more detail [18:43:05] past with rmoen is supporting us on [18:43:14] awjr: Yeah, as far as best practices, we always have the data analyst and/or PM check the log db on stat1 for events getting recorded. [18:43:30] You can also record events from labs, or your local setup with vagrant [18:43:43] kaldari: jgonera jdlrobson --^ [18:43:58] StevenW: yah - usually when we deploy stuff like that folks who know what's up are around to check stuff like that, but we were in a situation last night where it was just kaldari and me, and we didn't know what to do :) [18:44:20] tfinc: me sorry [18:44:29] s/with/what [18:45:20] StevenW: fwiw, vagrant's local one doesn't actually work (yet?) and I don't know who all has access to the labs ones [18:46:58] YuviPanda: idk how it works, but in our log for most recent GettingStarted test (GettingStartedOnRedirect_5944134 is the table) you can see events from 127.0.0.1 [18:47:14] StevenW: oh, sure, sending events *from* vagrant works, just not *to* vagrant [18:47:27] StevenW: so the only current way to easily test is to send them to production and then check from stat1 [18:47:28] Ah yes. [18:47:48] Yep [18:48:12] you can hack up a local instance trivially (2-3 commands, and they are even documented!) but obviously it won't be the same pipeline [18:48:45] Anyway tfinc awjr: DarTar should be able to walk you through how to check the production data pipeline step by step. But the skinny is basically that you need to go look on stat1. [18:49:16] dang, computer crashed again and ate my scrollback [18:49:43] ty StevenW, i think YuviPanda's additions to the documentation on wikitech was a big help [18:50:03] MaxSem: since we just had the branch cut anyway, do you think we should just do the LD from master (rather than cherrypicking)? [18:50:17] why not [18:50:20] kaity: ping [18:50:23] (if jcmish approves) [18:50:54] kaldari: kaity is AFK. She may be interviewing someone I think. [18:50:55] if nothing else gets merged today, it might not matter anyway [18:51:14] well apart from pushing things to en.wiki sooner [18:52:13] mhurd: YuviPanda: any final fixes I should poke at in the time before the meeting? [18:52:33] brion: nope [18:53:28] kaldari: what was your question? [18:53:46] jgonera: Is it cool if I pick up the AbuseFilter bug or is Rob working on that? [18:54:03] kaity: I have a small design tweak I want to run by you for approval. [18:54:16] I'll come over in a minute [18:54:19] thanks! [18:55:05] kaldari, I think he's not, but check with him [18:55:40] rmoen: is it cool if I take the AbuseFilter throttling issue or are you working on that? [18:56:54] brion: no fixes. i did however vastly refactor the uploading for cancel-ability but it needs further cleanup and testing [18:57:04] woohoo [18:57:14] ok we'll poke that into a bug fix release later as time permits [18:57:21] brion: ya. won't be ready in time for the release [18:57:38] brion: but hope to have something for you to look at this afternoon [18:57:43] spiff [18:57:55] brion: pretty big diff [18:58:00] hehe [18:58:22] brion: should finally be rid of that retain cycle warning :) [18:58:26] yay! [19:00:00] mhurd: that reminds me; for the new reader app YuviPanda and I are thinking of using an event bus for the Android version to allow different parts of the app to send notifications to each other. I think for iOS NSNotificationCenter can provide this role right? [19:00:17] brion: yes [19:00:21] excellent :D [19:00:31] that's one less external library we need [19:04:12] greg-g: I signed up for the Lightning deploy window again. Hope that's OK :) [19:05:22] (03CR) 10Jdlrobson: [C: 04-1] "(3 comments)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91532 (owner: 10Robmoen) [19:09:52] kaldari: sorry, please do :) [19:15:20] and... he's gone [19:17:02] and... he's back [19:17:06] kaldari: yep, that's fine :) [19:17:37] dr0ptp4kt: did you call me? I was having a wikimeating with st petersburg [19:17:43] lovelly crowd [19:17:49] (03CR) 10Jdlrobson: [C: 04-1] "It shouldn't actually use the browser window.history at all. See https://gerrit.wikimedia.org/r/91537 - instead I stub it so it never chan" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91635 (owner: 10Robmoen) [19:21:48] arg. R66 is booked right up to the review. can't prep it early [19:21:52] awjr_lunch: --^ [19:23:15] (03CR) 10Jdlrobson: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/89283 (owner: 10Jdlrobson) [19:23:27] MaxSem: Looking, ehm, right. I'll have to check the database if it's also working in Dutch now [19:23:55] (03CR) 10Jdlrobson: "(I will fix these things up if no objection)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/89283 (owner: 10Jdlrobson) [19:24:04] multichill|2, closing as WFM? [19:24:24] hold on, let's see if I can find some articles that were broken [19:25:04] (03PS1) 10Yuvipanda: Remove checklist rule prohibiting TODO items from code [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91655 [19:25:05] (03PS1) 10Yuvipanda: Add code to actuallty perform API request when asked [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91656 [19:25:07] brion: ^ [19:25:36] brion: these aren't 'true' unit tests since they hit testwiki, but eh [19:25:46] (03CR) 10Brion VIBBER: [C: 032] "yeah if we forbid todos we'll never get anything merged ;)" [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91655 (owner: 10Yuvipanda) [19:25:55] YuviPanda: let's just call them integration tests ;) [19:25:58] brion: hehe [19:26:09] ok tfinc, hopefully it will 'just work' [19:26:11] brion: so, there exists http://wiremock.org/ [19:26:20] YuviPanda: is jenkins doing stuff yet or should i v+2 and submit manually for now? [19:26:21] brion: which is quite nice, and I want to integrate it inside at least at some point [19:26:30] brion: for now V+2 ourselves. I'll poke hashar tomorrow [19:26:34] ok [19:26:46] (03CR) 10Brion VIBBER: [V: 032] Remove checklist rule prohibiting TODO items from code [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91655 (owner: 10Yuvipanda) [19:27:57] brion: http://kevinsawicki.github.io/http-request/ is the HTTP wrapper library I'm going to use... for now, but I'll try to make sure it doesn't 'leak' out at all, so we can replace it easily later if needed [19:28:11] ok [19:28:22] YuviPanda: oh what was the library you were looking at that supports SPDY and whatnot? [19:28:44] brion: OKHttp [19:28:52] brion: that's a little lower level, we can easily plug that into this [19:29:05] ok [19:29:06] brion: okhttp is a terrible name to google for [19:29:15] lol [19:29:22] MaxSem: MariaDB [nlwiki_p]> SELECT * FROm geo_tags WHERE gt_page_id=714 LIMIT 1; [19:29:50] i'll have to check if there's a nice abstraction layer on iOS or if we should stick in a quick wrapper class that we can switch the backend if we need to later [19:30:03] want to make sure we can do stuff like that easy instead of changing bits all over the place [19:30:09] MaxSem: Hmm, looks like something else is broken, gt_type is now empty for everything [19:31:28] multichill, I see non-empty gt_type in enwiki, template problems? [19:31:32] (03CR) 10Brion VIBBER: [C: 032 V: 032] ""all tests passed" on ApiTests :D" [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91656 (owner: 10Yuvipanda) [19:31:54] Probably, I haven't looked at that for months [19:33:13] I'm using https://nl.wikipedia.org/wiki/Haarlem as an example. It has {{Coördinaten|52_22_49_N_4_38_26_E_type:city(146753)_region:NL-NH|52° 22′ 49″ N, 4° 38′ 26″ O}} [19:33:27] That uses https://nl.wikipedia.org/w/index.php?title=Sjabloon:Co%C3%B6rdinaten&action=edit [19:33:44] (03PS1) 10Jdlrobson: Throw JS exception on badfile rather than PHP exception. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91658 [19:34:01] {{#invoke:AddNearby|coordinates|{{{1|}}}_name:{{PAGENAME}}_primary:1}} gets expanded to {{#invoke:AddNearby|coordinates|2_22_49_N_4_38_26_E_type:city(146753)_region:NL-NH_name:Haarlem_primary:1}} [19:34:17] That goes to https://nl.wikipedia.org/wiki/Module:AddNearby [19:34:56] (03PS2) 10Jdlrobson: Throw JS exception on badfile rather than PHP exception. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91658 [19:35:24] So it seems to be translated and split by '_'. Is 'type:city(14675)' valid MaxSem? Or should it be type=city(123)? [19:35:48] multichill, depends on how you're using it [19:36:16] (03CR) 10Jdlrobson: [C: 04-1] "Remove the unnecessary padding change and feel free to merge." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [19:36:53] I just explained how I'm using it :P [19:37:01] can be {{#coordinates:x|y|type=city|zoom=3}} or {{#coordinates:x|y|type:city_zoom:3}} [19:37:30] if it's {{#coordinates:x|y|type:city|zoom:3}} the result is undefined [19:37:47] mebbe zoom overwrites type [19:38:02] you're not supposed to have more than 1 geohack parameter [19:38:09] I'm not using zoom am I? [19:38:44] ok i'm gonna push iOS 1.0.10 up to store, we'll see how long they take to review [19:39:10] {{#coordinates:x|2|22|49|N|4|38|26|E|type:city(146753)|region:NL-NH|name:Haarlem|primary:1}} [19:39:18] That's what I seem to be submitting [19:39:27] (minus the x) [19:39:50] multichill, in such case you need to join the json-like parameters [19:40:07] or, ideally, convert them to name=value parameters [19:42:07] or I could allow more than 1 geohack parameter:P [19:43:03] " Status Waiting For Review " [19:43:11] MaxSem: Better: LUA hook [19:43:22] wat? [19:43:32] See https://nl.wikipedia.org/wiki/Module:AddNearby [19:43:39] return frame:callParserFunction( '#coordinates', coordargs ) [19:43:43] buh [19:46:34] awjr: room was empty to i'd like to do a quick hangout test [19:52:53] kaity, the color for divider lines on your overlay spec is #adaeb1 which is something we don't use anywhere else. do we want to reuse #e2e2e2, #cacaca or #ccc that we use elsewhere? [19:53:51] or are we switching to a new palette here (if I can call it like that since they're all mostly gray ;) [19:54:51] tfinc k [19:55:03] tfinc: did you see the update to the schedule? [19:55:18] (03PS1) 10Yuvipanda: Add tests proving that POST works [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91662 [19:55:43] brion: ^ [19:56:03] awjr: heh, the meeting starts at 2AM and ends at 5AM now for me [19:56:05] tfinc im staring at your laptop [19:56:09] YuviPanda: >_< [19:56:19] (03CR) 10Kaldari: "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [19:56:36] awjr: without DST, that would've been 3AM to 6AM, so hey... :D [19:56:51] (03CR) 10Brion VIBBER: [C: 032 V: 032] "confirmed passes tests" [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91662 (owner: 10Yuvipanda) [19:57:36] brion: so, I'm going to figure out error handling (and write tests for it!), and that's about it, I guess? [19:57:40] brion: we can add login later on [19:57:41] (03PS3) 10Kaldari: Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 [19:57:45] spiff [19:57:56] brion: also want to support server side cacheing, I think [19:58:09] ooh fancy [19:59:00] MaxSem: aren't there params you can pass to the API to make it emit cache headers? [19:59:04] MaxSem: are those documented somewhere? [19:59:11] brb grabbing a bite to eat before mtgf [19:59:13] *mtg [19:59:58] YuviPanda, maxage & smaxage [20:00:08] MaxSem: right. is it documented somewhere? [20:00:17] c-f cache on API:Main page gives me nothing [20:00:25] I guess - I see it in sandbox;) [20:01:04] wat [20:01:06] lookgin [20:01:20] ah, generic paraemeters [20:01:22] thanks MaxSem [20:11:36] yurik_, nothing urgent. just was on a conference call with christian to go over some analytics stuff [20:11:59] a,ok [20:33:34] rmoen: nice jacket boss :D [20:33:43] YuviPanda: Thanks! [20:33:52] rmoen: i waved, you can't see tho [20:34:22] YuviPanda: Yeah i take it you are in the hangout but I cannot see you :/ [20:34:33] rmoen: I'm the black rectangle [20:41:47] (03PS10) 10Jdlrobson: Beta: Add Special:History [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/89283 [20:47:20] (03PS11) 10Jdlrobson: Beta: Add Special:History [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/89283 [20:51:07] (03CR) 10Jdlrobson: [C: 032] Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [20:52:26] kaity|away: care to comment on https://bugzilla.wikimedia.org/55895 ? [20:53:10] (03CR) 10Robmoen: "Sure. At the time, this was actually modifying the address bar and not restoring it. Stubbing it does however make more sense." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91635 (owner: 10Robmoen) [20:53:39] (03Abandoned) 10Robmoen: Modify epoch test to restore original url on teardown [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91635 (owner: 10Robmoen) [20:56:19] (03Merged) 10jenkins-bot: Making last modified text darker so that it is readable. [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91640 (owner: 10Kaldari) [21:07:48] kaity: kaity kaity? https://bugzilla.wikimedia.org/show_bug.cgi?id=55895 [21:09:19] kaity: ? [21:10:30] jdlrobson: ? [21:18:27] (03PS1) 10Yuvipanda: Testing (DO NOT SUBMIT!) [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91770 [21:23:43] (03PS3) 10Robmoen: Hygiene: Eliminate api fixmes from PageApi [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91532 [21:25:04] (03CR) 10Robmoen: "(3 comments)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91532 (owner: 10Robmoen) [21:28:02] (03PS1) 10Yuvipanda: Don't fail mvn checkstyle:checkstyle when checkstyle fails [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91771 [21:29:52] brion: so close :D https://integration.wikimedia.org/ci/job/test-checkstyle-maven/5/console [21:29:55] -1s coming soon [21:30:01] whee [21:30:42] brion: i am going to keep the 'First sentence should end with a period.' rule on for javadoc. Can sometimes feel annoying, but I find that it helps me write good comment messages, so I'm going to leave it on [21:30:53] That's a good idea. [21:31:12] brion: yeah, so it follows git commit messages convention of 'summry line\n\noptherstuff' [21:36:35] brion: I'm looking at code coverage tools now [21:44:08] jdlrobson: does it seem like it would might be easy to plug the webcam into another laptop (or the macmini in the room?) and join the hangout? it would be nice if we could still see the room. it's really hard to follow only seeing slides and hearing audio - i think eseing the conf room would help [21:52:32] (03CR) 10Robmoen: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91537 (owner: 10Jdlrobson) [21:54:37] or jgonera, rmoen - someone in the conf room ^^ [21:56:04] I can try, not sure if it will break the setup or not, I also don't have an ethernet dongle handy [21:56:48] or perhaps we could switch back to the camera, the slides are not changing now [21:56:58] awjr: that makes sense. i can join the hangout and plug in the camera into my mac [21:57:20] rmoen: that would be great - if you do that, please mute your laptop :) [21:57:58] awjr: jgonera is handling it because he is closer to the camera :) [21:58:04] :D [21:58:07] thanks jgonera! [21:58:17] ahhhh [21:58:18] yes [21:58:20] thank you :D [21:58:23] jgonera: also if you want the ethernet, I can reach it to you [21:58:50] it's really nice to be able to switch back and forth [21:59:20] For sure, sorry I didn't notice your comment earlier [22:02:47] (03CR) 10Jdlrobson: [C: 031] "I'm happy with Max's changes but now I want to know if he's happy with my changes.." [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/89283 (owner: 10Jdlrobson) [22:06:21] woo, my iPhone 5s has made it to the united states -- package scanned in alaska [22:06:24] should arrive monday :D [22:07:01] brion: nice [22:07:03] i'm dying to stress-test that A7 processor [22:07:16] brion: I wonder if I'll have to come and pick up the Macbook Pro by hand [22:09:11] YuviPanda: looks like the latest model with nvidia GPU isn't available for in-store pickup (yet?), so may want to ship it to the office [22:09:23] brion: yeah, should poke tfinc again [22:09:51] after the meeting :D [22:16:44] (03CR) 10Jdlrobson: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91532 (owner: 10Robmoen) [22:17:06] jdlrobson, I need to know if you're OK with this asap: https://gerrit.wikimedia.org/r/#/c/90686/ [22:18:09] (03PS1) 10Jdlrobson: Regression: Ensure last modified link renders consistently when dynamically loaded [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91785 [22:18:21] jdlrobson, kaldari, also https://gerrit.wikimedia.org/r/#/c/91526/, https://gerrit.wikimedia.org/r/#/c/90068/ whenever you have a moment [22:18:26] jgonera: i'm ok with it ;) [22:18:33] ^ kaldari jgonera [22:18:48] jgonera: i need to have a look at it still but from what i've heard i don't think i will have many major complaints [22:18:54] jgonera: i'm just messing with you [22:18:55] I see [22:20:14] jgonera: i'll take a look at the overlays and panelist cleanup tomorrow - i'd rather it didn't go out on the deployment train and i'm not sure when they cut the branch [22:20:43] ok, I don't need you to merge them today ;) [22:21:11] (03CR) 10Robmoen: [C: 031] "I approve :)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/90686 (owner: 10JGonera) [22:27:33] (03PS1) 10Yurik: Changed warning message for Zero [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/91787 [22:27:51] awjr: I'm going to take my headphones off for a bit, think you can ping me on IRC when they get back? [22:28:06] suer thing YuviPanda :) [22:28:09] ty [22:28:14] er [22:28:17] YuviPanda: they're back [22:28:18] (03CR) 10Jdlrobson: [C: 04-2] "Kaity is thinking about putting a consistent place to put links to see uploads and user contribution. Let's hold off merging that till the" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/90930 (owner: 10MaxSem) [22:28:20] lol [22:28:24] :p [22:29:37] (03PS3) 10Yurik: New warning page [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/90881 [22:31:54] jgonera: I'm reviewing the notifications changes now [22:32:28] thanks kaldari [22:33:06] dfoy: ping [22:34:53] (03PS1) 10Yuvipanda: Rename tests so that maven picks them up [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91789 [22:35:25] (03PS1) 10Yuvipanda: Add .gitignore [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91790 [22:35:35] brion: 3 small patchsets :) [22:35:39] brion: mvn test will work now [22:35:43] sweet [22:35:47] i'll look in a moment [22:36:22] haha found my first-gen ipad. thing is HUGE [22:47:25] (03PS2) 10Yurik: Simplified banner rendering and fixed empty partner URL [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/90566 [22:48:10] (03CR) 10Brion VIBBER: [C: 032 V: 032] Don't fail mvn checkstyle:checkstyle when checkstyle fails [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91771 (owner: 10Yuvipanda) [22:49:00] (03CR) 10Brion VIBBER: [C: 032 V: 032] "\o/" [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91789 (owner: 10Yuvipanda) [22:49:40] (03CR) 10Brion VIBBER: [C: 032 V: 032] Add .gitignore [apps/android/java-mwapi] - 10https://gerrit.wikimedia.org/r/91790 (owner: 10Yuvipanda) [22:50:25] (03PS5) 10Yurik: Allow zero banner on ALL wiki sites (if configured) [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/90519 [22:50:38] brion: ty [22:52:44] (03CR) 10Kaldari: [C: 04-1] "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91526 (owner: 10JGonera) [22:53:48] (03CR) 10JGonera: "(1 comment)" [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91526 (owner: 10JGonera) [22:55:16] tfinc: I think it was useful for me to stick around till the end [22:59:57] dr0ptp4kt: i did a few more patch fixes (was working on ESI) most of the time, and now time to snooze, gnight :) [23:01:27] tfinc: ? [23:01:43] tfinc is nowhere to be found [23:01:49] ha [23:01:49] k [23:02:04] jdlrobson, jgonera, MaxSem: I still need https://gerrit.wikimedia.org/r/#/c/91401/ merged. Otherwise there is nothing to lightning deploy. [23:02:25] (03CR) 10MaxSem: [C: 032] Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91401 (owner: 10Kaldari) [23:02:31] yeehaw! [23:02:41] I wanted someone else to look, too [23:02:44] but meh [23:02:53] jcmish: yurik_ : ignore :) [23:03:01] MaxSem: setting up a deployment branch through the gerrit interface is so much easier than trying to do it from git :) [23:03:03] (03Merged) 10jenkins-bot: Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91401 (owner: 10Kaldari) [23:03:08] hahahah [23:03:12] so you don't love us? [23:03:16] we now are sad [23:03:57] * yurik_ is not sad, yurik is falling asleep [23:04:02] gnight [23:04:02] hahaha [23:04:07] see ya yurik_ [23:06:41] tfinc, you called? [23:09:56] (03PS4) 10Robmoen: Hygiene: Eliminate api fixmes from PageApi [extensions/MobileFrontend] - 10https://gerrit.wikimedia.org/r/91532 [23:15:11] Hey MaxSem: So I'm trying to do a cherry-pick again, but using a deployment branch created in gerrit... [23:15:30] yep [23:15:35] what's the problem? [23:15:45] First I created the deployent branch from master earlier today, which you can see at https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/MobileFrontend,branches [23:15:54] wmf/1.23wmf1 [23:16:12] I created a tracking branch in MobileFrontend locally... [23:17:08] Then I checked it out, did git pull, did git cherry-pick 2e0af8409a8ae9c54ea94a9d68f2bd2c80aa3aa4 (which seemed to work), and then tried to do git review…. [23:17:22] git review wmf/1.23wmf1 [23:17:22] But I get the error: ! [remote rejected] HEAD -> refs/publish/master/wmf/1.23wmf1 (change 91401 closed) [23:17:36] ah, lemme try that [23:17:49] (03PS1) 10Kaldari: Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] (wmf/1.23wmf1) - 10https://gerrit.wikimedia.org/r/91797 [23:18:02] MaxSem: awesome! thanks! [23:18:39] MaxSem: so is the documentation at https://wikitech.wikimedia.org/wiki/How_to_deploy_code#Now_that_your_extension_has_a_deployment_branch wrong? [23:19:10] looks like [23:19:18] OK, I'll fix it... [23:22:47] MaxSem: So do I need to merge https://gerrit.wikimedia.org/r/#/c/91797/ before I update the submodule in core? I would assume so. [23:23:01] (03CR) 10Yurik: [C: 032] "Great job Adam, its in!!! =)" [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/83133 (owner: 10Dr0ptp4kt) [23:23:11] kaldari, yes [23:23:18] dr0ptp4kt: ^ [23:23:24] (03Merged) 10jenkins-bot: Hide charge-inducing features. Intercept link interstitials sooner. [extensions/ZeroRatedMobileAccess] - 10https://gerrit.wikimedia.org/r/83133 (owner: 10Dr0ptp4kt) [23:23:32] Cool, I'll add that to the documentation as well... [23:23:35] yurik_ thx [23:24:42] yurik_ you're supposed to be sleeping, though! i see you submitted updated patches. will take a look. not sure if i'll make it to them tonight, but if time allows, will. [23:24:54] (03CR) 10Kaldari: [C: 032] Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] (wmf/1.23wmf1) - 10https://gerrit.wikimedia.org/r/91797 (owner: 10Kaldari) [23:25:49] dr0ptp4kt: ok, but see if you can go bug carolynne & yana - I would much rather get the i18n patch in once they check english text/grammar/explanation/etc. We could put a lot of good text in there [23:26:04] its a full screen warning, need to use it well [23:26:17] yurik_, you mean a little bit of REALLy good text [23:26:18] (not too much text, just enough to be easy to understand and clear) [23:26:24] correct [23:26:25] :) [23:26:33] ;) [23:27:00] dr0ptp4kt: try to get that done asap - translators will get to it over weekend [23:27:12] five dollars. i kid. [23:27:30] MaxSem: I'm trying to do the cherrypick deployment so I can make sure I know how to do this correctly in the future :) Sorry you have to babysit me through it though. [23:27:32] tfinc: i wonder if we should reschedule - it will be 1:30am saturday morning :) [23:27:51] MaxSem is god. all bow to max. [23:28:03] kaldari, np - we all had to learn some day:) [23:29:11] MaxSem: Might take minute though since I decided to start from a clean deployment branch [23:30:50] yurik_: sure. reschedule as needed [23:31:12] yurik_: i made it so that you can modify [23:32:04] ok [23:33:54] tfinc: fwiw, I found it useful to stick on till the end [23:35:42] YuviPanda: great [23:44:26] MaxSem: So when I do 'git review wmf/1.23wmf1', the 'wmf/1.23wmf1' part is the name of the remote/origin branch right? Not the name of my local tracking branch (although they are often the same thing). [23:45:02] yes - the branch you're pushing to [23:45:11] Cool, just wanted to check :) [23:45:48] (03PS1) 10Kaldari: Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91803 [23:46:21] (03CR) 10Kaldari: [C: 032] Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91803 (owner: 10Kaldari) [23:49:31] (03Merged) 10jenkins-bot: Set a 30 days cookie when someone edits on mobile [extensions/MobileFrontend] (wmf22) - 10https://gerrit.wikimedia.org/r/91803 (owner: 10Kaldari) [23:49:35] Jenkins is so slow today [23:50:46] brion: mhurd YuviPanda: how did our market release go ? [23:50:57] i've been anxiously waiting on a mail [23:51:08] tfinc: brion released android to market yesterday itself [23:51:30] tfinc: i think the iOS version is awaiting review [23:51:31] not beta [23:51:35] full market [23:52:46] tfinc: https://play.google.com/store/apps/details?id=org.wikimedia.commons is full market 1.0 [23:54:25] * YuviPanda should head to bed, it's past 5AM [23:54:46] Night Yuvi [23:56:11] kaldari: :D I'll stick on for a couple more minutes to see if tomasz replies [23:57:29] kaldari: how is the ld going? [23:57:49] awjr, MaxSem: new cookie-setting code should be on testwiki now [23:57:51] testing.... [23:57:54] w00t