[02:17:47] (03CR) 1020after4: "Why the strict treatment of Bug: lines?" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [02:19:50] (03CR) 10Alex Monk: "The bot won't pick it up otherwise. I've seen people who *really* should know better putting blank lines between Bug: lines and Change-Id:" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [02:35:37] Yippee, build fixed! [02:35:38] Project browsertests-WikiLove-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #692: FIXED in 2 min 36 sec: https://integration.wikimedia.org/ci/job/browsertests-WikiLove-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/692/ [02:41:53] Yippee, build fixed! [02:41:54] Project browsertests-PageTriage-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #656: FIXED in 53 sec: https://integration.wikimedia.org/ci/job/browsertests-PageTriage-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/656/ [04:31:40] Yippee, build fixed! [04:31:40] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-9-sauce build #559: FIXED in 39 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_7-internet_explorer-9-sauce/559/ [04:40:03] Yippee, build fixed! [04:40:04] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_8.1-internet_explorer-11-sauce build #565: FIXED in 33 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-windows_8.1-internet_explorer-11-sauce/565/ [08:19:03] (03PS3) 10TasneemLo: Add SpaceBeforeElseSniff to handle else structures [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/237733 (https://phabricator.wikimedia.org/T101311) [09:32:35] (03CR) 1020after4: "Gerrit is just so totally broken. I like phabricator's approach: much more thorough parsing of the commit message with multiple ways to sa" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [09:35:35] 10Gitblit-Deprecate: redirect gerrit repo paths to diffusion callsigns - https://phabricator.wikimedia.org/T110607#1633472 (10mmodell) @spage: upstream is working on path/to/repo support in diffusion. See [[ https://secure.phabricator.com/T4245 | upstream task ]]. [09:44:43] Yippee, build fixed! [09:44:43] Project browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce build #719: FIXED in 1 hr 34 min: https://integration.wikimedia.org/ci/job/browsertests-MultimediaViewer-en.wikipedia.beta.wmflabs.org-os_x_10.9-safari-sauce/719/ [10:20:51] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633545 (10Paladox) 3NEW [10:21:54] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633552 (10Paladox) And it will do until the system is re written to allow every one without compromising anything. [10:22:03] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633553 (10Paladox) [11:47:39] (03PS1) 10TasneemLo: Tests: Revamp tests to UnitTests for each Sniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/237899 (https://phabricator.wikimedia.org/T92751) [12:01:30] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633639 (10hashar) 5Open>3declined a:3hashar If you are whitelisted, your code is run. So a single repo whitelisted is the same as any repos. Additionally Zuul does not have such fin... [12:05:21] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633644 (10Paladox) Oh ok. [12:06:31] 10Continuous-Integration-Config: Add per extension/skin whitelisting for users - https://phabricator.wikimedia.org/T112367#1633645 (10Paladox) I am not whitelisted that is why I am asking for this support. or can you be whitelisted but blacklisted for the entire repos except the one you want to edit. [12:07:36] 10Continuous-Integration-Config: Create Jenkins bot for mediawiki/extensions/SafeDelete - https://phabricator.wikimedia.org/T112370#1633646 (10TTO) [12:13:32] (03CR) 10Hashar: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/236258 (owner: 10Paladox) [12:31:10] (03CR) 10Hashar: "We could add the set of rules as a Prolog in Gerrit to prevent submission of a patch. Would be done in All-Projects" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [12:52:50] (03PS1) 10TasneemLo: Sniff to check assignment in while & if [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/237902 (https://phabricator.wikimedia.org/T92744) [12:55:10] Project beta-scap-eqiad build #69685: FAILURE in 1 min 12 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/69685/ [13:05:10] Yippee, build fixed! [13:05:11] Project beta-scap-eqiad build #69686: FIXED in 1 min 11 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/69686/ [13:44:04] (03CR) 10Jforrester: "But… Prolog." [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [15:21:52] 10Differential, 15User-greg: Arcanist isn't packaged in Fedora, Debian, or Ubuntu - https://phabricator.wikimedia.org/T133#1633897 (10JanZerebecki) I don't think creating debian packages for our version of arcanist is useful, except the chain of trust and integrity it provides. Instead I would suggest as allud... [15:25:50] (03CR) 10Legoktm: "I don't think we want to prevent submission of patches. These are very easy issues to fix, and rejecting patches because of it would be ov" [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (owner: 10BryanDavis) [15:25:59] (03PS3) 10Legoktm: Add commit-message-validator.py tool [integration/jenkins] - 10https://gerrit.wikimedia.org/r/237719 (https://phabricator.wikimedia.org/T109119) (owner: 10BryanDavis) [15:26:12] 10Beta-Cluster, 10MediaWiki-extensions-OAuth, 10Pywikibot-OAuth, 7Pywikibot-network, 7Pywikibot-tests: "Nonce already used" regularly occurring on beta cluster - https://phabricator.wikimedia.org/T109173#1633903 (10jayvdb) 5Resolved>3Open Both jobs in the same build: https://travis-ci.org/jayvdb/pywi... [15:26:22] 10Beta-Cluster, 10MediaWiki-extensions-OAuth, 10Pywikibot-OAuth, 7Pywikibot-network, 7Pywikibot-tests: "Nonce already used" regularly occurring on beta cluster - https://phabricator.wikimedia.org/T109173#1633905 (10jayvdb) p:5Unbreak!>3Normal [15:26:51] 10Continuous-Integration-Config, 10pywikibot-core, 5Patch-For-Review: Check the style of the commit message - https://phabricator.wikimedia.org/T109119#1633906 (10Legoktm) @bd808 started working on this and neither of us were aware of this task I think :P [15:30:36] 10Continuous-Integration-Config, 10pywikibot-core, 5Patch-For-Review: Check the style of the commit message - https://phabricator.wikimedia.org/T109119#1633921 (10XZise) Hmm my main concern about such a checker is how we'd handle reasonable exceptions. Maybe there is some reason why something can violate the... [15:32:21] 6RelEng-Admin, 15User-greg: Write draft/strawman code-hosting exception guideline - https://phabricator.wikimedia.org/T109920#1633923 (10JanZerebecki) The draft currently conflates git hosting and review platform. Phabricator seems to support being a review platform for pull requests to a repo hosted anywhere.... [15:51:42] 6RelEng-Admin, 15User-greg: Write draft/strawman code-hosting exception guideline - https://phabricator.wikimedia.org/T109920#1633978 (10greg) Not now, thanks. No rush. I'll take a swing when I'm back at a real keyboard. [17:39:53] (03PS2) 10JanZerebecki: Replace composer shell snippets with builder [integration/config] - 10https://gerrit.wikimedia.org/r/237710 [17:40:24] (03CR) 10JanZerebecki: "PS2: rebase only" [integration/config] - 10https://gerrit.wikimedia.org/r/237710 (owner: 10JanZerebecki) [18:03:55] (03CR) 10JanZerebecki: "Deployed to Jenkins: (['mediawiki-phpunit-hhvm-composer', 'mediawiki-phpunit-zend-composer', 'mwext-Wikibase-client-tests-mysql-hhvm', 'mw" [integration/config] - 10https://gerrit.wikimedia.org/r/237710 (owner: 10JanZerebecki) [18:14:05] (03CR) 10JanZerebecki: [C: 032] "Example run: https://integration.wikimedia.org/ci/job/mwext-Wikibase-client-tests-mysql-hhvm/4972/console" [integration/config] - 10https://gerrit.wikimedia.org/r/237710 (owner: 10JanZerebecki) [18:15:24] (03Abandoned) 10JanZerebecki: [WIP] Change target for clone to src. [integration/config] - 10https://gerrit.wikimedia.org/r/232558 (https://phabricator.wikimedia.org/T90303) (owner: 10JanZerebecki) [18:15:52] (03Merged) 10jenkins-bot: Replace composer shell snippets with builder [integration/config] - 10https://gerrit.wikimedia.org/r/237710 (owner: 10JanZerebecki) [18:20:52] (03Abandoned) 10JanZerebecki: Run composer commands inside "src" for mediawiki-core [integration/config] - 10https://gerrit.wikimedia.org/r/193757 (owner: 10Legoktm) [18:39:35] 10Continuous-Integration-Infrastructure, 5MW-1.26-release, 5Patch-For-Review: Fetch dependencies using composer instead of cloning mediawiki/vendor for non-wmf branches - https://phabricator.wikimedia.org/T90303#1634150 (10JanZerebecki) Since https://gerrit.wikimedia.org/r/#/c/237710/2 the experimental jobs... [18:43:50] 10Beta-Cluster: New Accounts (Confirmed) Not Allowed to Use Twinkle - https://phabricator.wikimedia.org/T110897#1634154 (10Luke081515) @Skyllfully Confirmed is only local, at which wiki you try to use twinkle? [18:49:41] (03PS1) 10JanZerebecki: Make composer update more verbose [integration/config] - 10https://gerrit.wikimedia.org/r/237959 [18:53:39] (03CR) 10JanZerebecki: [C: 032] "Deployed to Jenkins: (['mediawiki-phpunit-hhvm-composer', 'mediawiki-phpunit-zend-composer', 'mwext-Wikibase-client-tests-mysql-hhvm', 'mw" [integration/config] - 10https://gerrit.wikimedia.org/r/237959 (owner: 10JanZerebecki) [18:55:19] (03Merged) 10jenkins-bot: Make composer update more verbose [integration/config] - 10https://gerrit.wikimedia.org/r/237959 (owner: 10JanZerebecki) [18:59:57] (03PS3) 10JanZerebecki: Ruby Ruby syntax check and RuboCop for PoolCounter [integration/config] - 10https://gerrit.wikimedia.org/r/237638 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:00:06] (03CR) 10JanZerebecki: [C: 032] Ruby Ruby syntax check and RuboCop for PoolCounter [integration/config] - 10https://gerrit.wikimedia.org/r/237638 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:00:58] (03Merged) 10jenkins-bot: Ruby Ruby syntax check and RuboCop for PoolCounter [integration/config] - 10https://gerrit.wikimedia.org/r/237638 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:02:24] (03PS3) 10JanZerebecki: Ruby Ruby syntax check and RuboCop for Gather [integration/config] - 10https://gerrit.wikimedia.org/r/237376 (https://phabricator.wikimedia.org/T112097) (owner: 10Zfilipin) [19:02:35] (03CR) 10JanZerebecki: [C: 032] Ruby Ruby syntax check and RuboCop for Gather [integration/config] - 10https://gerrit.wikimedia.org/r/237376 (https://phabricator.wikimedia.org/T112097) (owner: 10Zfilipin) [19:03:24] (03Merged) 10jenkins-bot: Ruby Ruby syntax check and RuboCop for Gather [integration/config] - 10https://gerrit.wikimedia.org/r/237376 (https://phabricator.wikimedia.org/T112097) (owner: 10Zfilipin) [19:13:47] (03PS2) 10JanZerebecki: Run Ruby syntax check for operations/puppet in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237375 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:14:06] (03CR) 10JanZerebecki: [C: 032] Run Ruby syntax check for operations/puppet in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237375 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:14:55] (03Merged) 10jenkins-bot: Run Ruby syntax check for operations/puppet in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237375 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:19:03] (03PS3) 10JanZerebecki: Run Ruby syntax check for mediawiki/vagrant in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237373 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:19:15] (03CR) 10JanZerebecki: [C: 032] Run Ruby syntax check for mediawiki/vagrant in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237373 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:20:11] (03Merged) 10jenkins-bot: Run Ruby syntax check for mediawiki/vagrant in experimental pipeline [integration/config] - 10https://gerrit.wikimedia.org/r/237373 (https://phabricator.wikimedia.org/T1361) (owner: 10Zfilipin) [19:21:11] (03CR) 10JanZerebecki: [V: 04-1] "Waiting for dependency." [integration/config] - 10https://gerrit.wikimedia.org/r/237492 (https://phabricator.wikimedia.org/T55354) (owner: 10Hashar) [19:21:26] (03CR) 10JanZerebecki: [V: 04-1] "Waiting for dependency." [integration/config] - 10https://gerrit.wikimedia.org/r/237617 (https://phabricator.wikimedia.org/T55354) (owner: 10Hashar) [19:41:19] 10Beta-Cluster: New Accounts (Confirmed) Not Allowed to Use Twinkle - https://phabricator.wikimedia.org/T110897#1634225 (10Skyllfully) [[ http://en.wikipedia.beta.wmflabs.org | en.wikipedia.beta.wmflabs.org ]] [19:48:42] !log reloading zuul for c7f17b5..a813107 [19:48:48] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [20:31:52] 10Beta-Cluster: New Accounts (Confirmed) Not Allowed to Use Twinkle - https://phabricator.wikimedia.org/T110897#1634270 (10Luke081515) @Skyllfully Ok, please try it again, you reported this at end of august, where you are only confirmed at deploymentwiki, not at enwiki, you are now confirmed there, so please try... [20:32:03] 10Beta-Cluster: New Accounts (Confirmed) Not Allowed to Use Twinkle - https://phabricator.wikimedia.org/T110897#1634271 (10Luke081515) a:3Luke081515 [23:33:24] 10Beta-Cluster, 10Traffic, 6operations, 5Patch-For-Review: Puppet failing on deployment-prep caches - https://phabricator.wikimedia.org/T104076#1634570 (10Luke081515)