Closed
Bug 1132209
Opened 10 years ago
Closed 10 years ago
Stop running robocop on Android x86
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gbrown, Assigned: gbrown)
References
Details
Attachments
(3 files)
1.08 KB,
patch
|
kmoir
:
review+
|
Details | Diff | Splinter Review |
4.59 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
1.78 KB,
patch
|
kmoir
:
review+
|
Details | Diff | Splinter Review |
In today's mobile testing meeting, we considered bug 1127916 -- Android x86 S4 is currently hidden, due to excessive robocop failures -- and asserted:
- we would like *some* x86 tests running and visible to avoid catastrophic x86 failures in releases
- we are having trouble finding time/people to resolve the robocop failures
- we don't expect significant differences in robocop (java) behavior between x86 and arm, so robocop is arguably of limited importance to x86.
The Android x86 S4 "set" consists of xpcshell, robocop-1, robocop-2, and robocop-3 test suites. If we stop running robocop-1/2/3, xpcshell tests should continue to run well and then S4 (which will be xpcshell only) can be un-hidden.
![]() |
Assignee | |
Comment 1•10 years ago
|
||
I think this is all that is necessary.
There's other "cleanup" to remove un-used robocop configuration, but maybe we should wait to see how this plays out before cleaning up everything?
Attachment #8563026 -
Flags: review?(kmoir)
Updated•10 years ago
|
Attachment #8563026 -
Flags: review?(kmoir) → review+
Comment 2•10 years ago
|
||
![]() |
Assignee | |
Comment 3•10 years ago
|
||
Verified on mozilla-inbound, mozilla-central, mozilla-beta and release: Robocop is no longer running in Android x86 S4.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
![]() |
Assignee | |
Comment 4•10 years ago
|
||
Sorry - I wanted to keep this open, to clean up robocop.ini, update robocop x86-specific bugs, and possibly clean up x86 mozharness configuration.
Assignee: nobody → gbrown
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
![]() |
Assignee | |
Comment 5•10 years ago
|
||
This updates robocop.ini to remove all the references to x86, to make the manifest easier to read.
I would also update all the referenced bugs, noting that we no longer run robocop on x86. Should I close those bugs?
Attachment #8572086 -
Flags: review?(snorp)
Comment 6•10 years ago
|
||
Comment on attachment 8572086 [details] [diff] [review]
clean up robocop.ini
Review of attachment 8572086 [details] [diff] [review]:
-----------------------------------------------------------------
I guess this means we won't try to fix it then? :/
Attachment #8572086 -
Flags: review?(snorp) → review+
![]() |
Assignee | |
Comment 7•10 years ago
|
||
Keywords: leave-open
Comment 8•10 years ago
|
||
![]() |
Assignee | |
Comment 9•10 years ago
|
||
(In reply to Geoff Brown [:gbrown] from comment #5)
> I would also update all the referenced bugs, noting that we no longer run
> robocop on x86. Should I close those bugs?
Updated. I closed a few bugs but many were cross-platform.
![]() |
Assignee | |
Comment 10•10 years ago
|
||
Just to keep it tidy!
Attachment #8573269 -
Flags: review?(kmoir)
Updated•10 years ago
|
Attachment #8573269 -
Flags: review?(kmoir) → review+
![]() |
Assignee | |
Comment 11•10 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
![]() |
Assignee | |
Updated•10 years ago
|
Keywords: leave-open
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•