Consider to disable emulator-kk (arm-based) tests on Treeherder

RESOLVED FIXED in Firefox 43

Status

Firefox OS
Emulator
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: hsinyi, Assigned: hsinyi)

Tracking

unspecified
FxOS-S7 (18Sep)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
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

3 years ago
NI Thomas, Josh, Edgar.
Flags: needinfo?(tzimmermann)
Flags: needinfo?(jocheng)
Flags: needinfo?(echen)
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

3 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

3 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

3 years ago
Assignee: nobody → htsai
Flags: needinfo?(jocheng)
(Assignee)

Comment 5

3 years ago
Created attachment 8652710 [details] [diff] [review]
1196651.patch

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

3 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+
https://hg.mozilla.org/mozilla-central/rev/c2f8b1c0743a
Status: NEW → RESOLVED
Last Resolved: 3 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.