[00:38:53] 10Phabricator, 10Phabricator-Upstream, 07Upstream: Add fields to support bug specific information - https://phabricator.wikimedia.org/T92708#2321641 (10mmodell) This is far more reasonable now that phabricator supports custom forms. We can fairly easily make custom task types with their own custom fields, ho... [00:58:02] 10Phabricator, 10Phabricator-Upstream, 07Upstream: Add support for task types - https://phabricator.wikimedia.org/T93499#2321726 (10mmodell) task types is fundamentally achievable now using a combination of custom forms and custom fields. [04:09:19] 10Diffusion, 05Gitblit-Deprecate, 10Phabricator: Update Module:Callsigns in mediawiki.org - https://phabricator.wikimedia.org/T135336#2321885 (10mmodell) this should no longer be necessary, just link from mediawiki.org to: phabricator.wikimedia.org/r/project/**${project_path}** [06:28:59] 10Diffusion, 05Gitblit-Deprecate, 10Phabricator: Update Module:Callsigns in mediawiki.org - https://phabricator.wikimedia.org/T135336#2321952 (10Paladox) Yep I have. Thanks for merging the other patch that allows us to link to /r/history/ [06:30:56] Hi, is there a way to check whether i.e. user qgil@wikimedia has +2 in any Gerrit repositories? I check the docs, could not find an answer. [06:32:19] 10Diffusion, 05Gitblit-Deprecate, 10Phabricator: Update Module:Callsigns in mediawiki.org - https://phabricator.wikimedia.org/T135336#2321953 (10TerraCodes) Shouldn't this be marked as invalided per @mmodell's comment? [07:18:33] 10Diffusion, 05Gitblit-Deprecate, 10Phabricator: Update Module:Callsigns in mediawiki.org - https://phabricator.wikimedia.org/T135336#2322004 (10Paladox) @terracodes maybe. @mmodell can you close this task if you think it is invalid please. [07:19:40] 10Diffusion, 05Gitblit-Deprecate: Update mediawiki.org templates to link to Diffusion, not gitblit - https://phabricator.wikimedia.org/T108864#2322006 (10mmodell) [07:19:44] 10Diffusion, 05Gitblit-Deprecate, 10Phabricator: Update Module:Callsigns in mediawiki.org - https://phabricator.wikimedia.org/T135336#2322005 (10mmodell) 05Open>03Invalid [07:20:14] qgil: I'm not sure how to tell [07:22:04] 10Diffusion, 10Phabricator: Diffusion replacement for tarfile download from git.wikimedia.org - https://phabricator.wikimedia.org/T111887#2322008 (10Paladox) What do people think of linking to GitHub for our downloads for now please. I've used GitHub to do https://phabricator.wikimedia.org/D234 which would mak... [07:29:50] it looks like it would be fairly complex to work out what rights any given user has [07:34:35] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2322012 (10mmodell) >>! In T135327#2318137, @Danny_B wrote:... [07:35:01] 10Differential, 05Gerrit-Migration, 10Phabricator, 07WorkType-NewFunctionality: Create conduit method to query the feed and return records with relevant details populated instead of just a bunch of phids - https://phabricator.wikimedia.org/T123417#1929298 (10mmodell) [07:35:02] 10Wikibugs, 07Tracking: Get rid of screen scraping in Wikibugs (tracking) - https://phabricator.wikimedia.org/T1175#2322014 (10mmodell) [07:35:09] 10Wikibugs, 07Tracking: Get rid of screen scraping in Wikibugs (tracking) - https://phabricator.wikimedia.org/T1175#20330 (10mmodell) T123417 will take care of this. [08:00:05] Krenair: I missed that one, thanks a lot for the heads-up [08:13:38] 10Differential, 10Phabricator: Doing arc patch dosent work properly - https://phabricator.wikimedia.org/T136069#2322044 (10Paladox) [08:14:16] 10Differential, 10Phabricator: Doing arc patch dosent work properly - https://phabricator.wikimedia.org/T136069#2322059 (10Paladox) [08:15:51] 10Differential, 10Phabricator: Doing arc patch dosent work properly, it dosent use the correct branch - https://phabricator.wikimedia.org/T136069#2322063 (10Paladox) [08:33:46] 10Differential, 10Phabricator: "arc patch" should be applied against the branch it is based on, instead of master branch - https://phabricator.wikimedia.org/T136069#2322090 (10Aklapper) [08:35:05] 10Differential, 10Phabricator: "arc patch" should be applied against the branch it is based on, instead of master branch - https://phabricator.wikimedia.org/T136069#2322044 (10Aklapper) Could you please edit "Actual results" by splitting it into several sentences with punctuation, so it is easier to read and u... [08:58:51] 10Phabricator, 13Patch-For-Review: Pastes don't allow lua syntax highlighting - https://phabricator.wikimedia.org/T100900#2322126 (10Aklapper) [08:59:58] 10Phabricator, 13Patch-For-Review: Pastes don't allow lua syntax highlighting - https://phabricator.wikimedia.org/T100900#1323164 (10Aklapper) (Assuming as per previous comment that Lua is not wanted as a default choice in upstream, I've prepared a downstream configuration patch, for the lulz.) [09:01:54] twentyafterfour, interesting. Would that improve with Phabricator/Owners? [09:02:49] Or maybe the problem is with "+2" while "project member" or "repository maintainer" is a different thing? [09:04:55] OK, this seems to tell something: https://phabricator.wikimedia.org/owners/query/XDvC251BUzZO/#R [09:05:54] Meh. https://gerrit.wikimedia.org/r/#/admin/groups/ is rights per group, not per person [09:06:00] e.g. https://gerrit.wikimedia.org/r/#/admin/groups/11,members is +2 in MW Core [09:07:51] nope, https://gerrit.wikimedia.org/r/Documentation/cmd-index.html doesn't list any "show groups for user X" command either [09:08:26] Are Differential owners and Gerrit +2s the same? [09:09:21] I also wonder whether there is an equivalent in Owners for Gerrit admin groups and their inheritances. [09:09:53] Anyway, not that important, just curious about how to know how involved is user X in the maintenance of repos. [09:10:57] "the same" as in "should be the same list of people" or "should mean the same" or "are the same list of people"? Definitely not the latter as I don't think anyone has tried transferring/copying such rights [09:11:08] Hm, another interested element seems to be that Owners correspond to Packages, and Packages are not fully equivalent to repositories. [09:11:52] andre__, I mean, when the Gerrit migration is completed, can we expect that whoever had +2 in repo X, will be Owner of the same repo in Differential? [09:12:12] "should mean the same"? [09:12:52] "Specify the files and directories which comprise this package." [09:13:00] Well, I hope someone(TM) will clean up and kick out inactive accounts that have +2 [09:13:08] Isn't this a big improvement over Gerrit? [09:13:33] and I don't think that people with +2 always also consider themselves maintainers / owners. It's all a social construct. [09:13:42] (I'm soooo looking forward to burying that whole stupid +1 and +2 idea/concept of Gerrit) [09:13:51] It seems that it is easier to watch i.e. only these two directories of MW core, or only readme files of this collection of extensions? [09:15:30] OK, so the desired scenario is that one day we have Differential + an updates list of Owners, where each owner can be watching one or more packages, meaning either one or more repository, or specific directories / files of specific repositories. [09:16:43] And using https://phabricator.wikimedia.org/owners/query/advanced/ anyone can have an idea of wither which owners are related with a repository, which packages own user X, or even more complex queries using keywords and paths. [09:17:20] If all this is true, this looks like a more sane picture, especially for new developers without a network for contacts, aiming to find the people that can review their patches. [09:17:34] I think so [09:17:36] (and following specific paths within a project is one thing that Gerrit is missing, IIRC - cf https://www.mediawiki.org/wiki/Gerrit/watched_projects ) [09:17:43] Also to answer the question of patch triagers: who could review this patch? [09:50:03] 10Differential, 10Phabricator: "arc patch" should be applied against the branch it is based on, instead of master branch - https://phabricator.wikimedia.org/T136069#2322168 (10Paladox) [09:51:17] 10Differential, 10Phabricator: "arc patch" should be applied against the branch it is based on, instead of master branch - https://phabricator.wikimedia.org/T136069#2322044 (10Paladox) And you should be able to apply it against the current branch your on so if to git checkout wmf/stable it should be applied ag... [09:52:35] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2322190 (10Qgil) [09:53:14] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2308100 (10Qgil) We have agreed on a rough plan to {T136071}. [10:04:33] andre__, have you seen Evan's last comments about https://secure.phabricator.com/T4768 (in our private chat)? I think this is the last point that needs to be clarified before agreeing on the statement of work for our contract with Phacility Inc. [10:05:14] Basically, he needs more information about what needs fixing, from his POV the problem seems to be resolved, there are no new comments in the task, etc. [10:05:42] Commenting in https://secure.phabricator.com/T4768 would be useful, and enough, since there is nothing private about the mid-air non-collision problems. [10:05:53] checking now, thanks [10:22:28] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2322265 (10Danny_B) >>! In T135327#2322012, @mmodell wrote:... [10:24:11] 10Phabricator, 10VisualEditor: Fix edit permissions of VisualEditor Phabricator projects - https://phabricator.wikimedia.org/T119622#2322267 (10Aklapper) Any reply to my question? :) [10:40:15] 10Phabricator, 10VisualEditor: Fix edit permissions of VisualEditor Phabricator projects - https://phabricator.wikimedia.org/T119622#2322342 (10Malyacko) p:05High>03Normal [10:40:22] 10Phabricator, 10VisualEditor: Fix edit permissions of VisualEditor Phabricator projects - https://phabricator.wikimedia.org/T119622#2322344 (10Aklapper) p:05Normal>03Low [10:52:43] {done} [11:41:13] Thanks! Very good and complete description. [11:52:34] 10Phabricator, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#2322527 (10Aklapper) @Danny_B: How is this related to #documentation? [12:03:38] 10Phabricator: Add fields to support bug specific information - https://phabricator.wikimedia.org/T92708#2322562 (10Aklapper) [12:04:53] 10Phabricator: Add support for task types - https://phabricator.wikimedia.org/T93499#2322563 (10Aklapper) [12:55:06] 10Phabricator, 10MediaWiki-Authentication-and-authorization: [Bug] [Android Chrome Dev] Logging into Phabricator with MediaWiki authorization causes infinite reload loop - https://phabricator.wikimedia.org/T136086#2322692 (10Niedzielski) [13:57:12] 10Phabricator, 10MediaWiki-Authentication-and-authorization: [Bug] [Android Chrome Dev] Logging into Phabricator with MediaWiki authorization causes infinite reload loop - https://phabricator.wikimedia.org/T136086#2322872 (10Aklapper) The steps above miss the exact outcome, but if I get that video right, Phabr... [13:59:24] 10Phabricator, 10MediaWiki-Authentication-and-authorization: [Bug] [Android Chrome Dev] Logging into Phabricator with MediaWiki authorization causes infinite reload loop - https://phabricator.wikimedia.org/T136086#2322873 (10Niedzielski) @Aklapper, that's right. The page enters a reload loop. I can stop the lo... [14:49:07] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2322957 (10Nemo_bis) [14:53:31] James_F: Hi do you mind if we use hashar workaround for migrating to npm-run-doc-node-4.3 please [14:53:39] Repos described at https://phabricator.wikimedia.org/T128091#2322951 [14:55:45] James_F: This workaround https://gerrit.wikimedia.org/r/#/c/280906/1/package.json [15:14:52] 10Phabricator, 10Phabricator-Upstream, 06Community-Liaisons (Apr-Jun-2016), 06Developer-Relations (Apr-Jun-2016), and 2 others: Decide Phabricator improvements to be funded by WMF Technical Collaboration - https://phabricator.wikimedia.org/T135327#2323102 (10Aklapper) [15:16:24] 10Phabricator, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#939268 (10Danny_B) @Aklapper Title mentions "Create a //plan//" - and plans should be documented so they can be followed... but DWYW... [17:45:48] o/ andre__. I just saw the progress on https://phabricator.wikimedia.org/T134651 [17:46:09] I see the last step is signoff from a c-level. Can you tell me what exactly I should be sending to a c-level for signature? [17:49:11] Hmm... Need to run. Any insights you can add to the task would be great.