Page MenuHomePhabricator

Reduce the number of six default URIs in Diffusion
Closed, DeclinedPublic

Description

When creating a new Diffusion repository on phabricator.wikimedia.org, a total of 6 different URIs are automatically added. See for example the latest creation: https://phabricator.wikimedia.org/diffusion/EACIL/manage/uris/ This is wholy unnecesary.

As stated at T244903 this seriously consumes our time. For that I'd like to request, if possible, that our Phabricator configuration be changed so that redundant URIs ain't added. We could start preventing non-https URIs as well as non-static URIs (those bearing the short name of the repo or the repo callsign, as those can be changed at any time, breaking the URI).

In short: new Diffusion repositories should only have 1 URI by default: a https:// one
(since ssh://vcs@git-ssh.wikimedia.org is dead since T296022: Deprecate git-ssh service on phabricator.wikimedia.org; see also T347408: Do not expose dead git-ssh.wikimedia.org service as repo Clone URLs (defined in diffusion.ssh-host setting) about hiding it by patching the code).
That's enough for our workflow and potential use of Diffusion.

Screenshot (the URIs the default ones; the I/O and visibility settings for each URI are not default):

Screenshot from 2023-10-25 11-23-19.png (718×1 px, 100 KB)

Event Timeline

Aklapper triaged this task as Low priority.May 30 2021, 10:05 AM
Aklapper renamed this task from Reduce the number of default URIs in Diffusion to Reduce the number of six default URIs in Diffusion.Oct 24 2023, 10:52 PM
Aklapper edited projects, added Phabricator (Upstream); removed Phabricator.

Just to understand a bit better, could unnecessary URIs just be hidden by default? or, this is already a thing (I see that in the mentioned example most URIs are already hidden), and it's not enough (?) Thanks :)

(Please make F40005688 visible - and sorry if Phabricator is so messed up with these things ihih - https://we.phorge.it/T15106)

I'm going to decline this as these six URIs are only listed in a view not accessible by average users thanks to our custom downstream patch for T361997: Expose only canonical Clone URI in Diffusion when repo hosted externally and Differential disabled, and as the time-consuming aspect of having to edit all of them as a member of Diffusion-Repository-Administrators in order to allow observing an external repository URI should be gone once T224753: Set new Diffusion repos URIs to No I/O by default has been merged and deployed in our instance.