Closed Bug 978233 Opened 8 years ago Closed 8 years ago

mozharness should use --manifest instead of --test-manifest for emulator-jb mochitests

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ahal, Assigned: ahal)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Mochitest manifests are still in a transition phase, and the command line for them is a little (ok a lot) hairy. Right now --test-manifest is used for the .json style manifests and --manifest is used for the .ini still. There's also --run-only-tests which is an alias for --test-manifest.

I think the harness should just have one commandline option and detect which type of manifest was passed in automatically. The problem here is that until we have 100% coverage with .ini style manifests we might actually need to pass in both! So for now I propose that we just add a little hack to mozharness to take care of this mess for us.
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
This should do it.
Attachment #8383898 - Flags: review?(jgriffin)
Comment on attachment 8383898 [details] [diff] [review]
mochitest manifest hack

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

We can unwind this later after it's not necessary.
Attachment #8383898 - Flags: review?(jgriffin) → review+
mozharness patch is merged and live in production! :)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Component: General Automation → Mozharness
You need to log in before you can comment on or make changes to this bug.