Page MenuHomePhabricator

Phabricator upgrade broke Vector CI - cannot access RAW text
Closed, ResolvedPublic

Description

Vector relies on raw assets to build documentation as part of one of its CI steps. Previously it was using github but migrated to Phabricator after being told that we should dog food our own tools.

This historically has come from a URI in Phabricator such as "https://phabricator.wikimedia.org/source/mediawiki/browse/master/resources/src/mediawiki.less/mediawiki.mixins.less?view=raw"

This no longer works for some reason, redirecting to
https://phabricator.wikimedia.org/file/view/31771082/source/

This means all Vector patches are blocked by CI for now.

Short term fix: disable the job
Longer term fix: Please provide a stable URI for accessing raw assets from repos.

(I'll be back to this on Monday!)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 22 2020, 12:21 AM
Jdlrobson triaged this task as Unbreak Now! priority.Apr 22 2020, 12:22 AM

I'm guessing blocking patches to our main skin might be UBN worthy and block any SWAT deploys so I wanted to make sure those impacted were aware of it :) - feel free to drop to high once aware.

Restricted Application added a subscriber: Liuxinyu970226. · View Herald TranscriptApr 22 2020, 12:22 AM
Jdlrobson renamed this task from Phabricator upgrade broke Vector CI to Phabricator upgrade broke Vector CI - cannot access RAW text.Apr 22 2020, 12:23 AM

Change 591580 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/skins/Vector@master] Load files from gerrit not phab

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

Change 591580 merged by jenkins-bot:
[mediawiki/skins/Vector@master] Load files from gerrit not phab

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

Jdlrobson closed this task as Resolved.Apr 22 2020, 5:20 AM
Jdlrobson claimed this task.

Thanks. Hadn't thought to use base64 on the raw assets. Bit weird but works!

Reedy claimed this task.Apr 22 2020, 3:39 PM
Edd0408 set Due Date to Apr 23 2020, 12:00 AM.Apr 23 2020, 8:35 AM

@Edd0408 These edits do not seem to be useful or correct, can I suggest that you desist, or seek guidance in the use of phabricator