Closed
Bug 969461
Opened 10 years ago
Closed 10 years ago
disable & delete mirror of gaia-ui-tests
Categories
(Developer Services :: Mercurial: hg.mozilla.org, defect)
Developer Services
Mercurial: hg.mozilla.org
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: hwine, Assigned: hwine)
References
Details
Attachments
(2 files)
1.34 KB,
patch
|
pmoore
:
review+
hwine
:
checked-in+
|
Details | Diff | Splinter Review |
990 bytes,
patch
|
pmoore
:
review+
hwine
:
checked-in+
|
Details | Diff | Splinter Review |
originally created in bug 832535, this process changed in Nov 2013. Time to dismantle the supporting infrastructure. Since the RoR was on github, the mirrors on both git.mozilla.org & hg.mozilla.org can be deleted. email from jgriffin: -------- Original Message -------- Subject: Re: status of gaia-ui-tests? Date: Fri, 07 Feb 2014 09:44:46 -0800 From: Jonathan Griffin To: Hal Wine, Stephen Donner Yes, kill them please. Gaia-ui-tests has been merged into the main gaia repo.
Comment 1•10 years ago
|
||
WFM!
Assignee | ||
Comment 2•10 years ago
|
||
Comment 3•10 years ago
|
||
Hi Hal, The patch looks good, but shouldn't we also delete the repo-sync-configs/b2g-gaia-ui-tests subdirectory too in this patch? Also, do we need to create bugs for I.T. to delete the mirror repos (blocked by this bug - since we should disable sync before IT remove the repos)? http://git.mozilla.org/?p=b2g/gaia-ui-tests.git http://hg.mozilla.org/integration/gaia-ui-tests/ Thanks Hal, Pete
Flags: needinfo?(hwine)
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Pete Moore [:pete][:pmoore] from comment #3) > The patch looks good, but shouldn't we also delete the > repo-sync-configs/b2g-gaia-ui-tests subdirectory too in this patch? No - that will be a separate patch due to timing. The job could be active at the time this patch is deployed, and would already have read gaia-ui-tests into the list of repos to process. If the configs are gone, errors would happen. This patch _only_ stops the conversion process. Once that is quiet, the removal patch and manual work can be done. > Also, do we need to create bugs for I.T. to delete the mirror repos (blocked > by this bug - since we should disable sync before IT remove the repos)? Yes, that will be a separate bug I'll open, after the configs have landed, and at least one cycle has occurred. There is no rush on this. > > http://git.mozilla.org/?p=b2g/gaia-ui-tests.git > http://hg.mozilla.org/integration/gaia-ui-tests/ > > Thanks Hal, Thanks for the lesson on why aki prepares all needed patches up front, even if they deploy one at a time. It's a better story for reviewers to follow.
Flags: needinfo?(hwine)
Comment 5•10 years ago
|
||
Comment on attachment 8376031 [details] [diff] [review] gaia-ui-tests.patch Review of attachment 8376031 [details] [diff] [review]: ----------------------------------------------------------------- Nice work Hal, all looks good. =)
Attachment #8376031 -
Flags: review?(pmoore) → review+
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8376031 [details] [diff] [review] gaia-ui-tests.patch http://hg.mozilla.org/users/hwine_mozilla.com/repo-sync-configs/rev/5fad72ef410e deployed to all production boxes
Attachment #8376031 -
Flags: checked-in+
Assignee | ||
Comment 7•10 years ago
|
||
Remove the (now unreferenced) configuration files. Testing: no reference in logs to gaia-ui-tests after prior patch deployed. After this patch is deployed, the repositories on the conversion box will be removed.
Attachment #8377999 -
Flags: review?(pmoore)
Comment 8•10 years ago
|
||
Comment on attachment 8377999 [details] [diff] [review] gaia-ui-tests.diff Review of attachment 8377999 [details] [diff] [review]: ----------------------------------------------------------------- Nice work Hal, lgtm
Attachment #8377999 -
Flags: review?(pmoore) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8377999 [details] [diff] [review] gaia-ui-tests.diff http://hg.mozilla.org/users/hwine_mozilla.com/repo-sync-configs/rev/534dc48d5543 deployed to all production boxes working directory deleted on gd3
Attachment #8377999 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Release Engineering → Developer Services
You need to log in
before you can comment on or make changes to this bug.
Description
•