1635136 - add a --push-to-try option

Status: RESOLVED FIXED

(Testing :: mozperftest, enhancement)

Description

[Tarek Ziadé (:tarek)]

This option will push all arguments for mach perftest in the try commit in a json file
and let the CI unwrap it for mach perftest.

This will be useful to run any perftest on bitbar

Comment 1

[Tarek Ziadé (:tarek)]

we can use push_to_try() from tools/tryselect/push.py and put all the options in try_task_config.json

and passing perftest parameters using the v2 schema https://searchfox.org/mozilla-central/source/taskcluster/taskgraph/decision.py#141

this will use a task that will rebuild the mach perftest call

Comment 2

[Tarek Ziadé (:tarek)]

CLI examples

push on g5:

./mach perftest --android --android-app-name org.mozilla.fennec_aurora --perfherder-metrics processLaunchToNavStart --android-install-apk fennec_aurora --hooks testing/performance/hooks_applink.py --push-to-try testing/performance/perftest_applink.js

push on pixel 2:

./mach perftest --android --android-app-name org.mozilla.fennec_aurora --perfherder-metrics processLaunchToNavStart --android-install-apk fennec_aurora --hooks testing/performance/hooks_applink.py --push-to-try  --try-platform p2 testing/performance/perftest_applink.js

Comment 3

[Tarek Ziadé (:tarek)]

Attachment: Bug 1635136 - add a --push-to-try option

Work in progress

Comment 4

[Tarek Ziadé (:tarek)]

try run nhttps://treeherder.mozilla.org/#/jobs?repo=try&tier=1%2C2%2C3&revision=6a21cdde831fdc8bb207c0af14088cf0adaa9b94

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/787b530a09fb
add a --push-to-try option r=aerickson

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/787b530a09fb

Comment 7

[Rail Aliiev [:rail]]

Looks like this line always evaluates to True in a boolean context, because is_linux becomes 'linux-bitbar' when worker['os'] != 'linux'

Comment 8

[Rail Aliiev [:rail]]

In other words it should be:
is_linux = worker['os'] == 'linux' or worker['os'] == 'linux-bitbar'