[00:11:51] (PS1) Hashar: operations-puppet-doc shell is now verbose [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96174 [00:17:50] (CR) Hashar: [C: 2] operations-puppet-doc shell is now verbose [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96174 (owner: Hashar) [00:18:09] (Merged) jenkins-bot: operations-puppet-doc shell is now verbose [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96174 (owner: Hashar) [00:39:11] chrismcmahon are you around to chat about Quality_Assurance/Browser_testing pages ? [00:40:10] spagewmf: sure. I want to talk to you about browser tests for Flow also [00:42:06] so I revived Running tests which is purely about running tests. I'd like to rename Running and writing tests to just Writing tests or maybe Writing test code since it's different from writing feature descriptions [00:43:04] I think "How to contribute" should be renamed "Writing feature descriptions", and I wonder if this is still "our foremost need" [00:43:37] chrismcmahon: ^does that sound reasonable? [00:44:23] spagewmf: it does sound reasonable [00:46:37] spagewmf: much of that should just point to the README https://github.com/wikimedia/qa-browsertests [00:50:48] spagewmf: let's chat tomorrow. I'd like to clean up those pages and I'd like to know more about what Flow will need, but I'm a bit east of you and it's dark out. :-) [00:51:05] no problem, see you then [08:43:51] (CR) Zfilipin: [C: 1] "Is this the thread at QA mailing list?" [qa/browsertests] - https://gerrit.wikimedia.org/r/96146 (owner: Cmcmahon) [08:48:38] hashar, James_F: is there an easy way to merge a commit that depends on a commit that was abandoned? [08:48:45] example: https://gerrit.wikimedia.org/r/#/c/96083/ [08:48:59] zeljkof: Oh, that? You need to rebase it on master. [08:49:09] James_F: will try [08:49:21] it will change the parent? let me try... [08:51:32] James_F: no [08:51:45] if I rebase the commit, it pulls the parent too [08:52:42] and looks like Chris and Jeff did not want the parent commit merged, for some reason (Gerrit does not say why) [08:53:02] let me try chaining parent commit… this will be fun [08:53:13] hello [08:53:35] zeljkof: you can cherry pick the commit [08:53:46] in Gerrit interface press the [Cherry-pick to] button [08:53:51] hashar: of course, I should have thought of that [08:53:54] hashar: thanks [08:53:55] you will be asked a branch to cherry pick to, fill in "master" [08:54:08] wow, there is even a way to do it in gerrit? [08:54:09] Gerrit will grab the patch apply it on master and forge a new patchset for you [08:54:21] yeah that is the "hacker" way :] [08:54:29] you can't rebase in this case [08:54:41] since the latest commit would always depends on the abandoned on [08:54:42] e [08:54:58] hashar: yes, I have tried rebasing [08:55:01] James_F: browser tests for VE / PArsoid, it is going to take a while sorry :/ [08:55:37] zeljkof: rebasing in Gerrit basically means: take the chain of unsubmitted patchset and reapply them on top of current master. [08:55:48] while cherry-pick act on a single commit [08:55:58] hashar: ugh, Code Review - Error, Could not create a merge commit during the cherry pick [08:56:05] will try cherry picking in terminal [08:56:06] conflict :-) [08:56:18] yeah you got to update your local master and cherry-pick [08:56:25] that will throw some error / conflict [08:59:28] hashar: :-( [08:59:40] hashar: BTW, Timo is working right now on getting VE passing in JSDuck 5. [08:59:50] hashar: Which is blocking oojs stuff. [09:00:04] hashar: But he says that JSDuck 5 isn't in our packages yet? [09:00:48] James_F: there is a bug for it iirc [09:00:54] Kk [09:01:22] yeah Timo filled a "upgrade jsduck" bug at https://bugzilla.wikimedia.org/show_bug.cgi?id=55753 [09:02:10] James_F: you need a RT ticket to get the new version packaged. The debian files are in operations/debs/ruby-jsduck.git [09:02:29] James_F: then ping ops till someone is assigned and able to update the package and upload it on apt.wikimedia.org [09:02:38] James_F: then one (ops / me) can manually upgrade [09:03:37] * James_F bugs Timo to file an RT ticket, then. :-) [09:04:03] James_F: are you project managing Parsoid ? [09:04:07] or is that gwicke ? [09:04:25] James_F: are you in Europe? it is pretty late SF time, right? [09:06:18] zeljkof: India. [09:06:42] hashar: gwicke is project managing, I'm in Product support but not much is normally needed of me and I focus on VE much more. [09:06:49] hashar: Why? Anything I can help with? [09:08:02] James_F: yeah gotta get Parsoid @master on beta. But will ping back by email :-] [09:08:39] gotta find a way to let the parsoid/VE team to manage those software on beta instead of mostly me :D [09:14:31] hashar: I managed to mess up the cherry pick of 96083 [09:14:35] will try again [09:16:43] hashar: I thought you did that last month? [09:17:38] James_F: the Parsoid extension code is updated but the parsoid instance itself runs from another copy of the code which is maintained manually. [09:17:46] Ah, yes, of course. [09:17:49] James_F: I though Roan / Gabriel / X would take care of it :D [09:18:06] This will be clearer when service/Parsoid.git is split from mediawiki/extension/Parsoid.git [09:18:30] if you can add that in your backlog and nail me about it, that would be nice (bug https://bugzilla.wikimedia.org/57233) [09:18:43] * James_F nods. [09:19:03] got to find out a way to have the Jenkins job to reload the parsoid server whenever code got updated [09:19:17] (and we don't have git-deploy in beta :/) [09:22:55] hashar: Well then, Parsoid is blocked on that. [09:23:05] hashar: 'Cos it's only ever been deployed using git-deploy. [09:26:09] James_F: we can hack up something with Jenkins / some script :-) [09:26:19] aka update the npm module and then restart parsoid. [09:26:45] * James_F nods. [09:27:00] "npm install && node js/api/server.js &" [09:27:03] It's easy. [09:27:05] ;-) [09:29:16] yup [09:29:22] integrating that is a bit more complicated though :D [09:29:34] I don't have a way to execute commands on a remote host :-( [09:29:40] (aka there is no salt in labs) [09:29:47] zeljkof: Yay. [09:29:51] zeljkof: Want me to +2? [09:30:00] hashar: Ah. :-( [09:30:13] anyway, would be nice to talk about it with the Parsoid team, they might find a solution. [09:30:31] I would loooovee to see VE/Parsoid teams maintaining that part of beta (me providing support of course) [09:30:36] * James_F nods. [09:30:46] wanna scale out beta/ci to moar folks. [09:32:10] James_F: I will leave it to Jeff and Chris to merge the commit [09:32:32] I just wanted to try how to make it merge-able :) [09:33:19] zeljkof: Ha, too late. :-) [09:33:26] hashar: I do too. [09:33:42] James_F: no harm done, I have probably not broken anything :) [09:34:38] zeljkof: It was byte-indentical to what Chris +2'ed, so it's his fault. :-) [10:19:42] zeljkof: are you attending tonight Release/QA meeting? [10:19:56] zeljkof: regardless, you might want to visit up https://etherpad.wikimedia.org/p/RelEngWeekly , I have added a few bugs related to browsertests [10:20:16] hashar: I will be there [10:48:56] accounting + lunch time .. [13:13:15] /topic https://www.mediawiki.org/wiki/Quality_Assurance | Publicly logged http://bots.wmflabs.org/~wm-bot/logs/%23wikimedia-qa/ [13:13:34] an op please do what above ^ public logging without stating it is against freenode policies [13:15:50] Tarawneh? sigh, I see, this is an hijacked Wikimedia Qatar channel :) [13:43:45] (PS1) Zfilipin: Documented which Jenkins jobs should use which passwords [selenium] - https://gerrit.wikimedia.org/r/96254 [14:13:30] (PS2) Cmcmahon: UW sometimes takes time to process. Discussed on QA mail list http://lists.wikimedia.org/pipermail/qa/2013-November/000692.html Change-Id: I9d6c9c2b512957876edf9f7642acbef14ca655bb [qa/browsertests] - https://gerrit.wikimedia.org/r/96146 [14:21:41] (CR) Hashar: [C: 2] "jobs updated" [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96044 (owner: Addshore) [14:24:43] (PS2) Hashar: Switch Query and QueryEngine to checks only [integration/zuul-config] - https://gerrit.wikimedia.org/r/96043 (owner: Addshore) [14:28:22] (CR) Hashar: [C: 2] "deploying" [integration/zuul-config] - https://gerrit.wikimedia.org/r/96043 (owner: Addshore) [14:28:31] (Merged) jenkins-bot: Switch Query and QueryEngine to checks only [integration/zuul-config] - https://gerrit.wikimedia.org/r/96043 (owner: Addshore) [14:29:08] [= [14:29:32] (CR) Hashar: "deployed, Zuul reloading as I write this." [integration/zuul-config] - https://gerrit.wikimedia.org/r/96043 (owner: Addshore) [14:30:18] (PS3) Hashar: Remove dependencies that are not on gerrit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96044 (owner: Addshore) [14:33:12] (CR) Hashar: [C: 2] "deployed" [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96044 (owner: Addshore) [14:33:31] (Merged) jenkins-bot: Remove dependencies that are not on gerrit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/96044 (owner: Addshore) [14:34:52] (CR) Addshore: [C: -1] "Now running checks only, this is no longer needed" [integration/zuul-config] - https://gerrit.wikimedia.org/r/93428 (owner: Jeroen De Dauw) [14:36:41] (Abandoned) Hashar: Do not run tests for Wikibase Query and QueryEngine [integration/zuul-config] - https://gerrit.wikimedia.org/r/93428 (owner: Jeroen De Dauw) [15:53:56] (CR) Jhall: [C: 1] Documented which Jenkins jobs should use which passwords [selenium] - https://gerrit.wikimedia.org/r/96254 (owner: Zfilipin) [15:54:19] (CR) Cmcmahon: [C: 2] "documentation" [selenium] - https://gerrit.wikimedia.org/r/96254 (owner: Zfilipin) [16:12:45] well [16:12:55] gonna setup a jenkins slave to auto update Parsoid on deployment-parsoid2 :D [16:15:56] zeljkof: in the hangout? [16:16:10] chrismcmahon: yes [16:16:14] did I disappear? [16:16:18] will reload [16:16:21] zeljkof: we don't see you [17:13:31] (CR) Jhall: [C: 1] UW sometimes takes time to process. Discussed on QA mail list http://lists.wikimedia.org/pipermail/qa/2013-November/000692.html Change-Id: I [qa/browsertests] - https://gerrit.wikimedia.org/r/96146 (owner: Cmcmahon) [17:36:01] hi JeffHall I have a ton of meetings today but I made a very simple outline of what I think are priorities for VE tests on your page: https://www.mediawiki.org/wiki/VisualEditor/Weekly_Test_Plans#Automated_testing [17:37:33] Looks good! If you don't mind, I might try to knock out the Lower Priority item first, with the goal of getting my branching and Gerrit review workflow bullet-proof [17:37:55] And then move onto the Higher priority items once I know the workflow is all good [17:46:44] JeffHall: sure [17:47:41] JeffHall: it would be worth brainstorming with James F also I think [17:50:34] chrismcmahon: Definitely, will plan to coordinate with JamesF once he's back in a familiar time zone! [17:57:56] greg-g I think we're missing hangout link? [18:05:30] JeffHall: are you coming to the RelEng/QA quarterly review? It's on your calendar [18:05:56] chrismcmahon: Yep, I'm in the room, but off-camera [20:33:37] (CR) JanZerebecki: [C: 1] "That change shown on mediawiki-core-regression-phpcs-HEAD is a noop compared to what is currently live. So it can be safely ignored." [integration/zuul-config] - https://gerrit.wikimedia.org/r/94239 (owner: JanZerebecki) [23:02:53] chrismcmahon, I'm readying myself... [23:03:31] hi spagewmf [23:06:22] (Abandoned) Jeroen De Dauw: Remove Wikibase Query and QueryEngine [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/95616 (owner: Jeroen De Dauw) [23:11:20] ohai [23:11:36] who wants to be the owner? [23:27:26] well, whatever :)