969461 - disable & delete mirror of gaia-ui-tests

Status: RESOLVED FIXED

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

Description

[hwine]

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

[Stephen Donner [:stephend] Not actively reading bugmail]

WFM!

Comment 2

[hwine]

Attachment: gaia-ui-tests.patch

Comment 3

[Pete Moore [:pmoore][:pete]]

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

Comment 4

[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.

Comment 5

[Pete Moore [:pmoore][:pete]]

Comment on attachment 8376031 [details] [diff] [review]
gaia-ui-tests.patch

Review of attachment 8376031 [details] [diff] [review]:
-----------------------------------------------------------------

Nice work Hal, all looks good. =)

Comment 6

[hwine]

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

Comment 7

[hwine]

Attachment: gaia-ui-tests.diff

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.

Comment 8

[Pete Moore [:pmoore][:pete]]

Comment on attachment 8377999 [details] [diff] [review]
gaia-ui-tests.diff

Review of attachment 8377999 [details] [diff] [review]:
-----------------------------------------------------------------

Nice work Hal, lgtm

Comment 9

[hwine]

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