Closed
Bug 1196651
Opened 10 years ago
Closed 10 years ago
Consider to disable emulator-kk (arm-based) tests on Treeherder
Categories
(Firefox OS Graveyard :: Emulator, defect)
Tracking
(firefox43 fixed)
RESOLVED
FIXED
FxOS-S7 (18Sep)
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: hsinyi, Assigned: hsinyi)
Details
Attachments
(1 file)
3.29 KB,
patch
|
garndt
:
review+
|
Details | Diff | Splinter Review |
Hi all,
I'd like to disable emulator-kk (arm) tests on Treeherder to save Treeherder resource and to focus developer effort on emulator-x86-kk.
I don't see harm given that emulator-kk (arm) tests has never been ready yet, but please let me know if you have concerns. Thanks!
Assignee | ||
Comment 1•10 years ago
|
||
NI Thomas, Josh, Edgar.
Flags: needinfo?(tzimmermann)
Flags: needinfo?(jocheng)
Flags: needinfo?(echen)
Comment 2•10 years ago
|
||
I don't think anyone ever cared about these tests. So I'm generally OK with the idea if it helps with emulator-x86-kk.
The only thing that must remain on Treeherder is the build step. We have to detect build failures for KK early.
Flags: needinfo?(tzimmermann)
Comment 3•10 years ago
|
||
+1 for disabling the tests running on emulator-kk since we now focus on emulator-x86-kk. Thank you.
Flags: needinfo?(echen)
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #2)
> I don't think anyone ever cared about these tests. So I'm generally OK with
> the idea if it helps with emulator-x86-kk.
>
> The only thing that must remain on Treeherder is the build step. We have to
> detect build failures for KK early.
Sure, I also managed to keep KK build.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → htsai
Flags: needinfo?(jocheng)
Assignee | ||
Comment 5•10 years ago
|
||
No one cares emu-kk tests and no one works emu-kk tests. I think it's beneficial to disable them and to save resources.
Attachment #8652710 -
Flags: review?(garndt)
Comment 6•10 years ago
|
||
Comment on attachment 8652710 [details] [diff] [review]
1196651.patch
Review of attachment 8652710 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good to me. Thanks!
Attachment #8652710 -
Flags: review?(garndt) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S7 (18Sep)
You need to log in
before you can comment on or make changes to this bug.
Description
•