799277 - merge day robocheck*, roboprovider, robopan reds issues on m-a

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[hwine]

Post uplift m-c -> aurora, various robocop tests running red.

Comment 1

[Phil Ringnalda (:philor)]

Because confusing in naming is everything, robocop is a sort of browser-chromeish kind of mochitest, which may or may not be permanently orange from timing out after a few tests, too few to tell yet, while robocheck(and 2 and 3), robopan and roboprovider are talos tests, which are permanently red from the infamous and opaque "browser non-zero return code (1) [browser_output.txt]"

Comment 2

[Phil Ringnalda (:philor)]

Oh, and since it's keeping a tree closed, BLOCKER by definition.

Comment 3

[Phil Ringnalda (:philor)]

And robocop did get better, so it's just the robotium talos suites that are permabusted.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: remove all instances of hardcoded org.mozilla.fennec from config file (1.0)

this resolve our problem, but we need to backout a mozdevice patch first.

Comment 5

[Phil Ringnalda (:philor)]

Comment on attachment 669357 [details] [diff] [review]
remove all instances of hardcoded org.mozilla.fennec from config file (1.0)

># HG changeset patch
># Parent bf0f2b04ab86a6659020e67a6c98f9b980a6e88c
>Bug 799277 - merge day robocheck*, roboprovider, robopan reds issues on m-a. r=philor
>
>diff -r bf0f2b04ab86 talos/remotePerfConfigurator.py
>--- a/talos/remotePerfConfigurator.py	Mon Oct 08 15:59:06 2012 -0400
>+++ b/talos/remotePerfConfigurator.py	Mon Oct 08 20:57:07 2012 -0400
>@@ -179,7 +179,7 @@
>         url = url.replace('webServer=', 'webServer=%s' % self.config['webserver'])
> 
>         # Take care of the robocop based tests
>-        url = url.replace('class org.mozilla.fennec.tests', 'class %s.tests' % self.config['browser_path'])
>+        url = url.replace('org.mozilla.fennec', self.config['browser_path'])
>         return url
> 
>     def buildRemoteManifest(self, manifestName):

Comment 6

[Justin Wood (:Callek)]

Comment on attachment 669357 [details] [diff] [review]
remove all instances of hardcoded org.mozilla.fennec from config file (1.0)

As per IRC, I'm not certain this will fix it, since I'm not up on the specifics of this talos code -- but I can assert the code is syntactically correct, and won't make things any worse than they are now.

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

landed in talos:
http://hg.mozilla.org/build/talos/rev/5f421eb0d1ed

Comment 8

[Alex Keybl [:akeybl]]

a=akeybl for pushing an empty cset to test this landing. Please make this a very high priority today - b2g landings and other fixes are blocked on getting this fixed.

Comment 9

[Jeff Hammel]

we still need to create a talos.zip and update testing/talos/talos.json to get this deployed

Comment 10

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

(In reply to Jeff Hammel [:jhammel] from comment #9)
> we still need to create a talos.zip and update testing/talos/talos.json to
> get this deployed

Anything left to do here? This bug is marked as blocker, but has no activity in a while...

Comment 11

[Justin Wood (:Callek)]

Joel, did we ever deploy said updated talos.zip? If so, lets resolve!

Comment 12

[Joel Maher ( :jmaher ) (UTC -8)]

I forgot to comment which would clear this flag.