[00:03:20] (03PS1) 10Paladox: [ContributionReporting] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271157 [00:08:28] 10Beta-Cluster-Infrastructure, 10Kartographer, 3Discovery-Maps-Sprint: Deploy Kartographer to beta cluster - https://phabricator.wikimedia.org/T126829#2033739 (10Yurik) 5Open>3Resolved [00:08:28] (03PS1) 10Paladox: [Nonlinear] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271158 [00:11:45] (03PS1) 10Paladox: [CloseWikis] Replace jslint with jshint and jsonlint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271159 [00:16:02] (03PS1) 10Paladox: [ArticleIndex] Replace jslint test with jshint and jsonlint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271160 [00:30:24] (03PS1) 10Paladox: Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271161 [00:30:53] (03PS2) 10Paladox: [Carp] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271161 [00:32:24] (03PS1) 10Paladox: [CategorySlideShow] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271162 [00:33:48] 7Browser-Tests, 10MediaWiki-extensions-RelatedArticles, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): Fix failed RelatedArticles Selenium Jenkins jobs - https://phabricator.wikimedia.org/T126589#2033806 (10Jdlrobson) [00:34:18] 7Browser-Tests, 10Continuous-Integration-Config, 10MediaWiki-extensions-RelatedArticles, 5WMF-deploy-2016-02-23_(1.27.0-wmf.15): RelatedArticles browser tests should run on a commit basis - https://phabricator.wikimedia.org/T120715#2033809 (10Jdlrobson) [00:35:06] (03PS1) 10Paladox: [CategorySortHeaders] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271163 [00:37:42] (03PS1) 10Paladox: [CategoryTests] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271166 [00:39:33] (03PS1) 10Paladox: [CentralLogging] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271167 [00:41:38] (03PS1) 10Paladox: [Checkpoint] Replaces jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271168 [00:43:09] (03PS1) 10Paladox: [Commentbox] Replaces jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271169 [01:01:48] !log attempting to turn off NFS on 52 instances on deployment-prep project [01:01:52] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [01:21:24] 10Beta-Cluster-Infrastructure, 6Labs: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2033926 (10yuvipanda) I've tried to unmount it from the following instances: ``` deployment-analytics03.eqiad.wmflabs deployment-analytics02.eqiad.wmflab... [01:25:27] * Krinkle makes a small happy dance to see https://secure.phabricator.com/T7628 resolved. [01:26:12] Not to mention https://secure.phabricator.com/T3670 which is much bigger (subprojects!) See the v3 Eratta linked for details [01:59:57] PROBLEM - Puppet failure on deployment-mediawiki02 is CRITICAL: CRITICAL: 62.50% of data above the critical threshold [0.0] [02:34:57] RECOVERY - Puppet failure on deployment-mediawiki02 is OK: OK: Less than 1.00% above the threshold [0.0] [02:43:02] PROBLEM - Free space - all mounts on deployment-fluorine is CRITICAL: CRITICAL: deployment-prep.deployment-fluorine.diskspace._srv.byte_percentfree (<62.50%) [02:50:02] 10Beta-Cluster-Infrastructure, 6Labs: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034059 (10yuvipanda) Everything except deployment-parsoid05 and deployment-sca01 has been handled. Parsoid seems to be writing logs to NFS >_> [03:02:05] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034076 (10yuvipanda) Disabled on deployment-parsoid05, logs are on `/var/log/parsoid` now. [03:07:06] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034078 (10yuvipanda) And an umount -f does the trick in deployment-sca01! [03:14:10] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034080 (10yuvipanda) So all instances that do not need NFS do not have NFS anymore! Woo! :D If someone is building a new server tha... [03:15:13] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034081 (10yuvipanda) 5Open>3Resolved w00t! [03:15:22] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034083 (10yuvipanda) p:5Triage>3Normal [07:07:57] RECOVERY - Free space - all mounts on deployment-fluorine is OK: OK: All targets OK [07:10:10] 10Continuous-Integration-Config: operations/mediawiki-config still running php53lint - https://phabricator.wikimedia.org/T127154#2034360 (10ori) 3NEW [07:15:54] 10Continuous-Integration-Config: operations/mediawiki-config still running php53lint - https://phabricator.wikimedia.org/T127154#2034372 (10Legoktm) a:3Legoktm [07:18:28] (03PS1) 10Legoktm: Switch operations/mediawiki-config to php55 [integration/config] - 10https://gerrit.wikimedia.org/r/271201 (https://phabricator.wikimedia.org/T127154) [07:35:10] (03CR) 10Legoktm: [C: 032] Switch operations/mediawiki-config to php55 [integration/config] - 10https://gerrit.wikimedia.org/r/271201 (https://phabricator.wikimedia.org/T127154) (owner: 10Legoktm) [07:36:36] (03Merged) 10jenkins-bot: Switch operations/mediawiki-config to php55 [integration/config] - 10https://gerrit.wikimedia.org/r/271201 (https://phabricator.wikimedia.org/T127154) (owner: 10Legoktm) [07:36:43] !log deploying https://gerrit.wikimedia.org/r/271201 [07:36:45] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [07:40:31] 10Continuous-Integration-Config, 5Patch-For-Review: operations/mediawiki-config still running php53lint - https://phabricator.wikimedia.org/T127154#2034430 (10Legoktm) 5Open>3Resolved Now using php55lint, verified with https://gerrit.wikimedia.org/r/#/c/271203/ [07:55:06] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: New labs instance fails running `block-for-project-export` before running mount - https://phabricator.wikimedia.org/T126568#2034488 (10yuvipanda) p:5Unbreak!>3Normal Is this still happening? I see that nfs-exports daemon is running fine on labstore... [08:11:40] (03PS6) 10Siebrand: Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 [08:11:58] (03CR) 10Siebrand: "Thanks for the help, Jan!" [integration/config] - 10https://gerrit.wikimedia.org/r/270887 (owner: 10Siebrand) [08:12:03] (03PS7) 10Siebrand: Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 [08:13:01] (03PS2) 10Siebrand: Skip puppet-lint max. 80 char/line check [integration/config] - 10https://gerrit.wikimedia.org/r/270991 [08:13:17] (03PS3) 10Siebrand: Skip puppet-lint max. 80 char/line check [integration/config] - 10https://gerrit.wikimedia.org/r/270991 [08:16:40] (03PS4) 10Hashar: Skip puppet-lint max. 80 char/line check [integration/config] - 10https://gerrit.wikimedia.org/r/270991 (https://phabricator.wikimedia.org/T116552) (owner: 10Siebrand) [08:17:14] (03CR) 10Hashar: "That is for T116552. Quoting myself there:" [integration/config] - 10https://gerrit.wikimedia.org/r/270991 (https://phabricator.wikimedia.org/T116552) (owner: 10Siebrand) [09:06:52] 10Beta-Cluster-Infrastructure, 10Kartographer, 3Discovery-Maps-Sprint: Deploy Kartographer to beta cluster - https://phabricator.wikimedia.org/T126829#2034577 (10hashar) I just went ahead and pushed CR+2 when Yurik asked yesterday. [09:08:21] 10Continuous-Integration-Config, 6Operations, 5Patch-For-Review, 7Puppet: translatewiki-puppetlint-strict does not honor puppet-lint.rc file in /puppet - https://phabricator.wikimedia.org/T116552#2034589 (10siebrand) >>! In T116552#1753624, @hashar wrote: > The Jenkins job is a template '{name}-puppetlint-... [09:08:45] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: New labs instance fails running `block-for-project-export` before running mount - https://phabricator.wikimedia.org/T126568#2034590 (10hashar) a:3chasemp I am assuming @chasemp fixed the script that generate the NFS export list by simply raising the... [09:08:51] 10Beta-Cluster-Infrastructure, 5Patch-For-Review: rebuild deployment-bastion on trusty - https://phabricator.wikimedia.org/T126537#2034593 (10hashar) [09:08:53] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: New labs instance fails running `block-for-project-export` before running mount - https://phabricator.wikimedia.org/T126568#2034592 (10hashar) 5Open>3Resolved [09:09:26] 10Beta-Cluster-Infrastructure, 5Patch-For-Review: rebuild deployment-bastion on trusty - https://phabricator.wikimedia.org/T126537#2034594 (10hashar) a:3thcipriani [09:09:34] 10Beta-Cluster-Infrastructure: Beta Cluster updates broken due to php incompatibilities - https://phabricator.wikimedia.org/T126573#2034597 (10hashar) [09:09:36] 10Beta-Cluster-Infrastructure, 5Patch-For-Review: rebuild deployment-bastion on trusty - https://phabricator.wikimedia.org/T126537#2034595 (10hashar) 5Open>3Resolved Looks all fine now. [09:14:59] 10Continuous-Integration-Config, 6Operations, 5Patch-For-Review, 7Puppet: translatewiki-puppetlint-strict does not honor puppet-lint.rc file in /puppet - https://phabricator.wikimedia.org/T116552#2034608 (10hashar) @siebrand eek sorry. So for translatewiki repo you can create: `/puppet/.puppet-lint.rc`: `... [09:21:05] 10MediaWiki-Codesniffer, 7Upstream: Generic.Functions.CallTimePassByReference.NotAllowed false positive when using php5.4 array syntax [] - https://phabricator.wikimedia.org/T127163#2034610 (10Legoktm) 3NEW [09:23:05] 10Continuous-Integration-Config, 6Operations, 5Patch-For-Review, 7Puppet: translatewiki-puppetlint-strict does not honor puppet-lint.rc file in /puppet - https://phabricator.wikimedia.org/T116552#2034618 (10siebrand) Thanks for the help, @hashar. That fixed it. [09:23:13] 10Continuous-Integration-Config, 6Operations, 5Patch-For-Review, 7Puppet: translatewiki-puppetlint-strict does not honor puppet-lint.rc file in /puppet - https://phabricator.wikimedia.org/T116552#2034619 (10siebrand) 5Open>3Resolved a:3siebrand [09:23:33] (03Abandoned) 10Siebrand: Skip puppet-lint max. 80 char/line check [integration/config] - 10https://gerrit.wikimedia.org/r/270991 (https://phabricator.wikimedia.org/T116552) (owner: 10Siebrand) [09:33:29] (03PS8) 10Siebrand: Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 [09:39:33] 10MediaWiki-Codesniffer, 7Upstream: Generic.Functions.CallTimePassByReference.NotAllowed false positive when using php5.4 array syntax [] - https://phabricator.wikimedia.org/T127163#2034650 (10Legoktm) Filed upstream at https://github.com/squizlabs/PHP_CodeSniffer/issues/897 [09:45:31] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2034657 (10hoo) 3NEW [09:46:02] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2034664 (10hoo) [10:01:59] 5Continuous-Integration-Scaling, 6Operations, 7Nodepool, 7WorkType-NewFunctionality: Backport python-shade from debian/testing to jessie-wikimedia - https://phabricator.wikimedia.org/T107267#2034676 (10hashar) [[ https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=804550 | Debian bug 804550 ]] got closed and... [10:04:58] (03PS1) 10Legoktm: Disable Generic.Functions.CallTimePassByReference.NotAllowed [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271216 (https://phabricator.wikimedia.org/T127163) [10:04:59] addshore: +2 ^ so I can release it? [10:07:01] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034687 (10hashar) The plugin has a few configuration settings in https://integration.wikimedia.org/ci/configure : * Max number... [10:07:18] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034689 (10hashar) [10:07:20] 10Continuous-Integration-Infrastructure, 5Continuous-Integration-Scaling, 7Jenkins, 7Nodepool: Postmortem: Nodepool can't add slaves to Jenkins due to config plugin directory reaching 32k inodes - https://phabricator.wikimedia.org/T127131#2034688 (10hashar) [10:07:30] 10Continuous-Integration-Infrastructure, 5Continuous-Integration-Scaling, 7Jenkins, 7Nodepool: Postmortem: Nodepool can't add slaves to Jenkins due to config plugin directory reaching 32k inodes - https://phabricator.wikimedia.org/T127131#2033616 (10hashar) [10:07:55] 10Continuous-Integration-Infrastructure, 5Continuous-Integration-Scaling, 7Jenkins, 7Nodepool: Postmortem: Nodepool can't add slaves to Jenkins due to config plugin directory reaching 32k inodes - https://phabricator.wikimedia.org/T127131#2033616 (10hashar) > [ ] Get rid of Jenkins configuration plugin for... [10:13:23] (03CR) 10Addshore: [C: 032] Disable Generic.Functions.CallTimePassByReference.NotAllowed [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271216 (https://phabricator.wikimedia.org/T127163) (owner: 10Legoktm) [10:13:32] {{done}} [10:13:45] ty! [10:14:02] epic work btw legoktm :P [10:14:07] (03Merged) 10jenkins-bot: Disable Generic.Functions.CallTimePassByReference.NotAllowed [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271216 (https://phabricator.wikimedia.org/T127163) (owner: 10Legoktm) [10:16:14] (03PS1) 10Legoktm: Release 0.6.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271218 [10:16:20] addshore: ty :) [10:16:25] addshore: ^ that one too plz? [10:22:11] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034721 (10hashar) I have manually deleted some old history from 2012/2013. The plugin no more save them under the job directory... [10:22:21] (03PS2) 10Legoktm: Release 0.6.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271218 [10:22:32] (03CR) 10Legoktm: [C: 032] Release 0.6.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271218 (owner: 10Legoktm) [10:22:57] legoktm: haha [10:23:02] ina meeting, didnt see the ping [10:23:15] (03Merged) 10jenkins-bot: Release 0.6.0 [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271218 (owner: 10Legoktm) [10:23:38] (03CR) 10Addshore: "+2!" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/271218 (owner: 10Legoktm) [10:26:35] 10Beta-Cluster-Infrastructure, 6Collaboration-Team-Backlog, 10DBA, 10Flow: Beta cluster update database is broken because of Flow db patch db_patches/patch-ref_target_not_null.sql - https://phabricator.wikimedia.org/T127040#2034729 (10jcrespo) There could be several issues here, some beta-related, some pot... [10:38:16] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034747 (10hashar) The regex is only uses for xml files directly in Jenkins home: ``` lang=java boolean isSaveable(final Sav... [11:10:34] (03PS1) 10Paladox: Migrate 6 repos from jslint to jshint and jsonlint [integration/config] - 10https://gerrit.wikimedia.org/r/271226 [11:13:15] (03PS2) 10Paladox: Migrate 6 repos from jslint to jshint and jsonlint [integration/config] - 10https://gerrit.wikimedia.org/r/271226 [11:16:43] 10Continuous-Integration-Infrastructure, 5Patch-For-Review: Convert operations/mediawiki-config to use composer for phpunit and php linting - https://phabricator.wikimedia.org/T85947#2034830 (10Paladox) I think @Krinkle made this possible but I'm not sure weather you have to use mediawiki/core to override whic... [11:26:36] 6Release-Engineering-Team, 5Patch-For-Review, 5WMF-deploy-2016-02-16_(1.27.0-wmf.14): MW 1.27.0-wmf.14 deployment blockers - https://phabricator.wikimedia.org/T125597#2034838 (10Florian) Current status, as far as I can see (just for documentation): wmf.14 is pre-deployed to testwiki, all other group0 wikis s... [11:30:05] legoktm: Is there anyway for you to run a bot to migrate all the repos to the new syntax please. [11:35:04] 7Browser-Tests, 10Wikidata, 5Patch-For-Review: Sitelink browser test sometimes fails with firefox - https://phabricator.wikimedia.org/T126585#2034857 (10Tobi_WMDE_SW) The failures seem unrelated to this. It seems that the some of the articles used for testing do not really exist on the corresponding test wik... [11:57:56] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034880 (10hashar) Plugin does not track slaves that are either `EphemeralNode` or `AbstractCloudSlave`. But that is for plugins... [12:19:37] 10Continuous-Integration-Infrastructure, 7Jenkins: Jenkins files under /var/lib/jenkins/config-history/config need to be garbage collected - https://phabricator.wikimedia.org/T126552#2034883 (10hashar) a:3hashar Since that has hit `/var/lib/jenkins/config-history/config/` and the nodes config as well, lets d... [12:43:17] 10Beta-Cluster-Infrastructure: Consider only running the Beta Cluster localisation caches with the content languages to make scaps much faster - https://phabricator.wikimedia.org/T123997#2034906 (10hashar) On beta cluster, scap is run via the job beta-scap-eqiad for which the build time trend can be seen at http... [12:44:02] 10Beta-Cluster-Infrastructure: Consider only running the Beta Cluster localisation caches with the content languages to make scaps much faster - https://phabricator.wikimedia.org/T123997#2034908 (10hashar) 5Open>3Resolved a:3hashar The issue was really: > l10n cache building really kicks in and makes the b... [12:45:48] 10Beta-Cluster-Infrastructure: beta equivalent of noc.wikimedia.org/conf - https://phabricator.wikimedia.org/T42819#2034913 (10hashar) 5Open>3declined a:3hashar The task has been opened back in 2012 and nothing happened since them. The idea might be worthwhile in theory, but in practice nobody needs it.... [12:54:12] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034919 (10hashar) Thank you @yuvipanda ! [12:54:17] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034928 (10hashar) [12:54:20] 10Beta-Cluster-Infrastructure, 6Labs: Completely remove Beta Cluster dependency on NFS - https://phabricator.wikimedia.org/T102953#2034927 (10hashar) [12:54:30] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#1992944 (10hashar) I marked it again as blocked on the root task {T102953}, it is part of it. [12:54:48] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2034931 (10hashar) And Swift is {T64835} itself a child task of the root task T102953. [12:56:29] 7Browser-Tests, 10Math: Math Selenium test fails with unable to locate element, using {:id=>"wpTextbox1", :tag_name=>"textarea"} (Watir::Exception::UnknownObjectException) - https://phabricator.wikimedia.org/T126463#2034933 (10hashar) [13:00:28] (03CR) 10Hashar: "CommentPages passes jslint test" [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:00:55] (03PS3) 10Hashar: Migrate 6 repos from jslint to jshint and jsonlint [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:02:58] (03CR) 10Hashar: [C: 032] Migrate 6 repos from jslint to jshint and jsonlint [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:04:14] greg-g: Hey. There are a bunch of FlaggedRev issues recently: https://phabricator.wikimedia.org/maniphest/query/C.DZQcus3nKL/#R - any idea who and if someone should take a look? aschulz maybe? [13:04:22] (03Merged) 10jenkins-bot: Migrate 6 repos from jslint to jshint and jsonlint [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:06:25] (03CR) 10Hashar: "I have deleted the mwext-*-jslint jobs. Thanks!" [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:06:35] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/271226 (owner: 10Paladox) [13:07:15] (03PS2) 10Paladox: [XMLContentExtension] Replace jslint test with jshint and jsonlint test [integration/config] - 10https://gerrit.wikimedia.org/r/271154 [13:07:33] (03PS2) 10Paladox: [ReplaceSet] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271156 [13:08:47] (03PS2) 10Paladox: [ContributionReporting] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271157 [13:09:47] (03PS2) 10Paladox: [CategorySlideShow] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271162 [13:09:55] (03PS3) 10Paladox: [Carp] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271161 [13:10:13] (03PS2) 10Paladox: [CategoryTests] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271166 [13:10:18] (03PS2) 10Paladox: [CentralLogging] Replace jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271167 [13:11:25] (03PS2) 10Paladox: [Commentbox] Replaces jslint test with jsonlint and jshint tests [integration/config] - 10https://gerrit.wikimedia.org/r/271169 [13:12:05] (03CR) 10Paladox: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/271168 (owner: 10Paladox) [13:12:12] (03CR) 10Paladox: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/271137 (owner: 10Paladox) [13:20:06] Project beta-update-databases-eqiad build #6547: 04FAILURE in 5.9 sec: https://integration.wikimedia.org/ci/job/beta-update-databases-eqiad/6547/ [13:42:40] 7Browser-Tests, 10Wikidata, 5Patch-For-Review: Sitelink browser test sometimes fails with firefox - https://phabricator.wikimedia.org/T126585#2035048 (10adrianheine) Locally this failed because of rate limiting and stopped failing after I changed config. I'll try again locally. [13:48:28] hashar: how bad is it if i force submit a patch in wikibase (on our branch) [13:48:50] i have 2 patches that fix independent issues and together they would pass [13:48:55] but they need each other [13:51:32] aude: eeek [13:51:36] cycles are not handled :( [13:51:41] aude: what are the patches? [13:51:42] there is no submit button [13:51:51] i might have to squash them or something eveil [13:52:06] https://gerrit.wikimedia.org/r/#/c/271243/ depends on https://gerrit.wikimedia.org/r/#/c/271246/ [13:52:09] and other way around [13:52:29] hmm [13:53:16] a crazy hack would be to propose them as a dependency chain. Propose a patch that merge them in master [13:53:30] CR+2 / V+2 all of them , get rid ofjenkins v-1 [13:53:34] and the merge commit should land [13:53:45] but an easier way is to squash them [13:53:47] or [13:53:49] just push :} [13:54:11] i dont want https://gerrit.wikimedia.org/r/#/c/271243/ for master [13:54:22] the fix is okay enough for a temporary fix, but is not that nice [13:54:45] going to handle it [13:54:48] i will try squash, so then we can see both test failures are resolved [13:55:03] ok [13:55:28] hold on [13:55:57] or Depends-On [13:56:00] aude the patches are for wmf.12 [13:56:04] yeah [13:56:05] but we no more have .12 [13:56:07] oh [13:56:09] so it's getting master [13:56:13] do we have wmf13? [13:56:14] we do [13:56:23] i can make wmf13 wikibase [13:56:39] I am confused by the versions [13:56:46] en has https://en.wikipedia.org/wiki/Special:Version .13 [13:57:05] this probably will work :) [13:57:19] * 3eb702c - all wikis to 1.27.0-wmf.13 (6 days ago) [13:57:26] that is what we have deployed [13:57:30] yeah so we are on .13 and this week is .14 (which is paused) [13:57:45] Hello everyone, i'm trying to use git/gerrit ; 'git log' return this https://wikitech.wikimedia.org/wiki/User:Youni_Verciti/Memo_cmd#git_log [13:57:48] so I guess abandon / retry on wmf.13 [13:57:51] yeah [13:58:16] Youni: that paste is unreadable :D [13:58:38] sorry ? [13:58:47] Youni: I speak french if it can help :-} [13:58:54] oh oui [13:59:07] je deviens fou avec gerrit [13:59:37] aude: going to handle some support question with Youni in French. Poke me anytime when the .13 patches are around, will be happy to push as needed [13:59:48] Youni: je regarde :) [14:00:53] Youni: quand tu te connectes à Gerrit, tu utilises le compte 'y-verciti' [14:01:06] Youni: mais tes commits ont pour auteur: Youni Verciti , qui nest pas toi [14:01:54] enfin en tout cas Gerrit ne te connais pas sous ce nom. Il faut donc modifier le champ auteur de chaque commit pour qu'ils aient l'adresse email enregistrée dans Gerrit [14:02:02] hashar: absoluement ceci date de mes premiers tests, je n'avais pas configuré git à cette epoque [14:02:18] oui mais je n'y arrive pas [14:02:35] git config --global user.name "Youni Verciti" [14:02:45] git config --global user.email "oliv71@laposte.net" [14:02:52] mais a priori ça c'est bon vu le dernier commit [14:02:58] j'ai fais cela [14:03:15] * hashar clone [14:04:17] le dépôt à déjà un commit 77da4f06 [14:04:27] de Christian Aistletner [14:04:34] mais apparement tu as des commits avant [14:05:18] il faudrait refaire une branche propre en partant du commit dans le dépot [14:05:29] puis y ajouter tes commits un par un [14:05:44] en suivant la doc j'ai créé un git dans le tool ensuite j'ai fais une requette de repo gerrit d'ou le commit de Christian [14:05:57] Pour faire une nouvelle branche partant du commit dans Gerrit: git checkout -b propre -t origin/master [14:06:21] ensuite tu peux prendre ton commit initial avec: git cherry-pick 477006577775a1048310f4509fa1008602ee5789 [14:06:49] et là je me perds car tout cela est très nouveau pour moi [14:06:53] 10Browser-Tests-Infrastructure, 5Patch-For-Review: Remove LoginPage from mediawiki_selenium Ruby gem - https://phabricator.wikimedia.org/T127042#2035105 (10zeljkofilipin) [14:07:07] (03PS9) 10JanZerebecki: Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 (owner: 10Siebrand) [14:07:15] Youni: oui c'est un peu compliqué mais on s'y fait :-} [14:07:25] l'idée la c'est de repartir d'une branche propre [14:07:28] j'essaie ta commande git checkout? [14:07:34] puis d'ajouter tes commits un par un et les corriger [14:07:36] (03CR) 10JanZerebecki: [C: 032] Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 (owner: 10Siebrand) [14:07:46] oui ca va te créer une nouvellebranche nommée "propre" [14:07:58] qui pointe vers ce qu'il y a dans Gerrit [14:08:29] a partir du compte tools.vocabulary-index j'y vais pour checkout, un moment! [14:08:46] (03Merged) 10jenkins-bot: Update test runners for translatewiki repo [integration/config] - 10https://gerrit.wikimedia.org/r/270887 (owner: 10Siebrand) [14:10:04] checkout plante! Comment te passer l'errur obtenue [14:10:45] Did you intend to checkout 'origin/master' which can not be resolved as commit? [14:10:45] hashar: so we have the same situation on wmf13 :/ [14:10:52] Youni: http://pastebin.com [14:11:00] aude: no problem. Just link me the .13 patches :) [14:11:06] ok [14:12:02] https://gerrit.wikimedia.org/r/#/c/271255/ and https://gerrit.wikimedia.org/r/#/c/271250/ [14:12:22] Youni: le depot distant a peut être un autre nom. git remote -v pour voir les noms [14:12:26] basically, DataTypes was never tested before in the build [14:12:37] Youni: et pour voir les branches distantes: git branch -r [14:12:40] and then there was an issue with qunit and uls [14:14:24] aude: you should be able to push them [14:14:35] yeah... evil but i can :P [14:14:41] aude: apparently the wikidata team has push rights [14:14:53] because we push the new branches (based on the previous one) [14:15:00] so push one [14:15:00] rebase [14:15:02] ok [14:15:03] push the other [14:15:15] the second one should work with gerrit [14:15:20] and Gerrit should notice the push and magically close the changes with something "Commit has been pushed to the repository by aude" [14:15:29] ok [14:15:48] j'ai passé les réponses dans un nouveau pastebin [14:16:19] Youni: me faut le lien :) [14:16:50] hashar: http://pastebin.com/55Uq3WtY [14:17:06] donc git remote -v [14:17:15] liste les depots externes. Dans ce cas il s'appelle "gerrit" [14:17:18] et non origin [14:17:34] git branch -r (-r == --remote == --distante), liste les branches sur les dépôts distant [14:17:46] ce qu'il y a dans gerrit est donc connu avec le nom gerrit/master [14:18:15] donc on peut refaire une branche "propre" avec: git checkout -b propre gerrit/master [14:18:26] ca va faire une branche nommée "propre" dont le commit est celui dans gerrit [14:18:40] i don't apparently have push rights in this case [14:18:47] git show devrait te montrer: [14:18:47] commit 77da4f066f4527b24844e23855f379f30489020f [14:18:47] Author: Christian Aistleitner [14:18:48] push is ok for a new branch [14:18:58] i will try squash [14:19:18] aude: I will push [14:19:23] hashar: je vais faire cela [14:19:24] hashar: if you want [14:20:35] Yippee, build fixed! [14:20:36] Project beta-update-databases-eqiad build #6548: 09FIXED in 32 sec: https://integration.wikimedia.org/ci/job/beta-update-databases-eqiad/6548/ [14:21:08] aude: https://gerrit.wikimedia.org/r/#/c/271250/ is in :) [14:21:13] hashar: la commande est passée, son resultat: http://pastebin.com/E5bqjkWJ [14:21:14] thanks [14:21:22] can the other one then go through gerrit? [14:21:49] yeah [14:21:52] ok [14:21:53] rebase button is missing [14:21:56] will refresh manually [14:21:59] thanks [14:22:22] at least we then know for sure, together the patches make jenkins happy [14:22:34] (though we still have the build step as another check( [14:22:35] ) [14:23:02] bah https://gerrit.wikimedia.org/r/#/c/271255/ lost its comment history [14:23:28] aude: it is being tested by zuul/jenkins etc [14:23:29] :/ but ok [14:23:36] thanks for helping with this [14:23:45] Youni: parfait [14:24:09] Youni: donc maintenant l'idée c'est de reprendre ton commit et le mettre sur cette branche 'propre' [14:24:12] effectivement git log me montre le commit de Christian [14:24:16] Youni: pour ça on utilise 'git cherry-pick' [14:24:35] donc pour prendre "Initial check-in": git cherry-pick 477006577775a1048310f4509fa1008602ee5789 [14:24:43] 10Browser-Tests-Infrastructure, 5Patch-For-Review: Remove LoginPage from mediawiki_selenium Ruby gem - https://phabricator.wikimedia.org/T127042#2035159 (10zeljkofilipin) [14:25:04] ensuite il faut corriger l'auteur en modifiant le commit: git commit --amend --author="Youni Verciti " [14:25:22] puis "git show" pour confirmer que l'auteur a changé. Ce nouveau commit devrait pouvoir atterrir dans Gerrit [14:25:52] aude: spread the world among wmde team. For anything related to wmf / gerrit / deploy etc do poke me as needed ;-} [14:26:37] aude: that is for the 1% cases when you get stuck somehow and helping deployment is definitely under our team responsibility [14:26:40] ouf! j'ai jamais utilisé git cherry-pick, je regarde... [14:26:46] Youni: commande magique ;-} [14:27:31] Youni: en fait il faut voir le dépôt git comme un arbre avec un tronc, des branches et des feuilles. Les commands git permettent de triturer l'arbre. git cherry-pick ca prend une feuille et la met sur une autre branche [14:27:32] Project browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #791: 04FAILURE in 1 min 32 sec: https://integration.wikimedia.org/ci/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/791/ [14:27:47] hashar: ok [14:28:17] it's a pretty nasty caching bug that is not nice when we are introducing new data types :/ [14:28:27] not the experience users should have [14:29:06] hashar: comment tu obtiens le numero du commit que je dois cherry-pick(er)? [14:29:15] zeljkof: can you retry the mediawiki/core daily browser tests ? https://gerrit.wikimedia.org/r/270932 landed that is for the remove LoginPage stuff ( https://phabricator.wikimedia.org/T127042 ) [14:29:32] Youni: du log sur https://wikitech.wikimedia.org/wiki/User:Youni_Verciti/Memo_cmd#git_log [14:29:58] Youni: quand un commit est créée git fait un hachage du message + les modifications et donne un identifiant unique [14:30:21] Youni: et on peut alors le prendre (git cherry-pick) et le poser là où on veut [14:30:40] soit je fais cela, un moment... [14:31:22] hashar: sure [14:31:41] Youni: et un super livre a avaler: https://git-scm.com/book/fr/v2 [14:32:47] hashar: running... https://integration.wikimedia.org/ci/view/BrowserTests/view/Core/job/browsertests-Core-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/905/console [14:33:09] :) [14:34:41] hashar: ok pour cherry-pick http://pastebin.com/ggLn8gjb [14:35:16] Youni: git show pour voir le commit [14:35:47] Youni: il faut corriger l'auteur avec: git commit --amend --author="Youni Verciti " [14:35:53] Youni: puis "git show" encore pour confirmer [14:36:22] aude: qunit fails on https://gerrit.wikimedia.org/r/#/c/271255/ :((( [14:36:57] ugh [14:36:57] oh my god [14:37:05] 14:29:19 + /srv/deployment/integration/slave-scripts/bin/mw-teardown-mysql.sh [14:37:06] 14:29:19 ERROR 1269 (HY000) at line 1: Can't revoke all privileges for one or more of the requested users [14:37:14] i got that on the build [14:37:16] comment je sors de git show? [14:37:28] i am giving a +2 and hope gate and submit doesn't have this issue [14:37:36] (to the build - wmf14) [14:37:45] Youni: tout dépend de l'éditeur. Si c'est vi il faut passer en mode commande avec ':' puis 'q' et taper entrer [14:38:00] ok j'en suis sorti via "q" [14:38:02] https://integration.wikimedia.org/ci/job/mwext-testextension-php55/1474/console [14:38:11] Youni: git show pour confirmer [14:38:23] Youni: il fallait peut être sauver en fait [14:39:03] aude: CI is broken in some terrible bad way :(( [14:39:07] ugh [14:39:28] pardon? je dois faire quoi dans vi apres git show? [14:40:03] Youni: est ce que dans git show tu vois le champs auteur modifier avec ton mail en laposte.net ? [14:40:15] aude: and the poor https://integration.wikimedia.org/ci/job/mwext-testextension-php55-composer/326/console is painfully slow [14:40:37] yeah, notiecd [14:40:47] non j'ai un soucis avec git commit --amend --author="Youni Verciti " [14:41:37] Youni: ca doit d'afficher l'éditeur te permettant de modifier le message de commit. Il faut ensuite sauver. [14:41:50] Youni: dans vim mode commande avec ':' puis 'x' pour sauver et quiter [14:42:10] la commande est passée effectivement je suis dans vim [14:44:10] là ça se complique vim prend un aspect different, je vois la chose suivante: [14:44:20] !log On Trusty slaves, reducing number of executors from 4 to 3 to leave room for Mysql/Java/Lua etc [14:44:25] ^O WriteOut [14:44:34] hashar: suppose fixing https://phabricator.wikimedia.org/T126274 would help (the scribunto tests) [14:44:40] aude: going to push https://gerrit.wikimedia.org/r/#/c/271255/ [14:44:54] ok, thanks [14:46:15] aude: yeah we should stop running tests of extensions that are there solely for a dependency [14:46:17] ok maintenant git show me montre l'utilisateur correct [14:46:22] aude: i.e. on Wikibase, there is no need to run Scribunto tests [14:46:30] Youni: parfait. Donc git-review devrait marcher [14:46:55] Youni: le commit à une adresse qui est associée à ton compte Gerrit et il devrait l'accepter [14:46:55] hashar: would be good, but we are running mwext-testextension [14:47:11] at least in the build [14:47:24] aude: yeah that one still run phpunit.php --testsuite extensions which blindly runs anything registered in the hook UnitTestList [14:47:40] aude: and that hook is populated by all the extensions, hence all extensions tests run [14:47:55] hashar: c'est passé [14:48:26] if they were fast, that would be ok and probalby nice [14:49:40] !sal [14:49:40] https://tools.wmflabs.org/sal/releng [14:49:44] hashar: http://pastebin.com/bmtBe2Qz [14:50:26] hashar: merci mille fois ; j'etais vraiement en galère depuis plusieurs jours avec ce problème. [14:51:23] Je vais essayer de consigner toutes ces indications dans mes notes, pui il faudra que je reproduise l'opération pour les commits suivant. [14:52:03] Youni: je te recommande vraiment de lire le bouquin https://git-scm.com/book/fr/v2 [14:53:40] Youni: ensuite pour modifier un changement dans gerrit tu le récupère via: git-review -d 123456 [14:53:56] Youni: tu fait tes modifications. A ajouter avec "git add" [14:54:06] Youni: puis tu modifies le commit existant avec: git commit --amend [14:54:11] Youni: et ensuite git-review [14:54:47] ca envoi le nouveau commit à Gerrit. Comme le champ Change-Id est le meme ton nouveau commit est reconnu par Gerrit comme étant le changement 123456 et l'interface affichera un nouveau patchset [14:55:15] J'ai commencé à parcourir cette doc, je continuerais, mais je suis bien content d'avoir un ami qui parle français et qui connaisse le sujet. [14:55:39] Youni: n'hésites pas. Je suis là tout les jours de la semaine de 10 à 18 environ [14:55:45] et parfois en soirée [14:56:58] merici hashar je suis au Brésil, j'espère que la suite sera plus facile, mais c'est passionnant de pouvoir utiliser ces outils. [14:57:38] A bientôt. [14:57:38] Youni: et tu dois pouvoir faire tout ça sur ta propre machine au besoin ;) [14:59:08] la triplicité entre ma machine le tools.account et le gerrit.account ça rend fou! Je copie tes indications et je fais un break, le coeur léger. A bientôt. [15:10:06] !log reloading zuul for bba1873..e945e92 [15:18:00] hashar: https://gerrit.wikimedia.org/r/#/c/271273/ is ready [15:18:13] (suppose it's ok if i go ahead and deploy, unless you want to) [15:18:34] involves updating core to use the new branch on wmf13 [15:18:43] err , no [15:18:50] this should be easy :) [15:22:27] eeek [15:23:07] ? [15:23:18] no new branch [15:23:24] we alreayd had wmf13 "wikidata" [15:23:35] so it is all about merging that wikidata change [15:23:46] yeah [15:23:50] then gerrit will bump the submodule in mediawiki/core and you deploy that ? [15:23:57] i gave +2 and see what happens with jenkins [15:24:03] yep [15:24:10] I never did such thing myself :-} [15:24:15] I am still a deployment rookie [15:24:20] ok [15:24:25] (03PS1) 10JanZerebecki: Skip php53 on translatewiki [integration/config] - 10https://gerrit.wikimedia.org/r/271277 [15:24:35] it fails :( https://integration.wikimedia.org/zuul/ [15:24:51] 00:00:38.424 Error: 1146 Table 'jenkins_u0_mw.filearchive' doesn't exist (127.0.0.1:3306) [15:24:52] pff [15:24:56] ugh [15:25:33] I have manually aborted the jobs [15:26:06] hashar: php jobs are failing here, but I only made a small change in ruby code :( https://gerrit.wikimedia.org/r/#/c/271266/ [15:26:19] and the php jobs are complaining about stuff not being installed [15:26:36] https://integration.wikimedia.org/ci/job/mwext-ZeroPortal-testextension-php55/4/console [15:26:42] 00:04:33.765 Warning: Could not find APCu, XCache or WinCache. [15:27:11] zeljkof: yeah ZeroPortal depends on ZeroBanner [15:27:24] and this one did not run any tests? [15:27:29] https://integration.wikimedia.org/ci/job/mwext-ZeroPortal-testextension-php55/4/console [15:27:34] 00:03:34.515 ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error? [15:27:37] :| [15:27:49] (03CR) 10JanZerebecki: [C: 032] Skip php53 on translatewiki [integration/config] - 10https://gerrit.wikimedia.org/r/271277 (owner: 10JanZerebecki) [15:28:06] zeljkof: https://gerrit.wikimedia.org/r/#/c/188115/ one year old patch grblblbl [15:28:51] :'( [15:29:28] (03Merged) 10jenkins-bot: Skip php53 on translatewiki [integration/config] - 10https://gerrit.wikimedia.org/r/271277 (owner: 10JanZerebecki) [15:32:13] !log reloading zuul for e945e92..52b90b2 [15:36:11] so suppose i just try again [15:44:23] zeljkof: I rebased your change [15:45:04] hashar: thanks! [15:45:04] aude: maybe one of our slaves MySQL is corrupt somehow [15:45:10] :/ [15:45:21] * aude hopes it gets merged this time before swat [15:45:37] otherwise i'll wait until after and not put the swatters through this [15:45:45] * hashar looks at too many bugs https://phabricator.wikimedia.org/tag/continuous-integration-infrastructure/board/ [15:46:03] :( [15:46:20] my update for wmf14 wikidata got merged [15:46:27] and now wmf13 \o/ [15:46:32] * aude deploys [15:47:09] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035332 (10hashar) [15:47:11] 10Continuous-Integration-Infrastructure: Passing jenkins job reported as failing - https://phabricator.wikimedia.org/T126810#2035331 (10hashar) [15:47:30] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2021271 (10hashar) From T126810: > In https://gerrit.wikimedia.org/r/#/c/270350/ PS1 the [[ https://integration.wikimedia.org/ci/job/me... [15:49:39] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035338 (10hashar) The ` Cannot find site jenkins_u0_mw in sites table` errors are unrelated. It is Wikibase assuming the site table sho... [15:49:48] aude: the mysql issue is https://phabricator.wikimedia.org/T126699 [15:51:45] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2035343 (10bd808) Running a sync-dir to deploy a configuration change that includes introducing a new `$wmg*` variable is not generally a good idea. Scap... [15:52:55] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035345 (10hashar) Random example from a Wikibase change against wmf.13 ( https://gerrit.wikimedia.org/r/#/c/271273/ ): update.php ([[... [15:54:25] deployed :) [15:56:49] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2035362 (10hoo) So I would be fine if I sync the new InitialiseSettings first, then touch it and sync it out again (with the full change including the ot... [16:00:17] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2035363 (10bd808) >>! In T127167#2035362, @hoo wrote: > So I would be fine if I sync the new InitialiseSettings first, then touch it and sync it out agai... [16:00:29] the bug looks fixed [16:06:43] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035374 (10hashar) On integration-slave-trusty-1018 I have dmesg filled with: ``` [Tue Feb 9 13:20:54 2016] init: mysql main process (... [16:31:34] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035506 (10hashar) The whole MySQL setup needs to be overhauled it is very messy. We have the datadir set to a tmpfs via: https://gerr... [16:53:04] (03Abandoned) 10Hashar: [operations/software] Add Jenkins tests [integration/config] - 10https://gerrit.wikimedia.org/r/267921 (https://phabricator.wikimedia.org/T125363) (owner: 10Paladox) [16:56:21] 10Continuous-Integration-Infrastructure, 5Patch-For-Review: Tests are not run on a change in operations/software - https://phabricator.wikimedia.org/T125363#2035603 (10hashar) 5Open>3declined a:3hashar I have removed the pep8/pyflakes jobs with 5c05c758d046ff14e636472a36c2ad812ea0475b because the reposit... [17:12:12] 10Continuous-Integration-Infrastructure, 6Release-Engineering-Team: Postmortem: CI incidents - week of Feb 8th - https://phabricator.wikimedia.org/T126634#2035670 (10hashar) a:3hashar [17:16:10] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035675 (10JanZerebecki) https://integration.wikimedia.org/ci/job/mwext-qunit-composer/804/consoleFull ``` 16:34:15 Building remotely on... [17:17:15] 10Continuous-Integration-Infrastructure, 5Continuous-Integration-Scaling, 5Patch-For-Review: MediaWiki gate takes 20 minutes for extensions tests and 1.5 hour for at least a patch - https://phabricator.wikimedia.org/T126274#2035690 (10greg) p:5Triage>3High [17:17:48] 10Continuous-Integration-Infrastructure, 5Continuous-Integration-Scaling, 5Patch-For-Review: MediaWiki gate takes 20 minutes for extensions tests and 1.5 hour for at least a patch - https://phabricator.wikimedia.org/T126274#2009830 (10greg) p:5High>3Normal Setting to Normal as we fixed the problem for no... [17:33:14] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035761 (10JanZerebecki) ``` root@integration-slave-trusty-1011:~# puppet agent -tv Info: Retrieving plugin Info: Loading facts in /var/... [17:33:16] 6Release-Engineering-Team, 15User-greg: SWAT grooming/recruitment - https://phabricator.wikimedia.org/T127210#2035762 (10greg) 3NEW a:3greg [17:35:56] 10Continuous-Integration-Infrastructure, 7Tracking: MySQL tuning on CI slaves (tracking) - https://phabricator.wikimedia.org/T96249#2035796 (10hashar) [17:35:58] 10Continuous-Integration-Infrastructure, 7developer-notice: Switch MySQL storage to tmpfs - https://phabricator.wikimedia.org/T96230#2035792 (10hashar) 5Resolved>3Open Most probably cause T126699 : i.e. mysql randomly restarting / loosing tables etc.. [17:36:06] 10Continuous-Integration-Infrastructure, 7developer-notice: Switch MySQL storage to tmpfs - https://phabricator.wikimedia.org/T96230#2035797 (10hashar) [17:36:08] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2021271 (10hashar) [17:39:08] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035805 (10hashar) Yup the override file is a craziness we inject which is also used by puppet service {}. From a comment I made on ht... [17:58:56] (03PS1) 10Jforrester: [OOjs UI] Only run composer for PHP5.5, so we can break 5.3 support [integration/config] - 10https://gerrit.wikimedia.org/r/271305 [17:59:42] (03CR) 10Jforrester: "(I'm guessing that this is how it works by looking at other repos' configuration.)" [integration/config] - 10https://gerrit.wikimedia.org/r/271305 (owner: 10Jforrester) [18:07:49] 10Deployment-Systems, 10MediaWiki-ResourceLoader, 10Wikimedia-General-or-Unknown, 7Easy: 404 for "poweredby_mediawiki" icon file in the footer on MediaWiki.org - https://phabricator.wikimedia.org/T127194#2035913 (10Aklapper) [18:11:18] !log updated cherry-pick https://gerrit.wikimedia.org/r/#/c/204528/14 on integration-puppetmaster T126699 [18:11:23] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2035931 (10Stashbot) {nav icon=file, name=Mentioned in SAL, href=https://tools.wmflabs.org/sal/log/AVLwbd50hQaf1CQcCfif} [2016-02-17T18:... [18:23:46] PROBLEM - Puppet failure on integration-slave-trusty-1016 is CRITICAL: CRITICAL: 12.50% of data above the critical threshold [0.0] [18:25:14] PROBLEM - Puppet failure on integration-slave-trusty-1001 is CRITICAL: CRITICAL: 60.00% of data above the critical threshold [0.0] [18:28:37] 10Deployment-Systems, 10Wikimedia-General-or-Unknown, 7Easy, 5Patch-For-Review: 404 for "poweredby_mediawiki" icon file in the footer on MediaWiki.org - https://phabricator.wikimedia.org/T127194#2035968 (10Krinkle) a:3Krinkle [18:29:26] 10Deployment-Systems, 10Wikimedia-General-or-Unknown, 7Easy, 5Patch-For-Review: 404 for "poweredby_mediawiki" icon file in the footer on MediaWiki.org - https://phabricator.wikimedia.org/T127194#2035971 (10Krinkle) 5Open>3Resolved [18:29:58] PROBLEM - Puppet failure on integration-slave-trusty-1023 is CRITICAL: CRITICAL: 11.11% of data above the critical threshold [0.0] [18:30:00] PROBLEM - Puppet failure on integration-slave-trusty-1017 is CRITICAL: CRITICAL: 33.33% of data above the critical threshold [0.0] [18:30:32] 10Deployment-Systems, 10Wikimedia-General-or-Unknown, 7Easy, 5Patch-For-Review: 404 for "poweredby_mediawiki" icon file in the footer on MediaWiki.org - https://phabricator.wikimedia.org/T127194#2035973 (10Stashbot) {nav icon=file, name=Mentioned in SAL, href=https://tools.wmflabs.org/sal/log/AVLwf27S-0X0I... [18:33:52] 3Scap3, 10scap, 6Phabricator, 7Blocked-on-Operations, 7WorkType-Maintenance: scap::target should use scap's debian package instead of trebuchet - https://phabricator.wikimedia.org/T127215#2035978 (10mmodell) 3NEW a:3mmodell [18:36:41] PROBLEM - Puppet failure on integration-slave-trusty-1006 is CRITICAL: CRITICAL: 80.00% of data above the critical threshold [0.0] [18:39:11] PROBLEM - Puppet failure on integration-slave-precise-1003 is CRITICAL: CRITICAL: 77.78% of data above the critical threshold [0.0] [18:39:54] 7Blocked-on-RelEng, 6Phabricator: iridium:/var/log/phd/daemons.log is growing too much (took 20% of filesystem space) - https://phabricator.wikimedia.org/T124651#2036005 (10mmodell) [18:40:29] 7Blocked-on-RelEng, 6Phabricator: iridium:/var/log/phd/daemons.log is growing too much (took 20% of filesystem space) - https://phabricator.wikimedia.org/T124651#1961567 (10mmodell) Might be fixed by rOPUPb8175c706507 [18:50:10] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036042 (10ssastry) We are fine dropping 0.8 and 0.10 support in Travis CI. Travis is covering us there already. https://travis-c... [18:50:24] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036044 (10cscott) I'm fine with moving CI to node 4.3 as long as we move production to node 4.3 as close as possible to the same... [18:53:32] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036052 (10ssastry) >>! In T126992#2036044, @cscott wrote: > I'm fine with moving CI to node 4.3 as long as we move production to... [18:57:30] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036090 (10ssastry) @hashar ruthenium which runs jessie has node v4.2.4 specifically. How come CI will get node v4.3 since you men... [18:59:27] !log updating chery-pick https://gerrit.wikimedia.org/r/#/c/204528/15 on integration-puppetmaster T126699 [18:59:31] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2036103 (10Stashbot) {nav icon=file, name=Mentioned in SAL, href=https://tools.wmflabs.org/sal/log/AVLwmfaZW8txF7J0uTsd} [2016-02-17T18:... [19:00:07] RECOVERY - Puppet failure on integration-slave-trusty-1001 is OK: OK: Less than 1.00% above the threshold [0.0] [19:01:54] James_F: sounds like a maybe on the phab upgrade, depends on whether twentyafterfour can get review from ops [19:05:00] RECOVERY - Puppet failure on integration-slave-trusty-1017 is OK: OK: Less than 1.00% above the threshold [0.0] [19:06:53] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036142 (10mobrovac) >>! In T126992#2036090, @ssastry wrote: > @hashar ruthenium which runs jessie has node v4.2.4 specifically. H... [19:08:36] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036158 (10ssastry) >>! In T126992#2036142, @mobrovac wrote: >>>! In T126992#2036090, @ssastry wrote: >> @hashar ruthenium which r... [19:08:48] RECOVERY - Puppet failure on integration-slave-trusty-1016 is OK: OK: Less than 1.00% above the threshold [0.0] [19:09:00] marxarelli: Thanks! [19:09:12] RECOVERY - Puppet failure on integration-slave-precise-1003 is OK: OK: Less than 1.00% above the threshold [0.0] [19:09:58] RECOVERY - Puppet failure on integration-slave-trusty-1023 is OK: OK: Less than 1.00% above the threshold [0.0] [19:10:03] 5Continuous-Integration-Scaling, 10Parsoid, 6Services, 7WorkType-NewFunctionality: Migrate Parsoid CI jobs from node 0.8/0.10 to 4.3 - https://phabricator.wikimedia.org/T126992#2036167 (10mobrovac) >>! In T126992#2036158, @ssastry wrote: >>>! In T126992#2036142, @mobrovac wrote: >>>>! In T126992#2036090, @... [19:11:44] RECOVERY - Puppet failure on integration-slave-trusty-1006 is OK: OK: Less than 1.00% above the threshold [0.0] [19:12:14] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2036195 (10JanZerebecki) PS15 fixes the problem [19:12:23] 10Continuous-Integration-Infrastructure: mediawiki jobs fail intermittently with "mw-teardown-mysql.sh: Can't revoke all privileges" - https://phabricator.wikimedia.org/T126699#2036197 (10JanZerebecki) 5Open>3Resolved a:3JanZerebecki [19:12:25] 10Continuous-Integration-Infrastructure, 7developer-notice: Switch MySQL storage to tmpfs - https://phabricator.wikimedia.org/T96230#2036199 (10JanZerebecki) [19:23:14] legoktm, https://integration.wikimedia.org/ci/job/parsoidsvc-php-parsertests/6853/console .. is that from the array code changes and enabling strict php linting? [19:34:21] 10Deployment-Systems, 6Operations, 6Performance-Team, 10Traffic, 5Patch-For-Review: Make Varnish cache for /static/$wmfbranch/ expire when resources change within branch lifetime - https://phabricator.wikimedia.org/T99096#2036304 (10Krinkle) [19:35:01] PROBLEM - Puppet failure on integration-slave-trusty-1018 is CRITICAL: CRITICAL: 22.22% of data above the critical threshold [0.0] [19:35:02] 10Beta-Cluster-Infrastructure, 6Collaboration-Team-Backlog, 10DBA, 10Flow: Beta cluster update database is broken because of Flow db patch db_patches/patch-ref_target_not_null.sql - https://phabricator.wikimedia.org/T127040#2036308 (10Mattflaschen) See also {T110446}, particularly my comment about CHARSET.... [19:35:08] 10Beta-Cluster-Infrastructure, 6Collaboration-Team-Backlog, 10DBA, 10Flow: Beta cluster update database is broken because of Flow db patch db_patches/patch-ref_target_not_null.sql - https://phabricator.wikimedia.org/T127040#2036310 (10Mattflaschen) 5Open>3Resolved [19:36:48] subbu: it is trying to run mediawiki master code under php 5.3 [19:37:27] subbu: i.e. that job needs to be fixed [19:37:36] ah ok .. i guess something releng / CI folks will fix? [19:37:37] to use php55 [19:37:54] ah i see .. ok. [19:41:15] subbu: please submit a patch or make a task in contint-config for it otherwise it might be lost in the log [19:41:31] yes, i am going to be doing that shortly. [19:48:14] 10Beta-Cluster-Infrastructure, 6Labs: Soft mount remaining NFS mounts on deployment-prep - https://phabricator.wikimedia.org/T127224#2036325 (10yuvipanda) 3NEW [19:49:14] (03Abandoned) 10Hashar: Add npm-node-4.2 to experimental: in template npm [integration/config] - 10https://gerrit.wikimedia.org/r/265854 (owner: 10Paladox) [19:53:26] (03PS1) 10Subramanya Sastry: Run parsoidsvc-php-parsertests with php5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/271335 [19:53:38] jzerebecki, ^ [19:54:44] thx [19:55:20] jzerebecki: If you're doing a CI deploy could you do https://gerrit.wikimedia.org/r/#/c/271305/ too? marxarelli [19:55:24] Bah, sorry marxarelli, mis-ping. [19:55:27] (03CR) 10jenkins-bot: [V: 04-1] Run parsoidsvc-php-parsertests with php5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/271335 (owner: 10Subramanya Sastry) [19:59:34] 10Beta-Cluster-Infrastructure, 6Labs: Soft mount remaining NFS mounts on deployment-prep - https://phabricator.wikimedia.org/T127224#2036347 (10chasemp) p:5Triage>3Normal [20:03:05] 6Release-Engineering-Team, 5Patch-For-Review, 5WMF-deploy-2016-02-16_(1.27.0-wmf.14): MW 1.27.0-wmf.14 deployment blockers - https://phabricator.wikimedia.org/T125597#2036354 (10hashar) 5Open>3stalled Yup wmf.14 is blocked until wmf.13 is pristine clear. There is a regression on it; T126700 [20:03:06] (03PS1) 10Ladsgroup: Add flake8 checks to pywikibot/bots/xqbot [integration/config] - 10https://gerrit.wikimedia.org/r/271339 [20:03:54] (03CR) 10jenkins-bot: [V: 04-1] Add flake8 checks to pywikibot/bots/xqbot [integration/config] - 10https://gerrit.wikimedia.org/r/271339 (owner: 10Ladsgroup) [20:07:33] 10Continuous-Integration-Infrastructure: mediawiki-extensions-hhvm jobs intermittently fail with "File not found: Filter.php" - https://phabricator.wikimedia.org/T127226#2036379 (10Catrope) 3NEW [20:08:26] greg-g: OK to push out https://gerrit.wikimedia.org/r/#/c/271144/ (mw-config speed-test patch)? [20:08:46] No sites affected, just performance testing purposes. [20:08:52] (03CR) 10Subramanya Sastry: "hmm .. why is jenkins unhappy?" [integration/config] - 10https://gerrit.wikimedia.org/r/271335 (owner: 10Subramanya Sastry) [20:09:34] 10Beta-Cluster-Infrastructure, 6Labs, 5Patch-For-Review: Disable /data/project for instances in deployment-prep that do not need it - https://phabricator.wikimedia.org/T125624#2036392 (10yuvipanda) Positive side of this is this graph, showing the NFS client drops: http://graphite.wikimedia.org/render/?width=... [20:09:45] James_F: oh, ok, sure [20:09:58] Thanks. [20:10:04] RoanKattouw: Go. [20:10:23] greg-g: ^ just a fyi I'm going to soft-mount NFS on deployment-prep later today. the effect of this would be that when NFS failures happen in labs, deployment-prep will only lose media serving rather than *all the things* [20:10:37] yuvipanda: neato [20:18:29] subbu|lunch: uhoh, I'll look into that [20:25:43] 10Deployment-Systems, 6Operations, 6Performance-Team, 10Traffic, 5Patch-For-Review: Make Varnish cache for /static/$wmfbranch/ expire when resources change within branch lifetime - https://phabricator.wikimedia.org/T99096#2036441 (10Stashbot) {nav icon=file, name=Mentioned in SAL, href=https://tools.wmfl... [20:29:47] hey again thcipriani! how do I specify an environment when deploying? [20:29:48] i've got [20:29:52] scap/environments/beta/eventbus [20:30:05] i think we figured this out before but i can't remember [20:30:09] i'm trying deploy -e beta [20:30:13] and also trying to give it paths [20:30:19] 10Deployment-Systems, 6Operations, 6Performance-Team, 10Traffic, 5Patch-For-Review: Make Varnish cache for /static/$wmfbranch/ expire when resources change within branch lifetime - https://phabricator.wikimedia.org/T99096#2036457 (10Krinkle) [20:30:20] ottomata: -e beta should work [20:30:27] naw [20:30:33] :( [20:30:44] it picks up the main (prod) targets [20:31:05] blerg. I wonder if scap is just out of date in production [20:31:10] i'm in beta [20:31:15] ? [20:31:20] ah, lemme check... [20:31:22] on deploment-tin [20:34:26] ottomata: ah, I see what's happening [20:35:02] the path to your dsh_targets file in /srv/deployment/eventlogging/eventbus is an absolute path [20:35:19] if you just make that path 'eventbus' [20:35:24] i think there was some bug with that before, but ok! [20:35:26] then give it -e beta...that should work [20:35:37] ottomata: yeah, there was, it's _hopefully_ fixed now :) [20:36:13] cool! it picked the right host [20:36:33] awesome! glad that works now [20:36:42] host key verification failed...i think that somebody changed beta deployment stuff [20:36:48] it used to be on deployment-bastion [20:36:58] and i notice that the git remote set on the target is wrong [20:36:59] yeah, I moved the host late last week [20:37:05] maybe I can just remove the target? [20:37:29] (03CR) 10JanZerebecki: [C: 032] [OOjs UI] Only run composer for PHP5.5, so we can break 5.3 support [integration/config] - 10https://gerrit.wikimedia.org/r/271305 (owner: 10Jforrester) [20:37:35] Thanks jzerebecki. [20:38:24] ottomata: fwiw, scap should reset the git remote on the target for each deploy [20:38:30] (03Merged) 10jenkins-bot: [OOjs UI] Only run composer for PHP5.5, so we can break 5.3 support [integration/config] - 10https://gerrit.wikimedia.org/r/271305 (owner: 10Jforrester) [20:38:40] ok [20:38:52] well, this is a dfferent problem with host keys anyway [20:38:54] you'll probably need to do an ssh-keyscan as your ssh_user to make sure host-key verification will work [20:38:59] Host key verification failed. [20:39:28] i see some keys [20:39:41] two keys are present [20:39:44] ssh-keyscan deployment-eventlogging04.deployment-prep.eqiad.wmflabs [20:40:48] ottomata: should work now, just did the ole: SSH_AUTH_SOCK=/run/keyholder/proxy.sock ssh -l eventlogging deployment-eventlogging04.deployment-prep.eqiad.wmflabs [20:40:51] and then accepted the key [20:41:03] hmm, ok [20:41:13] hmm, same reslut [20:41:19] Host key verification failed. [20:41:44] your ssh comand works for me too [20:41:44] oh [20:41:46] i accepted [20:41:47] trying now [20:41:54] theerre we go [20:41:55] i had to do that [20:42:26] heh, oh yeah, I guess that makes sense. [20:42:52] looking like its hanging on submodule stuff now [20:42:58] ntering 'config/schemas' [20:42:58] 20:41:51 [20:42:58] == DEFAULT == [20:42:58] :* deployment-eventlogging04.deployment-prep.eqiad.wmflabs [20:42:58] deploy_eventlogging/eventbus_fetch: 0% (ok: 0; fail: 0; left: 1) [20:43:27] 10Deployment-Systems, 10scap: HHVM doesn't always properly notice files changed via scap's sync-dir, touch needed - https://phabricator.wikimedia.org/T127167#2036489 (10hoo) Ok… shall we close this then? [20:43:54] ah, thcipriani the submodule remote is wrong [20:44:06] does scap know to change the submodule remote each time? [20:44:27] ottomata: should, but it doesn't look like it got that far [20:44:27] otto@deployment-eventlogging04:/srv/deployment/eventlogging/eventbus/config/schemas$ git remote -v [20:44:27] origin http://deployment-bastion.deployment-prep.eqiad.wmflabs/eventlogging/eventbus/.git/modules/config/schemas (fetch) [20:44:27] origin http://deployment-bastion.deployment-prep.eqiad.wmflabs/eventlogging/eventbus/.git/modules/config/schemas (push) [20:44:32] looks lit it was trying to fetch from tin [20:45:03] Starting new HTTP connection (1): tin.eqiad.wmnet [20:45:08] ? [20:45:41] huh [20:45:58] why would it do that? [20:45:59] ottomata: could you change git_server in your scap.cfg to deployment-tin.deployment-prep.eqiad.wmflabs [20:46:00] !log reloading zuul for 52b90b2..e11a9ff [20:46:05] k [20:46:06] 10Continuous-Integration-Config, 6Operations, 5Patch-For-Review, 7Puppet: translatewiki-puppetlint-strict does not honor puppet-lint.rc file in /puppet - https://phabricator.wikimedia.org/T116552#2036500 (10hashar) Neat! Well done @siebrand and congratulations for passing puppet-lint! [20:46:19] ahh hm [20:46:21] yeah [20:46:24] for wmflabs [20:46:25] ? [20:46:39] yup. [20:46:46] right [20:46:54] James_F: done [20:47:19] jzerebecki: Thanks! [20:47:56] thcipriani: before I pinged you i manually tried changing the remote for the git repo at /srv/deployment/eventlogging/eventbus-cache/cache on deployment-eventlogging04 [20:48:03] because I noticed it was pointing at bastion [20:48:07] not sure if that messed things up [20:48:20] but, ja, it does seem like the submodule update is hanging [20:48:37] ottomata: nah, shouldn't. Every time it deploys it should try to replace the git remote [20:48:49] aye, and the submodule's remote too? [20:49:06] hmm [20:49:07] still [20:49:07] - Starting new HTTP connection (1): tin.eqiad.wmnet [20:49:13] but whyyyy!? [20:50:26] thcipriani: i'm going to change the[wmnet] one too and see what it does [20:50:52] ottomata: kk, I think it's working fine on the deploy side, .git/DEPLOY_HEAD has the right info [20:52:55] we're still running php53lint on CI? I thought we've moved to 5.5? [20:53:14] hah thcipriani still!~ [20:53:15] - Starting new HTTP connection (1): tin.eqiad.wmnet [20:53:48] dunno where its getting tin from! [20:54:12] ottomata: mind if I try something on the remote...would run a deploy if it worked... [20:54:15] please [20:54:17] its fine [20:54:18] kk [20:54:58] legoktm jzerebecki its seems php53 tests are run on the master branch here https://gerrit.wikimedia.org/r/#/c/265212/ [20:56:28] ottomata: give it a try now [20:56:55] (03PS1) 10Paladox: Fix phplint53 tests running on master branch [integration/config] - 10https://gerrit.wikimedia.org/r/271347 [20:56:58] yeehaw [20:57:00] what was it? [20:57:08] it's a scap.cfg problem [20:57:25] we moved the default /etc/scap.cfg to puppet so we could package scap [20:57:31] (03CR) 10Paladox: "See https://gerrit.wikimedia.org/r/#/c/265212/ please." [integration/config] - 10https://gerrit.wikimedia.org/r/271347 (owner: 10Paladox) [20:57:38] and we didn't set the git_server for...well...anywhere. [20:57:41] ah [20:57:43] hm ok [20:58:04] cool, welp, all is well then! [20:58:07] thank you! [20:58:12] so, on the targets, you can now run: deploy-local --repo '[reponame]' [20:58:40] ottomata: thank you, as always, for being a guinea pig :) [20:59:14] legoktm ive fixed it here https://gerrit.wikimedia.org/r/#/c/271347/ [21:01:37] hehe, np! [21:01:49] thcipriani: you can do do that on any target to deploy just to it? [21:02:01] deploy-local --repo eventlogging/eventbus [21:02:01] ? [21:02:18] that's the idea [21:02:30] we made that for use with the scap provider in puppet [21:02:36] that's nice [21:03:21] yeah, you'd have to be the right user to run it, of course. So it's a little complicated, but seems to work well enough. [21:03:36] in fact, that's what deploy is doing now, ssh to target and run that command [21:04:14] (03PS1) 10JanZerebecki: Skip php53 on translatewiki in a better way [integration/config] - 10https://gerrit.wikimedia.org/r/271348 [21:04:26] fetches all information from the deployment host + deployment_dir + .git/DEPLOY_HEAD. So: /srv/deployment/eventlogging/eventbus/.git/DEPLOY_HEAD [21:07:32] (03CR) 10Paladox: "Your prevous patch broke the script for php53. Since php53 was running on the master branch for the BlueSky skin." [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:08:17] (03CR) 10JanZerebecki: [C: 032] Skip php53 on translatewiki in a better way [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:08:59] (03CR) 10Paladox: "Yes this looks better but users who arnt whitelisted carnt run any phplint now." [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:09:17] (03Merged) 10jenkins-bot: Skip php53 on translatewiki in a better way [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:10:50] (03CR) 10JanZerebecki: "Yes they do, look at the template definition." [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:11:51] !log reloading zuul for e11a9ff..d0914a7 [21:16:33] legoktm, i did push https://gerrit.wikimedia.org/r/#/c/271335/ so you could start there. jzerebecki helpfully guided me to the fact that we needed a php5.5 requirement there. [21:23:48] (03CR) 10Paladox: "Oh sorry." [integration/config] - 10https://gerrit.wikimedia.org/r/271348 (owner: 10JanZerebecki) [21:33:44] (03PS2) 10Legoktm: Run parsoidsvc-php-parsertests with php5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/271335 (owner: 10Subramanya Sastry) [21:36:10] subbu: https://integration.wikimedia.org/ci/job/parsoidsvc-php-parsertests/6855/console [21:36:30] (03CR) 10Legoktm: [C: 032] "Deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/271335 (owner: 10Subramanya Sastry) [21:36:45] legoktm, \o/ [21:41:11] (03Merged) 10jenkins-bot: Run parsoidsvc-php-parsertests with php5.5 [integration/config] - 10https://gerrit.wikimedia.org/r/271335 (owner: 10Subramanya Sastry) [21:42:31] 10MediaWiki-Codesniffer: phpcs should error on unclosed @codingStandardsIgnoreStart annotations - https://phabricator.wikimedia.org/T127241#2036732 (10Legoktm) 3NEW [21:43:44] (03PS4) 10Paladox: Add new hhvm lint and use it [integration/config] - 10https://gerrit.wikimedia.org/r/270938 (https://phabricator.wikimedia.org/T97889) [21:47:48] > Exception: No PHID found for slug #mw1.27.0-wmf.14! [21:48:48] legoktm: that's not a valid project name [21:49:14] afaik [21:49:16] yeah, it has #mw1.27.0-wmf14 [21:49:39] legoktm: Oh, yeah, my fault. [21:49:55] legoktm: Or, rather, whoever in RelEng made the mistake of making both wmf.12 and wmf12 [21:50:04] hmm [21:50:06] 2016-02-17 21:02:18,064: INFO - https://gerrit.wikimedia.org/r/269411: merged in branch master, Task T121950, needs slugs ['mw1.27.0-wmf15'] [21:50:06] 2016-02-17 21:02:20,358: INFO - https://gerrit.wikimedia.org/r/271009: merged in branch wmf/1.27.0-wmf.14, Task T99096, needs slugs ['mw1.27.0-wmf.14'] [21:50:06] T121950: Rephrase Flow notifications, move content from body into header - https://phabricator.wikimedia.org/T121950 [21:50:07] T99096: Make Varnish cache for /static/$wmfbranch/ expire when resources change within branch lifetime - https://phabricator.wikimedia.org/T99096 [21:50:11] shush stashbot [21:50:16] Krenair: do you know what 'deployment-upload' is used for? [21:51:09] interesting [21:51:10] wmf_parts[1] = wmf_parts[1].replace('.', '') [21:51:14] legoktm: Fixed. [21:51:41] thanks, it'll run in 9 minutes [21:52:20] yuvipanda, not off the top of my head [22:27:53] legoktm and krinkle i keep getting this [17-Feb-2016 22:18:11 Europe/London] PHP Warning: strpos(): Empty needle in /home/pal2015/public_html/includes/OutputPage.php on line 3873 in my error log. Does that mean there is a bug and that i have to file a bug for that. [22:45:12] paladox: Sounds like a probelm in your mediawiki configuration. Both parameters to that function have default values in Setup.ph [22:45:21] Maybe $wgResourceBasePath is assigned an invalid value in your LocalSettings.php? [22:45:38] Krinkle: I have it set like $wgResourceBasePath = $wgScriptPath; [22:45:50] paladox: in LocalSettings or Setup? [22:45:58] paladox: Can you figure out which parametervalues are passed to strpos() [22:46:04] When the error happens [22:46:09] Krinkle: In LocalSettings.php [22:46:22] How would i find out if parametervalues are passed to strpos() [22:46:31] It happends when i load http://wikisaur.tk [22:46:34] paladox: $remotePath = ...; [22:46:36] then add this: [22:46:55] Ok. [22:47:02] if ($$remotePath ) { error_log( 'remotePath is: ' . $remotePath ); } [22:47:05] if (!$remotePath ) { error_log( 'remotePath is: ' . $remotePath ); } [22:47:23] and check your error log [22:47:35] Krinkle: Ok thanks [22:47:44] paladox: You should not do '$wgResourceBasePath = $wgScriptPath;' in LocalSettings.php, that is the default already. [22:48:05] Krinkle: It is set when you do the web updater. [22:48:09] OK [22:48:21] I guess it needs to re-use that variable, ok [22:48:43] Krinkle: Ok. [22:49:43] paladox: Ah, you're using a root install? [22:49:56] It's doing /index.php and /skins [22:50:01] instead of /w/index.php and /w/skins [22:50:06] Krinkle: Um not sure what you mean by root [22:50:13] root directory [22:50:21] Krinkle: Oh yes. [22:50:40] Krinkle i am using pretty urls. It isent installed in a w directory or folder. [22:51:03] Krinkle After doing what you said by adding that code i get this in the error log [22:51:04] [17-Feb-2016 22:49:17 Europe/London] PHP Warning: strpos(): Empty needle in /home/pal2015/public_html/includes/OutputPage.php on line 3873 [22:51:47] Krinkle i think that the message may be different in a newer php version since i am using php 5.4 on that wiki and i am aware that the message was confusing so was updated. [22:51:57] https://bugs.php.net/bug.php?id=63943 [22:52:43] That bug is unrelated [22:52:53] The problem is that the variable contains the empty string [22:53:00] It's not supposed to be empty [22:53:21] You can have pretty urls for wiki pages and still have mediawiki in a subdirectory internally [22:53:50] Basically move public_html to public_html/w and be sure to check your Apache rewrite urls to make sure "/" is rewritten to /w/index.php [22:53:55] Krinkle: I have it in the public_html folder but have pretty urls turned on. [22:54:02] and set wgScriptPath = /w [22:54:09] Krinkle: Ok. [22:54:25] MediaWIki supports using the root for pages, but I don't think it supports using the root for scriptPath [22:55:30] Krinkle: Oh is there a bug on that since it should support root. [22:56:03] paladox: There is no benefit to supporting script path as root, I think that's a bad idea. [22:56:09] Krinkle: Is there anyway for me to redirect after one from using the normal http://wikisaur.tk to http://wikisaur.tk/w/ [22:56:17] Krinkle: Oh. [22:56:20] paladox: No need to redirect [22:56:26] it will not change the pretty urls [22:56:31] it will stay as .tk/wiki/Foo [22:57:03] Krinkle: Ok but i mean because users visiting wikisaur.tk will recicive the note found page unless they go to wikisaur.tk/w [22:57:30] paladox: I said earlier "check your Apache rewrite urls to make sure "/" is rewritten to /w/index.php" [22:57:40] Krinkle: Ok. [22:57:43] It will work just like now. wikisaur.tk -> wikisaur.tk /wiki/Main Page automatically [22:57:51] Thanks. [23:00:03] Krinkle: It works now. I think the errors have gone [23:00:49] paladox: See https://www.mediawiki.org/wiki/Manual:Short_URL/Apache for more examples and infromation [23:02:19] Krinkle: Ok thanks also do you know what kind of differences users would have if it is not put into a sub path. [23:19:01] paladox: I'm not sure what you mena [23:19:28] Krinkle: What would the difference between having the wiki set in the root compared to in a sub folder. [23:19:53] paladox: well, the short answer is that the difference is that one version works and the other one doesn't :) [23:20:04] I don't have a good answer unfortunately. [23:20:07] Krinkle: Oh ok. [23:20:33] I agree this should be better documented and ideally detected by the installer. [23:24:57] Krinkle ive tryed updating my rules for the url but they are not working. Actually it has caused my wiki to not show now. [23:25:24] This is what i have set in .htaccess inside the /w folder. [23:25:25] RewriteEngine On [23:25:25] RewriteRule ^/?wiki(/.*)?$ %{DOCUMENT_ROOT}/w/index.php [L] [23:25:25] RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} !-f [23:25:25] RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} !-d [23:25:25] RewriteRule ^/?images/thumb/[0-9a-f]/[0-9a-f][0-9a-f]/([^/]+)/([0-9]+)px-.*$ %{DOCUMENT_ROOT}/w/thumb.php?f=$1&width=$2 [L,QSA,B] [23:25:28] RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} !-f [23:25:30] RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} !-d [23:25:32] RewriteRule ^/?images/thumb/archive/[0-9a-f]/[0-9a-f][0-9a-f]/([^/]+)/([0-9]+)px-.*$ %{DOCUMENT_ROOT}/w/thumb.php?f=$1&width=$2&archived=1 [L,QSA,B] [23:25:37] And this is what i have set in root [23:25:39] RewriteEngine On [23:25:41] RewriteRule ^$ /w/index.php [L] [23:25:53] paladox: they should all be in the root .htaccess [23:26:08] and images becomes w/images [23:26:09] Krinkle: Oh. [23:27:14] Krinkle: wikis works now but when i go to for example simple-random-wikisaur.tk it redirects to /wwwwwww/ [23:28:19] I don't know what that website is [23:28:28] Is it pointing at the same server? [23:28:44] It would presumably be handled by a different virtualhost [23:29:08] paladox: make sure to remove the one in /w/ [23:29:33] Follow the examples at https://www.mediawiki.org/wiki/Manual:Short_URL/Apache#Setting_up_the_rewrite_rules and https://www.mediawiki.org/wiki/Manual:Thumb.php#MediaWiki_.3E.3D_1.20 [23:29:36] Krinkle: Yes pointing to the same server. And im not sure what you mean by remove the one in /w/ [23:29:37] ok [23:29:49] paladox: maybe you still have an old copy of htaccess in /w/ [23:29:57] anyway, I'm going back to work. Ask in #mediawiki for general mediawiki support. [23:30:02] Krinkle Ok. [23:30:04] This is not an issue with the software or releng [23:32:43] ok. [23:46:52] 3Scap3, 10scap, 5Patch-For-Review: Make puppet provider for scap3 - https://phabricator.wikimedia.org/T113072#2037132 (10chasemp) [23:56:03] could anybody point me to which settings CI uses for globals when running CI tests? [23:56:12] I mean mediawiki CI tests of course