Page MenuHomePhabricator

Mirroring mediawiki/core to GitHub from diffusion does not work
Closed, ResolvedPublic

Description

Steps to reproduce

The repo is enabled in diffusion and gerrit.

But this https://github.com/wikimedia/mediawiki/commit/d27a597efa5a926b662df48b058fbf31881b5da7

Should be working. It's an open change but phabricator pushes everything from refs/*

But with metrolook which is enabled this works https://github.com/wikimedia/mediawiki-skins-Metrolook/commit/68fbfced9fbaa26a9f38916251dc271ec860ec47

Which is an open change. So I'm wondering why mediawiki core is not working from diffusion mirroring. We tryed this a few months ago and it didn't work but with metrolook working it is strange mediawiki core isent.

Event Timeline

Paladox added a subscriber: demon.

@demon can I add you to task please.

In just wondering why metrolook works and mediawiki core dosent.

Could it be a bug in phabricator that if it is a big repo it carnt mirror it with tons of changes.

@Danny_B nope because you set the mirror link in diffusion for each repo. Any repo can have a different name to the mirror.

@Aklapper should we keep the gerrit-migration project here due to the fact mirroring for this repo is not working in diffusion.

Workaround is to manually add refs/changes to the mirror.

The diffusion will start mirroring because it would not need to mirror all of it since it would be there.

Ive tested with my test mirror and it works.

So for any big repos with a ton of changes in refs/changes/ would need to follow this workaround.

Workaround was provided by @demon. Its GitHub that is broken.

I guess we could look for a way to choose the refs we want to mirror on a per repo basis but this would be a request for upstream.

demon triaged this task as Medium priority.Sep 1 2016, 10:23 PM

This should work now (errr, in about 20 minutes or so when my bash script finishes). Basically iterated over all the refs/changes/{00..99}/* and pushed them one by one.

Paladox claimed this task.

Upstream fixed this issue I think a week ago so it can push all refs at once instead of in batches.

As I've said before, there's no need to push them?