1133603 - See if treeherder-service:remotes/origin/pr/28 can be removed so the clone using the Github client isn't 400MB

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

In order to decide whether we should keep treeherder-service (ie rename it) rather than either renaming treeherder-ui or creating a branch new github repo for bug 1056877 I wanted to see if we could fix this issue first. 

After many attempts to do this myself, it appears as though it needs Github intervention.

I sent the following email to Github support:

{
A while ago we accidentally committed some large log files to our repo, as part of this pull request:
https://github.com/mozilla/treeherder-service/pull/28/
(Added in 6e821b81ae53b3d19931627d446cccf5f376a555, removed in 16bf48d4f2af0eea0896433e22da074d3acb8a1f)

History was rewritten to remove/squash those add & remove commits, and the old branch for the PR (job-test-refactor) deleted on Github.

This worked fine if doing a standard clone of the repo - however on the Github for Windows/Mac client (or if you manually add this line so you can checkout PRs), this line is present in the config:
  fetch = +refs/pull/*/head:refs/remotes/origin/pr/*
...which causes the (presumably Github behind the scenes) branches for PRs to be checked out as well.

This means the repo takes up 400MB on disk for people using Github for Windows/Mac (vs the ~70MB it should) and means their initial clone takes a _lot_ longer. I know we can tell them to remove the line from their git config, however it would be great to not have to have this manual step - especially given it gets rewritten whenever the Github for Windows copy of msysgit gets updated, which you often don't realise has happened until you've already re-fetched the (in our case) several hundred MB of remotes/origin/pr/*.

I've tried force pushing our current master (which has the commits rewritten) to the old branch name 'job-test-refactor' in the hope that the 'remotes/origin/pr/28' branch will re-sync and drop the older commits - however this hasn't helped.

Would it be possible to delete remotes/origin/pr/28 on the Github side (or at least force push master to it, so the PR appears as a no-op or something)?
}

Comment 1

[Ed Morley [:emorley]]

> Hi Ed,
> 
> We can remove the ref, but we'll also need to delete pull request 28 from the repository's web interface.
> 
> https://github.com/mozilla/treeherder-service/pull/28/
> 
> If that's OK, let us know and we'll proceed.
> 
> Cheers,
> John

> Hi John
> 
> That's absolutely fine - thank you! :-)
> 
> Many thanks,
> 
> Ed

> Hi Ed,
> 
> Thanks. We have deleted the pull request and its refs.
> 
> A test clone using the GitHub for Mac client was about 70MB.
> 
> Cheers,
> John

-> confirmed locally :-)