Page MenuHomePhabricator

Striker: Always expose observed external URI when creating Diffusion repositories
Open, LowPublic

Description

Per T405596: Disable IO for diffusion repositories, we want folks not to clone from Diffusion but ideally from the canonical source. Thus expose the URI.

(FYI we replace any gerrit-replica URIs set with gerrit URIs but only for display in the UI since T406553)

Event Timeline

Aklapper triaged this task as Low priority.

Change #1196901 had a related patch set uploaded (by Aklapper; author: Aklapper):

[labs/striker@master] Always expose observed external URI when creating Diffusion repos

https://gerrit.wikimedia.org/r/1196901

Change #1196901 merged by jenkins-bot:

[labs/striker@master] Always expose observed external URI when creating Diffusion repos

https://gerrit.wikimedia.org/r/1196901

This still seems not to be deployed

This still seems not to be deployed

Deploying a new version of Striker is done by adjusting the profile::wmcs::striker::docker::instances:striker:version and profile::wmcs::striker::docker::instances:striker-toolsbeta:version hiera settings to choose a newer container version in operations/puppet.git:/hieradata/role/eqiad/wmcs/openstack/eqiad1/cloudweb.yaml. It looks like we are currently running a build from July 2025.

Change #1213546 had a related patch set uploaded (by BryanDavis; author: Bryan Davis):

[operations/puppet@production] striker: Bump container version

https://gerrit.wikimedia.org/r/1213546

@bd808 Sorry, I did not want to sound pushy, was more of a note to myself. Plus this likely the patch for T409341 should also get in for some future deploy :-/

Plus this likely the patch for T409341 should also get in for some future deploy :-/

The patch on that task states that it has not been tested before being submitted. It is unlikely I will have time to review any unreviewed patches in that repository any time soon.

Yeah, I'm not sure how / where I'm supposed to test, sorry.