981707 - browser_vimemacs.js is perma failing on linux debug tests when ran on amazon ec2 slaves

Status: NEW

(DevTools :: Source Editor, defect)

Description

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

I am not sure of the reason for this, but in order for us to switch our tests to ec2, we need to disable the perma failing tests.  This will be done in bug 981701, it would be great to get this figured out and turned back on.

Comment 1

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

here is a link to a logfile:
https://tbpl.mozilla.org/php/getParsedLog.php?id=35820713&tree=Cedar&full=1

Comment 2

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

the original author of the bug [e-mail], is an invalid email address, :harth, I see you were a reviewer of this test, couldyou take a look at this?

Comment 3

[Heather Arthur [:harth]]

Added in bug 941725. I didn't review that section of the patch, I think Mihai did.

Comment 4

[Mihai Sucan [:msucan]]

Attachment: add codemirror test runner reports

The TBPL log is as cryptic as it can be. The mozilla tests only load the codemirror test pages and, when done, they report a binary 'tests failed/passed'. This is far from optimal. 

I have updated the tests to include progress reports from within the codemirror test runner. This is something we need in any case.

Try push: https://tbpl.mozilla.org/?tree=Try&rev=a7cd172b4fa1

Joel: with this patch landed we should see where the test gets stuck, or if it fails to start.

Comment 5

[Heather Arthur [:harth]]

Comment on attachment 8395941 [details] [diff] [review]
add codemirror test runner reports

Review of attachment 8395941 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for taking a look!

Comment 6

[Mihai Sucan [:msucan]]

Thanks Heather for r+! Try push was green. This can land.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/fx-team/rev/8577346377ff

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8577346377ff