Closed Bug 979008 Opened 10 years ago Closed 9 years ago

Intermittent Jetpack tests breaking on Linux ASAN builds with | testRmdirNonempty, test-tab-events | Test output exceeded timeout (300s).

Categories

(Add-on SDK Graveyard :: General, defect, P1)

x86_64
Linux
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: KWierso, Unassigned)

References

Details

(Keywords: intermittent-failure)

Flags: needinfo?(dtownsend+bugmail)
Dave, Erik, any ideas? ASAN doesn't run on the Jetpack tree, so this might be hard to diagnose.
Summary: Jetpack tests breaking on Linux ASAN builds with | testRmdirNonempty, test open -> pin -> unpin, or test open -> close | Test output exceeded timeout (300s). → Jetpack tests breaking on Linux ASAN builds with | testRmdirNonempty, test-tab-events | Test output exceeded timeout (300s).
Whiteboard: https://tbpl.mozilla.org/php/getParsedLog.php?id=35552482&tree=Fx-Team
Whiteboard: https://tbpl.mozilla.org/php/getParsedLog.php?id=35552482&tree=Fx-Team
Summary: Jetpack tests breaking on Linux ASAN builds with | testRmdirNonempty, test-tab-events | Test output exceeded timeout (300s). → Intermittent Jetpack tests breaking on Linux ASAN builds with | testRmdirNonempty, test-tab-events | Test output exceeded timeout (300s).
Jordan is looking into the 300s timeout problems.
Flags: needinfo?(dtownsend+bugmail)
Out of the commits here https://github.com/mozilla/addon-sdk/compare/2d29a417803206a25c87764b6a7ca1d6cfab9afc...036238d8bd1bfbbbc2a44467be646711e3aeed19

I think these two are the only ones that may have affected the test failures here

https://github.com/mozilla/addon-sdk/commit/3117486febbdf7f1e19f832380a7ba456e699591#diff-cfdb8a4b887b2e50acc107e0c1c98a7e

https://github.com/mozilla/addon-sdk/commit/fb04bb48821469fa371b28fdb89de97d672d8989#diff-053c0ba2ea06dd310ecc8032c5ae0b03

The former seems more likely and it didn't seem to have the desired affect in bug 926264, so I think we may as well revert it and see if that helps here.

If that doesn't work then we could try reverting the other patch.
Flags: needinfo?(evold) → needinfo?(dtownsend+bugmail)
(In reply to Erik Vold [:erikvold] [:ztatic] from comment #5)
> Out of the commits here
> https://github.com/mozilla/addon-sdk/compare/
> 2d29a417803206a25c87764b6a7ca1d6cfab9afc...
> 036238d8bd1bfbbbc2a44467be646711e3aeed19
> 
> I think these two are the only ones that may have affected the test failures
> here
> 
> https://github.com/mozilla/addon-sdk/commit/
> 3117486febbdf7f1e19f832380a7ba456e699591#diff-
> cfdb8a4b887b2e50acc107e0c1c98a7e
> 
> https://github.com/mozilla/addon-sdk/commit/
> fb04bb48821469fa371b28fdb89de97d672d8989#diff-
> 053c0ba2ea06dd310ecc8032c5ae0b03
> 
> The former seems more likely and it didn't seem to have the desired affect
> in bug 926264, so I think we may as well revert it and see if that helps
> here.
> 
> If that doesn't work then we could try reverting the other patch.

Agreed, let's get that first one backed out.
Flags: needinfo?(dtownsend+bugmail)
Inactive; closing (see bug 1180138).
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.