Closed Bug 969461 Opened 8 years ago Closed 8 years ago

disable & delete mirror of gaia-ui-tests

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hwine, Assigned: hwine)

References

Details

Attachments

(2 files)

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.
Assignee: nobody → hwine
Status: NEW → ASSIGNED
Attachment #8376031 - Flags: review?(pmoore)
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)
(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 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+
Depends on: 974222
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 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+
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+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: Release Engineering → Developer Services
You need to log in before you can comment on or make changes to this bug.