Rename marionette.py's run-marionette action to run-tests

RESOLVED FIXED

Status

Release Engineering
Mozharness
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ahal, Assigned: ahal)

Tracking

unspecified

Firefox Tracking Flags

(firefox49 fixed)

Details

MozReview Requests

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

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Armen pointed out that marionette doesn't have a "run-tests" actoin (it's instead called "run-marionette"). This means that some of the tools from bug 1250904 won't work when interactively debugging tests with taskcluster (they assume there is a "run-tests" action).

We should just rename run-marionette to run-tests to make this work and to be consistent with every other mozharness script.
(Assignee)

Comment 1

2 years ago
Created attachment 8755916 [details]
MozReview Request: Bug 1275308 - [mozharness] Rename the run-marionette action to run-tests, r?armenzg

In bug 1262260 and 1250904, we are building some tools to make debugging tests in
interactive jobs easier. Part of this includes making it easier to run mozharness
scripts with various configurations. Some of these tools depend on a "run-tests"
action being present, but in marionette.py, this action is called "run-marionette"
instead.

Rather than make these tools special case marionette, let's just make marionette.py
consistent with every other mozharness script.

Review commit: https://reviewboard.mozilla.org/r/54884/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54884/
Attachment #8755916 - Flags: review?(armenzg)

Comment 2

2 years ago
Comment on attachment 8755916 [details]
MozReview Request: Bug 1275308 - [mozharness] Rename the run-marionette action to run-tests, r?armenzg

https://reviewboard.mozilla.org/r/54884/#review51530

Excellent!
Attachment #8755916 - Flags: review?(armenzg) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/3449c8ff31c9

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3449c8ff31c9
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.