[00:03:20] o/ [08:22:48] hi [08:22:53] jdlrobson: hi [09:27:31] d3r1ck: jdlrobson is using irccloud, which is a permanently – well, almost permanently – connected irc bouncer [09:27:39] he may have forgotten to mark himself as afk [09:37:52] o/ [13:15:45] phuedx: are you there? [13:16:09] d3r1ck: i'm about to go to lunch [13:16:16] is there anything i can help you with? [13:16:21] can you review a patch? [13:18:32] d3r1ck: i can certainly take a look [13:18:42] which project is it against? [13:19:27] Mobile FE [13:19:32] can i send you the link? [13:20:14] phuedx: let me just add you as a reviewer? [13:20:22] what is your name on Gerrit [13:20:22] d3r1ck: just add me as a reviewer [13:20:33] d3r1ck: take a wild guess ;) [13:20:36] (it's phuedx) [13:21:18] phuedx: i have added you. [13:25:42] d3r1ck: after quick look it looks ok [13:25:52] has the thanks extension been updated to not use the toast module? [13:28:59] phuedx: well i don't understand [13:29:08] just +2 the one i added you to [13:29:16] so it can be merged by Gerrit bot [13:29:24] i want you to review another one [13:29:28] i have just two [13:31:33] d3r1ck: https://gerrit.wikimedia.org/r/#/c/240611/4/resources/mobile.toast/toast.js <- the change to that file appears to break another extension [13:32:09] ohhh, so i should undo it? [13:32:31] d3r1ck: yes and make a note that the Thanks extension will have to be updated [13:32:56] phuedx: ok [13:33:12] d3r1ck: the comment from jenkins-bot on sep 24, 9:19 pm as what i was alluding to earlier [13:33:22] (as did jdlrobson on the patch itself) [13:33:34] where it says "Gate pipeline build failed" [13:33:58] the build marked as failed reports that the thanks extension's unit tests failed [13:34:08] because a file is trying to require the "toast" module [13:37:23] ok [13:37:29] phuedx: i will do that now [13:43:38] phuedx: wit [13:43:41] phuedx: wait [13:44:46] phuedx: done. Check it out now. [13:49:48] phuedx: you can +2 it now. [13:49:58] so gerrit bot can just merge it. [13:55:25] d3r1ck: what exactly did you change? [13:56:35] i see no changes to the toast.js file [13:56:40] wait [13:56:41] mistake [13:58:39] phuedx: done [14:00:09] d3r1ck: not quite -- you appear to have submitted the wrong patch in place of the original [14:00:13] :) [14:04:35] phuedx: yes i want to do that now [14:08:08] phuedx: check it out now. [14:09:22] phuedx: can you cherry pick it to master? [14:09:27] please it is not going to me. [14:09:35] phuedx: do i need to rebase? [14:10:44] d3r1ck: i suggest that you abandon those two patches that you've just submitted and work of this one [14:10:46] https://gerrit.wikimedia.org/r/#/c/240611/4 [14:12:24] that one you requested that i review [14:12:32] it only requires a minor change [14:12:46] phuedx: ok [14:13:29] dbrant: any reason for me not to hit the big button and merge maps in nearby: tng? [14:14:32] phuedx: hope you will merge it [14:14:33] :) [14:14:43] mdholloway: well, it has a soft dependency on https://gerrit.wikimedia.org/r/240286 , since it hasn't been tested on GB... [14:14:57] d3r1ck: i'll take a look [14:15:38] mdholloway: but... any reason we're hesitating merging https://gerrit.wikimedia.org/r/240286 ? [14:15:46] niedzielski: ^ [14:15:48] d3r1ck: how familiar are you with git and gerrit (git-review specifically)? [14:16:11] dbrant: yeah, i've had my eye on that one. does it makes sense to get quiet crash reporting in first? /cc niedzielski [14:16:47] phuedx: i am very good at it. [14:17:17] dbrant: that's the only thing i'm holding back on [14:17:22] phuedx: you can look at it now. [14:17:24] d3r1ck: ah, i've just seen that you've submitted a new patch [14:18:05] phuedx: yes [14:21:15] dbrant: i'm fine with that patch as well, except that (as you noted) the possibility of an infinite crash/restart loop should be addressed [14:24:55] phuedx: waiting [14:26:02] mdholloway: i believe we've already agreed that GB will remain on ACRA crash reporting (but niedzielski can confirm). [14:26:10] mdholloway: the last crash report we've received from 2.3.x was over a month ago, and there have been no reports from 2.3 for the last production version. [14:26:42] d3r1ck: adding a deprecation warning is far less destructive than removing one -- any remaining clients relying on the old module name will break [14:27:26] i'm verifying that all of the extensions that i have cloned locally (which, honestly, isn't that representative a sample) don't rely on the old module names [14:27:50] which, as you've noted, is taking a little time [14:28:47] unit tests help us here [14:29:08] but we don't run all unit tests for all extensions every build [14:30:21] phuedx: ok. I am waiting [14:30:24] dbrant: i must have missed that. nothing blocking a +2 for https://gerrit.wikimedia.org/r/240286 for me, then. [14:30:24] to see the results [14:41:43] phuedx: i see all is well [14:41:57] i just hope jenkins will be happy and gerrit bot will merge [14:43:00] i ran the integration tests myself, so jenkins should be ok there [14:44:58] dbrant mdholloway: confirm [14:45:22] dbrant mdholloway: quiet crash reporting will definitely be post-gb [14:45:41] dbrant mdholloway: still waiting on an agreement to take shape [14:45:58] niedzielski: dbrant: cool. if you have any objection to my +2'ing the version code bump, speak now [14:46:14] mdholloway: go for it! [14:47:01] DONE [14:48:08] phuedx: yes it is fine [14:48:13] phuedx: it is merged [14:48:31] :) [14:48:33] phuedx: so it seems the bug can be marked as resolved because i have done all [14:51:55] niedzielski: hi [14:53:12] d3r1ck: i see 34 calls to deprecate in the mobilefrontend codebase [14:53:15] d3r1ck: hello [14:53:43] (using ag -a '\.deprecate' resources/ | wc -l) [14:54:11] niedzielski: you cool with the maps patch going in? [14:54:21] phuedx: hey, meaning they are not yet finished? [14:54:27] niedzielski: i understand there's to be some follow-up UI work [14:54:28] mdholloway:yep! [14:54:33] niedzielski: cool [14:54:49] phuedx: it seems ... Wait let me check again [14:55:03] http://media.giphy.com/media/8fen5LSZcHQ5O/giphy.gif [14:55:17] :) [14:55:44] womp womp [14:56:09] merge failed. [14:56:27] rebasing... [15:01:23] phuedx: have you updated your repo? [15:01:34] git fetch origin master :) [15:04:19] phuedx: i have made another patch [15:04:26] mdholloway: lock and load [15:04:52] phuedx: check it out an test, the probably +2 for gerrit to merge [15:06:29] d3r1ck: git fetch won't merge refs/objects into the branch that you're working on [15:07:04] it'll only fetch them from the remote that you specify [15:07:31] (just in case) [15:07:40] phuedx: so must do git pull? [15:07:49] but check the patch i sent [15:07:53] you can test it. [15:09:22] d3r1ck: git pull will git fetch and then git merge fetch_head, so yes [15:10:05] niedzielski: yeah, i noticed the same a few days ago, cool stuff! [15:10:40] brb -- fetching shopping in from the car [15:11:10] niedzielski: dbrant: i don't think we have a ticket for it yet, but it would be nice to move to an on-demand location permission when using nearby instead of at install time [15:11:27] mdholloway:i think that would be covered by bearND|afk's m list [15:12:53] niedzielski: so whats up? [15:13:17] d3r1ck: oh just going through the morning patches. you? [15:13:23] niedzielski: i remember him talking about making a list but if it's in phab i must not be subscribed to it, looking now [15:18:06] it may have been a wiki page... [15:18:12] mdholloway: https://www.mediawiki.org/wiki/Notes_from_Google_I/O_15#Action_Items [15:18:27] sorry, i was lost in a sea of wikis [15:19:27] niedzielski: mdholloway: I'll be sure to make tasks out of those items [15:19:33] niedzielski: well nothing muchni [15:19:37] niedzielski: no worries, thanks! [15:19:38] niedzielski: can you checkout mine? [15:20:38] d3r1ck: er, which repo is that? [15:20:45] niedzielski: i have added you as reviewer [15:20:51] niedzielski: Mobile Frontend [15:22:14] d3r1ck: does this affect android? [15:22:43] niedzielski: i don't think [15:22:51] d3r1ck: not everyone here is a mobile-web engineer. niedzielski and I work on the Android app. [15:23:03] dbrant: ohhhh, i see [15:23:23] dbrant: then jdlrobson and phuedx will look at it when they come back [15:23:28] d3r1ck: yeah, i don't think this one would be a good one for me review [15:24:02] d3r1ck: yep! they're quite busy; give them some time... [15:32:16] dbrant: yes [15:32:34] phuedx: hmmm, i no longer see afk on your handle :) [15:35:58] d3r1ck: yep, i'm here [15:36:25] i'm finishing off a piece of work that i've been working on for a while at the moment [15:36:56] keep submitting the patches and they'll be reviewed -- but not necessarily within 10 minutes ;) [15:37:07] ok [15:41:34] phuedx: i need to fix like 50 bugs GSoC next year :) [15:54:03] d3r1ck: yowza -- that seems like a whole lot [15:54:38] phuedx: a whole lot of what? [15:54:49] phuedx: i have updated using git pull. [15:54:59] i think that is the last patch [16:03:48] d3r1ck: ag -l --js '\.deprecate' resources/ https://www.irccloud.com/pastebin/8rfCQ38K/ [16:04:01] cross-post from -dev [16:04:38] phuedx: have you did git pull? [16:04:49] on the MobileFrontend extensions? [16:05:00] d3r1ck: yes [16:05:23] phuedx: and you still see that? [16:06:19] d3r1ck: here's a similar search on github: https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/search?utf8=%E2%9C%93&q=.deprecate [16:06:38] you can ignore the mobile.modules file, which is the definition of the function [16:06:56] and there are a few instances of the word deprecate in documentation there [16:07:13] yes i know [16:07:19] phuedx: i have submitted a patch already [16:07:28] merge it and then pull [16:08:57] phuedx: so when you merge it then i think all is well then [16:10:36] d3r1ck: you're talking about https://gerrit.wikimedia.org/r/#/c/241052/ ? [16:10:43] or have i missed another patch? [16:11:43] phuedx: yes [16:11:47] that is it. [16:13:48] ok, then i think there'll still be 16 ish occurrences of deprecate to clear up [16:14:12] phuedx: wait [16:14:14] they are [16:14:29] i used: git grep -E ".deprecate" [16:15:42] and i got no call [16:16:04] to the deprecated module [16:16:17] let me pastebin it and show you. [16:17:02] phuedx: http://pastebin.com/VY1NNhQs [16:23:49] d3r1ck: here's what i get from a new clone of the mobilefrontend extension https://www.irccloud.com/pastebin/2DczO8PT/ [16:24:06] have you submitted all of your changes to gerrit? [16:24:10] and have they all been merged? [16:24:38] d3r1ck: i've got to go and have dinner but i'll be back later [16:25:38] jdlrobson might be coming online soon [16:25:46] so he /may/ be able to help you out in the interim [16:26:03] okie poke -- bbiab [16:26:15] ok [16:27:39] jdlrobson: hey [16:27:57] awesome jdlrobson, welcome [16:30:46] hi there [16:31:33] dbrant|brb: niedzielski mdholloway|afk bearND Sprint name? "Holmium"? [16:31:51] jdlrobson: so what have you been working on? [16:31:54] mbinder: you got it! [16:32:18] not much unfortunately. Mostly code review and defining future work :) [16:32:43] jdlrobson: hahaha.... :) [16:33:06] jdlrobson: i am planning to reach the level of reviewing code. I want to help you when you are not there. [16:33:07] +1 [16:33:31] this sprint is about to get sprung [16:34:06] niedzielski: what? [16:34:29] d3r1ck: was chatting with the android devs :) [16:34:55] ok [16:35:38] hi jdlrobson :P [16:35:48] d3r1ck: there's a few more deprecated function calls. Would be great to get those fixed up [16:36:07] hey FlorianSW :) We broke Monobook on Wikivoyage :) https://phabricator.wikimedia.org/T113746 [16:36:11] (well mostly me ;-)) [16:36:13] jdlrobson: ok. Let me check it out. [16:36:24] d3r1ck: you'll need to install the Thanks extension [16:36:47] jdlrobson: i have it already [16:36:48] jdlrobson: bähh :P [16:37:23] d3r1ck: awesome. [16:37:36] jdlrobson: if you rebase, I'll check and merge :) [16:37:36] i think we'll be able to wrap that bug up and get it closed today ! yay! :) [16:37:44] jdlrobson: remember i fixed a bug for them back some few days ago [16:37:47] FlorianSW: on it [16:38:35] dbrant mdholloway|afk bearND: would one of you devs mind reviewing this when you can? https://gerrit.wikimedia.org/r/#/c/237024/ [16:38:42] jdlrobson: when i run: git grep -E ".deprecate" [16:38:46] jdlrobson: i have this [16:38:49] jdlrobson: http://pastebin.com/VY1NNhQs [16:38:58] niedzielski: looking now [16:39:01] FlorianSW: done! :) [16:39:04] which calls should i remove from there? [16:39:08] dbrant: thanks :) [16:39:15] * FlorianSW is checking jdlrobson's change [16:39:22] d3r1ck: it's possible you didn't submit a change [16:39:28] are you on a clean master branch? [16:39:30] git checkout master [16:40:02] jdlrobson: i am there [16:40:06] https://gist.github.com/jdlrobson/5a4b6b9844041c182a24 > d3r1ck that's what i am seeing [16:41:00] jdlrobson: ok [16:41:03] let me fix those [16:41:18] d3r1ck: you almost certainly have something on your master branch that hasn't been pushed to gerrit. If you do a `git log` do you see any patches that haven't been merged? [16:42:32] jdlrobson: i see it. [16:42:53] jdlrobson: fixing it now. But toast breaks the Thanks extension. I will have to fix that too. [16:44:00] d3r1ck: there's this bug if you're interested afterwards - https://phabricator.wikimedia.org/T113667 - should be an easy fix. We'll have you up to speed and reviewing in no time :) [16:44:21] niedzielski: looking too [16:44:33] bearND: cool thanks [16:50:20] jdlrobson: let me look [16:54:05] jdlrobson: let me fix the calls to deprecated modules too [16:54:23] when i finish up with that, then i will go for the one you just asked [16:54:38] yeh you'll need to do those first. Ping me when patches are up and i'll merge them asap [16:56:06] ok [17:13:42] jdlrobson: checkout the one i just send [17:13:45] *sent [17:16:45] jdlrobson: check it out [17:20:29] jdlrobson: are you there? [17:20:58] lets finish up with this one and then let me fix the other one. :0 [17:21:02] :) [17:34:37] d3r1ck: yup taking a look now [17:44:38] jdlrobson: i don't even know how that file got added. [17:44:44] but what should i do now [17:48:21] jdlrobson: done, you can review now [17:55:50] d3r1ck: back to you [17:56:23] jdlrobson: yes, gerrit bot will surely merge it [17:56:38] jdlrobson: lets resolve the bug now. [17:57:19] jdlrobson: hey we have a problem [17:57:24] qunit refused the patch [17:57:26] mediawiki-extensions-qunit https://integration.wikimedia.org/ci/job/mediawiki-extensions-qunit/12998/console : FAILURE in 1m 32s [17:58:26] jdlrobson: can you do a recheck? [17:58:57] d3r1ck: click `full log`and you'll see there is a qunit failure in Actual: ext.thanks.mobilediff,tests.ext.thanks.mobilediff [17:59:02] you'll have to update Thanks first [17:59:09] it's preventing us from deploying broken code so this is good it caught it [18:00:00] jdlrobson: wait, so how do i update Thanks? [18:00:13] The Thanks extension is referring to the old deprecated names [18:00:21] jdlrobson: can you walk me through? [18:00:35] so you'll have to install Thanks [18:00:39] jdlrobson: i have it [18:00:46] I suspect if you visit a diff page on mobile you'll see warnings in the JavaScript console [18:00:51] Do what the warnings ask you [18:01:49] jdlrobson: wait, i don't get it. [18:03:00] bearND: kaity_ retro? [18:04:31] jdlrobson: i see it. There is a call to toast [18:04:36] which is deprecated. [18:04:48] d3r1ck: exactly. So you'll need to update a patch to avoid the deprecation warning [18:04:58] jdlrobson: so i need to submit a patch for Thanks too? [18:05:03] yup [18:05:06] men will you review it? [18:05:15] yes i will review it if you link me to it [18:05:55] ok [18:06:02] jdlrobson: so what is the module name? [18:06:22] is it: 'mobile.toast/toast' [18:06:25] in [18:06:38] M.define( 'mobile.toast/toast', new Toast() ) [18:06:48] or i just have to put the whole line of code? [18:08:07] jdlrobson: so from the link of code i pasted, what is the module name? [18:13:50] jdlrobson: done. The call has been updated. :) [18:13:55] let me link you to the patches [18:22:16] d3r1ck: the module name is the first argument to M.define [18:22:25] 'mobile.toast/toast' [18:23:29] i'm not around for long -- just checking in [18:24:50] phuedx: it is done. [18:25:01] phuedx: mooney is revieing it [18:25:16] but i have submitted another patch with updated toast issue [18:25:27] it should be fine now jdlrobson is busy :) [18:27:05] \o phuedx go enjoy your weekend :P [18:27:50] jdlrobson: :) then you will do the work :D [18:45:53] d3r1ck: take a look a the report (click full log at top) [18:45:57] https://www.irccloud.com/pastebin/QR6fjUTK/ [18:46:59] jdlrobson: ok [18:50:46] jdlrobson: i don't still seem to figure out how to fix thatpart [18:50:48] *part [18:52:53] jdlrobson: checkt his out [18:52:56] *this [18:52:57] 18:42:58 Chromium 45.0.2454 (Ubuntu 0.0.0) ERROR: 'Error: Module not found: loggingSchemas/SchemaMobileWebClickTracking', Error{} [18:53:11] I don't know how to go about this. [19:06:28] mhurd: http://hatjitsu.wmflabs.org/58033 [19:18:34] There will be clues in the log d3r1ck [19:18:42] i suspect SchemaMobileWebClickTracking is being used in Thanks as well [19:19:02] Part of the fun is working out where the problem lies :) [19:26:43] dbrant|brb: got worktype tags for items for next sprint? [21:09:31] bearND dbrant mdholloway: ok it looks like the last commit is merged. are we doing a release then? [21:10:00] bearND dbrant mdholloway: also, is the next version / name 2.1.131? [21:11:58] niedzielski: nothing really blocking a release, but I'd really like us to look at the sqlite upgrade bug a bit more. Maybe release on Monday? [21:12:18] bearND dbrant mdholloway: sounds good [21:12:32] 2.1.131 sounds good to me [21:12:58] yeah, monday is probably better. Do we also want to get the HA stuff in? [21:14:39] bearND: i'd love to but we're still waiting on the agreements :/ [21:16:02] bearND: last response from ha didn't sound favorable so i'm guessing it needs to cook for a bit [21:16:17] bearND: that was yesterday [21:16:51] niedzielski: ok, good to know. It wasn't clear in the patch. Maybe -1 it so we don't accidentally merge it [21:17:10] bearND:will do [21:17:45] dbrant: i think https://phabricator.wikimedia.org/T110686 should go to Design review column so kaity can take a look [21:18:38] bearND: good call [23:26:26] aude, around? are the maps in nearby done? how can i play with it? [23:26:49] dbrant|brb, ^? [23:27:04] bearND, you also commented on it [23:27:15] (was looking at https://gerrit.wikimedia.org/r/#/c/212922/