Add -dsds to existing Jenkins jobs

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jlorenzo, Assigned: jlorenzo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

46 bytes, text/x-github-pull-request
Bebe
: review+
RobertC
: review+
Details | Review
46 bytes, text/x-github-pull-request
Bebe
: review+
RobertC
: review+
Details | Review
Assignee

Description

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

5 years ago
Assignee: nobody → jlorenzo
QA Whiteboard: [fxosqa-auto-s4][fxosqa-auto-points=1]
Assignee

Comment 1

5 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: 5 years ago
Flags: needinfo?(dave.hunt)
Resolution: --- → FIXED
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

5 years ago
If dsds is not set in the manifest file, all the tests are skipped.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee

Comment 4

5 years ago
Attachment #8522269 - Flags: review?(robert.chira)
Attachment #8522269 - Flags: review?(florin.strugariu)
Attachment #8522269 - Flags: review?(dave.hunt)
Attachment #8522269 - Flags: review?(robert.chira) → review+
Assignee

Comment 5

5 years ago
Posted file Gaia PR 2.1
Same thing for 2.1
Attachment #8522272 - Flags: review?(robert.chira)
Attachment #8522272 - Flags: review?(florin.strugariu)
Attachment #8522272 - Flags: review?(dave.hunt)
Attachment #8522272 - Flags: review?(robert.chira) → review+
Attachment #8522269 - Flags: review?(florin.strugariu) → review+
Attachment #8522272 - Flags: review?(florin.strugariu)
Attachment #8522272 - Flags: review+
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Assignee

Updated

5 years ago
Depends on: 1099041
Assignee

Comment 8

5 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 → ---
I removed the -dsds from the V2.1 jobs to be able to run the tests
Assignee

Comment 10

5 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: 5 years ago5 years ago
Resolution: --- → FIXED
Assignee

Comment 11

5 years ago
I checked flame-kk-319.b2g-inbound.ui.functional.smoke today, it's running with -dsds and detects 42 tests.

Comment 12

5 years ago
Reverted the Jenkins changes in v2.1 to match the revert in bug 1099041.
Assignee

Updated

5 years ago
Blocks: 1093589
Assignee

Updated

5 years ago
Blocks: 1093608
You need to log in before you can comment on or make changes to this bug.