1083208 - postrelease should fail if we didn't push to mirrors

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, defect, P2)

Description

[Massimo Gervasini [:massimo]]

postrelease should fail if the push to mirror did not happen/succeeded.
 
see: https://bugzilla.mozilla.org/show_bug.cgi?id=1083127#c2

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

It would be good to catch this. Might just need a simple test on .../<stage-product>/releases/<version>/ existing prior to modifying the symlink.

Comment 2

[Massimo Gervasini [:massimo]]

Attachment: [tools] Bug 1083208 - postrelease should fail if we didn't push to mirrors.patch

Hi Nick,

here's a fix (tested on dev-master) but I am not sure the push_to_mirrors_dir value is the right one for production.

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 8510542 [details] [diff] [review]
[tools] Bug 1083208 - postrelease should fail if we didn't push to mirrors.patch

>diff --git a/scripts/release/stage-tasks.py b/scripts/release/stage-tasks.py
>index 53b4a72..2d5cb19 100755
>--- a/scripts/release/stage-tasks.py
>+++ b/scripts/release/stage-tasks.py
>@@ -193,6 +193,19 @@ def deleteIndexFiles(cleanup_dir, stageServer, stageUsername,
> 
> def updateSymlink(productName, version, stageServer, stageUsername,
>                   stageSshKey, target):
>+    # step 1 check if we have already push to mirrors (bug 1083208)

Nit, s/push/pushed/

>+    push_to_mirrors_dir = '/pub/mozilla.org/%s/releases/%s' % (productName, version)

You can use 
   release_dir = makeReleasesDir(productName, version)
for this.

>+    try:
>+        # check if the remote dir exists
>+        run_remote_cmd('test -d %s' % push_to_mirrors_dir,

We normally write this as ['test', '-d', release_dir]

>+                       server=stageServer, username=stageUsername,
>+                       sshKey=stageSshKey)
>+    except CalledProcessError:
>+            log.error('ERROR: push to mirrors directory, %s, does not exists on %s'
>+                      % (push_to_mirrors_dir, stageServer))

Nit, 'ERROR: release directory, %s, doesn't exist on %s'.

r+ with those fixed and retested.

Comment 4

[Massimo Gervasini [:massimo]]

Comment on attachment 8510542 [details] [diff] [review]
[tools] Bug 1083208 - postrelease should fail if we didn't push to mirrors.patch

Thanks Nick.

Updated, tested in staging and landed.