[00:04:09] legoktm: Browser tests broken? [00:04:32] it doesn't even run them [00:04:48] James_F: like, there are no tests in the repo afais [00:05:56] legoktm: Oh, in that case, disable the job and I'll merge? [00:06:36] James_F: they appear to exist on the "dev" branch? https://gerrit.wikimedia.org/r/#/c/235050/ I don't get what's going on here [00:06:49] legoktm: I can force-merge instead? [00:07:02] jdlrobson: Do you know? [00:11:11] legoktm: Also, apparently BoilerPlate, WikiGrok, CentralAuth, ExtensionDistributor, MassMessage, MsUpload and SyntaxHighlight_Pygments, and the Blueprint skin are all awaiting upgrading too. ;-) [00:11:43] hrm [00:12:02] According to User:Legoktm/ci ;-) [00:12:04] WikiGrok is "archived" in CI now, and the tests were already failing, so I skipped [00:12:09] That's fair. [00:12:16] BoilerPlate also had something broken about it [00:12:16] Maybe de-list it from the dashboard? [00:12:36] I probably have local changes in CA, ED, and MM which is why the script skipped them [00:12:43] MsUpload is also failing [00:12:49] * James_F nods. [00:12:49] and _Pygments should just be deleted [00:12:55] I'll file a bug for that [00:15:05] legoktm: WFM. [01:37:24] PROBLEM - Free space - all mounts on deployment-bastion is CRITICAL deployment-prep.deployment-bastion.diskspace._var.byte_percentfree (<33.33%) [02:33:30] 10Deployment-Systems, 6Release-Engineering, 6Performance-Team, 6operations, 7HHVM: Translation cache exhaustion caused by changes to PHP code in file scope - https://phabricator.wikimedia.org/T103886#1605232 (10Krinkle) [02:45:51] 10Gerrit-Migration, 10Differential, 10Phabricator-Upstream: No obvious way to remove self from differential reviewers - https://phabricator.wikimedia.org/T106371#1605245 (10greg) [02:49:14] 10Deployment-Systems, 10MediaWiki-Internationalization, 6Performance-Team, 7Performance: Experiment with plain .php files for l10n cache instead of CDB - https://phabricator.wikimedia.org/T99740#1605271 (10Krinkle) 5Open>3Resolved a:3Krinkle Per T108638, this seems resolved. [02:49:18] 10Deployment-Systems, 10Wikimedia-Site-Requests, 5Patch-For-Review: Convert wikiversions from CDB to static array file - https://phabricator.wikimedia.org/T108638#1525674 (10Krinkle) [02:49:22] 10Deployment-Systems, 10MediaWiki-Internationalization, 6Performance-Team, 7Performance: Experiment with plain .php files for l10n cache instead of CDB - https://phabricator.wikimedia.org/T99740#1605278 (10Krinkle) [03:07:30] (03PS1) 10Legoktm: Add zuul-cloner map for parsoidsvc jobs [integration/jenkins] - 10https://gerrit.wikimedia.org/r/235964 (https://phabricator.wikimedia.org/T111469) [03:10:12] (03PS1) 10Legoktm: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) [03:10:42] (03PS2) 10Legoktm: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) [03:10:56] (03CR) 10Legoktm: "Depends on Ida89f44dde9668edca3af48cfdcd308b3de46df0 in integration/jenkins" [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [03:18:23] Project browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #806: FAILURE in 36 min: https://integration.wikimedia.org/ci/job/browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce/806/ [03:25:58] 10Deployment-Systems, 10ReleaseTaggerBot, 5Patch-For-Review: Rename "Forrestbot" to "ReleaseTaggerBot" - https://phabricator.wikimedia.org/T100811#1605357 (10Ricordisamoa) ```lang=javascript function rename( str ) { return str.replace( /Forrest(\w)/, function ( m, a ) { return 'ReleaseTagger' + a.toUpperC... [03:51:04] (03PS1) 10Krinkle: Remove unused qunit-mobile jobs [integration/config] - 10https://gerrit.wikimedia.org/r/235969 [03:51:37] legoktm: There's several old -qunit jobs (non-mobile) as well that don't use plain. need to check those as well. [03:51:39] I'll do that tomorrow [03:51:43] Or you can :) [03:51:54] sure [04:03:02] (03CR) 10Krinkle: [C: 032] Remove unused qunit-mobile jobs [integration/config] - 10https://gerrit.wikimedia.org/r/235969 (owner: 10Krinkle) [04:03:42] (03CR) 10Krinkle: "No layout difference. Deleted mwext-MobileFrontend-qunit-mobile and mwext-Thanks-qunit-mobile in Jenkins." [integration/config] - 10https://gerrit.wikimedia.org/r/235969 (owner: 10Krinkle) [04:04:51] (03Merged) 10jenkins-bot: Remove unused qunit-mobile jobs [integration/config] - 10https://gerrit.wikimedia.org/r/235969 (owner: 10Krinkle) [05:35:23] Yippee, build fixed! [05:35:24] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-11-sauce build #531: FIXED in 33 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-11-sauce/531/ [06:37:25] RECOVERY - Free space - all mounts on deployment-bastion is OK All targets OK [06:41:38] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1605471 (10Qgil) If this is not Resolved, then I would consider resolving it as Declined. We are not going to work on those packages ourselves, but also... With Wikimedia Phabricator following upstream'... [07:41:44] PROBLEM - Puppet staleness on deployment-restbase01 is CRITICAL 100.00% of data above the critical threshold [43200.0] [07:48:14] 10Browser-Tests: Tutorial to setup a Raita dev environment - https://phabricator.wikimedia.org/T111493#1605559 (10hashar) 3NEW [08:10:07] 10Browser-Tests: Investigate distribution of browser test run time - https://phabricator.wikimedia.org/T104396#1605581 (10hashar) Awesome @dduvall nanoseconds totally make sense :-} *giggles*. I am subscribing to T102536 right now! [08:11:55] 10Browser-Tests: Display date/time and timing info for Raita builds - https://phabricator.wikimedia.org/T102536#1605583 (10hashar) From Raita `db/mappings/build.json` the duration is available as "result.duration": ``` lang=json { "build": { "properties": { "number": { "type": "string" },... [08:21:04] 5Continuous-Integration-Scaling, 7Nodepool: Nodepool should send metrics to statsd - https://phabricator.wikimedia.org/T111496#1605602 (10hashar) 3NEW [08:37:55] 5Continuous-Integration-Scaling, 7Nodepool: Nodepool should send metrics to statsd - https://phabricator.wikimedia.org/T111496#1605642 (10hashar) a:3hashar [08:58:12] (03CR) 10Hashar: [C: 04-1] "Great idea! The .apk build should be done after the change has been merged by using the pipeline 'postmerge'." (032 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [08:59:54] (03PS2) 10Hashar: Reduce Android emulator boot time [integration/config] - 10https://gerrit.wikimedia.org/r/235833 (owner: 10Niedzielski) [09:02:19] (03CR) 10Hashar: [C: 032] "I have cherry picked the patch on the tip of master, the previous parent patch ( https://gerrit.wikimedia.org/r/#/c/231624/2 Publish Andro" [integration/config] - 10https://gerrit.wikimedia.org/r/235833 (owner: 10Niedzielski) [09:03:57] (03CR) 10Hashar: "I manually triggered a couple build of the job (1447 and 1448). The build time trend should show an enhancement at:" [integration/config] - 10https://gerrit.wikimedia.org/r/235833 (owner: 10Niedzielski) [09:04:02] (03Merged) 10jenkins-bot: Reduce Android emulator boot time [integration/config] - 10https://gerrit.wikimedia.org/r/235833 (owner: 10Niedzielski) [09:08:14] 10Gerrit-Migration: Allow viewing diffs of single commits within a Differential - https://phabricator.wikimedia.org/T125#1605692 (10QChris) >>! In T125#1603573, @greg wrote: > Is this simply https://phabricator.wikimedia.org/D1?vs=15&id=16&whitespace=ignore-most#toc ? (That shows the diff between commit 15 and 1... [09:12:02] 10Gerrit-Migration: Allow viewing diffs of single commits within a Differential - https://phabricator.wikimedia.org/T125#1605697 (10QChris) [09:20:54] 5Continuous-Integration-Scaling, 7Nodepool: Nodepool should send metrics to statsd - https://phabricator.wikimedia.org/T111496#1605758 (10hashar) [09:22:03] 5Continuous-Integration-Scaling, 7Nodepool: Nodepool should send metrics to statsd - https://phabricator.wikimedia.org/T111496#1605765 (10hashar) 5Open>3stalled Per discussion on https://gerrit.wikimedia.org/r/#/c/235989/ Nodepool sends too many metrics which is going to overload our Statsd server eventual... [09:22:15] 10Beta-Cluster, 10MediaWiki-extensions-OAuth, 10Pywikibot-OAuth, 7Pywikibot-network, 7Pywikibot-tests: Nonce already used regularly occurring on beta cluster - https://phabricator.wikimedia.org/T109173#1605767 (10jayvdb) https://travis-ci.org/jayvdb/pywikibot-core/jobs/78718812#L721 [09:23:23] 5Continuous-Integration-Scaling, 7Nodepool: Document Nodepool statsd metrics - https://phabricator.wikimedia.org/T111503#1605770 (10hashar) 3NEW a:3hashar [09:24:25] 5Continuous-Integration-Scaling, 7Nodepool: Teach Nodepool to not send statsd metrics per jobs - https://phabricator.wikimedia.org/T111504#1605779 (10hashar) 3NEW a:3hashar [09:24:36] 5Continuous-Integration-Scaling, 7Nodepool: Teach Nodepool to not send statsd metrics per jobs - https://phabricator.wikimedia.org/T111504#1605779 (10hashar) a:5hashar>3None [09:24:43] 5Continuous-Integration-Scaling, 7Nodepool: Document Nodepool statsd metrics - https://phabricator.wikimedia.org/T111503#1605770 (10hashar) a:5hashar>3None [09:25:09] 5Continuous-Integration-Scaling, 7Nodepool: Nodepool should send metrics to statsd - https://phabricator.wikimedia.org/T111496#1605790 (10hashar) I have filled sub tasks: {T111503} {T111504} [09:33:14] 6Release-Engineering: Create a Releng dashboard on Grafana - https://phabricator.wikimedia.org/T110313#1605822 (10hashar) [09:42:35] 10Continuous-Integration-Infrastructure, 10Wikibase-Quality-External-Validation, 10Wikidata: WikibaseQualityExternalValidation Jenkins jobs fail due to a composer error - https://phabricator.wikimedia.org/T111505#1605831 (10hashar) @Amire80 #jenkins is solely used for issues impacting the Jenkins software it... [09:42:56] 10Continuous-Integration-Infrastructure, 10Wikibase-Quality-External-Validation, 10Wikidata: WikibaseQualityExternalValidation Jenkins jobs fail due to a composer error - https://phabricator.wikimedia.org/T111505#1605912 (10aude) this is related to T111322 [09:48:13] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1605915 (10scfc) I think that belittles the work distribution packagers do. If I install a package, someone has made sure that it works, and that it works with the distribution I am using. It will rece... [09:52:06] Project browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox » fa,contintLabsSlave && UbuntuTrusty build #66: ABORTED in 5 min 59 sec: https://integration.wikimedia.org/ci/job/browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox/LANGUAGE_SCREENSHOT_CODE=fa,label=contintLabsSlave%20&&%20UbuntuTrusty/66/ [09:59:31] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1605951 (10Qgil) I guess distribution packagers will test Arcanist against the Phabricator snapshot they are shipping in the same distribution? If this is true, would we willing to rely in those packages... [10:09:51] 10Continuous-Integration-Infrastructure, 7Documentation, 7Zuul: Zuul can't merge two patches at once on integration/config.git because of fast-forward only Gerrit strategy - https://phabricator.wikimedia.org/T111426#1605991 (10hashar) 5Invalid>3Open p:5Triage>3Normal [10:11:20] 10Continuous-Integration-Infrastructure, 7Documentation, 7Zuul: Zuul can't merge two patches at once on integration/config.git because of fast-forward only Gerrit strategy - https://phabricator.wikimedia.org/T111426#1603585 (10hashar) I have reopened the task and rephrased it. It is worth documenting somewh... [10:17:37] 10Continuous-Integration-Infrastructure, 10Wikibase-Quality-External-Validation, 10Wikidata: WikibaseQualityExternalValidation Jenkins jobs fail due to a composer error - https://phabricator.wikimedia.org/T111505#1606037 (10aude) [10:22:14] 10Beta-Cluster, 6operations, 7Graphite, 7Shinken: Delete specific deployment-prep graphite datapoints - https://phabricator.wikimedia.org/T104091#1606044 (10fgiunchedi) 5Open>3Resolved {{done}} ``` root@labmon1001:/var/lib/carbon/whisper/deployment-prep# rm -rfv deployment-elastic05/diskspace/_var dep... [10:25:32] 10Continuous-Integration-Config, 3Mobile-App-Sprint-65-Android-Terbium, 5Patch-For-Review, 3Wikipedia-Android-App: Jenkins should run tests for the Wikipedia app before merge - https://phabricator.wikimedia.org/T62720#1606061 (10hashar) [10:25:34] 10Continuous-Integration-Config, 6Release-Engineering, 3Mobile-App-Sprint-65-Android-Terbium, 5Patch-For-Review, 3Wikipedia-Android-App: Create jenkins slave instance dedicated to Android runs - https://phabricator.wikimedia.org/T107336#1606058 (10hashar) 5Open>3Resolved a:3hashar @Niedzielski than... [10:26:08] 10Continuous-Integration-Config, 3Mobile-App-Sprint-65-Android-Terbium, 5Patch-For-Review, 3Wikipedia-Android-App: Jenkins should run tests for the Wikipedia app before merge - https://phabricator.wikimedia.org/T62720#1285927 (10hashar) [10:28:51] 10Continuous-Integration-Config, 3Mobile-App-Sprint-65-Android-Terbium, 5Patch-For-Review, 3Wikipedia-Android-App: Jenkins should run tests for the Wikipedia app before merge - https://phabricator.wikimedia.org/T62720#1606078 (10hashar) 5stalled>3Resolved I removed T110243 as a blocker since there is a... [10:30:33] 10Continuous-Integration-Config, 3Wikipedia-Android-App: Add code coverage report for Android app code - https://phabricator.wikimedia.org/T108737#1606096 (10hashar) [10:30:37] 10Continuous-Integration-Infrastructure, 10VisualEditor: language screenshot jobs are failing - https://phabricator.wikimedia.org/T111510#1606097 (10Amire80) 3NEW a:3zeljkofilipin [10:32:05] 10Continuous-Integration-Infrastructure, 7Documentation, 7Zuul: Zuul can't merge two patches at once on integration/config.git because of fast-forward only Gerrit strategy - https://phabricator.wikimedia.org/T111426#1606112 (10hashar) And poked the QA list about it https://lists.wikimedia.org/pipermail/qa/20... [10:33:36] 10Continuous-Integration-Infrastructure, 10VisualEditor: language screenshot jobs are failing - https://phabricator.wikimedia.org/T111510#1606118 (10hashar) Can you please enhance the task by: * giving the job URL * a copy paste of the console that shows the failure [10:34:09] zeljkof-meeting: I guess you are pairing with Amir aren't you ? [10:34:25] zeljkof-meeting: would explain the very short task details on https://phabricator.wikimedia.org/T111510#1606097 :D [10:34:26] hashar: no, just finished that [10:34:49] but yes, we were pairing on that when I had to go to another meeting, hence the phab task :) [10:38:12] 10Continuous-Integration-Infrastructure, 10VisualEditor: language screenshot jobs are failing - https://phabricator.wikimedia.org/T111510#1606128 (10Amire80) This should be useful: https://integration.wikimedia.org/ci/view/BrowserTests/view/VisualEditor/job/browsertests-VisualEditor-language-screenshot-os_x_10... [10:39:57] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606129 (10zeljkofilipin) [10:45:18] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606148 (10hashar) Yup seems the credentials are wrong :-( Apparently the username is LanguageScreenshotB... [10:57:01] (03CR) 10Filippo Giunchedi: [C: 04-1] "some more comments around host filtering, LGTM overall" (033 comments) [tools/scap] - 10https://gerrit.wikimedia.org/r/235385 (owner: 10Thcipriani) [11:45:28] 6Release-Engineering, 7Tracking: Update repositories that use mediawiki_selenium Ruby gem 1.x - https://phabricator.wikimedia.org/T94083#1606247 (10zeljkofilipin) [12:44:08] 6Release-Engineering, 7Tracking: Update repositories that use mediawiki_selenium Ruby gem 1.x - https://phabricator.wikimedia.org/T94083#1606291 (10zeljkofilipin) [12:50:06] 6Release-Engineering, 7Tracking: Update repositories that use mediawiki_selenium Ruby gem 1.x - https://phabricator.wikimedia.org/T94083#1606292 (10zeljkofilipin) [12:51:41] 10Browser-Tests: Investigate distribution of browser test run time - https://phabricator.wikimedia.org/T104396#1606295 (10zeljkofilipin) a:5zeljkofilipin>3None [12:57:23] !log Compiling Textual v5.2.0 to support latest bd808 theme ( https://github.com/bd808/Textual-Theme-bd808.git ) [12:57:27] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [12:57:33] yeah I am abusing the team log [12:57:51] Project browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #774: FAILURE in 25 min: https://integration.wikimedia.org/ci/job/browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce/774/ [13:18:43] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606319 (10zeljkofilipin) @hashar: I think `LanguageScreenshotBot` is used only for uploading files to Com... [13:21:11] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606330 (10zeljkofilipin) Hm, but looks like the password is really wrong: https://integration.wikimedia.... [13:23:52] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606332 (10zeljkofilipin) Oh, looks like (at least the browser) uses `Selenium user` as username, but `Lan... [13:28:56] bd808: thank you for the port of your Textual theme toward 5.x :-} [13:42:59] (03PS1) 10Zfilipin: WIP Fix language screenshot Jenkins job [integration/config] - 10https://gerrit.wikimedia.org/r/236019 (https://phabricator.wikimedia.org/T111510) [13:46:33] (03CR) 10Filippo Giunchedi: [C: 031] "LGTM, the commit message still references .scaprc btw" [tools/scap] - 10https://gerrit.wikimedia.org/r/224374 (owner: 10Thcipriani) [14:03:14] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor, 5Patch-For-Review: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606420 (10zeljkofilipin) Looks like a recent change to [[ https://phabricator.wikime... [14:04:16] (03PS2) 10Zfilipin: Fix language screenshot Jenkins job [integration/config] - 10https://gerrit.wikimedia.org/r/236019 (https://phabricator.wikimedia.org/T111510) [14:04:55] (03CR) 10Zfilipin: "This commit does fix the problem:" [integration/config] - 10https://gerrit.wikimedia.org/r/236019 (https://phabricator.wikimedia.org/T111510) (owner: 10Zfilipin) [14:07:10] 10Browser-Tests, 10VisualEditor: No such file or directory - i18n/af.json (Errno::ENOENT) - https://phabricator.wikimedia.org/T111525#1606436 (10zeljkofilipin) 3NEW [14:08:30] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor, 5Patch-For-Review: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606097 (10zeljkofilipin) I have created subtask {T111525} [14:09:03] 10Browser-Tests, 10VisualEditor: No such file or directory - i18n/af.json (Errno::ENOENT) - https://phabricator.wikimedia.org/T111525#1606450 (10Amire80) a:3Amire80 Oh drat, we didn't implement fallback in Ruby. [14:16:12] (03CR) 10JanZerebecki: [C: 032] Add zuul-cloner map for parsoidsvc jobs [integration/jenkins] - 10https://gerrit.wikimedia.org/r/235964 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [14:16:45] (03Merged) 10jenkins-bot: Add zuul-cloner map for parsoidsvc jobs [integration/jenkins] - 10https://gerrit.wikimedia.org/r/235964 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [14:18:26] (03CR) 10JanZerebecki: [C: 031] Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [14:20:58] does anyone know if/how to run qunit tests with karma in vagrant? [14:21:07] is there a role for that? [14:21:53] in non-vagrant, i get "Unexpected token ILLEGAL [14:21:54] " [14:22:12] and so am trying vagrant to see if it might work there [14:36:26] 10Continuous-Integration-Infrastructure, 6operations, 7Jenkins: Please refresh Jenkins package on apt.wikimedia.org to 1.609.3 - https://phabricator.wikimedia.org/T111327#1606548 (10hashar) Confirmed: ``` $ ssh gallium.wikimedia.org apt-cache policy jenkins jenkins: Installed: 1.609.1 Candidate: 1.609.3... [14:37:17] zeljkof: still around ? [14:37:24] zeljkof: will get Jenkins upgraded on monday :-} [14:37:37] hashar: still here, but not for long [14:37:44] great [14:38:01] I saw that ops pushed new jenkins for us [14:43:51] 10Continuous-Integration-Infrastructure, 7Jenkins: Upgrade Jenkins to 1.609.3 - https://phabricator.wikimedia.org/T111326#1606556 (10hashar) a:3hashar The package is available indeed: ``` $ ssh gallium.wikimedia.org apt-cache policy jenkins jenkins: Installed: 1.609.1 Candidate: 1.609.3 Version table:... [14:45:15] !log Jenkins upgrade on Monday Sep. 7th at 8:00am UTC ( https://phabricator.wikimedia.org/T111326#1606556 ) [14:45:18] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [14:45:32] zeljkof-away: we will pair the upgrade on monday :} [15:00:11] 10Continuous-Integration-Infrastructure, 10Wikidata: Intermittent qunit failures (MessageBlobStore::insertMessageBlob failed, database locked during setup) - https://phabricator.wikimedia.org/T111530#1606567 (10aude) 3NEW [15:19:54] * ostriches mumbles something about sudo_check_call, cries, goes back to work. [15:19:56] lol thcipriani ^ [15:20:45] ostriches: :) sounds like a good time [15:25:38] thcipriani: Yeah, I decided " deploy-local failed: Command '' returned non-zero exit status 1" is an unacceptable form of output [15:25:56] Since you have to then by hand to figure out why it failed. [15:26:50] indeed. it's more than some systems give you, but would be nice to see the actual exit. [15:27:57] And I know sudo_check_call has it on hand, just a matter of smashing it into the caller's logger I guess. [15:31:06] heh, "smashing it into" always sounds like a good plan, but yeah, I think that's the right approach. [15:32:57] (03CR) 10JanZerebecki: "Please leave clicking the rebase button to the person giving the +2." (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [15:33:50] (03CR) 10Paladox: "Ok I will." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [15:37:08] (03CR) 10Paladox: Use jshint in Cargo extension (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [15:45:56] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1606686 (10JanZerebecki) Please correct me if i'm wrong: For every installation of phabricator you need to use that specific arcanist, as there is no compatibility intended between any two versions. So i... [16:07:54] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1606763 (10Legoktm) >>! In T133#1605471, @Qgil wrote: > With all our code contributors having Git enabled in their systems, it is also the simplest way to support them with documentation and advice. I d... [16:20:20] 10Browser-Tests: Tutorial to setup a Raita dev environment - https://phabricator.wikimedia.org/T111493#1606811 (10dduvall) Gladly! There's a Vagrant role that you can start with—it includes test data—but we can definitely pair on it as well. [16:29:23] (03CR) 10JanZerebecki: Use jshint in Cargo extension (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:32:40] (03PS1) 10Dduvall: Release minor version 0.5.0 [ruby/api] - 10https://gerrit.wikimedia.org/r/236038 [16:34:31] (03CR) 10Paladox: "Should I also add jshint to check so like jsonlint being in check and in its own template should the same be for jshint." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:35:34] (03PS10) 10Paladox: Use jshint in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228472 [16:37:58] (03CR) 10JanZerebecki: "There is something unnecessary in the patch. Trying if our CI complains:" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:38:06] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:41:12] (03CR) 10JanZerebecki: [C: 04-1] "Look up what the templates do that you used, then you might already understand what I meant is unnecessary." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:42:26] 10Differential: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1606870 (10greg) >>! In T133#1606686, @JanZerebecki wrote: > Please correct me if i'm wrong: For every installation of phabricator you need to use that specific arcanist, as there is no compatibility int... [16:43:16] (03CR) 10Paladox: "Oh so I doint need check: jsonlint because that will duplicate what name: jsonlint does." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:44:45] (03PS3) 10Niedzielski: Publish Android app on postmerge & lint on submit [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) [16:45:42] (03CR) 10Niedzielski: "responses inline. ready 4 review" (032 comments) [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [16:47:44] (03CR) 10Paladox: "jshint does js files only and jsonlint does json files only and jslint does js and json files." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:47:54] 10Beta-Cluster, 6operations, 7Graphite, 7Shinken: Delete more specific deployment-prep graphite datapoints - https://phabricator.wikimedia.org/T111540#1606894 (10Krenair) 3NEW [16:48:02] 6Release-Engineering, 10Wikimedia-Mailing-lists: Admins for QA-alerts mailing list need to be updated - https://phabricator.wikimedia.org/T111013#1606904 (10greg) Feel free to add me to any list as an admin. I use `listadmin` to quickly process moderation queues via the CLI for the below list already, which I... [16:51:15] 10Browser-Tests, 10Continuous-Integration-Infrastructure, 10VisualEditor, 5Patch-For-Review: browsertests-VisualEditor-language-screenshot-os_x_10.10-firefox jenkins job failing - https://phabricator.wikimedia.org/T111510#1606908 (10JanZerebecki) Isn't the bug that the wrong password is used? As it shouldn... [16:53:13] 10Gerrit-Migration, 10Gitblit-Deprecate, 6Release-Engineering: Import all gerrit.wikimedia.org repositories with Diffusion - https://phabricator.wikimedia.org/T616#1606913 (10demon) [16:55:21] (03CR) 10JanZerebecki: "Yes." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [16:56:55] 10Beta-Cluster, 7Varnish: deployment-parsoidcache02 fails puppet: Role::Cache::Statsd/Varnish::Logging::Statsd[default]/Base::Service_unit[varnishstatsd-default]/Service[varnishstatsd-default]: Could not evaluate: Could not find init script or upstart conf file... - https://phabricator.wikimedia.org/T103660#1606935 [17:01:41] RECOVERY - Puppet staleness on deployment-restbase01 is OK Less than 1.00% above the threshold [3600.0] [17:04:18] (03CR) 10Paladox: "So I remove the" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [17:04:40] (03CR) 10Paladox: "That should say should I remove" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [17:06:34] (03CR) 10JanZerebecki: "Why are you unsure?" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [17:10:10] (03CR) 10Paladox: "I am not unsure I just want to make sure should I remove it." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [17:48:48] 10Browser-Tests, 10Deployment-Systems: Run @integration tests on new deploy branch creation - https://phabricator.wikimedia.org/T111545#1607160 (10greg) 3NEW [17:54:02] (03PS2) 10Dduvall: Release minor version 0.5.0 [ruby/api] - 10https://gerrit.wikimedia.org/r/236038 [17:55:06] (03CR) 10Dduvall: [C: 032] Release minor version 0.5.0 [ruby/api] - 10https://gerrit.wikimedia.org/r/236038 (owner: 10Dduvall) [17:55:22] (03Merged) 10jenkins-bot: Release minor version 0.5.0 [ruby/api] - 10https://gerrit.wikimedia.org/r/236038 (owner: 10Dduvall) [17:57:06] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1607571 (10greg) 3NEW [18:03:25] 6Release-Engineering, 10Gather, 10MobileFrontend, 10Reading Web Planning, and 2 others: [EPIC] Encourage developers to increase code coverage - https://phabricator.wikimedia.org/T100294#1607689 (10greg) Should we use this task as the main tracking task across all projects? I made {T111546} to track that, b... [18:14:02] (03PS11) 10Thcipriani: Add service deploy via scap [tools/scap] - 10https://gerrit.wikimedia.org/r/224374 [18:20:14] (03CR) 10Paladox: "Hi I am not unsure. I just want to make sure that is what you meant for me to find and is it correct." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [18:32:43] 10Beta-Cluster, 3Collaboration-Team-Current, 7WorkType-Maintenance: update.php broken in Beta - https://phabricator.wikimedia.org/T111267#1609068 (10Mattflaschen) a:3Mattflaschen [18:36:31] 10Beta-Cluster: Output of wmf-beta-update-databases.py is not clear on errors - https://phabricator.wikimedia.org/T110407#1609333 (10Mattflaschen) [18:41:50] (03PS1) 10Paladox: Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 [18:42:19] (03PS2) 10Paladox: Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 [18:54:35] (03PS1) 10Paladox: Use npm for jshint [integration/config] - 10https://gerrit.wikimedia.org/r/236068 [18:56:02] (03PS2) 10Paladox: Use npm for jshint [integration/config] - 10https://gerrit.wikimedia.org/r/236068 [18:57:14] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1610114 (10hashar) For MediaWiki core, the PHP code coverage is done twice per day because it takes two hours to generate (... [18:57:22] (03CR) 10Paladox: "Please see https://gerrit.wikimedia.org/r/#/c/236067/ jslint test which dosent fail so this can be merged safely." [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [18:57:58] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1610129 (10hashar) Which is essentially what I said on the parent task T100294#1322186 :-D [18:58:51] 6Release-Engineering, 10Gather, 10MobileFrontend, 10Reading Web Planning, and 2 others: [EPIC] Encourage developers to increase code coverage - https://phabricator.wikimedia.org/T100294#1309866 (10hashar) Better to merge / decline {T111546} imho. Until we get a coverage build that is fast enough, I don't... [18:58:57] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1610157 (10greg) yeah, sorry, this might be a straight dupe of that one, the tasks relating to this topic are a bit over-la... [18:59:39] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1610185 (10hashar) Creating a task is cheap so there is no problem in abusing the create button :-] [18:59:45] 10Continuous-Integration-Config, 6Release-Engineering, 10MediaWiki-Unit-tests: [EPIC] Add a voting test code coverage report to gate pipeline - https://phabricator.wikimedia.org/T111546#1610190 (10greg) [18:59:47] 6Release-Engineering, 10Gather, 10MobileFrontend, 10Reading Web Planning, and 2 others: [EPIC] Encourage developers to increase code coverage - https://phabricator.wikimedia.org/T100294#1610191 (10greg) [18:59:56] :) [19:08:01] 6Release-Engineering, 15User-greg: Investigate https://scrutinizer-ci.com/pricing WMF logo use - https://phabricator.wikimedia.org/T110396#1610308 (10hashar) >>! In T110396#1604327, @greg wrote: >>>! In T110396#1595661, @greg wrote: >> Did we give them explicit permission? Did they ask? > > @hashar do you kno... [19:16:40] (03CR) 10Hashar: [C: 04-1] "Excellent, I think it will works fine :-}" (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:23:37] (03PS4) 10Niedzielski: Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) [19:24:28] 10Continuous-Integration-Infrastructure, 10Deployment-Systems: Unify deployment of integration/config.git changes using the official Wikimedia deployment system - https://phabricator.wikimedia.org/T111559#1610383 (10hashar) 3NEW [19:27:37] (03CR) 10Hashar: [C: 032] "All great thank you very much !" [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:28:48] (03CR) 10jenkins-bot: [V: 04-1] Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:29:01] (03CR) 10Hashar: [C: 032] Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:30:40] (03CR) 10jenkins-bot: [V: 04-1] Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:31:21] (03CR) 10Hashar: [C: 04-1] "Bah:" (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:35:14] (03PS5) 10Niedzielski: Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) [19:35:52] (03CR) 10Niedzielski: "patch ammended with response inline. ready 4 review" (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:39:36] (03CR) 10Hashar: [C: 032] "Let see" [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:41:28] (03Merged) 10jenkins-bot: Publish Android app on postmerge [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:47:03] (03CR) 10Hashar: "I have deleted all the apps-android-wikipedia-gradlew* jobs." [integration/config] - 10https://gerrit.wikimedia.org/r/231624 (https://phabricator.wikimedia.org/T99115) (owner: 10Niedzielski) [19:50:02] 10Continuous-Integration-Config, 3Mobile-App-Sprint-65-Android-Terbium, 3Wikipedia-Android-App: Publish Jenkins built Android APK to well known URL - https://phabricator.wikimedia.org/T99115#1610462 (10hashar) Stephen patch should do it; Will be confirmed next time a patch is merged. [19:50:46] 10Beta-Cluster, 3Collaboration-Team-Current, 5Patch-For-Review, 7WorkType-Maintenance: update.php broken in Beta - https://phabricator.wikimedia.org/T111267#1610463 (10Mattflaschen) ``` 2015-09-04 19:26:30 deployment-bastion enwiki Flow INFO: Flow\Model\WikiReference::makeTitle: Invalid title. Namespace:... [19:51:08] (03CR) 10Hashar: "\O/" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/235964 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [19:58:20] 10Beta-Cluster, 3Collaboration-Team-Current, 5Patch-For-Review, 7WorkType-Maintenance: update.php broken in Beta - https://phabricator.wikimedia.org/T111267#1610475 (10Mattflaschen) Title::makeTitleSafe fails, but Title::makeTitle returns a Title. This is because MWNamespace::exists( 100 ) is false (there... [20:03:19] (03CR) 10JanZerebecki: Use npm for jshint (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:03:28] (03CR) 10JanZerebecki: [C: 04-1] Use npm for jshint [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:04:55] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:06:32] (03CR) 10jenkins-bot: [V: 04-1] Use npm for jshint [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:11:54] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:12:19] (03PS3) 10Paladox: Use npm for jshint [integration/config] - 10https://gerrit.wikimedia.org/r/236068 [20:12:29] (03CR) 10jenkins-bot: [V: 04-1] Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:12:40] (03CR) 10Paladox: "Ok thanks for spotting that and done." [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:13:36] 6RelEng-Admin, 15User-greg: Write draft/strawman code-hosting exception policy - https://phabricator.wikimedia.org/T109920#1610500 (10greg) [20:13:54] (03CR) 10JanZerebecki: "Yes" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:14:05] (03CR) 10Paladox: "Needs rebase so it will fail." [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:15:38] (03PS3) 10Paladox: Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 [20:15:48] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:16:17] (03CR) 10Paladox: "Removed whitespace." [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:16:46] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:17:43] (03CR) 10Paladox: "Ok thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:18:39] (03PS11) 10Paladox: Use jshint in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228472 [20:18:57] (03CR) 10Paladox: "Done." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:24:29] (03CR) 10JanZerebecki: [V: 04-1] "Waiting for https://gerrit.wikimedia.org/r/#/c/236067/" [integration/config] - 10https://gerrit.wikimedia.org/r/236068 (owner: 10Paladox) [20:30:24] (03CR) 10JanZerebecki: [C: 032] Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:31:44] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:33:34] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:34:57] (03Merged) 10jenkins-bot: Add jenkings tests for ParsoidBatchAPI [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [20:36:04] (03PS12) 10JanZerebecki: Use jshint in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:36:13] (03CR) 10JanZerebecki: [C: 032] Use jshint in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:37:43] (03PS9) 10Paladox: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 [20:37:52] (03Merged) 10jenkins-bot: Use jshint in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:38:32] (03CR) 10Paladox: "I have removed jshint test for now so this can be merged I will open a separate patch that uses npm for jshint for when js files are added" [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [20:38:39] (03PS10) 10Paladox: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 [20:38:57] (03PS11) 10Paladox: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 [20:39:23] (03CR) 10Paladox: "This can be merged now since it wont break the extension." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [20:39:49] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [20:43:27] (03PS3) 10JanZerebecki: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [20:43:52] (03PS7) 10Paladox: Use npm in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228473 [20:45:53] (03CR) 10Paladox: "This can be merged since jshint test dosent fail. Please see https://gerrit.wikimedia.org/r/#/c/235913/ and look at jslint test to see it." [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [20:53:04] 10Continuous-Integration-Config, 3Mobile-App-Sprint-65-Android-Terbium, 3Wikipedia-Android-App: Publish Jenkins built Android APK to well known URL - https://phabricator.wikimedia.org/T99115#1610585 (10Niedzielski) Ugh, looks like things aren't working out down on the farm[0]. I think an UbuntuTrusty node sh... [20:55:06] Argh wikitech. [20:55:16] It seems like I get logged out every few minutes. [20:59:50] (03PS4) 10JanZerebecki: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [21:00:48] 6RelEng-Admin, 15User-greg: Write draft/strawman code-hosting exception guideline - https://phabricator.wikimedia.org/T109920#1610605 (10greg) [21:04:18] (03PS8) 10JanZerebecki: Use npm in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:04:38] (03CR) 10JanZerebecki: "There was no composer added." [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:05:47] (03CR) 10JanZerebecki: [C: 04-1] "Revert Jenkins job because of failure: https://integration.wikimedia.org/ci/job/parsoidsvc-php-parsertests/5781/console" [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [21:06:24] (03CR) 10JanZerebecki: [C: 032] Use npm in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:07:46] (03CR) 10Paladox: "Ok thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:12:52] (03Merged) 10jenkins-bot: Use npm in Cargo extension [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:16:03] (03PS12) 10Paladox: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 [21:16:14] !log reloading zuul for 3740b19..1219cc3 [21:16:17] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [21:18:08] How long should I wait before making a component project I've proposed? I forget if there's a cooling-off period? [21:18:49] (03CR) 10JanZerebecki: "Test triggered in: https://gerrit.wikimedia.org/r/#/c/236159/" [integration/config] - 10https://gerrit.wikimedia.org/r/236064 (owner: 10Paladox) [21:19:33] (03CR) 10JanZerebecki: "Test triggered in: https://gerrit.wikimedia.org/r/#/c/235913/" [integration/config] - 10https://gerrit.wikimedia.org/r/228472 (owner: 10Paladox) [21:19:48] (03CR) 10JanZerebecki: "Test triggered in: https://gerrit.wikimedia.org/r/#/c/235913/" [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:22:20] James_F|Away: eh? [21:22:37] (03PS5) 10Legoktm: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) [21:22:44] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228473 (owner: 10Paladox) [21:23:34] (03PS13) 10JanZerebecki: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:23:41] (03CR) 10JanZerebecki: [C: 032] Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:23:45] (03CR) 10Paladox: "Should I rebase this." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:24:30] (03Merged) 10jenkins-bot: Add Jenkins test for SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:24:33] greg-g, I think he's talking about phabricator [21:24:37] not sure why here though [21:24:41] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:26:58] (03PS1) 10Paladox: Add npm test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/236203 [21:28:05] James_F|Away: oh, uh, not long? if it's just a component, all you need to do is record you made it [21:28:24] (03CR) 10Paladox: "Please merge this patch. This adds a json test to test json files which the extension have." [integration/config] - 10https://gerrit.wikimedia.org/r/225291 (owner: 10Paladox) [21:28:50] jzerebecki: I'm testing out the parsoid job now, thanks for testing it earlier ;) [21:28:52] :)* [21:28:58] (03CR) 10Paladox: "Sorry wrong extension that message is for." [integration/config] - 10https://gerrit.wikimedia.org/r/225291 (owner: 10Paladox) [21:29:15] "(CR) Paladox: "Should I rebase this." [integration/config] " ... you mean he's finally going to ask if it's worth rebasing something??? :O [21:29:34] (03CR) 10Paladox: "Please merge this patch. This adds a json test to test json files which the extension have." [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:30:12] 21:29:54 GitCommandError: 'git clone -v https://gerrit.wikimedia.org/r/p/mediawiki/core src/mediawiki/core' returned with exit code 128 [21:30:12] 21:29:54 stderr: 'fatal: destination path 'src/mediawiki/core' already exists and is not an empty directory. [21:30:13] ugh [21:30:51] * legoktm fiddles with salt [21:32:14] Lcawte: I considered removing the right for that button. But after I asked him why he does so and explanations and asking him to let me do it instead when I merge, he has not since pressed it needlessly. [21:36:51] (03CR) 10JanZerebecki: "Please check this patch again." [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:38:23] PROBLEM - Free space - all mounts on deployment-bastion is CRITICAL deployment-prep.deployment-bastion.diskspace._var.byte_percentfree (<44.44%) [21:40:14] (03PS11) 10Paladox: Add check for json in TwitterLogin [integration/config] - 10https://gerrit.wikimedia.org/r/225711 [21:40:35] (03CR) 10Paladox: "Ok done." [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:42:55] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:45:03] (03CR) 10JanZerebecki: [C: 032] Add check for json in TwitterLogin [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:45:21] (03PS6) 10Legoktm: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) [21:45:52] (03Merged) 10jenkins-bot: Add check for json in TwitterLogin [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [21:47:09] (03Abandoned) 10Paladox: Add npm test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/228623 (owner: 10Paladox) [21:47:22] 10Beta-Cluster, 3Collaboration-Team-Current, 5Patch-For-Review, 5WMF-deploy-2015-09-08_(1.26wmf22), 7WorkType-Maintenance: update.php broken in Beta - https://phabricator.wikimedia.org/T111267#1610780 (10Mattflaschen) It does exist on prod, so that patch should restore the config on Labs so that's a vali... [21:48:41] (03PS1) 10Paladox: Add composer-test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/236210 [21:51:18] greg-g, Reedy, bd808, legoktm, am I right that there is no way to merge a sub-array (e.g. specify a couple $wgExtraNamespace['enwiki'] in Labs, but also inherit the full list from production automatically)?: https://gerrit.wikimedia.org/r/#/c/236206/1/wmf-config/InitialiseSettings-labs.php [21:51:28] That should work, but will get out of sync if someone changes something in prod. [21:52:44] (03CR) 10Paladox: "Hi it seems when I upload a patch with json files the test is not run I think in template it wont run but if it is also in check: it runs " [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:53:00] matt_flaschen: It seems like I looked at this once and decided the same thing you did (that recursive merges were not supported) [21:54:04] You might be able to do something in CommonSettings-labs.php instead? [21:54:46] bd808, yeah, that's a good idea. I would have to put it separately from the normal place, but it would be future-proof if someone changes prod. [21:55:24] 10Deployment-Systems, 3Scap3: Scap3 should support rollback - https://phabricator.wikimedia.org/T109514#1610807 (10dduvall) p:5Triage>3Normal a:3dduvall [21:56:26] (03PS7) 10Legoktm: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) [21:56:36] (03CR) 10Legoktm: [C: 032] "Deployed!" [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [21:56:38] (03PS2) 10Paladox: Add composer-test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/236210 [21:58:11] (03PS2) 10Paladox: Add npm test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/236203 [21:58:16] (03Merged) 10jenkins-bot: Convert parsoidsvc-php-parsertests job to use zuul-cloner [integration/config] - 10https://gerrit.wikimedia.org/r/235965 (https://phabricator.wikimedia.org/T111469) (owner: 10Legoktm) [21:58:35] 10Continuous-Integration-Infrastructure, 10Gitblit-Deprecate, 5Patch-For-Review, 7Technical-Debt: Remove CI dependencies on git.wikimedia.org - https://phabricator.wikimedia.org/T74001#1610831 (10Legoktm) [21:58:37] 10Continuous-Integration-Config, 5Patch-For-Review, 7Technical-Debt: Migrate 'parsoidsvc-php-parsertests' job to use zuul-cloner - https://phabricator.wikimedia.org/T111469#1610830 (10Legoktm) 5Open>3Resolved [21:59:23] (03CR) 10JanZerebecki: "This is not yet live, I still have to deploy it." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [21:59:54] 10Continuous-Integration-Infrastructure, 10Gitblit-Deprecate, 5Patch-For-Review, 7Technical-Debt: Remove CI dependencies on git.wikimedia.org - https://phabricator.wikimedia.org/T74001#758725 (10Legoktm) parsoidsvc-php-parsertests now uses zuul-cloner. [22:00:49] 10Gerrit-Migration, 15User-greg: Write draft "business case" for migration from Gerrit to Differential - https://phabricator.wikimedia.org/T111250#1610861 (10greg) [22:01:11] (03PS1) 10Legoktm: Remove unused get-mw-core macro [integration/config] - 10https://gerrit.wikimedia.org/r/236215 (https://phabricator.wikimedia.org/T74001) [22:02:37] (03PS1) 10Legoktm: Remove unused mw-core-get.sh [integration/jenkins] - 10https://gerrit.wikimedia.org/r/236216 (https://phabricator.wikimedia.org/T74001) [22:03:06] (03CR) 10Paladox: "Oh ok sorry." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [22:03:22] (03CR) 10Legoktm: [C: 032] "no-op" [integration/config] - 10https://gerrit.wikimedia.org/r/236215 (https://phabricator.wikimedia.org/T74001) (owner: 10Legoktm) [22:03:48] (03PS3) 10JanZerebecki: Add composer-test to SpamDiffTool [integration/config] - 10https://gerrit.wikimedia.org/r/236210 (owner: 10Paladox) [22:05:02] (03Merged) 10jenkins-bot: Remove unused get-mw-core macro [integration/config] - 10https://gerrit.wikimedia.org/r/236215 (https://phabricator.wikimedia.org/T74001) (owner: 10Legoktm) [22:05:34] (03CR) 10Legoktm: [C: 032] "No uses left that I can see" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/236216 (https://phabricator.wikimedia.org/T74001) (owner: 10Legoktm) [22:06:08] (03Merged) 10jenkins-bot: Remove unused mw-core-get.sh [integration/jenkins] - 10https://gerrit.wikimedia.org/r/236216 (https://phabricator.wikimedia.org/T74001) (owner: 10Legoktm) [22:06:58] !log reloading zuul for 1219cc3..1b53ca9 [22:07:01] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [22:08:58] (03CR) 10JanZerebecki: "Test triggered for https://gerrit.wikimedia.org/r/#/c/236165/" [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [22:09:43] (03CR) 10JanZerebecki: "Test triggered for https://gerrit.wikimedia.org/r/#/c/233557/" [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [22:16:22] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/228622 (owner: 10Paladox) [22:18:54] (03CR) 10Paladox: "Thanks" [integration/config] - 10https://gerrit.wikimedia.org/r/225711 (owner: 10Paladox) [22:22:10] (03PS17) 10Paladox: Update CheckUser tests [integration/config] - 10https://gerrit.wikimedia.org/r/225182 [22:22:51] (03PS18) 10Paladox: Change jslint to jshint in CheckUser [integration/config] - 10https://gerrit.wikimedia.org/r/225182 [22:23:03] (03PS19) 10Paladox: Change jslint test to jshint test in CheckUser [integration/config] - 10https://gerrit.wikimedia.org/r/225182 [22:31:43] greg-g: OK if I (well, Krenair) does an emergency prod deploy for citoid? T93800 has been a UBN for months and we've now got a bandaid we think hides the impact from users. Would be really good to get it out. [22:32:53] James_F: Krenair sure [22:33:05] Thanks. [22:38:28] 10Gerrit-Migration, 15User-greg: Write draft "business case" for migration from Gerrit to Differential - https://phabricator.wikimedia.org/T111250#1610951 (10greg) [23:01:45] 6RelEng-Admin, 6Team-Practices: Do 90 day review of post-offsite actions in early August '15 - https://phabricator.wikimedia.org/T103917#1611003 (10demon) Summary sent to internal team list for consumption, hoping to post here next week. [23:19:25] 10Gerrit-Migration, 15User-greg: Write draft "business case" for migration from Gerrit to Differential - https://phabricator.wikimedia.org/T111250#1611027 (10greg) [23:33:57] 6RelEng-Admin, 15User-greg: Write draft/strawman code-hosting exception guideline - https://phabricator.wikimedia.org/T109920#1611036 (10greg) Draft at https://www.mediawiki.org/wiki/Wikimedia_Release_Engineering_Team/Code_review_platform_guideline [23:37:02] greg-g: It's almost like you think MW is better for writing documents than Phabricator's task description field! :-) [23:37:16] James_F: I do.... ;) [23:37:24] * James_F grins. [23:37:33] to be honest, I draft both those in gedit [23:37:48] so I think gedit, with my by memory wikitext, is better than phab markup [23:40:07] Tsk. [23:40:16] Use VE! [23:40:17] ;-) [23:41:11] I know I know, I will from now on, honestly