Don't try to package the steeplechase tests if webrtc is disabled

RESOLVED FIXED in Firefox 39, Firefox OS v2.2

Status

()

P2
normal
Rank:
25
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: RyanVM, Assigned: RyanVM)

Tracking

Trunk
mozilla39
Points:
---
Bug Flags:
firefox-backlog +

Firefox Tracking Flags

(firefox39 fixed, b2g-v2.2 fixed, b2g-master fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
We were hitting bustage on Emulator L builds during packaging because webrtc was disabled and it was trying to package the steeplechase tests. We shouldn't try to do so unless webrtc is enabled.
(Assignee)

Comment 1

4 years ago
Created attachment 8576696 [details] [diff] [review]
patch

Try confirms that this works as expected. Nils, speak now or forever hold your peace if there's any reason to not do this :)
Assignee: nobody → ryanvm
Status: NEW → ASSIGNED
Attachment #8576696 - Flags: review?(gps)
Attachment #8576696 - Flags: feedback?(drno)
Comment on attachment 8576696 [details] [diff] [review]
patch

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

If I understand the idea correct this only means: don't packages steeplechase (plus its tests) if MOZ_WEBRTC is not defined. That sounds perfectly reasonable.

My only question/concern is: does this affect the *tests.zip file we publish together with the binary tarballs?
Because our current steeplechase setup in the MV QA lab extracts steeplechase from these test zip files. So excluding steeplechase from that would cause major trouble for us (my + assumes that your patch is not going that far :-) ).
Attachment #8576696 - Flags: feedback?(drno) → feedback+
(Assignee)

Comment 3

4 years ago
I don't see how it would affect tests.zip except for platforms where packaging them would have been useless anyway (i.e. they wouldn't be able to run on a webrtc-disabled build anyway). Below are my Try pushes if you want to take a look at the packages and verify, though.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a65bea9f145a
https://treeherder.mozilla.org/#/jobs?repo=try&revision=721313d119fd
Rank: 25
Flags: firefox-backlog+
Priority: -- → P2
Attachment #8576696 - Flags: review?(gps) → review+
(Assignee)

Comment 5

4 years ago
https://hg.mozilla.org/mozilla-central/rev/b3ae8d7b0426
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.