Closed Bug 790613 Opened 9 years ago Closed 9 years ago

Make sut_tools/ output failures in the format "Automation Error: " so TBPL's parser recognises them


(Release Engineering :: General, defect)

Not set


(Not tracked)



(Reporter: emorley, Assigned: emorley)


(Blocks 1 open bug)


(Whiteboard: [sheriff-want])


(1 file, 1 obsolete file)

(Bah, I should have done this as part of bug 787019)

Failures of the type bug 781419 require the log to be opened, since TBPL doesn't recognise the error string and so doesn't show it in the annotated summary box.

We can either change the string to match one of the ones at:

Or add our own, after changing from 'WARNING: ' since that's too generic & will match against other parts of the log.

I suggest changing:
WARNING: Unable to ping tegra after 5 attempts
to (the already recognised):
Automation Error: Unable to ping tegra after 5 attempts

(Whilst we're at it, we might as well change other error strings in at the same time).
Assignee: nobody → bmo
Status: NEW → ASSIGNED uses a lot of:
"Remote Device Error: foo"
...perhaps we should add that to the parser in addition to "Automation Error: " ?
Attached patch Patch v1 (obsolete) — Splinter Review
Prefixes a number of failures with "Automation Error:" or "Remote Device Error:" so TBPL's parser can recognise them.

The "Remote Device Error:" case will require a TBPL change, I'll file that now.

Had to make best guess as to which errors to put under which category, tweaks welcome :-)
Attachment #660415 - Flags: review?(bugspam.Callek)
Depends on: 790618
Depends on: 781341
Comment on attachment 660415 [details] [diff] [review]
Patch v1

From a releng perspective just changing these strings are fine.

Add'l r? to jmaher since he has code that this will bitrot, and for the determination of which strings are correct.
Attachment #660415 - Flags: review?(jmaher)
Attachment #660415 - Flags: review?(bugspam.Callek)
Attachment #660415 - Flags: review+
Comment on attachment 660415 [details] [diff] [review]
Patch v1

Review of attachment 660415 [details] [diff] [review]:

r- because I am concerned about the error for all cases when we are not installing robocop.apk.

This is from a jsreftest log on m-c:
return code [0]
WARNING (robocop): zip file not as expected: There is no item named 'bin/robocop.apk' in the archive (("There is no item named 'bin/robocop.apk' in the archive",))
WARNING (robocop): robocop.apk will not be installed
/builds/sut_tools/ DeprecationWarning: BaseException.message has been deprecated as of Python 2.6
  print "WARNING (robocop): zip file not as expected: %s (%s)" % (e.message,
program finished with exit code 0
========= Finished Install App on Device (results: 0, elapsed: 2 mins, 2 secs) (at 2012-09-12 14:32:14.329316) =========

::: sut_tools/
@@ +85,2 @@
>                                                                          str(e.args))
> +                print "Remote Device Error: (robocop) robocop.apk will not be installed"

these are not really remote device errors, they would be 'Automation Error'

@@ +89,2 @@
>                                                                        str(matches))
> +            print "Remote Device Error: (robocop) robocop.apk will not be installed"

Won't this occur when we are testing without robocop.apk?  This really seems problematic.
Attachment #660415 - Flags: review?(jmaher) → review-
Attached patch Patch v2Splinter Review
As before except without the changes to the robocop.apk warnings (I had forgotten we indiscriminately display that warning in non-robocop runs).

Those strings weren't the ones I was most fussed about anyway; the bug 781419 case is the primary goal - I was just trying to do some others whilst I was there :-)
Attachment #660415 - Attachment is obsolete: true
Attachment #660890 - Flags: review?(jmaher)
Comment on attachment 660890 [details] [diff] [review]
Patch v2

Review of attachment 660890 [details] [diff] [review]:

this looks pretty good.
Attachment #660890 - Flags: review?(jmaher) → review+
Blocks: 791477
(In reply to Ed Morley (Away 18th-20th) [:edmorley UTC+1] from comment #7)
> Comment on attachment 660890 [details] [diff] [review]
> Patch v2
> Thank you :-)

Deployed to all foopies just now.
Closed: 9 years ago
Resolution: --- → FIXED
(In reply to Justin Wood (:Callek) from comment #8)
> Deployed to all foopies just now.

Thank you, working great on TBPL now :-)
No longer depends on: 781341
Product: → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.