Closed Bug 1402955 Opened 4 years ago Closed 4 years ago

bouncer submission failed for firefox 57.0b3

Categories

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

defect
Not set
normal

Tracking

(firefox58 fixed)

RESOLVED FIXED
Tracking Status
firefox58 --- fixed

People

(Reporter: kmoir, Assigned: rail)

References

Details

Attachments

(1 file)

I think this is because of an extra bracket here before "partials": 

https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-beta-noarch/release-mozilla-beta_firefox_bncr_sub-bm72-build1-build28.txt.gz

diff --git a/testing/mozharness/configs/releases/bouncer_firefox_beta.py b/testing/mozharness/configs/releases/bouncer_firefox_beta.py
--- a/testing/mozharness/configs/releases/bouncer_firefox_beta.py
+++ b/testing/mozharness/configs/releases/bouncer_firefox_beta.py
@@ -100,44 +100,16 @@ config = {
                     "bouncer-platform": "win",
                 },
                 "win64": {
                     "path": "/firefox/releases/%(version)s/update/win64/:lang/firefox-%(version)s.complete.mar",
                     "bouncer-platform": "win64",
                 },
             },
         },
-        "complete-mar-bz2": {
-            "product-name": "Firefox-%(version)s-Complete-bz2",
-            "check_uptake": True,
-            "ssl-only": False,
-            "add-locales": True,
-            "paths": {
-                "linux": {
-                    "path": "/firefox/releases/%(version)s/update/linux-i686/:lang/firefox-%(version)s.bz2.complete.mar",
-                    "bouncer-platform": "linux",
-                },
-                "linux64": {
-                    "path": "/firefox/releases/%(version)s/update/linux-x86_64/:lang/firefox-%(version)s.bz2.complete.mar",
-                    "bouncer-platform": "linux64",
-                },
-                "macosx64": {
-                    "path": "/firefox/releases/%(version)s/update/mac/:lang/firefox-%(version)s.bz2.complete.mar",
-                    "bouncer-platform": "osx",
-                },
-                "win32": {
-                    "path": "/firefox/releases/%(version)s/update/win32/:lang/firefox-%(version)s.bz2.complete.mar",
-                    "bouncer-platform": "win",
-                },
-                "win64": {
-                    "path": "/firefox/releases/%(version)s/update/win64/:lang/firefox-%(version)s.bz2.complete.mar",
-                    "bouncer-platform": "win64",
-                },
-            },
-        },
     },
     },
     "partials": {
         "releases-dir": {
             "product-name": "Firefox-%(version)s-Partial-%(prev_version)s",
             "check_uptake": True,
             "ssl-only": False,
missed this in the first comment

hg diff -r 55d0a2aa99de -r 69932f66faf2 testing/mozharness/configs/releases/bouncer_firefox_beta.py

Rail, should we land a patch for this and run a build from a new revision or should we run the bouncer submission manually?
Flags: needinfo?(rail)
(In reply to Kim Moir [:kmoir] ET from comment #1)
> missed this in the first comment
> 
> hg diff -r 55d0a2aa99de -r 69932f66faf2
> testing/mozharness/configs/releases/bouncer_firefox_beta.py
> 
> Rail, should we land a patch for this and run a build from a new revision or
> should we run the bouncer submission manually?

Do you mean land a tree patch and wait for the builds to run or specify just the Mozharness Revision with the fix and keep the current revision? If latter, this doesn't always work, it's more of a dice roll. 

Another solution could be to hack releasetasks to only send a graph with just one task - the bouncer submission. I did something like that for Fennec once.
Attached patch bug1402955.patchSplinter Review
Attachment #8911970 - Flags: review?(nthomas)
•nthomas> last time we hit a syntax error r.ail landed the fix, cloned the task and updated it to use the new rev. No downstream tasks so this works fin
Comment on attachment 8911970 [details] [diff] [review]
bug1402955.patch

Oops, my bad. I'm going to land it, uplift and fix the current build. Sorry for the churn.
Flags: needinfo?(rail)
Attachment #8911970 - Flags: review?(nthomas) → review+
Assignee: nobody → rail
Pushed by raliiev@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a172badf4ab6
Fix bouncer_firefox_beta.py syntax. r=rail DONTBUILD
I did the same for ZPZz9WCHQr6qUXHve4TMLg. After the cloned task became green, I rerun the original task and resolved it asap:

taskcluster-cli task rerun ZPZz9WCHQr6qUXHve4TMLg && taskcluster-cli task complete ZPZz9WCHQr6qUXHve4TMLg
See Also: → 1403131
Did similar thing for bouncer aliases in https://tools.taskcluster.net/groups/GmRau2v0TXmDi0j_wwXSWQ/tasks/GmRau2v0TXmDi0j_wwXSWQ/details and rerun/completed the original task.
You need to log in before you can comment on or make changes to this bug.