Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get rid of tc-vcs completely #43

Merged
merged 2 commits into from
Feb 7, 2018
Merged

Get rid of tc-vcs completely #43

merged 2 commits into from
Feb 7, 2018

Conversation

djmitche
Copy link
Contributor

@djmitche djmitche commented Feb 7, 2018

We no longer use tc-vcs for Gecko, but there are a few stragglers still using it. We have hooks set up to re-generate it periodically, and sometimes those fail. Let's get rid of it.

Bug 1364032 was opened on the same topic, but nobody volunteered to do the work :)

@djmitche djmitche added the ktlo label Apr 5, 2017
@djmitche djmitche added the sunset label Jul 3, 2017
@djmitche
Copy link
Contributor Author

djmitche commented Jul 5, 2017

In discussions at SF there was no disagreement here.

Alternatives were to patch esr52 to not use it anymore and kill it, or just remove caches for everything but esr52 (to prevent accidental use) and wait for the ESR to die in a year.

@djmitche
Copy link
Contributor Author

@gregarndt I think we finally settled on the last bit (remove all but esr52 caches). Did that get done?

@gregarndt
Copy link

gregarndt commented Sep 20, 2017

Hasn't been done yet. I have to get time to audit what might still be using it somehow. It's one of the remaining pieces for deprecating the scheduler too.

I think if esr52 is the only thing left using it, we can cache that repo and keep the artifact from expiring for a year and turn tc-vcs caching off completely.

@djmitche
Copy link
Contributor Author

https://bugzilla.mozilla.org/show_bug.cgi?id=1409260 is removing some uses of tc-vcs that remain in trunk.

@djmitche
Copy link
Contributor Author

djmitche commented Feb 5, 2018

@djmitche djmitche merged commit c7132e7 into master Feb 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants