Remove external media tests from tree and disable them across all platforms

RESOLVED FIXED in Firefox 56

Status

RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: bryce, Assigned: bryce)

Tracking

(Blocks: 2 bugs)

Trunk
mozilla56
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(4 attachments)

(Assignee)

Description

2 years ago
As per bug 1362252, the media team has been sunsetting the external media tests. I am seeing local issues with the tests and the harness. As per our policy of removing tests as issues arise with them, it is time to remove the tests and see that they are disabled.

A follow up bug will be raised to track removal of documentation around the tests.
(Assignee)

Updated

2 years ago
Blocks: 1362252
(Assignee)

Updated

2 years ago
Assignee: nobody → bvandyk
(Assignee)

Updated

2 years ago
Depends on: 1372053
(Assignee)

Updated

2 years ago
Blocks: 1372059
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 6

2 years ago
mozreview-review
Comment on attachment 8877756 [details]
Bug 1372051 - Remove task cluster configuration for external media tests.

https://reviewboard.mozilla.org/r/149186/#review153630
Attachment #8877756 - Flags: review?(dustin) → review+

Comment 7

2 years ago
mozreview-review
Comment on attachment 8877757 [details]
Bug 1372051 - Remove external media tests from /testing, /build, /python.

https://reviewboard.mozilla.org/r/149188/#review154622

A few more places to clean up: 

testing/tools/autotry/autotry.py
build/virtualenv_packages.txt
python/mozbuild/mozbuild/action/test_archive.py
Attachment #8877757 - Flags: review?(mjzffr) → review-

Comment 8

2 years ago
mozreview-review
Comment on attachment 8877759 [details]
Bug 1372051 - Remove external media test source.

https://reviewboard.mozilla.org/r/149192/#review154624
Attachment #8877759 - Flags: review?(mjzffr) → review+
(Assignee)

Comment 9

2 years ago
mozreview-review-reply
Comment on attachment 8877757 [details]
Bug 1372051 - Remove external media tests from /testing, /build, /python.

https://reviewboard.mozilla.org/r/149188/#review154622

Cheers! Updated.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 13

2 years ago
mozreview-review
Comment on attachment 8877758 [details]
Bug 1372051 - Remove external media tests from mach modules.

https://reviewboard.mozilla.org/r/149190/#review155342
Attachment #8877758 - Flags: review?(gps) → review+

Comment 14

2 years ago
mozreview-review
Comment on attachment 8877757 [details]
Bug 1372051 - Remove external media tests from /testing, /build, /python.

https://reviewboard.mozilla.org/r/149188/#review155372
Attachment #8877757 - Flags: review?(mjzffr) → review+

Comment 15

2 years ago
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 228836b6c26a -d 7abd82919be6: rebasing 402704:228836b6c26a "Bug 1372051 - Remove task cluster configuration for external media tests. r=dustin"
merging taskcluster/ci/test/test-platforms.yml
merging taskcluster/ci/test/test-sets.yml
merging taskcluster/ci/test/tests.yml
merging taskcluster/taskgraph/try_option_syntax.py
warning: conflicts while merging taskcluster/ci/test/tests.yml! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
(Assignee)

Comment 16

2 years ago
Comment on attachment 8877756 [details]
Bug 1372051 - Remove task cluster configuration for external media tests.

Removed some extra lines from TC config that have been introduced since initial request. Requesting second review since initial request has changed (albeit in a straight forward fashion).

Lines removed are for external tests and of the format:

> windows7-32-vm/debug: built-projects
> windows7-32/debug: built-projects
Attachment #8877756 - Flags: review+ → review?(dustin)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 21

2 years ago
mozreview-review
Comment on attachment 8877756 [details]
Bug 1372051 - Remove task cluster configuration for external media tests.

https://reviewboard.mozilla.org/r/149186/#review156762
Attachment #8877756 - Flags: review+

Comment 22

2 years ago
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s e7505dce8c70 -d 95369f9de1ec: rebasing 403576:e7505dce8c70 "Bug 1372051 - Remove task cluster configuration for external media tests. r=dustin"
merging taskcluster/ci/test/test-platforms.yml
merging taskcluster/ci/test/test-sets.yml
merging taskcluster/ci/test/tests.yml
warning: conflicts while merging taskcluster/ci/test/test-platforms.yml! (edit, then use 'hg resolve --mark')
warning: conflicts while merging taskcluster/ci/test/test-sets.yml! (edit, then use 'hg resolve --mark')
warning: conflicts while merging taskcluster/ci/test/tests.yml! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 27

2 years ago
Pushed by bvandyk@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a91bf05d67bf
Remove task cluster configuration for external media tests. r=dustin
https://hg.mozilla.org/integration/autoland/rev/459d039c70b6
Remove external media tests from /testing, /build, /python. r=maja_zf
https://hg.mozilla.org/integration/autoland/rev/4da69bcf9d4b
Remove external media tests from mach modules. r=gps
https://hg.mozilla.org/integration/autoland/rev/619f14ae3bc6
Remove external media test source. r=maja_zf

Comment 28

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a91bf05d67bf
https://hg.mozilla.org/mozilla-central/rev/459d039c70b6
https://hg.mozilla.org/mozilla-central/rev/4da69bcf9d4b
https://hg.mozilla.org/mozilla-central/rev/619f14ae3bc6
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56

Updated

a year ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.