[00:00:39] greg-g: none in particular, i know the deployment schedule. but i like linking to that page when telling people when a given fix will be deployed. [00:01:20] (03PS4) 10Paladox: [LinkSuggest2] Update Jenkins test to more advanced tests [integration/config] - 10https://gerrit.wikimedia.org/r/239681 [00:04:47] MatmaRex: gotcha, yeah, that's understandable, if it wasn't a pain to edit I'd do it more. I already manually edit [[deployments]] each week, which is enough pain in the butt :) [00:05:22] MatmaRex: also, I don't want to give people a promise we can't keep (ie: the schedule changing due to a bad deploy or a holiday (that I forgot about ;) ) or what not [00:10:33] 10Differential, 10Differential-Beta, 6Phabricator: Diffusion unable to view PNG files in commit - https://phabricator.wikimedia.org/T117628#1780243 (10Krinkle) 3NEW [00:11:03] 10Differential, 10Differential-Beta, 6Phabricator: Diffusion unable to view PNG files in commit - https://phabricator.wikimedia.org/T117628#1780252 (10Krinkle) [00:32:08] 10Differential, 6Phabricator: Differential unable to view PNG files in commit - https://phabricator.wikimedia.org/T117628#1780354 (10greg) [00:57:01] Notice: Undefined variable: wmgContentTranslationCXServerAuthKey in /srv/mediawiki/wmf-config/CommonSettings.php on line 2701 [00:57:05] When I start eval.php on mira [00:58:07] //This is set in PrivateSettings.php [00:58:07] 'key' => $wmgContentTranslationCXServerAuthKey, [00:58:08] Ugh. [00:58:41] kart_ did it [01:04:15] https://phabricator.wikimedia.org/T116134 [01:39:09] Yippee, build fixed! [01:39:09] Project browsertests-Wikidata-SmokeTests-linux-firefox-sauce build #429: 09FIXED in 22 min: https://integration.wikimedia.org/ci/job/browsertests-Wikidata-SmokeTests-linux-firefox-sauce/429/ [01:39:19] Project browsertests-Wikidata-PerformanceTests-linux-firefox build #1: 04FAILURE in 18 sec: https://integration.wikimedia.org/ci/job/browsertests-Wikidata-PerformanceTests-linux-firefox/1/ [01:50:30] PROBLEM - Free space - all mounts on deployment-bastion is CRITICAL: CRITICAL: deployment-prep.deployment-bastion.diskspace._var.byte_percentfree (<12.50%) [03:11:08] Project browsertests-Wikidata-WikidataTests-linux-firefox build #1: 04FAILURE in 1 hr 13 min: https://integration.wikimedia.org/ci/job/browsertests-Wikidata-WikidataTests-linux-firefox/1/ [03:25:42] Project browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #869: 04FAILURE in 43 min: https://integration.wikimedia.org/ci/job/browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce/869/ [03:34:22] Project beta-scap-eqiad build #77181: 04FAILURE in 9 min 4 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77181/ [03:52:01] !log sudo rm /var/log/account/pacct.?* on deployment-bastion for the usual reason [03:52:04] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [03:57:59] !log sudo rm /var/log/atop.log.? on deployment-bastion [03:58:02] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [04:00:44] !log sudo rm /var/log/apache2/*.??.gz on deployment-bastion [04:00:48] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [04:01:21] !log sudo rm /var/log/*.??.gz on deployment-bastion [04:01:25] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [04:05:23] RECOVERY - Free space - all mounts on deployment-bastion is OK: OK: All targets OK [04:15:47] !log Handy cleanup script for when /var is full on deployment-bastion -- https://phabricator.wikimedia.org/P2273 [04:15:51] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [04:28:30] 6Release-Engineering-Team, 3Scap3, 10Wikimedia-Developer-Summit-2016: Scap3: updates, upgrades, and challenges - https://phabricator.wikimedia.org/T114045#1780757 (10Babygirl.md7565) p:5Triage>3High [04:31:38] 6Release-Engineering-Team, 3Scap3, 10Wikimedia-Developer-Summit-2016: Scap3: updates, upgrades, and challenges - https://phabricator.wikimedia.org/T114045#1780766 (10Babygirl.md7565) [04:32:21] twentyafterfour: phab vandal?? ^ [04:35:30] 6Release-Engineering-Team, 3Scap3, 10Wikimedia-Developer-Summit-2016: Scap3: updates, upgrades, and challenges - https://phabricator.wikimedia.org/T114045#1780773 (10greg) p:5High>3Normal [04:35:53] 6Release-Engineering-Team, 3Scap3, 10Wikimedia-Developer-Summit-2016: Scap3: updates, upgrades, and challenges - https://phabricator.wikimedia.org/T114045#1683210 (10greg) [04:54:07] Krenair: that need to fix in beta, right? [05:06:02] uh. I don't have permission for sync-file on scap? [05:06:09] on beta* [05:06:25] sudo to the jenkins-deploy user [05:06:37] sudo -u jenkins-deploy -s [05:07:10] beta cluster has some kind of funny stuff for the ssh keys [05:12:05] oh [05:13:24] bd808: thanks! [05:13:35] yw [05:14:01] ok. It doesn't log anywhere? [05:14:07] sync finished. [05:14:08] nope [05:14:43] Noted. [05:15:06] Just !log yourself if you are doing something that folks should know about [05:17:20] #til rsync barfs when .swp files disappear from under it. [05:17:25] https://phabricator.wikimedia.org/D34 uploaded to deal. [05:17:30] We should just skip them [05:23:15] 10Deployment-Systems, 15User-bd808: Special:Version on Wikimedia wikis shows outdated commit hashes for submodules - https://phabricator.wikimedia.org/T116345#1780814 (10bd808) [05:29:08] ostriches: LGTM [05:29:18] what flow are you folks using for landing? [05:29:39] does the author do it or the reviewer? [05:30:39] Easier to have the author do it, if they have push rights. [05:48:28] 10Differential, 5Gerrit-Migration, 6translatewiki.net, 7I18n: Train L10n-bot to work with Differential in Wikimedia Phabricator - https://phabricator.wikimedia.org/T92493#1780871 (10revi) >>! In T92493#1112872, @demon wrote: > I actually don't think it'll need to speak arc/differential. In l18n-bot's case... [05:50:47] Yippee, build fixed! [05:50:47] Project beta-scap-eqiad build #77192: 09FIXED in 5 min 15 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77192/ [05:50:50] 10Differential, 5Gerrit-Migration, 6translatewiki.net, 7I18n: Train L10n-bot to work with Differential in Wikimedia Phabricator - https://phabricator.wikimedia.org/T92493#1780874 (10Legoktm) >>! In T92493#1112872, @demon wrote: > I actually don't think it'll need to speak arc/differential. In l18n-bot's ca... [05:56:16] 10Differential, 5Gerrit-Migration, 6translatewiki.net, 7I18n: Train L10n-bot to work with Differential in Wikimedia Phabricator - https://phabricator.wikimedia.org/T92493#1780878 (10demon) >>! In T92493#1780874, @Legoktm wrote: >>>! In T92493#1112872, @demon wrote: >> I actually don't think it'll need to s... [05:57:05] heh [05:59:34] :p [06:11:25] 10Continuous-Integration-Config: mwext-testextension-zend should load extension mathsearch after math - https://phabricator.wikimedia.org/T117659#1780887 (10Physikerwelt) 3NEW [06:15:57] duh [07:24:20] 10Differential, 5Gerrit-Migration, 6translatewiki.net, 7I18n: Train L10n-bot to work with Differential in Wikimedia Phabricator - https://phabricator.wikimedia.org/T92493#1780936 (10Nikerabbit) >>! In T92493#1780878, @demon wrote: > Long term I'd love a world where we don't have to make gigantic daily comm... [07:46:33] 10Browser-Tests, 10Wikidata: Wikidata various features: edit label, description, alias goes to the non-js version - https://phabricator.wikimedia.org/T117584#1780952 (10Sjoerddebruin) Yup, this is since they changed the way Javascript is loaded. [09:38:29] Project browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #662: 04FAILURE in 1 min 27 sec: https://integration.wikimedia.org/ci/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-chrome-sauce/662/ [09:54:02] 10Beta-Cluster-Infrastructure, 6Labs, 10Labs-Infrastructure, 7Graphite, 7Shinken: Delete more specific deployment-prep graphite datapoints - https://phabricator.wikimedia.org/T111540#1781112 (10fgiunchedi) diamond discovers the mount point locally and starts pushing metrics for those [09:54:47] PROBLEM - Puppet staleness on integration-dev is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [43200.0] [10:04:43] (03PS1) 10Adrian Lang: Add qunit jobs for data-values/value-view [integration/config] - 10https://gerrit.wikimedia.org/r/250937 [10:10:26] (03CR) 10JanZerebecki: [C: 032] Add qunit jobs for data-values/value-view [integration/config] - 10https://gerrit.wikimedia.org/r/250937 (owner: 10Adrian Lang) [10:18:19] (03Merged) 10jenkins-bot: Add qunit jobs for data-values/value-view [integration/config] - 10https://gerrit.wikimedia.org/r/250937 (owner: 10Adrian Lang) [10:21:27] !log reloading zuul for b8808f8..0f620bd [10:21:30] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [10:44:35] Krenair: can you sync PrivateSettings.php on labs? It seems erorr is still there despite of adding variable and running sync-file. [10:48:57] kart_, is it fixed now? [10:49:36] I synchronised it as jenkins-deploy [11:01:51] 10Browser-Tests, 10Wikidata, 7Tracking: [tracking] make Wikidata browsertests non-flaky - https://phabricator.wikimedia.org/T92619#1781265 (10JanZerebecki) [11:03:59] 10Browser-Tests, 10Wikidata: Wikidata Smoketest Headless: XHR didn't load - https://phabricator.wikimedia.org/T117591#1781268 (10JanZerebecki) [11:06:29] 10Browser-Tests, 10Wikidata: Wikidata Smoketest Headless: XHR didn't load - https://phabricator.wikimedia.org/T117591#1781275 (10Tobi_WMDE_SW) [11:14:39] 10Browser-Tests, 10Wikidata, 7Tracking: [tracking] make Wikidata browsertests non-flaky - https://phabricator.wikimedia.org/T92619#1781315 (10JanZerebecki) [11:15:42] Krenair: thanks. I did same, but was throwing same error. [11:16:36] 6Release-Engineering-Team, 3Scap3, 10Wikimedia-Developer-Summit-2016: Scap3: updates, upgrades, and challenges - https://phabricator.wikimedia.org/T114045#1781317 (10Qgil) [11:22:36] 10Browser-Tests, 10Wikidata: Wikidata Smoketest Headless: XHR didn't load - https://phabricator.wikimedia.org/T117591#1781341 (10JanZerebecki) This is not a failed XHR request: Feature: Property smoke test Scenario: Click statement add button, moving it to a different bug. [11:25:11] 10Browser-Tests, 10Wikidata: Wikidata Feature: Property smoke test Scenario: Click statement add button; button doesn't appear - https://phabricator.wikimedia.org/T117677#1781350 (10JanZerebecki) 3NEW [11:26:18] 10Browser-Tests, 10Wikidata: Wikidata Feature: Edit sitelinks Scenario Outline: Save sitelink; save doesn't activate - https://phabricator.wikimedia.org/T117591#1781360 (10JanZerebecki) [11:27:00] 10Browser-Tests, 10Wikidata, 7Regression, 3Wikidata-Sprint-2015-11-03: AuthorityControl gadget browsertest fail - https://phabricator.wikimedia.org/T117564#1777775 (10JanZerebecki) [11:27:02] 10Browser-Tests, 10Wikidata, 7Tracking: [tracking] make Wikidata browsertests non-flaky - https://phabricator.wikimedia.org/T92619#1781362 (10JanZerebecki) [11:36:32] 10Browser-Tests, 10Wikidata, 3Wikidata-Sprint-2015-11-03: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781369 (10thiemowmde) 3NEW [11:37:08] 10Browser-Tests, 10Wikidata, 7Easy, 7Need-volunteer, 3Wikidata-Sprint-2015-11-03: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781376 (10thiemowmde) [11:45:02] 10Browser-Tests, 10Wikidata, 7Easy, 7Need-volunteer, 3Wikidata-Sprint-2015-11-03: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781390 (10thiemowmde) [12:28:45] RECOVERY - Host deployment-parsoidcache02 is UP: PING OK - Packet loss = 0%, RTA = 1.12 ms [12:56:01] 10Browser-Tests, 10Wikidata, 7Easy, 7Need-volunteer, 3Wikidata-Sprint-2015-11-03: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781527 (10JanZerebecki) Maybe this is also a good idea for qunit tests. [12:56:50] 10Browser-Tests, 10Wikidata, 7Need-volunteer: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781529 (10JanZerebecki) [12:57:03] Project beta-scap-eqiad build #77235: 04FAILURE in 2 min 7 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77235/ [12:57:30] 10Browser-Tests, 10Wikidata, 7Need-volunteer: [Task] Disable all jQuery animations while running browser tests - https://phabricator.wikimedia.org/T117678#1781369 (10JanZerebecki) To qualify for easy it first needs to be explained where to set this. [13:02:10] Yippee, build fixed! [13:02:10] Project browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #838: 09FIXED in 30 min: https://integration.wikimedia.org/ci/job/browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce/838/ [13:08:08] Yippee, build fixed! [13:08:08] Project beta-scap-eqiad build #77236: 09FIXED in 2 min 54 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77236/ [13:24:01] (03PS1) 10Thiemo Mättig (WMDE): Avoid in_array for performance reasons [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 [13:28:03] (03CR) 10jenkins-bot: [V: 04-1] Avoid in_array for performance reasons [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 (owner: 10Thiemo Mättig (WMDE)) [14:28:39] Project browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #679: 04FAILURE in 2 min 37 sec: https://integration.wikimedia.org/ci/job/browsertests-Echo-en.wikipedia.beta.wmflabs.org-linux-firefox-sauce/679/ [14:50:33] (03PS2) 10Phedenskog: Test the desktop site at WPT.org [integration/config] - 10https://gerrit.wikimedia.org/r/250672 [15:17:39] 10Browser-Tests, 10Wikidata: Wikidata Feature: Creating statements Scenario Outline: Cancel statement; button never shows - https://phabricator.wikimedia.org/T117703#1781871 (10JanZerebecki) 3NEW [15:20:01] 10Browser-Tests, 10Wikidata: Wikidata Feature: Creating statements Scenario Outline: Cancel statement; button never shows - https://phabricator.wikimedia.org/T117703#1781880 (10JanZerebecki) Err the add button wasn't yet clicked. Probably the same as T117677. [15:20:25] 10Browser-Tests, 10Wikidata: Wikidata Feature: Creating statements Scenario Outline: Cancel statement; statement add button never shows - https://phabricator.wikimedia.org/T117703#1781882 (10JanZerebecki) [15:54:22] zeljkof / marxarelli https://gerrit.wikimedia.org/r/#/c/250978/ :) [15:54:38] +1 or -1 as you see fit :) [15:57:16] greg-g: looks good to me, +1d [15:57:24] because we need more spam here ;) [15:57:30] :) [15:57:41] we can discuss that if people dislike it :) [15:57:42] thcipriani: [15:57:43] set -x [15:57:45] exec tox -e venv -- jenkins-jobs --conf jenkins_jobs.ini update config/jjb "$@" [15:57:55] thcipriani: usage: ./jjbupdate.sh *tox* [16:01:59] 10Continuous-Integration-Infrastructure: some tests run from mwext-testextension-hhvm will pick up files from extensions that were not checked out for this job - https://phabricator.wikimedia.org/T117710#1782002 (10JanZerebecki) 3NEW [16:04:23] 5Release-Engineering-Epics, 10Browser-Tests-Infrastructure, 7Epic, 7Tracking: Fix or delete failing browser tests Jenkins jobs - https://phabricator.wikimedia.org/T94150#1782025 (10zeljkofilipin) [16:04:44] hashar: just fyi, gcal isn't working, but https://plus.google.com/hangouts/_/wikimedia.org?authuser=0 will give you the url to our hangout [16:05:50] (03PS2) 10Thiemo Mättig (WMDE): Avoid in_array for performance reasons [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 [16:07:21] greg-g: okkk [16:07:27] greg-g: it is in 23 minutes right? [16:07:31] yup [16:08:49] 10Beta-Cluster-Infrastructure, 10Browser-Tests-Infrastructure, 7WorkType-NewFunctionality: Make selenium users use botflags at beta-cluster - https://phabricator.wikimedia.org/T116027#1782048 (10zeljkofilipin) a:5zeljkofilipin>3None [16:08:50] was talking to tyler about https://ring.cx/ a decentralized OSS chat system that seems to have video [16:09:00] based on DHT (Distributed Hash Table) [16:09:16] with open source clients for mac / linux / windows :-} [16:09:39] yeah! I installed that a little while ago, does it do groups or just 1:1? [16:09:46] no clue [16:10:20] hmm, thought I did "which ring" gives me nothing [16:11:57] just installed, only took a minute, looks nice. [16:13:52] the dev cert is not valid :( [16:22:25] ah [16:22:27] ring:ce4ea266077905813600d9186978bbaf367975e8 ! [16:27:41] anyone know what the differential url is? [16:29:03] (03PS1) 10JanZerebecki: ext:Wikidata repo: switch to generic and add hhvm [integration/config] - 10https://gerrit.wikimedia.org/r/250990 (https://phabricator.wikimedia.org/T117544) [16:30:01] niedzielski: you mean https://phabricator.wikimedia.org/differential/ ? or something else? [16:30:21] niedzielski: you can add the Differential app to your phab sidebar [16:30:45] niedzielski: https://phabricator.wikimedia.org/differential/ [16:30:50] greg-g: hm, maybe i just don't have access to any repos? [16:31:18] (03PS2) 10JanZerebecki: ext:Wikidata repo: switch to generic and add hhvm [integration/config] - 10https://gerrit.wikimedia.org/r/250990 (https://phabricator.wikimedia.org/T117544) [16:31:20] greg-g hashari just wanted to see some projects currently using differential and mirrors of projects not would be nice too [16:31:26] (03CR) 10JanZerebecki: [C: 032] ext:Wikidata repo: switch to generic and add hhvm [integration/config] - 10https://gerrit.wikimedia.org/r/250990 (https://phabricator.wikimedia.org/T117544) (owner: 10JanZerebecki) [16:31:33] niedzielski: it is only scap for now [16:31:49] hashar: would it be publicly visible? [16:31:55] yep [16:31:57] niedzielski: should be already [16:33:20] niedzielski: looking for the actual repos? https://phabricator.wikimedia.org/diffusion/ or an example of a differential diff looks for scap https://phabricator.wikimedia.org/D31 [16:33:58] thcipriani: ah thanks! yeah, i was looking for repos like https://phabricator.wikimedia.org/diffusion/MSCA/repository/master/ [16:34:46] weird. i swear https://phabricator.wikimedia.org/diffusion/ wasn't showing _any_ repos previously [16:34:57] (03Merged) 10jenkins-bot: ext:Wikidata repo: switch to generic and add hhvm [integration/config] - 10https://gerrit.wikimedia.org/r/250990 (https://phabricator.wikimedia.org/T117544) (owner: 10JanZerebecki) [16:35:07] https://phabricator.wikimedia.org/differential/query/all/ [16:35:26] all differentials (aka diffs, patchset, whatever) ^ [16:35:51] I landed D34 last night, tldr: .swp files are silly. [16:37:14] greg-g thcipriani hashar: thanks all! [16:47:39] PROBLEM - Host deployment-parsoidcache02 is DOWN: CRITICAL - Host Unreachable (10.68.16.145) [16:47:55] ostriches: are pushes to diffusion working now? [16:51:41] marxarelli: Ya I sorted the perms last night. [16:51:50] If it doesn't work, lmk and I'll look again [16:51:51] xlnt! [16:52:03] just https though, right? [16:52:12] SSH should work. I haven't tried tho. [16:52:16] oh, nice [16:52:19] i'll give it a shot [16:55:27] 10Differential, 6Release-Engineering-Team, 15User-bd808: Setup differential for https://github.com/wikimedia/composer-merge-plugin - https://phabricator.wikimedia.org/T117293#1782278 (10mmodell) ok I got the repo to import, finally [16:58:04] Bleh. [16:58:07] twentyafterfour: Pull of "GCMP" failed: git fetch failed with error #128: stdout: stderr:fatal: unable to access 'https://github.com/wikimedia/composer-merge-plugin.git/': Failed to connect to github.com port 443: Connection timed out [16:58:28] Ah, puppet got in the way [16:58:36] Refreshed gitconfig [16:58:59] yeah I was testing by overwriting the file and then trying it before puppet could put it back [16:59:35] Let's go ahead and merge the puppet change then, we can always fiddle with it later. [16:59:46] the file that's in my gerrit patch works, afaik [16:59:50] twentyafterfour: when you get a chance, could you look at https://github.com/wikimedia/composer-merge-plugin/pull/92 and see if it looks like the right stuff for adding .arcconfig? [17:00:33] bd808: looks good to me :) [17:00:39] lgtm [17:02:14] should we have some kind of webrequest hook on the github side that tells diffusion when the repo is updated? [17:02:35] oh, looks like it figured it out pretty fast [17:03:00] bd808: it polls but it's smart about it ... phabricator calculates the average commit frequency and schedules it's polling frequency to match that [17:03:18] it's pretty variable yeah but usually quick for often updated things [17:04:26] Once I/we figure out the PR to diff mechanics we can switch things around the other way (diff as authority) and get rid of that problem [17:04:57] there is cli tooling for on demand update [17:05:08] we could do a thing such as this fyi but probably not needed or sane atm [17:05:43] I casually poked in #hhvm-dev and didn't get any response about the FB bot that does their imports. I think I'll send an email soon to the folks who were called out in thier blog post for making that tooling better [17:06:24] they seem to have something that works well and maybe I can talk them into publishing it [17:06:38] * bd808 me can be very convincing ;) [17:07:11] !log reloading zuul for 0f620bd..b3a8e99 [17:07:14] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [17:07:37] bd808: I'm planning to test the other direction (replication -> github) today. [17:08:53] (03PS1) 10Thiemo Mättig (WMDE): Remove dead code from SpaceBeforeSingleLineCommentSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250996 [17:09:40] phab's markdown parser seems to be teh suck [17:10:35] it is failing on some pretty basic stuff like relative links-- https://phabricator.wikimedia.org/diffusion/GCMP/#plugin-configuration [17:10:49] (03PS1) 10Thiemo Mättig (WMDE): Simplify existing regular expressions [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250998 [17:12:12] :/ [17:14:59] 10Continuous-Integration-Config, 10Wikidata, 3Wikidata-Sprint-2015-11-03: [Task] Add Wikidata to Jenkins job mediawiki-extensions-hhvm - https://phabricator.wikimedia.org/T96264#1782463 (10JanZerebecki) [17:15:00] 10Continuous-Integration-Config, 10Wikidata, 5Patch-For-Review, 3Wikidata-Sprint-2015-11-03: [Bug] the changed job configuration extension-unittests -> extension-unittests-generic for Wikidata.git makes it not run all tests and fail - https://phabricator.wikimedia.org/T95897#1782462 (10JanZerebecki) 5Open... [17:15:06] ostriches: can we get our github credentials added into phabricator's passphrase app so that we can mirror to github from phabricator? [17:15:48] bd808: phabricator markup isn't quite markdown it's their own proprietary markup [17:15:51] bd808: it's not markdown it's remarkup! https://secure.phabricator.com/book/phabricator/article/remarkup/ [17:15:55] https://secure.phabricator.com/T2849 [17:16:09] see also: relevant xkcd which I can't recall [17:16:26] https://xkcd.com/927/ [17:16:36] :) [17:17:21] very relevant [17:17:22] twentyafterfour: Already done, see K12/K13. [17:17:35] (Admin-only, natch) [17:17:40] K? [17:17:58] key [17:18:00] probably [17:18:07] Kredential? [17:18:19] Kphredential [17:18:41] come to think of it [17:18:54] it should have been conphidential [17:18:57] speaking of catchy names, if I build a github->differential bot it is going to be called axel [17:18:59] really missed the boat on that one [17:19:14] rose? [17:19:16] connecting the hub to the differential [17:19:43] s/axel/axle/ [17:19:56] used to work at a place where everything was 'big lebowski' related [17:20:00] bd808: it is a great name, yes [17:20:01] so the main db was called 'the rug' [17:20:03] etc [17:20:37] What was Donnie? The QA system? [17:21:14] Yay replication to github works! [17:21:20] Shouting "shut the fuck up Donnie" is very cathartic [17:21:35] twentyafterfour: Set up a remote pointing to HTTPS github, use K13. [17:21:58] bd808: it was layers of references I can't remember what donnie was unfortunately :) [17:23:16] In theory, we can have Phab take over replication to Github from Gerrit. [17:23:23] (now) [17:24:29] 10Differential, 6Release-Engineering-Team, 15User-bd808: Setup differential for https://github.com/wikimedia/composer-merge-plugin - https://phabricator.wikimedia.org/T117293#1782555 (10bd808) a:3mmodell [17:24:36] 10Differential, 6Release-Engineering-Team, 15User-bd808: Setup differential for https://github.com/wikimedia/composer-merge-plugin - https://phabricator.wikimedia.org/T117293#1782556 (10bd808) 5Open>3Resolved [17:35:56] I think I'm gonna have to mess around with ./bin/repo some more, there's no way to set these up on a wide scale. [17:37:59] ostriches: if you want custom conduit coding help I can assist. [17:38:57] 10Differential, 6Release-Engineering-Team, 15User-bd808: Setup differential for https://github.com/wikimedia/composer-merge-plugin - https://phabricator.wikimedia.org/T117293#1782699 (10mmodell) [17:55:24] twentyafterfour: I'm about to get Evan to land my other bin/repo changes, should be nbd to add more. [17:55:39] He said we can basically upstream w/e there since I'm marking it as deprecated by conduit. [17:56:34] cool [18:01:25] Also, I just got commit access without even asking, heh :) [18:03:49] ostriches: nice [18:06:56] Project beta-scap-eqiad build #77267: 04FAILURE in 2 min 10 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77267/ [18:14:26] Yippee, build fixed! [18:14:26] Project browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce build #870: 09FIXED in 38 min: https://integration.wikimedia.org/ci/job/browsertests-MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox-sauce/870/ [18:17:37] Yippee, build fixed! [18:17:38] Project beta-scap-eqiad build #77268: 09FIXED in 2 min 23 sec: https://integration.wikimedia.org/ci/job/beta-scap-eqiad/77268/ [18:20:12] twentyafterfour: Ok, my new options for ./bin/repo edit have landed. [18:20:24] ostriches: sweet! [18:38:17] 10Continuous-Integration-Config, 10Differential: Allow `doc-publish` to be run without zuul dependency - https://phabricator.wikimedia.org/T117770#1783035 (10thcipriani) 3NEW [18:51:40] 10Differential, 10Diffusion: Support full text url instead of random names for url - https://phabricator.wikimedia.org/T111947#1783070 (10Paladox) Yes. It would be easier since it would be harder to remember a callsign. [18:54:25] 10Gitblit-Deprecate, 10Diffusion: redirect gerrit repo paths to diffusion callsigns - https://phabricator.wikimedia.org/T110607#1783081 (10greg) [18:55:25] 10Differential, 10Diffusion: Support full text url instead of random names for url - https://phabricator.wikimedia.org/T111947#1783090 (10greg) This is basically {T110607} and upstream's https://secure.phabricator.com/T4245 [18:55:58] 10Differential, 10Diffusion: Support full text url instead of random names for url - https://phabricator.wikimedia.org/T111947#1783096 (10greg) In fact, I'll just merge into T110607 [18:56:08] 10Differential, 10Diffusion: Support full text url instead of random names for url - https://phabricator.wikimedia.org/T111947#1783097 (10greg) [18:56:11] 10Gitblit-Deprecate, 10Diffusion: redirect gerrit repo paths to diffusion callsigns - https://phabricator.wikimedia.org/T110607#1582193 (10greg) [19:03:16] 10Differential, 6Phabricator: Differential unable to view PNG files in commit - https://phabricator.wikimedia.org/T117628#1783146 (10greg) p:5Triage>3Normal [19:03:58] 10Differential, 5Gerrit-Migration: Differential notification emails lack headers for repository and state change - https://phabricator.wikimedia.org/T117186#1783149 (10greg) p:5Triage>3Low [19:05:02] 10Differential, 5Gerrit-Migration: Document git-review -> arc mapping. - https://phabricator.wikimedia.org/T112967#1783160 (10greg) p:5Triage>3Normal [19:07:58] 10Differential, 10Diffusion: Support one-click creation of reverts in Differential/Diffusion - https://phabricator.wikimedia.org/T117774#1783185 (10greg) 3NEW [19:09:03] (03PS1) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/251023 (https://phabricator.wikimedia.org/T88343) [19:09:25] 10Differential: Support simple ("one-click") generation of cherry-pick Differentials between branches - https://phabricator.wikimedia.org/T117776#1783206 (10greg) 3NEW [19:10:33] 10Differential: Support "one-click" rebasing of a Differential onto current master - https://phabricator.wikimedia.org/T117779#1783232 (10greg) 3NEW [19:11:10] (03Abandoned) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/251023 (https://phabricator.wikimedia.org/T88343) (owner: 10Krinkle) [19:11:13] 5Gerrit-Migration, 7Epic: Align Wikimedia and Phabricator code review processes - https://phabricator.wikimedia.org/T167#1783254 (10greg) [19:11:14] 10Differential, 5Gerrit-Migration, 15User-greg: Support automatic commit generation in Phabricator - https://phabricator.wikimedia.org/T182#1783246 (10greg) 5Open>3Invalid a:3greg This task was a collection of subtasks really. I've created those tasks and am closing this one as invalid since it's not r... [19:11:15] 10Differential, 5Gerrit-Migration, 7Documentation: Create example workflows for differential showing old way and new way side by side - https://phabricator.wikimedia.org/T117058#1783253 (10greg) [19:11:45] (03PS9) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:11:51] (03PS10) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:12:07] 10Differential: Support "one-click" rebasing of a Differential onto current master - https://phabricator.wikimedia.org/T117779#1783232 (10greg) [19:12:08] 10Differential: Support simple ("one-click") generation of cherry-pick Differentials between branches - https://phabricator.wikimedia.org/T117776#1783262 (10greg) [19:12:10] 10Differential, 5Gerrit-Migration, 15User-greg: Support automatic commit generation in Phabricator - https://phabricator.wikimedia.org/T182#1783264 (10greg) [19:12:12] 10Differential, 10Diffusion: Support one-click creation of reverts in Differential/Diffusion - https://phabricator.wikimedia.org/T117774#1783185 (10greg) [19:12:14] 10Differential, 5Gerrit-Migration, 7Documentation: Create example workflows for differential showing old way and new way side by side - https://phabricator.wikimedia.org/T117058#1765720 (10greg) [19:12:25] sorry for phab spam [19:12:37] (03CR) 10Krinkle: [C: 032] Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:13:42] RECOVERY - Host deployment-parsoidcache02 is UP: PING OK - Packet loss = 0%, RTA = 1.53 ms [19:14:25] (03CR) 10jenkins-bot: [V: 04-1] Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:14:52] (03PS11) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:16:46] (03PS12) 10Krinkle: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:16:53] 10Differential, 5Gerrit-Migration: Running `arc diff` twice on the same commit creates another redundant diff within the same differential - https://phabricator.wikimedia.org/T137#1783283 (10greg) p:5Low>3Lowest [19:17:29] (03CR) 10Krinkle: "Fixed syntax as it used job-template which doesn't work in this case as the template is not used. It's a job, not a template for another j" [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:18:23] 10Differential, 5Gerrit-Migration: "Open Revisions Affecting These Files" does not indicate direction of dependency - https://phabricator.wikimedia.org/T179#1783290 (10greg) Can we get an updated repo case for this task? This was imported from the test phab instance and the Differentials references are not the... [19:18:43] (03CR) 10Krinkle: [C: 032] Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:19:04] 10Differential, 5Gerrit-Migration: "Open Revisions Affecting These Files" does not indicate direction of dependency - https://phabricator.wikimedia.org/T179#1783293 (10greg) p:5Low>3Triage [19:19:56] 10Continuous-Integration-Infrastructure, 10Differential, 5Gerrit-Migration: Connect Differential code review with continuous integration - https://phabricator.wikimedia.org/T31#1783313 (10greg) p:5Low>3Normal [19:21:01] (03Merged) 10jenkins-bot: Add jsduck jobs for EventLogging extension [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [19:21:46] 10Differential, 5Gerrit-Migration: Provide example .arclist a standard set of lints/checks relevant to our community - https://phabricator.wikimedia.org/T170#1783328 (10greg) [19:21:53] 10Differential, 5Gerrit-Migration: Provide example .arclint a standard set of lints/checks relevant to our community - https://phabricator.wikimedia.org/T170#2287 (10greg) [19:22:37] PROBLEM - Host deployment-parsoidcache02 is DOWN: CRITICAL - Host Unreachable (10.68.16.145) [19:24:16] 10Differential, 5Gerrit-Migration: Arc does not allow to land a commit from a local master branch that is ahead of the remote's master onto the remote's master - https://phabricator.wikimedia.org/T134#1783349 (10greg) >>! In T134#1306228, @greg wrote: > I don't think this is a blocker but should inform our doc... [19:24:23] 10Differential: Arc does not allow to land a commit from a local master branch that is ahead of the remote's master onto the remote's master - https://phabricator.wikimedia.org/T134#1783351 (10greg) [19:26:26] 10Differential: Changing the Git remote of a repository locks it - https://phabricator.wikimedia.org/T120#1783366 (10greg) [19:27:14] 10Differential, 5Gerrit-Migration: Define an equivalent to Gerrit's +-1 +-2 for code review evaluation - https://phabricator.wikimedia.org/T138#1783375 (10greg) p:5Low>3Normal [19:33:23] !log Reloading Zuul to deploy https://gerrit.wikimedia.org/r/246773 [19:33:26] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [19:33:49] 10Differential: Allow self-reviewing changes; make test plan optional - https://phabricator.wikimedia.org/T114575#1783458 (10greg) Just to be explicit: You can self-merge and push changes already (if you have push permissions, obviously). You're right though, I don't see a way for the diff author to self-review... [19:34:03] 10Differential: Allow self-reviewing changes; make test plan optional - https://phabricator.wikimedia.org/T114575#1783460 (10greg) [19:34:05] 10Differential, 5Gerrit-Migration: Define an equivalent to Gerrit's +-1 +-2 for code review evaluation - https://phabricator.wikimedia.org/T138#1970 (10greg) [19:34:42] and... the #differential board is cleaned with only 3 untriaged (on purpose) https://phabricator.wikimedia.org/project/board/1487/?order=priority [19:41:47] 10Gitblit-Deprecate, 10Diffusion: Mirror unmerged changes in Diffusion from Gerrit - https://phabricator.wikimedia.org/T117625#1783500 (10greg) p:5Triage>3Low [19:59:51] (03PS1) 10Krinkle: doc: Add link to EventLogging [integration/docroot] - 10https://gerrit.wikimedia.org/r/251035 (https://phabricator.wikimedia.org/T88343) [20:00:02] (03CR) 10Krinkle: [C: 032] doc: Add link to EventLogging [integration/docroot] - 10https://gerrit.wikimedia.org/r/251035 (https://phabricator.wikimedia.org/T88343) (owner: 10Krinkle) [20:02:01] (03Merged) 10jenkins-bot: doc: Add link to EventLogging [integration/docroot] - 10https://gerrit.wikimedia.org/r/251035 (https://phabricator.wikimedia.org/T88343) (owner: 10Krinkle) [20:03:43] 10Continuous-Integration-Config, 10Analytics-EventLogging: Set up jsduck test job for EventLogging - https://phabricator.wikimedia.org/T88343#1783607 (10Krinkle) 5Open>3Resolved p:5Lowest>3Low a:3Paladox [20:03:47] 10Continuous-Integration-Config, 10Analytics-EventLogging: Set up jsduck test job for EventLogging - https://phabricator.wikimedia.org/T88343#1009758 (10Krinkle) Now live at . [20:22:46] PROBLEM - Puppet staleness on deployment-restbase01 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [43200.0] [20:23:20] 10Gitblit-Deprecate, 10Diffusion: Mirror unmerged changes in Diffusion from Gerrit - https://phabricator.wikimedia.org/T117625#1783670 (10demon) [20:23:21] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783671 (10demon) [20:25:43] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783680 (10demon) We're fetching them but they don't show up in Phab, hrm. [20:41:41] (03PS1) 10Thcipriani: Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) [20:42:57] (03CR) 10jenkins-bot: [V: 04-1] Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) (owner: 10Thcipriani) [20:43:19] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783728 (10Paladox) Would that be because links are shown for gitblit instead of phabricator in gerrit. I propose we show two links one for gitblit and one for phabricator. [20:45:12] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783742 (10demon) >>! In T89940#1783728, @Paladox wrote: > Would that be because links are shown for gitblit instead of phabricator in gerrit. > No, I was just searching for the raw sha1s in Pha... [20:46:09] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783746 (10Paladox) It may be possible if we create a plugin like we have for gitblit. [20:51:57] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783760 (10demon) We don't have a plugin for Gitblit. [21:01:44] 10Gitblit-Deprecate, 10Diffusion: Replicate open patchsets to diffusion - https://phabricator.wikimedia.org/T89940#1783787 (10Paladox) Oh well then how does gitblit replicate patches that haven't been merged yet. [21:06:05] (03PS2) 10Thcipriani: Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) [21:06:23] (03CR) 10Paladox: "Thanks." [integration/config] - 10https://gerrit.wikimedia.org/r/246773 (https://phabricator.wikimedia.org/T88343) (owner: 10Paladox) [21:20:48] Project browsertests-QuickSurveys-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce build #59: 04FAILURE in 4 min 46 sec: https://integration.wikimedia.org/ci/job/browsertests-QuickSurveys-en.m.wikipedia.beta.wmflabs.org-linux-chrome-sauce/59/ [21:28:15] 10Continuous-Integration-Config, 10Wikidata, 3Wikidata-Sprint-2015-11-03: [Task] Add Wikidata to Jenkins job mediawiki-extensions-hhvm - https://phabricator.wikimedia.org/T96264#1783853 (10JanZerebecki) This requires also adding it to mediawiki-extensions-qunit which is currently failing for Wikidata: https:... [21:33:56] (03PS1) 10JanZerebecki: add mwext-qunit to experimental of ext:Wikidata [integration/config] - 10https://gerrit.wikimedia.org/r/251126 [21:35:48] (03CR) 10JanZerebecki: [C: 032] add mwext-qunit to experimental of ext:Wikidata [integration/config] - 10https://gerrit.wikimedia.org/r/251126 (owner: 10JanZerebecki) [21:39:48] (03Merged) 10jenkins-bot: add mwext-qunit to experimental of ext:Wikidata [integration/config] - 10https://gerrit.wikimedia.org/r/251126 (owner: 10JanZerebecki) [21:41:10] !log reloading zuul for 8db8417..d49e21a [21:41:15] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [21:47:50] 10Continuous-Integration-Config, 10Wikidata, 3Wikidata-Sprint-2015-11-03: [Task] Add Wikidata to Jenkins job mediawiki-extensions-hhvm - https://phabricator.wikimedia.org/T96264#1783928 (10JanZerebecki) [21:48:09] (03PS1) 10JanZerebecki: add extension-qunit-generic to ext:Wikidata [integration/config] - 10https://gerrit.wikimedia.org/r/251134 (https://phabricator.wikimedia.org/T96264) [21:48:23] (03CR) 10Polybuildr: [C: 032] Simplify existing regular expressions (031 comment) [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250998 (owner: 10Thiemo Mättig (WMDE)) [21:49:01] (03Merged) 10jenkins-bot: Simplify existing regular expressions [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250998 (owner: 10Thiemo Mättig (WMDE)) [21:49:40] twentyafterfour: hey! To make sure this new extension rides the train - https://gerrit.wikimedia.org/r/#/c/251130/ - what do you need from me? [21:49:48] (cc greg-g ) [21:56:26] (03PS2) 10Legoktm: Remove dead code from SpaceBeforeSingleLineCommentSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250996 (owner: 10Thiemo Mättig (WMDE)) [21:56:31] (03CR) 10Legoktm: [C: 032] Remove dead code from SpaceBeforeSingleLineCommentSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250996 (owner: 10Thiemo Mättig (WMDE)) [21:57:09] (03Merged) 10jenkins-bot: Remove dead code from SpaceBeforeSingleLineCommentSniff [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250996 (owner: 10Thiemo Mättig (WMDE)) [21:58:15] (03CR) 10Polybuildr: "If we tried to fix it instead of removing it, would changing the * to a + suffice?" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250996 (owner: 10Thiemo Mättig (WMDE)) [21:59:28] (03PS1) 10JanZerebecki: add Wikidata to extension-gate [integration/config] - 10https://gerrit.wikimedia.org/r/251138 (https://phabricator.wikimedia.org/T96264) [22:00:22] (03CR) 10JanZerebecki: [C: 04-1] "It's still failing mediawiki-extensions-qunit." [integration/config] - 10https://gerrit.wikimedia.org/r/251138 (https://phabricator.wikimedia.org/T96264) (owner: 10JanZerebecki) [22:01:19] jzerebecki: Won't https://gerrit.wikimedia.org/r/#/c/251138/ massively slow down merged in core? :-( [22:01:30] (03CR) 10Polybuildr: "I was hoping you'd already implemented this. :P" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/247565 (owner: 10Hashar) [22:02:28] James_F: probably less than 4min [22:02:48] jzerebecki: An additional 4 minutes on top of an existing 8 minute chain? [22:02:55] (At the best of times.) [22:03:09] it's not going to slow down phpunit-zend [22:03:14] which is the current bottleneck [22:03:22] Oh, we'd just run in HHVM? [22:03:34] no, it'll be added to mw-extensions-zend as well [22:03:41] but the mw-phpunit-zend job is the slow one [22:03:45] * jzerebecki -> rfc office hour [22:03:50] Huh. OK. [22:04:10] matanya: hi [22:04:36] jdlrobson: it needs to be added to the list in tools/release:/make-wmf-branch/config.json [22:07:53] (03PS1) 1020after4: add QuickSurveys extension to make-wmf-branch conf [tools/release] - 10https://gerrit.wikimedia.org/r/251140 [22:09:28] (03PS3) 10Legoktm: Avoid in_array for performance reasons [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 (owner: 10Thiemo Mättig (WMDE)) [22:10:51] (03CR) 10Legoktm: "PS3: Rebase" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 (owner: 10Thiemo Mättig (WMDE)) [22:18:45] hey twentyafterfour thanks lemme take a look [22:19:43] We're a little hosed, this repo is not actually gate-and-submitting: https://gerrit.wikimedia.org/r/#/c/250746/1 [22:19:44] jdlrobson: super simple. That should cause the new extension to get added to next week's branch. [22:19:56] I'd prefer to not force submit cos I hear that breaks zuul. [22:20:02] (03PS1) 10Jdlrobson: Add QuickSurveys branch [tools/release] - 10https://gerrit.wikimedia.org/r/251144 (https://phabricator.wikimedia.org/T110661) [22:20:10] twentyafterfour: awesome. [22:24:50] (03CR) 10Legoktm: [C: 032] "Thanks!" [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 (owner: 10Thiemo Mättig (WMDE)) [22:28:40] (03CR) 1020after4: [C: 032] update deploy-promote for dblist location change [tools/release] - 10https://gerrit.wikimedia.org/r/246093 (owner: 1020after4) [22:38:52] (03Merged) 10jenkins-bot: Avoid in_array for performance reasons [tools/codesniffer] - 10https://gerrit.wikimedia.org/r/250956 (owner: 10Thiemo Mättig (WMDE)) [22:38:54] (03Merged) 10jenkins-bot: update deploy-promote for dblist location change [tools/release] - 10https://gerrit.wikimedia.org/r/246093 (owner: 1020after4) [22:40:12] man I dislike how gerrit mangles urls.... [22:41:12] I dislike how gerrit does everything [22:41:23] I can't even gerrit. [22:46:58] :) :) [22:49:04] 6Release-Engineering-Team, 15User-greg: [transparency placeholder] Write up thoughts on "CTO group" "self" "management" and "prioritization" - https://phabricator.wikimedia.org/T116327#1784061 (10greg) 5Open>3Resolved [22:50:15] "I think, I'm awesome" [22:55:44] gerrit: not even once [22:57:58] (03PS3) 1020after4: Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) (owner: 10Thcipriani) [22:58:41] (03CR) 1020after4: [C: 032] Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) (owner: 10Thcipriani) [23:00:02] twentyafterfour: ummm, I think that code is probably broken. I should have put a WIP on it. [23:00:45] thcipriani: hmm [23:00:51] :) [23:01:09] I'll verify and fix any brokenness [23:01:30] do I need to run the job builder or did I just inadvertently run it by merging? [23:01:42] marxarelli: had some question about whether it was valid to pass UUIDs between jobs like I was. I hadn't tested yet [23:02:28] twentyafterfour: I think you have to manually update jenkins-jobs from jjb. [23:02:47] the jobs won't be updated until you run `jenkins-jobs update` [23:02:58] ok cool [23:03:27] usually the protocol for jjb patches is to wait for a +1, update the job(s), test, +2 if everything is kosher [23:03:28] don' [23:03:42] RECOVERY - Host deployment-parsoidcache02 is UP: PING OK - Packet loss = 0%, RTA = 1.16 ms [23:03:48] i wouldn't update all the jobs affected by that patch, though [23:03:54] there are many [23:04:20] and never run `jenkins-jobs update` without giving it specific jobs to udpate [23:04:23] update [23:04:35] * marxarelli looks for the docs [23:05:10] https://www.mediawiki.org/wiki/Continuous_integration/Jenkins_job_builder [23:06:01] should be able to update the mw-tools-scap-tox-doc-publish job without too much worry. [23:06:06] it's kinda wonky as is. [23:15:11] (03Merged) 10jenkins-bot: Publish scap docs without $ZUUL_UUID [integration/config] - 10https://gerrit.wikimedia.org/r/251118 (https://phabricator.wikimedia.org/T117770) (owner: 10Thcipriani) [23:18:44] !log Running `jenkins-jobs update config/ 'mw-tools-scap-tox-doc-publish'` to test Ib4753ad493115682d902cf15136199fd2083b8e5 [23:18:49] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [23:20:01] marxarelli: I don't think it'll work without updating all the downstream jobs. [23:20:26] which is what all the other *-tox-doc-publish's rely on... [23:22:17] thcipriani: ah, you're right. we should revert then [23:22:35] yeah, I'll post up a revert. [23:24:55] (03PS1) 10Thcipriani: Revert "Publish scap docs without $ZUUL_UUID" [integration/config] - 10https://gerrit.wikimedia.org/r/251160 [23:25:32] ^ marxarelli [23:27:29] (03CR) 10Dduvall: [C: 032] Revert "Publish scap docs without $ZUUL_UUID" [integration/config] - 10https://gerrit.wikimedia.org/r/251160 (owner: 10Thcipriani) [23:28:15] that's lame :-/ [23:28:38] thcipriani, twentyafterfour: one way i can think of to test this change without effecting too many jobs would be to create temporary duplicates of doc-publish and publish-on-gallium [23:29:02] marxarelli: that sounds like a good plan [23:29:15] +1 [23:29:29] (03Merged) 10jenkins-bot: Revert "Publish scap docs without $ZUUL_UUID" [integration/config] - 10https://gerrit.wikimedia.org/r/251160 (owner: 10Thcipriani) [23:34:01] twentyafterfour, thcipriani: rad. i'm keen to pair on it if you want