Bug 1097752, don't update latest directories for log uploads, r=bhearsum
authorNick Thomas <nthomas@mozilla.com>
Tue, 18 Nov 2014 10:31:40 +1300 (2014-11-17)
changeset 3959 5bf45e9cc8c2afe312c23a5812f3f5df5b9977c3
parent 3946 c0185f80b7fad7a92dbdbc2d5a2814d26ea420a1
child 3961 7488410fce9b10f31a80800e4aed1877f6a7a380
push id3268
push usernthomas@mozilla.com
push dateMon, 17 Nov 2014 21:31:54 +0000 (2014-11-17)
reviewersbhearsum
bugs1097752
Bug 1097752, don't update latest directories for log uploads, r=bhearsum
bin/log_uploader.py
process/factory.py
--- a/bin/log_uploader.py
+++ b/bin/log_uploader.py
@@ -271,16 +271,17 @@ if __name__ == "__main__":
                     files=[logfile], remote_dir=remote_tmpdir)
 
                 remote_files = [os.path.join(
                     remote_tmpdir, os.path.basename(f)) for f in [logfile]]
 
             uploadArgs = dict(
                 branch=options.branch,
                 product=options.product,
+                log=True,
             )
 
             # Make sure debug platforms are properly identified
             # Test builders don't have the '-debug' distinction in the platform
             # string, so check in the builder name to make sure.
             platform = options.platform
             if platform:
                 if '-debug' in builder_path and '-debug' not in platform:
--- a/process/factory.py
+++ b/process/factory.py
@@ -127,16 +127,17 @@ def postUploadCmdPrefix(upload_dir=None,
                         to_dated=False,
                         to_latest=False,
                         to_try=False,
                         to_candidates=False,
                         to_mobile_candidates=False,
                         nightly_dir=None,
                         as_list=True,
                         signed=False,
+                        log=False,
                         ):
     """Returns a post_upload.py command line for the given arguments.
 
     If as_list is True (the default), the command line will be returned as a
     list of arguments.  Some arguments may be WithProperties instances.
 
     If as_list is False, the command will be returned as a WithProperties
     instance representing the entire command line as a single string.
@@ -162,17 +163,18 @@ def postUploadCmdPrefix(upload_dir=None,
     if revision:
         cmd.extend(['--revision', revision])
     if who:
         cmd.extend(['--who', who])
     if builddir:
         cmd.extend(['--builddir', builddir])
     if to_tinderbox_dated:
         cmd.append('--release-to-tinderbox-dated-builds')
-        cmd.append('--release-to-latest-tinderbox-builds')
+        if not log:
+            cmd.append('--release-to-latest-tinderbox-builds')
     if to_tinderbox_builds:
         cmd.append('--release-to-tinderbox-builds')
     if to_try:
         cmd.append('--release-to-try-builds')
     if to_latest:
         cmd.append("--release-to-latest")
     if to_dated:
         cmd.append("--release-to-dated")