disable jetpack test job

REOPENED
Unassigned

Status

Testing
General
REOPENED
11 months ago
11 months ago

People

(Reporter: jmaher, Unassigned)

Tracking

Trunk
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

11 months ago
now that web extensions will be default in firefox 57, I would like to ensure we are disabling tests that are not needed- specifically the jetpack job.
(Reporter)

Comment 1

11 months ago
:andym- does this bug sound correct?
Flags: needinfo?(amckay)

Comment 2

11 months ago
Sounds correct thanks, I think :kmag wanted to turn off the jetpack tests in the 57 cycle to save some test cycles for everyone.
Blocks: 1371065
Flags: needinfo?(amckay)

Comment 3

11 months ago
Yes, I was planning to write a patch to disable all SDK tests other than the loader tests today. The loader tests can run in a regular mochitest job, though.

Updated

11 months ago
Blocks: 1352075
Comment hidden (mozreview-request)
(Reporter)

Comment 5

11 months ago
Created attachment 8893016 [details] [diff] [review]
remove jetpack job from taskcluster configs
Attachment #8893016 - Flags: review?(kmaglione+bmo)

Comment 6

11 months ago
Comment on attachment 8893016 [details] [diff] [review]
remove jetpack job from taskcluster configs

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

::: taskcluster/ci/test/tests.yml
@@ -906,5 @@
> -                linux.*:
> -                    - unittests/linux_unittest.py
> -                    - remove_executables.py
> -        extra-options:
> -            - --mochitest-suite=jetpack-package

We still need to run the jetpack-package suite, for now, since the devtools still use the SDK loader. After my other patch, the loader tests will be the only ones in that suite, though, so they can be moved to another mochitest job without too much disruption.
Attachment #8893016 - Flags: review?(kmaglione+bmo) → review-

Comment 7

11 months ago
mozreview-review
Comment on attachment 8893010 [details]
Bug 1386694: Part 1 - Disable most SDK tests.

https://reviewboard.mozilla.org/r/164012/#review169352
Attachment #8893010 - Flags: review?(dtownsend) → review+
(Reporter)

Updated

11 months ago
Depends on: 1386776
(Reporter)

Updated

11 months ago
No longer depends on: 1386776

Comment 9

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f6cdb28c2f2a
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
(Reporter)

Comment 10

11 months ago
I believe there is a bit more work to do here to shift the remaining jetpack-package suite into regular mochitests.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
You need to log in before you can comment on or make changes to this bug.