Closed
Bug 1098279
Opened 10 years ago
Closed 10 years ago
Add -dsds to existing Jenkins jobs
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Firefox OS Graveyard
Gaia::UI Tests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
Details
Attachments
(2 files)
As :davehunt suggested, if we want to have a dedicated job for DSDS tests, we need to exclude them from the other jobs. In order to do so, we have to append "-dsds" at the end of "--type=b2g*"
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jlorenzo
QA Whiteboard: [fxosqa-auto-s4][fxosqa-auto-points=1]
Assignee | ||
Comment 1•10 years ago
|
||
Done on every UI except flame-kk-319.mozilla-central.ui.marketplace-tests because it runs a different suite.
Dave, are you okay with leaving this one as is?
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(dave.hunt)
Resolution: --- → FIXED
Comment 2•10 years ago
|
||
That job doesn't appear to be using --type, and anyway you'd need to set dsds = false in the main manifest of the marketplace-tests-gaia if you wanted to make that change. I doubt it's worth the effort.
Flags: needinfo?(dave.hunt)
Assignee | ||
Comment 3•10 years ago
|
||
If dsds is not set in the manifest file, all the tests are skipped.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8522269 -
Flags: review?(robert.chira)
Attachment #8522269 -
Flags: review?(florin.strugariu)
Attachment #8522269 -
Flags: review?(dave.hunt)
Updated•10 years ago
|
Attachment #8522269 -
Flags: review?(robert.chira) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Same thing for 2.1
Attachment #8522272 -
Flags: review?(robert.chira)
Attachment #8522272 -
Flags: review?(florin.strugariu)
Attachment #8522272 -
Flags: review?(dave.hunt)
Updated•10 years ago
|
Attachment #8522272 -
Flags: review?(robert.chira) → review+
Updated•10 years ago
|
Attachment #8522269 -
Flags: review?(florin.strugariu) → review+
Comment 6•10 years ago
|
||
Comment on attachment 8522269 [details] [review]
Gaia PR - Avoid skipping all the tests
merged in https://github.com/mozilla-b2g/gaia/commit/a83edf2bfbb1987e24618b9f895955ac4dfc807d
Attachment #8522269 -
Flags: review?(dave.hunt)
Updated•10 years ago
|
Attachment #8522272 -
Flags: review?(florin.strugariu)
Attachment #8522272 -
Flags: review+
Comment 7•10 years ago
|
||
Comment on attachment 8522272 [details] [review]
Gaia PR 2.1
merged in https://github.com/mozilla-b2g/gaia/commit/af6533781356acc62b0f40c9e040aa5b47d3b709
Attachment #8522272 -
Flags: review?(dave.hunt)
Updated•10 years ago
|
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•10 years ago
|
||
Adding these changes in the general manifest is not enough. We need to add this manifest as a dependency for gaiatest/tests/functional/manifest.ini. It'll be done in bug 1099041
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 9•10 years ago
|
||
I removed the -dsds from the V2.1 jobs to be able to run the tests
Assignee | ||
Comment 10•10 years ago
|
||
Now that bug 1099041 is fixed, I added -dsds to each job one by one. I checked every job execution and if it didn't skip the tests, I aborted the job.
One job I couldn't check was flame-kk-319.b2g-inbound.ui.functional.smoke because it was always in use.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•10 years ago
|
||
I checked flame-kk-319.b2g-inbound.ui.functional.smoke today, it's running with -dsds and detects 42 tests.
Comment 12•10 years ago
|
||
Reverted the Jenkins changes in v2.1 to match the revert in bug 1099041.
You need to log in
before you can comment on or make changes to this bug.
Description
•