[04:40:49] Yippee, build fixed! [04:40:49] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_8.1-internet_explorer-11-sauce build #461: FIXED in 33 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_8.1-internet_explorer-11-sauce/461/ [06:17:58] 10Browser-Tests, 10MediaWiki-extensions-DonationInterface: Write browser tests for DonationInterface - https://phabricator.wikimedia.org/T99955#1324881 (10Qgil) [06:18:04] 10Browser-Tests, 10MediaWiki-extensions-DonationInterface: Write browser tests for DonationInterface - https://phabricator.wikimedia.org/T99955#1302602 (10Qgil) [06:33:41] 6Release-Engineering: Ready-to-use Docker package for MediaWiki - https://phabricator.wikimedia.org/T92826#1324929 (10Qgil) Did anyone work on this task at #Wikimedia-Hackathon-2015? If so, please share an update. If not, remove the label. [06:34:35] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1324931 (10Qgil) Did anyone work on this task at #Wikimedia-Hackathon-2015? If so, please share an update. If not, remove the label. [06:35:29] 10Deployment-Systems, 6Release-Engineering, 10ReleaseTaggerBot, 7Epic: EPIC: Code Deploy Dashboard - https://phabricator.wikimedia.org/T280#1324933 (10Qgil) Did anyone work on this task at #Wikimedia-Hackathon-2015? If so, please share an update. If not, remove the label. [06:36:47] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1324935 (10Legoktm) [06:37:31] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1321372 (10Legoktm) >>! In T100812#1324931, @Qgil wrote: > Did anyone work on this task at #Wikimedia-Hackathon-2015? If so, please share an upda... [06:38:37] 10Deployment-Systems, 6Release-Engineering, 10ReleaseTaggerBot, 7Epic: EPIC: Code Deploy Dashboard - https://phabricator.wikimedia.org/T280#1324938 (10Legoktm) >>! In T280#1324933, @Qgil wrote: > Did anyone work on this task at #Wikimedia-Hackathon-2015? If so, please share an update. If not, remove the la... [06:43:40] RECOVERY - Free space - all mounts on deployment-videoscaler01 is OK All targets OK [07:02:29] 6Release-Engineering: Ready-to-use Docker package for MediaWiki - https://phabricator.wikimedia.org/T92826#1324960 (10Mglaser) @thcipriani and I worked on this task. We agreed on technology (use salt inside a docker container as a first start), on the environment (esp use of MariaDB) and the desired first featur... [07:20:00] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1324980 (10valhallasw) Unfortunately, this is currently non-trivial; the API does allow project creation, but does not allow adding extra tags, o... [07:35:26] addshore: hello :) got a minute? [07:36:02] I have aproximately 25 of them :P [07:36:08] 24! [07:36:17] alright. :P [07:36:21] so, two things. [07:36:28] one, legoktm's https://gerrit.wikimedia.org/r/#/c/193766/ [07:36:44] hashar pointed out that there is a sniff that already exists upstream [07:37:33] however, two concerns. 1) the upstream sniff only checks for spaces in function *declarations*. There is a test for function calls too, but that's a PEAR standard, which we don't use. [07:38:03] 2) legoktm's patch, if I'm reading it correctly, checks for everything being spacey, not just function declarations or calls [07:38:10] control statements, etc. too [07:39:22] hmmm [07:39:25] *reads the change* [07:41:19] polybuildr: I agree [07:41:53] addshore: okay, excellent. That means there's no need to include the upstream sniff in the ruleset and this patch is good to be merged, right? [07:42:12] probably, I'll add a comment and see what hashar says :) [07:42:16] Assuming the output of running it against core is satisfactory to the user. [07:42:57] addshore: Hold on, I'll write the comment. Until then, could you please take a look at https://phabricator.wikimedia.org/T92754 ? [07:42:59] (03CR) 10Addshore: [C: 031] "The sniff you refer to hashar only checks spacing around function declarations!" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/193766 (owner: 10Legoktm) [07:43:08] okay, cool. :P [07:43:23] mediawiki/tools/codesniffer should pass phpcs haha [07:44:22] horrific, seriously. :P [07:46:42] polybuildr: addshore good morning [07:47:05] polybuildr: are the only remaining issues 1 and 2? classname and long comments? [07:47:26] good morning hashar [= [07:48:50] addshore: as far as I can tell, yes. you can take a look at the console output in the test run for that patch. [07:49:21] (03CR) 10Polybuildr: [C: 04-1] "In addition, this sniff tests for spaceyness everywhere, right? Functions, control statements. Anywhere there's a `(.*)`. The commit messa" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/193766 (owner: 10Legoktm) [07:49:21] so the standard of class naming is the PEAR standard? [07:49:37] quick away [07:49:45] tbh, the PHPCS repo should continue doing what PHPCS upstream is doing [07:49:45] addshore: I'm not sure what it's called. :P Lemme confirm. [07:50:04] Yes, PEAR. [07:50:33] Right, so how do we get codesniffer to be partial to itself? :P [07:50:34] *looks upstream* [07:50:56] https://github.com/squizlabs/PHP_CodeSniffer/blob/master/CodeSniffer/Standards/PSR2/Sniffs/Classes/ClassDeclarationSniff.php [07:51:13] looks like upstream has changed [07:51:17] so we could change? :) [07:51:32] changed how?... [07:51:40] no more _s [07:51:50] class PSR2_Sniffs_Classes_ClassDeclarationSniff extends PEAR_Sniffs_Classes_ClassDeclarationSniff ? [07:51:56] https://github.com/squizlabs/PHP_CodeSniffer/blob/master/CodeSniffer/Standards/PSR2/Sniffs/Classes/ClassDeclarationSniff.php#L33 [07:52:45] Their _ is not exactly PEAR. It's an attempt at namespacing, in some sense. [07:52:53] *facepalm* sorry, I was expecting the class in the file to have the same name as the file [07:52:55] but of course not.. [07:53:08] oh :P [07:53:30] well, I say we should stick with what upstream is doing :/ [07:53:45] it just means our PHPCS repo doesnt actually follow our MW CS rules xD [07:53:48] Personally, I agree. However, I don't know how we can get codesniffer to pass against itself, though. [07:53:51] exactly. :P [07:54:04] mhhhm [07:54:15] Though changing it to normal CamelCase probably won't cause any issues. [07:54:25] Or it might. [07:54:31] I don't know how phpcs processes the sniffs. [07:54:41] I havn't looked at the CS code in a while but I believe it might :P [07:54:45] Also, the line length issue still remains. Or is that okay, since it's just a warning? [07:55:08] line length is probably okay, which line / lines are they and how many are they? [07:55:26] I've noted them in the comment, I think. [07:55:27] Two kinds. [07:55:34] One is the class declaration lines. [07:55:43] Second is looong URLs in comments at the top of the file. [07:55:54] Almost every class declaration, I think. [07:55:58] haha [07:56:00] I fixed any other minor instances there might have been. [07:56:38] How does one add linebreaks to 1) class declarations 2) URLs :P [07:57:10] well, class declrations would be after the class name, stick the extends / implements on a new line [07:57:23] I would hope that would solve most of the class line lengths [07:58:01] URLs, the best you could probably do is make sure the URL is on a line all by itself to keep the length to a minimum [07:58:07] They already are. :P [07:58:11] if it is still too long there isnt really anything you could do [07:58:28] Also, phpcs returns a non zero status code even if there are only warnings. [07:58:30] and I'm sorry but your 24 mins are up ;P I'll be back latere!!!! [07:58:39] So if it was a voting test, it would still fail. [07:58:46] alright. o/ [07:59:01] I'm guessing you're on the conference call. :P [07:59:17] A different conference call, but yes :P [07:59:44] okay :P later then :) [08:00:07] for PHP_CodeSniffer patches, we should probably run the proposed still against mw/core @ master [08:00:10] as a non voting job [08:00:16] might help review [08:00:45] and I am in a call with zeljko [08:01:29] hasharConfcall: legoktm's spacey patch has been run against mw/core, though maybe not too recently. He's checked the output and it seemed legit. [08:01:46] polybuildr: yeah I am sure he handled it properly :-} [08:02:14] but if Jenkins does it for you, that saves time to reviewer [08:02:18] I should fill a task about it [08:02:27] In which case the commit message and the comment at the top of the sniff needs a minor edit and it's ready to merge. [08:02:33] hasharConfcall: Oh. [08:02:38] for every codesniffer patch [08:02:48] jenkins will run it against core? [08:02:49] yeah [08:02:53] that's pretty cool :D [08:03:04] we would do a shallow clone of mw/core @ master [08:03:12] then clone our code sniffer style [08:03:14] checkout the patch [08:03:22] and run phpcs using the patched standard [08:03:33] That would be great. [08:03:36] yup [08:05:26] filled https://phabricator.wikimedia.org/T100966 [08:09:31] (03PS5) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [08:09:48] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [08:10:58] zeljkof: https://phabricator.wikimedia.org/T100655 [08:17:57] (03PS6) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [08:18:10] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [08:18:15] !log Jenkins: upgrading git plugin from 1.5.0 to latest [08:18:19] Logged the message, Master [08:19:04] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce build #617: ABORTED in 9 min 4 sec: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce/617/ [08:20:26] addshore: Done, broke the class declaration lines. URLs and CamelCase only pending issues. [08:25:06] hashar: You submit new patch sets that have no line changes by changing the permissions, right? [08:33:19] polybuildr: the tests failed? :( [08:34:18] RECOVERY - Puppet staleness on deployment-restbase01 is OK Less than 1.00% above the threshold [3600.0] [08:37:14] addshore: Which tests? [08:37:29] The new patch has an updated composer. [08:37:33] It's already testing against itself. [08:37:37] ahhh! [08:37:45] so still have to fix the lines ;p [08:37:49] yep :P [08:37:56] or figure out a way to do something differently. [08:41:34] 10Continuous-Integration-Infrastructure, 7Regression: ERROR: Failed to notify endpoint 'HTTP:http://127.0.0.1:8001/jenkins_endpoint' - https://phabricator.wikimedia.org/T93321#1325055 (10hashar) [08:41:35] 10Continuous-Integration-Infrastructure: Upgrade Jenkins git related plugins - https://phabricator.wikimedia.org/T100655#1325053 (10hashar) 5Open>3Resolved We have upgraded the git plugin from 1.5.0 to 2.3.5. Seems jobs are all happy about it :-] [08:43:16] [alias] [08:43:16] lg = log --graph --pretty=format:'%C(auto)%h%Creset -%C(auto)%d%Creset %s %Cgreen(%cr) %C(cyan)<%an>%Creset' --abbrev-commit --date=relative [08:43:22] zeljkof-meeting: ^^^ in your ~/.gitconfig [08:43:57] hehe, i have the same alias [08:54:12] addshore: any suggestions? [08:54:39] polybuildr: *looks at whcih lines are causing issues* [08:55:39] polybuildr: what happened in here? :P https://gerrit.wikimedia.org/r/#/c/214792/6/phpunit.bootstrap.php [08:55:56] lines 6 looks broken ;p [08:55:56] addshore: OUCH. :P [08:56:43] (03PS7) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [08:57:07] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [08:57:18] addshore: Thanks for catching that one. [08:57:28] A random test I was doing, only deleted half of the change. [08:59:18] heh, there isnt really much you can do about this one .... https://github.com/wikimedia/mediawiki-tools-codesniffer/blob/master/TestHelper.php#L4 [08:59:29] parallel-lint caught it though. I didn't realise because I wasn't paying too close attention to failing tests. :P [08:59:36] yeah. :/ [08:59:39] >> https://github.com/cakephp/cakephp-codesniffer/blob/7abb71ed17/CakePHP/tests/TestHelper.php [08:59:48] I even though about using a URL shortener :P [09:00:05] I guess the more important thing is to figure out how to get it to not fail for warnings? [09:00:08] the above link will leave it under 100 chars ;p [09:00:22] addshore: hahahaha :D [09:00:45] you can probably do the same with all of the github urls polybuildr [09:01:15] addshore: Shorten the blobs? [09:01:29] if not, afaik https://git.io/ would probably also be acceptable [09:02:00] but yes, shorten the hashes to 10 chars and all the links should still work [09:02:29] addshore: on it. [09:02:39] =] [09:04:13] addshore: https://github.com/cakephp/cakephp-codesniffer/blob/015919e550/CakePHP/tests/CakePHPStandardTest.php still 103. :/ [09:04:23] wahahahaa [09:04:30] Cutting down 3 of the last characters works though. :P [09:04:34] If we want to take the risk. [09:04:48] mhhhm, probably better to use git.io urls [09:05:13] they shouldnt go anywhere unless github does, and if github does then the targets are useless anyway [09:05:34] okay. Also maybe keep a link to the repo above that, so that it gives an idea of where it came from without having to expand the git.io url? [09:05:47] yeh, could do! [09:06:02] on it [09:08:22] how about manual:cc/php#something ? [09:08:58] addshore: "Per https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP#Alternative_syntax_for_control_structures" [09:10:14] Sorry I don't follow! [09:10:40] There's a link to that ^ [09:10:56] Coincidentally, I put it there in the first place. :P [09:11:02] And it needs to be shortened. [09:11:15] I could remove the #fragment. [09:11:32] Or I could shorten it to manual:cc/php or something. [09:11:36] If your suggestion works then go for that! [09:12:02] Removing the fragment? [09:12:31] manual:cc/php#something ? :P [09:13:01] I haveanother call now, tata for now! :P [09:13:12] addshore: No, it's not an actual link. [09:13:17] It's just a convention I've seen on wikis. [09:14:54] (03PS8) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [09:15:10] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [09:22:26] PROBLEM - Puppet failure on deployment-mx is CRITICAL 100.00% of data above the critical threshold [0.0] [09:41:59] addshore: Only class names left. :) [09:42:07] Then it shall pass against itself. :D [09:47:13] away afk [10:04:17] polybuildr|away: yay! [10:04:44] Yippee, build fixed! [10:04:44] Project browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-monobook-sauce build #449: FIXED in 43 min: https://integration.wikimedia.org/ci/job/browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-monobook-sauce/449/ [12:34:41] PROBLEM - Free space - all mounts on deployment-videoscaler01 is CRITICAL deployment-prep.deployment-videoscaler01.diskspace._var.byte_percentfree (<30.00%) [13:58:54] thanks hashar_ for checking the logs :) https://phabricator.wikimedia.org/T91764#1325582 [14:01:16] Nemo_bis: and I have added some columns on the Workboard https://phabricator.wikimedia.org/tag/wikimedia-log-errors/ :} [14:34:48] Project browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #520: FAILURE in 8 min 47 sec: https://integration.wikimedia.org/ci/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/520/ [14:44:45] PROBLEM - Free space - all mounts on deployment-bastion is CRITICAL deployment-prep.deployment-bastion.diskspace._var.byte_percentfree (<10.00%) [15:03:03] 10Continuous-Integration-Infrastructure, 6Collaboration-Team, 10Flow: integration-slave-precise-1012 as a corrupted workspace for mediawiki-extensions-zend causing SpecialMobileDiffTest::testNames to fail with MWException: Diff not implemented for Flow\Conte... - https://phabricator.wikimedia.org/T100891#1325962 [15:04:48] RECOVERY - Free space - all mounts on deployment-bastion is OK All targets OK [15:11:59] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1326003 (10mmodell) Upstream is willing to consider a patch to add color/symbol to the project creation api. Probably would accept extra tags as... [15:25:27] 10Continuous-Integration-Infrastructure: Fix "PHP Warning: Module 'apc' already loaded" on zend slaves - https://phabricator.wikimedia.org/T99413#1326035 (10Krinkle) This warning impairs readability of the test resutls. Especially things like phplint which have no other output normally and are exclusively run o... [15:27:30] 10Continuous-Integration-Infrastructure, 6Labs: Designate should support split horizon resolution to yield private IP of instances behind a public DNS entry - https://phabricator.wikimedia.org/T95288#1326046 (10Andrew) [15:32:18] (03PS1) 10Hashar: operations-puppet-typos now handle missing /typos [integration/config] - 10https://gerrit.wikimedia.org/r/215042 (https://phabricator.wikimedia.org/T100989) [15:47:57] (03PS1) 10Hashar: Bind operations-puppet-typos to Trusty [integration/config] - 10https://gerrit.wikimedia.org/r/215047 (https://phabricator.wikimedia.org/T100989) [15:56:11] (03PS2) 10Hashar: operations-puppet-typos now handle missing /typos [integration/config] - 10https://gerrit.wikimedia.org/r/215042 (https://phabricator.wikimedia.org/T100989) [15:56:57] (03CR) 10Hashar: [C: 032] "Ignore grep exit code via set+e set-e. We handle it in the switch/case." [integration/config] - 10https://gerrit.wikimedia.org/r/215042 (https://phabricator.wikimedia.org/T100989) (owner: 10Hashar) [15:59:00] (03Merged) 10jenkins-bot: operations-puppet-typos now handle missing /typos [integration/config] - 10https://gerrit.wikimedia.org/r/215042 (https://phabricator.wikimedia.org/T100989) (owner: 10Hashar) [16:00:37] (03PS2) 10Hashar: Bind operations-puppet-typos to Trusty [integration/config] - 10https://gerrit.wikimedia.org/r/215047 (https://phabricator.wikimedia.org/T100989) [16:02:00] (03CR) 10Hashar: [C: 032] Bind operations-puppet-typos to Trusty [integration/config] - 10https://gerrit.wikimedia.org/r/215047 (https://phabricator.wikimedia.org/T100989) (owner: 10Hashar) [16:03:59] (03Merged) 10jenkins-bot: Bind operations-puppet-typos to Trusty [integration/config] - 10https://gerrit.wikimedia.org/r/215047 (https://phabricator.wikimedia.org/T100989) (owner: 10Hashar) [16:09:41] (03PS1) 10Hashar: operations-puppet-typos now grep with colors [integration/config] - 10https://gerrit.wikimedia.org/r/215051 (https://phabricator.wikimedia.org/T100995) [16:09:51] (03CR) 10Hashar: [C: 032] operations-puppet-typos now grep with colors [integration/config] - 10https://gerrit.wikimedia.org/r/215051 (https://phabricator.wikimedia.org/T100995) (owner: 10Hashar) [16:12:01] (03Merged) 10jenkins-bot: operations-puppet-typos now grep with colors [integration/config] - 10https://gerrit.wikimedia.org/r/215051 (https://phabricator.wikimedia.org/T100995) (owner: 10Hashar) [16:14:14] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1326202 (10mmodell) https://secure.phabricator.com/D13098 [16:18:33] 10Continuous-Integration-Infrastructure, 6Collaboration-Team, 10Flow: integration-slave-precise-1012 has a corrupted workspace for mediawiki-extensions-zend causing SpecialMobileDiffTest::testNames to fail with MWException: Diff not implemented for Flow\Conte... - https://phabricator.wikimedia.org/T100891#1326205 [16:18:39] 10Continuous-Integration-Infrastructure, 6Collaboration-Team, 10Flow: integration-slave-precise-1012 has a corrupted workspace for mediawiki-extensions-zend causing SpecialMobileDiffTest::testNames to fail with MWException: Diff not implemented for Flow\Conte... - https://phabricator.wikimedia.org/T100891#1322902 [17:03:50] 6Release-Engineering, 7user-notice: Shorten/Simplify MW train deploy cadence to Tu->W->Th - https://phabricator.wikimedia.org/T97553#1326344 (10Nemo_bis) I have troubles understanding the word "simplify". What's simpler? The difference from quickest to slowest, which is currently 100 %, will become 233 %. Ther... [17:12:48] 10Beta-Cluster, 6Labs, 10Labs-Infrastructure, 6operations, 6WMF-NDA: On Beta Cluster, add warning text + labs Term of Uses link to the motd files - https://phabricator.wikimedia.org/T100837#1326373 (10greg) [17:14:44] 10Browser-Tests, 6Release-Engineering: Track state of browser tests before Thursday branch cut - https://phabricator.wikimedia.org/T437#1326377 (10greg) >>! In T437#1324518, @mmodell wrote: > Shouldn't it be `tuesday branch cut` not `thursday branch cut`? now yes. [17:14:48] 10Beta-Cluster, 6Labs, 10Labs-Infrastructure, 6operations, 6WMF-NDA: On deployment-prep, add warning text + labs Term of Uses link to the motd files - https://phabricator.wikimedia.org/T100837#1326378 (10ori) [17:14:55] 10Browser-Tests, 6Release-Engineering: Track state of browser tests before new wmfXX branch cut - https://phabricator.wikimedia.org/T437#1326379 (10greg) [17:16:39] (03PS1) 10Aude: Add Paladox to the whitelist [integration/config] - 10https://gerrit.wikimedia.org/r/215064 [17:17:45] aude: noooooooo :/ [17:18:23] :/ [17:19:48] 10Beta-Cluster, 6Labs, 10Labs-Infrastructure, 6operations, 6WMF-NDA: On deployment-prep, add warning text + labs Term of Uses link to the motd files - https://phabricator.wikimedia.org/T100837#1326392 (10Joe) Proposal: ``` oo... [17:21:03] 6Release-Engineering, 7user-notice: Shorten/Simplify MW train deploy cadence to Tu->W->Th - https://phabricator.wikimedia.org/T97553#1326394 (10greg) >>! In T97553#1320304, @hashar wrote: > I have added {T437} as a blocker since we want to run smoke browser tests at some point. This isn't a blocker for this c... [17:21:14] 6Release-Engineering, 7user-notice: Shorten/Simplify MW train deploy cadence to Tu->W->Th - https://phabricator.wikimedia.org/T97553#1326395 (10greg) [17:21:15] 10Browser-Tests, 6Release-Engineering: Track state of browser tests before new wmfXX branch cut - https://phabricator.wikimedia.org/T437#1326396 (10greg) [17:21:42] 6Release-Engineering, 7user-notice: Shorten/Simplify MW train deploy cadence to Tu->W->Th - https://phabricator.wikimedia.org/T97553#1326397 (10greg) 5Open>3Resolved This is announced, scheduled, and on it's way for next week. Resolving now. [17:21:50] 10Beta-Cluster, 6Labs, 10Labs-Infrastructure, 6operations, 6WMF-NDA: On deployment-prep, add warning text + labs Term of Uses link to the motd files - https://phabricator.wikimedia.org/T100837#1326400 (10yuvipanda) Yes, and everytime someone breaks beta we can factually say they clubbed a baby seal to de... [17:29:28] (03CR) 10Krinkle: [C: 04-1] "User is sufficiently inexperienced that I would not trust him to not accidentally compromise our CI infrastructure." [integration/config] - 10https://gerrit.wikimedia.org/r/215064 (owner: 10Aude) [17:31:07] (03CR) 10Aude: "okay to wait on this :/" [integration/config] - 10https://gerrit.wikimedia.org/r/215064 (owner: 10Aude) [17:31:09] * aude sad [17:32:07] has to do recheck [18:00:21] 10Deployment-Systems, 10ReleaseTaggerBot: Have ReleaseTaggerBot automatically create new deploy branch (release) projects - https://phabricator.wikimedia.org/T100812#1326554 (10Aklapper) a:5Forrestbot>3None [Unassigning this task from a non-human assignee] [18:02:27] 10Deployment-Systems, 6Release-Engineering: Make sync-wikiverisons check that a valid localisation cache exists when syncing new versions - https://phabricator.wikimedia.org/T100573#1326582 (10mmodell) a:3mmodell [18:04:55] 10Deployment-Systems: scap eats underlying commands output (such as maintenance script stacktrace) - https://phabricator.wikimedia.org/T97140#1326593 (10mmodell) p:5Triage>3Normal [18:05:19] 10Deployment-Systems, 7Puppet: Trebuchet master should be separate from scap - https://phabricator.wikimedia.org/T96042#1326597 (10thcipriani) p:5Triage>3Low [18:08:18] legoktm: Do you know where the " # timeout=10" comments come from in jenkins outpout recently? [18:08:20] https://integration.wikimedia.org/ci/job/mwext-VisualEditor-npm/8597/console [18:08:24] The job hasn't changed. [18:08:44] Krinkle: I don't, I was wondering about that too. [18:08:54] 00:00:05.477 > git tag -a -f -m Jenkins Build #8597 jenkins-mwext-VisualEditor-npm-8597 # timeout=10 [18:08:56] that too [18:09:43] maybe a jenkins upgrade? [18:09:52] Looks like it also broke submodule updating [18:11:19] 10Deployment-Systems, 6Release-Engineering: Make sync-wikiversions check that a valid localisation cache exists when syncing new versions - https://phabricator.wikimedia.org/T100573#1326612 (10Aklapper) [18:15:01] (03CR) 10Paladox: "Hi what is this whitelist. and what does it do." [integration/config] - 10https://gerrit.wikimedia.org/r/215064 (owner: 10Aude) [18:17:30] (03CR) 10Legoktm: "Ping :)" [tools/scap] - 10https://gerrit.wikimedia.org/r/214288 (https://phabricator.wikimedia.org/T100600) (owner: 10Legoktm) [18:18:25] 10Beta-Cluster, 10Traffic, 6operations: Upgrade beta-cluster caches to jessie - https://phabricator.wikimedia.org/T98758#1326622 (10thcipriani) [18:21:31] PROBLEM - Puppet failure on integration-zuul-packaged is CRITICAL 100.00% of data above the critical threshold [0.0] [18:21:49] 10Deployment-Systems, 6Release-Engineering: make-release should run a syntax checker over the generated tarball - https://phabricator.wikimedia.org/T100599#1326632 (10thcipriani) p:5Triage>3Normal [18:23:25] (03CR) 10Ori.livneh: "It'd be better to have a dictionary which maps file extensions to validators, then have scap walk the file hierarchy that is to be synced," [tools/scap] - 10https://gerrit.wikimedia.org/r/214288 (https://phabricator.wikimedia.org/T100600) (owner: 10Legoktm) [18:26:29] (03CR) 10Aude: "@paladox the whitelist allows jenkins to run all the phpunit tests automatically." [integration/config] - 10https://gerrit.wikimedia.org/r/215064 (owner: 10Aude) [18:28:02] (03CR) 10Ori.livneh: [C: 031] "(This is fine for now, though.)" [tools/scap] - 10https://gerrit.wikimedia.org/r/214288 (https://phabricator.wikimedia.org/T100600) (owner: 10Legoktm) [18:30:45] addshore: there? [18:30:58] legoktm: there? [18:31:05] polybuildr: hey [18:31:22] the codesniffer repository *should* pass against itself in the next patch that I'm about to submit [18:31:34] (03PS9) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [18:31:53] (03CR) 10jenkins-bot: [V: 04-1] [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [18:31:57] Okay. :P [18:32:14] legoktm: Ah, no. This is an HHVM issue again. [18:32:17] The rest is passing! :D [18:34:02] (03CR) 10Paladox: "Ok." [integration/config] - 10https://gerrit.wikimedia.org/r/215064 (owner: 10Aude) [18:36:01] Hey, can I have some company in being excited here? :P [18:36:13] legoktm, addshore :P [18:36:57] Also, this needs a check: https://integration.wikimedia.org/ci/job/php-composer-test-hhvm/902/console [18:37:03] yayayaya!!! [18:37:17] wat [18:37:26] it's not dieing properly? [18:37:39] polybuildr: I think we should just pin to 2.3.0 for now, which doesn't have the bug [18:37:57] thank you, now I have company :P [18:38:39] legoktm: I'm missing something here. For all the other patches, the die works fine. But when the composer test is running for itself, that's when it fails. [18:38:53] hmm [18:39:01] let me look at your patch [18:40:15] polybuildr: oh, because phpcs itself is failing, and that doesn't call the phpunit bootstrap [18:40:55] polybuildr: so my hack only fixed it for phpunit [18:40:55] polybuildr: patch incomfing [18:40:55] incoming even [18:41:10] The die is output too :/ [18:41:12] I'm still confused. :P [18:41:13] But okay. [18:41:15] Patch patch patch. [18:43:15] Okaaay! I just understood. Both are getting output because while the bootstrap is dying, phpcs still fails. Okay. -.- [18:43:21] That took too long to figure out. [18:43:32] (03PS1) 10Legoktm: Use upstream codesniffer 2.3.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/215084 [18:45:09] legoktm: okay, so that works. Now someone needs to merge. [18:45:15] polybuildr: +1 it? [18:46:06] legoktm: gerrit puts the "," in the link too -.- [18:46:42] (03CR) 10Polybuildr: [C: 031] Use upstream codesniffer 2.3.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/215084 (owner: 10Legoktm) [18:46:43] legoktm: done :) [18:47:18] (03CR) 10Legoktm: [C: 032] "Per polybuildr" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/215084 (owner: 10Legoktm) [18:52:06] polybuildr: you probably need to rebase your patch on top of mine now [18:52:13] legoktm: willdo [18:52:17] waiting for jenkins to merge. [18:52:45] csteipp: Did my Phabricator task (for releases access) submit last Monday? I don't see anything on my side, just wanted to check before I try creating a new task. [18:53:13] Lcawte: I don't think I've seen it [18:54:01] what security setting was this? [18:54:06] (03Merged) 10jenkins-bot: Use upstream codesniffer 2.3.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/215084 (owner: 10Legoktm) [18:55:00] 10Beta-Cluster, 6Release-Engineering: Determine weekly triage meeting for Beta Cluster - https://phabricator.wikimedia.org/T98204#1326761 (10mmodell) 5Open>3Resolved [18:55:02] Lcawte, ^ [18:56:34] Pretty sure it was normal, I think Phabricator was just being weird and/or not paying attention and hitting a wrong button or something. It was pretty late in the evening after the hackathon had finished at that point. [18:57:06] 10Beta-Cluster, 7Blocked-on-RelEng, 10ContentTranslation-Deployments, 10MediaWiki-extensions-ContentTranslation, and 3 others: Setup new wikis in Beta Cluster for Content Translation - https://phabricator.wikimedia.org/T90683#1326767 (10mmodell) /poke what's the status of this? [18:57:10] (03PS10) 10Polybuildr: [WIP] Make phpcs pass against itself [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [18:57:21] I can't find anything like that, so either it didn't go in or I can't see it [18:57:50] I don't really know what that 'Access Request' security option does in terms of policy [18:57:59] 10Beta-Cluster, 10Continuous-Integration-Infrastructure, 10pywikibot-core: Run pywikibot test suite regularly on beta cluster as part of MediaWiki/Wikimedia CI - https://phabricator.wikimedia.org/T100903#1326768 (10mmodell) p:5Triage>3Normal [18:58:07] I assume it causes a private review blocker to be made, but I don't really know much more [18:59:42] Its fine, I'll create a new one. This is the instruction page I should be looking at isn't it? https://wikitech.wikimedia.org/wiki/Volunteer_NDA [19:00:41] legoktm: And we're done. :D [19:06:02] 10Deployment-Systems, 10Traffic, 6operations: Varnish cache busting desired for /static/$VERSION/ resources which change within the lifetime of a WMF release branch - https://phabricator.wikimedia.org/T99096#1326808 (10hashar) [19:06:04] 10Beta-Cluster, 10MediaWiki-ResourceLoader, 3Mobile-Web-Sprint-48-Voyage-of-the-Damned: ResourceLoader urls with certain version serving different content on beta cluster - https://phabricator.wikimedia.org/T100542#1315239 (10hashar) [19:08:33] 10Deployment-Systems, 10Traffic, 6operations: Varnish cache busting desired for /static/$VERSION/ resources which change within the lifetime of a WMF release branch - https://phabricator.wikimedia.org/T99096#1326811 (10ori) Note that /static/$VERSION/ resources are not used to serve JavaScript, unless Resour... [19:08:54] 10Beta-Cluster, 10MediaWiki-JobQueue: Job queue can't insert jobs on Beta Cluster - https://phabricator.wikimedia.org/T99311#1326812 (10mmodell) p:5Triage>3Normal [19:09:42] 10Beta-Cluster: Request for admin rights on http://fa.wikipedia.beta.wmflabs.org/ - https://phabricator.wikimedia.org/T100251#1326818 (10mmodell) p:5Triage>3Normal [19:10:26] 10Beta-Cluster: Add RecentActivityFeed extension to beta labs - https://phabricator.wikimedia.org/T71785#717948 (10hashar) @Ragesoss is RecentActivityFeed still a project or has it been abandoned? [19:11:36] 10Beta-Cluster: Add RecentActivityFeed extension to beta labs - https://phabricator.wikimedia.org/T71785#1326830 (10Ragesoss) @hashar: it's been abandoned. [19:12:21] 10Beta-Cluster: Add RecentActivityFeed extension to beta labs - https://phabricator.wikimedia.org/T71785#1326834 (10Ragesoss) [19:12:59] 10Beta-Cluster: Add RecentActivityFeed extension to beta labs - https://phabricator.wikimedia.org/T71785#1326836 (10Ragesoss) 5Open>3Invalid [19:14:02] polybuildr: woot, I'll look after lunch [19:14:25] legoktm: sure :) reviewing it myself once and then removing the [WIP] from the commit message [19:14:54] 10Deployment-Systems, 10Traffic, 6operations: Varnish cache busting desired for /static/$VERSION/ resources which change within the lifetime of a WMF release branch - https://phabricator.wikimedia.org/T99096#1326839 (10ori) Alternately ResourceLoader should use load.php to fetch resources even in debug mode. [19:16:27] 10Beta-Cluster, 10MediaWiki-JobRunner, 10MediaWiki-extensions-TimedMediaHandler, 6Multimedia: deployment-videoscaler01 PHP Fatal error: /srv/mediawiki/wikiversions-labs.cdb has no version entry for `webVideoTranscode%2Fdewiki/webVideoTranscode%2Fdewiki`.#0... - https://phabricator.wikimedia.org/T100694#1326858 [19:17:06] 10Beta-Cluster, 10MediaWiki-JobRunner, 10MediaWiki-extensions-TimedMediaHandler, 6Multimedia: deployment-videoscaler01 PHP Fatal error: /srv/mediawiki/wikiversions-labs.cdb has no version entry for `webVideoTranscode%2Fdewiki/webVideoTranscode%2Fdewiki`.#0... - https://phabricator.wikimedia.org/T100694#1318564 [19:18:02] 10Beta-Cluster: en.wikipedia.beta.wmflabs.org create account contains - https://phabricator.wikimedia.org/T100800#1326862 (10mmodell) p:5Triage>3Low [19:18:32] 10Beta-Cluster: Meeting: Automatic deployment of backend services on beta cluster - https://phabricator.wikimedia.org/T100099#1326867 (10hashar) a:5hashar>3None [19:19:30] 10Beta-Cluster, 6Labs: Make it possible to run public queries against databases on beta cluster - https://phabricator.wikimedia.org/T99456#1326869 (10mmodell) p:5Triage>3Normal [19:19:57] 10Beta-Cluster, 6Labs: Make it possible to run public queries against databases on beta cluster - https://phabricator.wikimedia.org/T99456#1326873 (10yuvipanda) What's the use case? [19:20:51] (03PS11) 10Polybuildr: Make mediawiki/tools/codesniffer pass phpcs [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [19:21:19] (03CR) 10Polybuildr: "Ready for review. :)" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [19:22:28] 10Beta-Cluster, 7Epic: Meeting: Automatic deployment of backend services on beta cluster - https://phabricator.wikimedia.org/T100099#1326881 (10hashar) This should be paired within #releng . Probably enough work to do to qualify it as an Epic. I have added it for our weekly team meeting on June 2nd 2015. [19:25:28] 10Beta-Cluster, 5Patch-For-Review: /var/lib/l10nupdate fills up deployment-bastion /var partition - https://phabricator.wikimedia.org/T95564#1326883 (10mmodell) 5Open>3Resolved a:3mmodell [19:25:53] 10Beta-Cluster, 5Patch-For-Review: /var/lib/l10nupdate fills up deployment-bastion /var partition - https://phabricator.wikimedia.org/T95564#1326885 (10hashar) I have originally filled this bug because the files were written at the wrong place. The deployment-bastion /var being too small is a different issue. [19:46:15] 10Deployment-Systems: Define Future Deployment Tooling Interface - https://phabricator.wikimedia.org/T101022#1326911 (10thcipriani) 3NEW [19:48:01] 10Deployment-Systems: Define Future Deployment Tooling Transport Mechanism - https://phabricator.wikimedia.org/T101023#1326918 (10thcipriani) 3NEW [19:49:06] 10Deployment-Systems: Future Deployment Tooling Modules - https://phabricator.wikimedia.org/T101024#1326926 (10thcipriani) 3NEW [19:58:44] (03CR) 10Legoktm: Make mediawiki/tools/codesniffer pass phpcs (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) (owner: 10Polybuildr) [20:08:06] (03PS12) 10Polybuildr: Make mediawiki/tools/codesniffer pass phpcs [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/214792 (https://phabricator.wikimedia.org/T92754) [20:10:42] legoktm: sorry, didn't notice. done. [20:21:10] wikibugs is too spammy [20:21:38] polybuildr: I have been busy triaging log errors all day [20:21:49] polybuildr: seems you are making progress on code sniffer \O/ [20:23:54] thanks for dealing with those hashar [20:24:30] Krenair: going to talk about it during our releng weekly meeting [20:24:50] probably need a community wide attention :-} [20:24:56] hashar: Glad to be of help. The patch I've been working on for a couple of days isn't really helping, though. :P [20:25:19] PROBLEM - Puppet staleness on deployment-restbase01 is CRITICAL 11.11% of data above the critical threshold [43200.0] [20:25:42] hashar: But yes, I intend to improve codesniffer. [20:25:42] polybuildr: is that part of some google summer of code ? [20:25:48] What log errors were these? [20:26:14] hashar: I am a GSoC student, but this is not my project. [20:26:20] :-} [20:26:23] My project is the mediawiki/extensions/SmiteSpam extension. [20:26:39] at wikimedia, the backend error logs are sent to an utility named LogStash [20:26:46] which uses ElasticSearch has a backend [20:26:54] that make it trivial to filter / find log files :} [20:27:08] we look at them and manually fill bug for such errors. The result can be seen at https://phabricator.wikimedia.org/tag/wikimedia-log-errors/ [20:28:08] WOW. [20:28:19] to give you an idea, over the last 15 minutes we received 900 000 lines of log [20:28:44] there is a bunch of debug / info lines though [20:28:54] Still a really big number. :P [20:29:08] PROBLEM - Puppet staleness on deployment-restbase02 is CRITICAL 100.00% of data above the critical threshold [43200.0] [20:29:23] Okay, must leave. I'll talk to you later. :) [20:30:10] hey releng [20:30:42] i see this comment: "I have poked RelEng internal mailing list about it." from hashar [20:30:57] regarding shell on gerrit server [20:31:08] hashar: hi [20:31:18] can i go forward and merge your patch? [20:31:26] got an ok in ops meeting [20:32:30] goes ahead [20:33:46] mutante: hello :-} [20:34:21] mutante: glad to hear it has been approved during ops meeting! [20:34:34] mutante: we will probably need some additional sudo rights later on but I have no clue which yet [20:35:05] hashar: i merged the "just shell for log access" then [20:35:20] runs puppet [20:36:20] hashar: ok, we can later adjust the rights of "gerrit admin" group, yea [20:37:33] mutante: should run puppet on ytterbium.wikimedia.org and antimony.wikimedia.org [20:37:51] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1327058 (10Dzahn) a:3Dzahn [20:39:26] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1315740 (10Dzahn) Has been brought up in ops meeting today. No concerns have been raised. (for non-root acc... [20:40:02] hashar: yes, surprisingly no change yet on ytterbium [20:41:29] mutante: maybe hiera needs to be poked [20:42:35] mutante: gotta sleep now, but you can follow up on https://phabricator.wikimedia.org/T100565 [20:42:36] :-} [20:43:31] hashar: users created on antimony [20:43:39] hashar: will do. good night [20:48:16] mutante: antimony works for me :) [20:48:29] ytterbium pending :-} [20:48:31] I am off see you [20:48:34] and thanks [20:48:53] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1327076 (10Dzahn) This created the users on node "antimony" (which includes role gitblit and role subversio... [20:49:18] yea, so we have an admin group called gerrit-admin [20:49:30] which creates users on a node that doesn't include a gerrit role [20:49:38] but does not create them on the node that has the gerrit role [20:49:42] :o [20:49:57] 10Beta-Cluster, 7Epic: Meeting: Automatic deployment of backend services on beta cluster - https://phabricator.wikimedia.org/T100099#1327079 (10hashar) [20:49:58] 10Beta-Cluster, 6Release-Engineering, 10Continuous-Integration-Config, 10Parsoid: Parsoid patches don't update Beta Cluster automatically -- only deploy repo patches seem to update that code - https://phabricator.wikimedia.org/T92871#1327078 (10hashar) [20:50:58] 10Beta-Cluster, 6Release-Engineering, 10Continuous-Integration-Config, 10Parsoid: Parsoid patches don't update Beta Cluster automatically -- only deploy repo patches seem to update that code - https://phabricator.wikimedia.org/T92871#1122266 (10hashar) Some of us had a meeting about it during the hackathon... [20:53:39] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1327084 (10Dzahn) hiera data: role/common/gitblit.yaml: - gerrit-admin role/common/gerrit/production.yaml... [20:54:34] 10Beta-Cluster, 7Epic: Meeting: Automatic deployment of backend services on beta cluster - https://phabricator.wikimedia.org/T100099#1327086 (10cscott) I found probably mention here the `auto mode` I implemented for git-deploy, which would allow its use for beta deploys. [20:55:42] JohnFLewis: eh.. now it works [20:55:51] delayed somehow [20:57:42] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1327087 (10Dzahn) 5Open>3Resolved it was just delayed somehow. now also on ytterbium: Notice: /Stage[... [21:04:21] 6Release-Engineering, 10Ops-Access-Requests, 6operations, 5Patch-For-Review: Grant access for aklapper to phab-admins - https://phabricator.wikimedia.org/T97642#1327112 (10chasemp) @aklapper, are you back? can you generate some ssh keys for prod? [21:07:05] 6Release-Engineering, 10Ops-Access-Requests, 10Wikimedia-Git-or-Gerrit, 6operations, 5Patch-For-Review: Add all Release-Engineering team as Gerrit admins - https://phabricator.wikimedia.org/T100565#1327120 (10hashar) I can log on both hosts. Thank you @Dzahn [21:09:05] Project browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #656: FAILURE in 43 min: https://integration.wikimedia.org/ci/job/browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce/656/ [21:14:57] 6Release-Engineering: "Welcome to WMF" information for new devs about releasing, CI, and QA - https://phabricator.wikimedia.org/T101031#1327164 (10greg) 3NEW [21:17:26] PROBLEM - Puppet staleness on deployment-eventlogging02 is CRITICAL 100.00% of data above the critical threshold [43200.0] [21:23:06] 10Deployment-Systems, 6Release-Engineering: Clean up erroneously created wmf/1.20wmf21 branches - https://phabricator.wikimedia.org/T92501#1327206 (10mmodell) 5Open>3Resolved Done.. [21:25:51] 10Deployment-Systems, 6Release-Engineering, 5Patch-For-Review: scap should lint JSON files - https://phabricator.wikimedia.org/T100600#1327212 (10mmodell) >>! In T100600#1316976, @hashar wrote: > maybe we need a generic linter job that would be run on all repositories? +1 Differential does that client side... [22:06:33] 10Deployment-Systems, 6Release-Engineering, 5Patch-For-Review: scap should lint JSON files - https://phabricator.wikimedia.org/T100600#1327293 (10Legoktm) >>! In T100600#1327279, @gerritbot wrote: > Change 215216 had a related patch set uploaded (by Legoktm): > Fix Echo and extension1 creation in addWiki.php... [22:13:46] 6Release-Engineering: "Welcome to WMF" information for new devs about releasing, CI, and QA - https://phabricator.wikimedia.org/T101031#1327299 (10hashar) First day checklist for new devs: [ ] desk and laptop [ ] mentor assigned [ ] deploy code More seriously, maybe we can ask the recent hires. They still hav... [22:14:49] [ ] take down cluster [22:27:11] 10Beta-Cluster, 6Labs: Make it possible to run public queries against databases on beta cluster - https://phabricator.wikimedia.org/T99456#1327335 (10hashar) 5Open>3declined a:3hashar Beta contains private data such as users IP addresses and emails. That is covered by the https://wikitech.wikimedia.org/w... [22:27:19] 6Release-Engineering: "Welcome to WMF" information for new devs about releasing, CI, and QA - https://phabricator.wikimedia.org/T101031#1327338 (10greg) I just spoke with Stephen from WMF Legal about something completely different and I brought this up to get his feedback, and we thought it might also be good to... [22:41:31] (03PS1) 10Dduvall: Render tags and links to SauceLabs sessions [integration/raita] - 10https://gerrit.wikimedia.org/r/215221 [22:41:41] (03CR) 10jenkins-bot: [V: 04-1] Render tags and links to SauceLabs sessions [integration/raita] - 10https://gerrit.wikimedia.org/r/215221 (owner: 10Dduvall) [23:05:29] Project browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #690: FAILURE in 54 min: https://integration.wikimedia.org/ci/job/browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/690/ [23:42:52] (03PS1) 10Dduvall: Highlight step arguments [integration/raita] - 10https://gerrit.wikimedia.org/r/215237 [23:43:03] (03CR) 10jenkins-bot: [V: 04-1] Highlight step arguments [integration/raita] - 10https://gerrit.wikimedia.org/r/215237 (owner: 10Dduvall) [23:54:07] 10Browser-Tests, 6Release-Engineering, 6Mobile-Web: Introduce @skip tag in mediawiki selenium - https://phabricator.wikimedia.org/T101062#1327642 (10Jdlrobson) 3NEW