[00:37:40] twentyafterfour: and greg-g i think i have made a installer that can install arc onto windows and set the enviroment variable making it one click install maybe a few clicks but a user dosen't need to go through the hassel of adding arc bin to the path. [00:39:14] Yippee, build fixed! [00:39:14] Project selenium-Flow » chrome,beta,Linux,,contintLabsSlave && UbuntuTrusty build #12: 09FIXED in 23 min: https://integration.wikimedia.org/ci/job/selenium-Flow/BROWSER=chrome,MEDIAWIKI_ENVIRONMENT=beta,PLATFORM=Linux,VERSION=,label=contintLabsSlave%20&&%20UbuntuTrusty/12/ [01:10:03] RECOVERY - Puppet run on deployment-ores-web is OK: OK: Less than 1.00% above the threshold [0.0] [01:37:33] RECOVERY - Puppet run on deployment-sca01 is OK: OK: Less than 1.00% above the threshold [0.0] [02:03:24] PROBLEM - Puppet run on deployment-sca01 is CRITICAL: CRITICAL: 60.00% of data above the critical threshold [0.0] [03:12:58] Yippee, build fixed! [03:12:58] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #931: 09FIXED in 22 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/931/ [03:16:15] !log manually rebooted deployment-ores-web and deployment-sca01 [03:16:21] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [04:24:32] Yippee, build fixed! [04:24:32] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-9-sauce build #770: 09FIXED in 32 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-9-sauce/770/ [04:32:59] 10Beta-Cluster-Infrastructure, 6Revision-Scoring-As-A-Service, 10ores, 13Patch-For-Review: Setup ORES service in beta cluster - https://phabricator.wikimedia.org/T130404#2173816 (10Ladsgroup) Everything is okay now. We only need these patches merged to go to prod [06:20:35] Project browsertests-Core-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce-T89342 build #1: 09SUCCESS in 8 min 34 sec: https://integration.wikimedia.org/ci/job/browsertests-Core-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce-T89342/1/ [06:35:59] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2173836 (1001tonythomas) Is there a chance to get this done during the #wikimedia-hackathon-2016 ? I think most of the blockers are resolved for deploy... [06:50:15] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2173859 (10ori) >>! In T127297#2173836, @01tonythomas wrote: > Is there a chance to get this done during the #wikimedia-hackathon-... [06:53:50] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2173877 (1001tonythomas) >>! In T127297#2173859, @ori wrote: >>>! In T127297#2173836, @01tonythomas wrote: >> Is there a chance to... [06:58:34] legoktm: Any chance I can bully you into merge/deploying an integration/config patch? :-) [06:59:08] (Specifically, https://gerrit.wikimedia.org/r/#/c/281079/ so we can merge VE support for Wikisource.) [06:59:56] (03PS2) 10Legoktm: Adds to ProofreadPage a dependence to the VisualEditor [integration/config] - 10https://gerrit.wikimedia.org/r/281079 (owner: 10Tpt) [07:00:28] (03CR) 10Legoktm: [C: 032] Adds to ProofreadPage a dependence to the VisualEditor [integration/config] - 10https://gerrit.wikimedia.org/r/281079 (owner: 10Tpt) [07:01:48] (03Merged) 10jenkins-bot: Adds to ProofreadPage a dependence to the VisualEditor [integration/config] - 10https://gerrit.wikimedia.org/r/281079 (owner: 10Tpt) [07:01:59] !log deploying https://gerrit.wikimedia.org/r/281079 [07:02:02] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [07:02:48] James_F: done [07:03:08] legoktm: You're awesome. :-) [08:32:55] Yippee, build fixed! [08:32:56] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce build #929: 09FIXED in 22 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce/929/ [09:10:28] 10Deployment-Systems, 3Scap3: Create `deploy-init` command for scap3 - https://phabricator.wikimedia.org/T129906#2174726 (10mobrovac) Oh, right. Good point @mmodell. Maybe running `deploy --init` can be done as part of the initial clone on the deployment server. In that case, we'd just need to force a Puppet r... [09:34:55] PROBLEM - Host integration-trusty-1026 is DOWN: CRITICAL - Host Unreachable (10.68.17.98) [11:22:32] 10Continuous-Integration-Config, 10MediaWiki-extensions-Scribunto, 10Wikidata: [Task] Add Scribunto to extension-gate in CI - https://phabricator.wikimedia.org/T125050#2175411 (10Lydia_Pintscher) p:5Triage>3Normal [11:31:02] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2175441 (10ori) a:3ori I'm still waiting to hear back from the Security team regarding whether or not my security review is suff... [11:35:08] twentyafterfour: greg-g: The installer i created for arc works. It will need updating with the files when ever arc is updated but it now installs the required arc files, php and git is up to the user to install. But this installer also registers the path. [11:35:12] 7Browser-Tests, 10MediaWiki-extensions-WikibaseRepository, 10Wikidata: [Task] enable rubocop in Wikibase - https://phabricator.wikimedia.org/T131139#2175459 (10Lydia_Pintscher) p:5Triage>3Normal [11:38:26] Dereckson: Hi i managed to create a windows installer for arc that works. [11:39:38] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2175474 (10ori) Done in https://gerrit.wikimedia.org/r/#/c/281288/ , waiting for scap to run on beta. [11:39:58] paladox: you install Arcanist + PHP ? [11:39:59] paladox: have you tested it [11:41:22] FlorianSW: Yes works, Dereckson It only installs Arcanist. Doing php would require alot more time since alot of coding involved with Detecting your current php if you have one, then choosing the right version so x86 or x64 [11:42:24] Dereckson: Maybe you could help because i actually tryed to do that but was confussing because i doint know how to detect if the key php exist in the enviromental variable [11:43:49] Detecting x86/x64 is trivial [11:44:05] detecting PHP is potentially complex [11:44:07] if in path, easy [11:44:12] If it's not, you could literally have it anywhere [11:44:26] You could ask for a path to php.exe [11:44:52] Reedy: I could install it, but then extracting it from a zip file. <- php [11:45:23] Reedy: I think we can re use code from composer since they do that and use the same installer im using. [11:50:48] It's okay not to install it, and actually it would be more flexible, but installers tend to check if all the requirements for an app are met. [11:52:00] ofc, for all it'd probably matter, you could just do x86 php unconditionally [11:52:03] To ask for php.exe like Reedy offers another benefit: you can add the PHP directory in the path, or you can tweak the arc.bat launcher so it uses the full path instead of "php" [11:52:11] 10Continuous-Integration-Infrastructure, 10MediaWiki-extensions-Scribunto, 10Wikidata: [Bug] Scribuntu fails CI on 1.26 branch on php 5.3 - https://phabricator.wikimedia.org/T129854#2118067 (10Lydia_Pintscher) p:5Triage>3Normal [11:52:18] Dereckson: Oh, how would i do that since composer has huge files and im not sure which bits i should do. [11:52:22] Or just include the zip yourslf, and control the versions [11:52:50] PHabricator staff doesn't like Composer, so no hope there. [11:52:59] Reedy and Dereckson, i can get it to download, and get the user to unzip it and add it to path. [11:53:22] See https://secure.phabricator.com/T6012 for Composer [11:54:03] Dereckson, im talking about the php bit, composer can detect php. [11:54:15] well [11:54:24] Composer is in PHP, not compiled afaik [11:54:52] Oh [11:55:31] yeah [11:55:33] it's a phar [11:55:35] php archive [11:55:44] it's just loads of php files that you can move around as one file [11:55:47] for ease [11:56:04] ok [11:56:37] 7Browser-Tests, 10Wikidata: [Task] Find a sensible subset of browsertests that could be run on each commit - https://phabricator.wikimedia.org/T130019#2122791 (10Lydia_Pintscher) p:5Triage>3High [11:57:08] 7Browser-Tests, 10Browser-Tests-Infrastructure, 10Continuous-Integration-Config, 10Wikidata, 3Wikidata-Sprint-2016-03-01: [Task] Remove Wikidata performance-tests jobs - https://phabricator.wikimedia.org/T130017#2122670 (10Lydia_Pintscher) p:5Triage>3Normal [11:59:14] Reedy and Dereckson something like http://www.jrsoftware.org/ishelp/topic_isxfunc_regkeyexists.htm detecting if php is added to your path if not add it. [12:01:26] http://stackoverflow.com/questions/23337207/how-do-i-check-if-registry-key-exists-and-exit-setup [12:03:29] (03CR) 10Daniel Kinzler: "Anyone willing to give a V+2 in addition to the CR+2?" [tools/code-utils] - 10https://gerrit.wikimedia.org/r/190825 (owner: 10Daniel Kinzler) [12:03:44] (03CR) 10Reedy: [C: 032 V: 032] Utility script for trimming i18n files. [tools/code-utils] - 10https://gerrit.wikimedia.org/r/190825 (owner: 10Daniel Kinzler) [12:05:15] 10Continuous-Integration-Config, 10OOjs-UI (OOjs-UI-0.16.5), 13Patch-For-Review: Demos is totally broken; files not present in the rsync? - https://phabricator.wikimedia.org/T130678#2175644 (10Jdforrester-WMF) [12:16:52] Reedy, Dereckson, FlorianSW see https://github.com/paladox/Arcanist-installer-for-windows/tree/master/Arcanist and https://github.com/paladox/Arcanist-installer-for-windows/releases please. [12:17:00] twentyafterfour and greg-g ^^ [12:20:50] Project beta-update-databases-eqiad build #7583: 04FAILURE in 49 sec: https://integration.wikimedia.org/ci/job/beta-update-databases-eqiad/7583/ [12:44:25] 7Browser-Tests, 10VisualEditor, 5WMF-deploy-2016-03-15_(1.27.0-wmf.17), 5WMF-deploy-2016-03-22_(1.27.0-wmf.18): Fix language screenshots browser test on VisualEditor - https://phabricator.wikimedia.org/T119482#2175841 (10Jdforrester-WMF) [12:49:50] Dereckson ive uploaded the setup.exe file at https://github.com/paladox/Arcanist-installer-for-windows/releases [12:56:49] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2175895 (10Luke081515) 5Open>3Resolved Checked out at deploymentwiki: Works. [13:18:21] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 10Wikimedia-Hackathon-2016, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2176014 (10Aklapper) [13:20:48] Yippee, build fixed! [13:20:48] Project beta-update-databases-eqiad build #7584: 09FIXED in 47 sec: https://integration.wikimedia.org/ci/job/beta-update-databases-eqiad/7584/ [13:45:42] Reedy and Dereckson and FlorianSW i have updated the docs at with this section now, https://www.mediawiki.org/wiki/Phabricator/Arcanist#Windows_installation [13:45:53] ;) [13:45:55] :) [13:46:16] i Have't tested it on Windows 32bit due to me not having a 32bit pcs but 64-bit it works. [13:46:42] twentyafterfour greg-g ^^ [13:51:12] You won't have `arc anoid`. [13:51:50] paladox: Can you import the other libraries via submodules rather than just arbitarily copying them in? [13:52:10] Reedy yes. [13:54:37] Reedy i will do that now [13:54:53] Dereckson why [13:55:13] Python [13:56:08] Dereckson oh, would arc land work because it is failing for me with something to do proc [15:06:18] Reedy and Dereckson ive added the use of submodules at https://github.com/paladox/Arcanist-installer-for-windows and ive also release the 1.0.1 update to include them. [15:13:17] twentyafterfour and greg-g im getting this error on windows when using arc land [15:13:17] [2016-04-03 08:06:28] EXCEPTION: (Exception) Failed to passthru proc_open(): proc_open(): CreateProcess failed, error code - 87 at [\src\future\exec\PhutilExecPassthru.php:99] [15:13:18] arcanist(), phutil() [15:13:18] #0 PhutilExecPassthru::execute() called at [\src\future\exec\execx.php:50] [15:13:18] #1 phutil_passthru(string, string, string) called at [\src\repository\api\ArcanistGitAPI.php:46] [15:13:20] #2 ArcanistGitAPI::execPassthru(string, string, string) called at [\src\land\ArcanistGitLandEngine.php:139] [15:13:25] #3 ArcanistGitLandEngine::fetchTarget() called at [\src\land\ArcanistGitLandEngine.php:14] [15:13:28] #4 ArcanistGitLandEngine::execute() called at [\src\workflow\ArcanistLandWorkflow.php:296] [15:13:31] #5 ArcanistLandWorkflow::run() called at [\scripts\arcanist.php:392] [15:13:40] Dereckson ^^ [15:16:25] According to https://msdn.microsoft.com/en-us/library/ms681382(v=vs.85).aspx it is ERROR_INVALID_PARAMETER [15:19:51] FlorianSW: Ive created the topic here https://www.mediawiki.org/wiki/Talk:Phabricator/Arcanist [15:26:45] paladox: you're going to port every piece of software for Windows by the way? There are grunt, gulp, phpcs, selenium, etc. :D [15:27:09] It could be a Danaides' barrel. [15:27:45] Dereckson: Oh im not sure, but we could if you want, but even if we did there would be no point since npm already does that and composer. [15:27:58] Dereckson do you know why arc land is not working for me. [15:29:12] proc_open is when it tries to open a process, could be a path issue [15:30:26] Dereckson oh [15:48:07] PROBLEM - App Server bits response on deployment-mediawiki02 is CRITICAL: CRITICAL - Socket timeout after 10 seconds [16:04:26] Dereckson it seems phabricator doint offically support php 7. https://secure.phabricator.com/T10717 [18:05:32] PROBLEM - Host cache-rsync is DOWN: CRITICAL - Host Unreachable (10.68.23.165) [18:38:01] twentyafterfour: Hi im having problems with arc land on windows. [19:02:49] Dereckson i fixed the problem. :) [19:03:29] Dereckson just i have to do arc diff first then arc land after. [19:47:32] 10Beta-Cluster-Infrastructure, 10MediaWiki-extensions-Newsletter, 10Wikimedia-Hackathon-2016, 13Patch-For-Review: Add the Newsletter extension to the Beta Cluster - https://phabricator.wikimedia.org/T127297#2176405 (10Qgil) THIS IS IMPRESSIVE! Thank you very much to all the regular and spontaneous contribu... [20:47:45] Project browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #974: 09SUCCESS in 21 min: https://integration.wikimedia.org/ci/job/browsertests-Flow-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce/974/ [21:12:23] (03PS1) 10Jean-Frédéric: Add jsonlint and php55lint pipelines for heritage [integration/config] - 10https://gerrit.wikimedia.org/r/281330 (https://phabricator.wikimedia.org/T131597) [21:23:59] 10Continuous-Integration-Config, 13Patch-For-Review: Whitelist Lokal Profil for CI - https://phabricator.wikimedia.org/T131558#2176605 (10Lokal_Profil) [22:16:35] 5Gerrit-Migration, 10Differential, 10Phabricator, 13Patch-For-Review: Allow liberal close in Differential - https://phabricator.wikimedia.org/T131623#2176795 (10mmodell) During our use of differential for developing scap, we have landed other people's patches without issue. @demon, @dduvall, @thcipriani: h... [22:17:04] 5Gerrit-Migration, 5Gitblit-Deprecate, 6Release-Engineering-Team, 10releng-201516-q3, and 4 others: [RfC]: Migrate code review / management to Phabricator from Gerrit - https://phabricator.wikimedia.org/T119908#2176799 (10JanZerebecki) T130949 implies that zuul will be dropped. Will its features like prepa... [22:38:08] 5Gerrit-Migration, 5Gitblit-Deprecate, 6Release-Engineering-Team, 10releng-201516-q3, and 4 others: [RfC]: Migrate code review / management to Phabricator from Gerrit - https://phabricator.wikimedia.org/T119908#1839726 (10Paladox) @JanZerebecki probally because harbormaster's is not fully feature ready plu... [22:52:36] 10Continuous-Integration-Config, 6Operations, 13Patch-For-Review: Switch CI from jsduck deb package to a gemfile/bundler system - https://phabricator.wikimedia.org/T109005#2176878 (10Krinkle) I'd like to offer an alternative to adding a Gemfile everywhere. Repositories should not have multiple test entry po... [22:54:14] (03PS1) 10Krinkle: doc.wikimedia.org: Add link to TemplateData [integration/docroot] - 10https://gerrit.wikimedia.org/r/281378 (https://phabricator.wikimedia.org/T126951) [22:54:34] (03CR) 10Krinkle: [C: 032] doc.wikimedia.org: Add link to TemplateData [integration/docroot] - 10https://gerrit.wikimedia.org/r/281378 (https://phabricator.wikimedia.org/T126951) (owner: 10Krinkle) [22:55:17] (03Merged) 10jenkins-bot: doc.wikimedia.org: Add link to TemplateData [integration/docroot] - 10https://gerrit.wikimedia.org/r/281378 (https://phabricator.wikimedia.org/T126951) (owner: 10Krinkle) [23:11:49] (03CR) 10Krinkle: "See T109005" [integration/config] - 10https://gerrit.wikimedia.org/r/278443 (owner: 10Paladox) [23:12:52] (03Abandoned) 10Paladox: [MobileFrontend] Add test npm-run-doc [integration/config] - 10https://gerrit.wikimedia.org/r/278443 (owner: 10Paladox) [23:56:42] Yippee, build fixed! [23:56:43] Project browsertests-CentralNotice-en.wikipedia.beta.wmflabs.org-windows_7-chrome-sauce build #380: 09FIXED in 41 sec: https://integration.wikimedia.org/ci/job/browsertests-CentralNotice-en.wikipedia.beta.wmflabs.org-windows_7-chrome-sauce/380/