Commit Graph

579 Commits

Author SHA1 Message Date
Marian Raiciof
e915869171 Bug 1614275 - Don't report "no raptor test results were found" if page load tests time out r=perftest-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D63887

--HG--
extra : moz-landing-system : lando
2020-02-28 17:02:56 +00:00
Florin Strugariu
e3573630c8 Bug 1617561 - Raptor benchmark tests bimodal results r=perftest-reviewers,AlexandruIonescu
Differential Revision: https://phabricator.services.mozilla.com/D63843

--HG--
extra : moz-landing-system : lando
2020-02-27 13:32:42 +00:00
Henrik Skupin
e9234a3491 Bug 1617909 - [raptor] Unify browser handling in webextension. r=perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D64352

--HG--
extra : moz-landing-system : lando
2020-02-27 13:46:57 +00:00
Henrik Skupin
a4f55ce018 Bug 1617909 - [raptor] Change variable declarations to const or let. r=perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D64351

--HG--
extra : moz-landing-system : lando
2020-02-27 13:37:36 +00:00
Henrik Skupin
cc66a7e720 Bug 1617909 - [raptor] Add log level argument to raptorLog method. r=perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D64349

--HG--
extra : moz-landing-system : lando
2020-02-26 20:25:34 +00:00
Henrik Skupin
cecdf86e9f Bug 1617909 - [raptor] Fix Raptor webextension for async code. r=perftest-reviewers,sparky
Various code in the Raptor webextension that is written
asynchronously isn't used with 'await' or at least a
Promise chain. As result there will be overlapping
attempts to send messages to the control server that
can even vary in their order, and causing tests
to fail.

With that patch the amount of extra console logging
will be reduced to only those cases where sending the
message to the control server failed.

Differential Revision: https://phabricator.services.mozilla.com/D64348

--HG--
extra : moz-landing-system : lando
2020-02-27 13:38:19 +00:00
octavian.negru
4124e5bed3 Bug 1616818 - Get rid of bool_from_str method. Use distutils.util.strtobool instead. r=perftest-reviewers,Marian,sparky
Differential Revision: https://phabricator.services.mozilla.com/D63462

--HG--
extra : moz-landing-system : lando
2020-02-27 13:51:05 +00:00
Tarek Ziadé
55c69c1a46 Bug 1602657 - Set appropriate value for post_startup_delay in CI r=perftest-reviewers,Bebe
Differential Revision: https://phabricator.services.mozilla.com/D57550

--HG--
extra : moz-landing-system : lando
2020-02-26 16:24:19 +00:00
Gregory Mierzwinski
f96c310580 Bug 1616065 - Add stability optimizations to browsertime arguments. r=perftest-reviewers,whimboo
This patch adds some stability and comparison optimizations to the Browsertime arguments. It adds a viewport setting to ensure that all browsers we will test on use the same browser size (only affects desktop). The pageLoadStrategy is also set to `none` so that we can use the `pageCompleteCheckStartWait` and `pageCompleteCheckPollTimeout` options. Those options determine how much Selenium polls (and when it starts polling) to check if there are results available.

The size of the browser is also set to a specific size to prevent regressions/improvements (alerts) from occuring when we update Browsertime and the default screen resolution changes. This patch also disables the Browsertime Firefox extension since we don't need it.

Differential Revision: https://phabricator.services.mozilla.com/D63182

--HG--
extra : moz-landing-system : lando
2020-02-25 13:22:45 +00:00
Bogdan Tara
16c6f3244e Backed out changeset 026538af0f95 (bug 1616818) on sparky's request 2020-02-24 18:24:54 +02:00
Gregory Mierzwinski
45affe7b0a Bug 1616332 - Add all fenix pageload tests to raptor-browsertime. r=perftest-reviewers,ahal,AlexandruIonescu
This patch adds all Fenix pageload tests to raptor-browsertime. To handle how the task is specified (specifying run-on-projects by raptor-test) the raptor.py taskgraph transforms had to be modified.

The transform modifications change a couple things. First, the `raptor-subtests` is now the first thing to have it's keyed-by entries handled because it's values are used as keyed-by entries on other fields such as run-on-projects. Once it's values are resolved, we immediately split/clone the task into one per subtest entry (adding a `chunk-number` field that is used for more setup in the next transforms). After this, the rest of the fields have their keyed-by entries resolved.

Differential Revision: https://phabricator.services.mozilla.com/D63222

--HG--
extra : moz-landing-system : lando
2020-02-24 12:57:20 +00:00
octavian.negru
7b3ff520e4 Bug 1616818 - Get rid of bool_from_str method. Use distutils.util.strtobool instead. r=perftest-reviewers,Marian,sparky
Differential Revision: https://phabricator.services.mozilla.com/D63462

--HG--
extra : moz-landing-system : lando
2020-02-24 10:21:06 +00:00
Gregory Mierzwinski
921314da22 Bug 1565324 - Use ADB commands to enable/disable charging in android power tests. r=bc
This patch allows us to use the enable/disable charging commands for power tests, and moves the android power tests onto the primary perf bitbar devices. It also removes the use of the --host argument that was previously required to run power tests. Disabling charging occurs when the device is being setup, and when the device is being setup within raptor (to accomadate different entry points). Enabling charging occurs when the mach command finishes, or during the clean_up stage in raptor in normal, passing executions. In case any errors occur within raptor, and due to the multiple entry points, charging is enabled where those errors are caught as well.

Differential Revision: https://phabricator.services.mozilla.com/D42373

--HG--
extra : moz-landing-system : lando
2020-02-21 13:47:28 +00:00
Marian Raiciof
2a17b95cb1 Bug 1616066 - Remove type checks for str/bool in BrowsertimeResultsHandler. r=perftest-reviewers,octavian_negru
Differential Revision: https://phabricator.services.mozilla.com/D63456

--HG--
extra : moz-landing-system : lando
2020-02-20 14:22:22 +00:00
Nick Alexander
df11faa804 Bug 1607851 - Bump browsertime version to sitespeedio/browsertime@v8.0.1+. r=tarek
This patch upgrades the major browsertime version used in-tree from 4 to 8 (including some additional fixes to fix some failing tests on our end).

We also add the node v10 requirement in this patch. Also, there were some changes in the browsertime repo's visualmetrics.py script that made it necessary to change where we find the file.

Differential Revision: https://phabricator.services.mozilla.com/D59235

--HG--
extra : moz-landing-system : lando
2020-02-17 20:20:18 +00:00
Marian Raiciof
0f0e9aa0b5 Bug 1615559 - Perma browsertime AttributeError: 'bool' object has no attribute 'lower' r=perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D63068

--HG--
extra : moz-landing-system : lando
2020-02-17 14:38:45 +00:00
octavian.negru
41750e345f Bug 1532315 - Allow Raptor test INI settings to use 'False/false' or 'True/true' r=perftest-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D62563

--HG--
extra : moz-landing-system : lando
2020-02-14 08:28:40 +00:00
Marian Raiciof
526482fc87 Bug 1615062 - Youtube playback tests set "false" for "lowerIsBetter" of subtests r=perftest-reviewers,whimboo,AlexandruIonescu
Differential Revision: https://phabricator.services.mozilla.com/D62729

--HG--
extra : moz-landing-system : lando
2020-02-13 13:18:47 +00:00
alexandru.ionescu
db9a041cfb Bug 1599099 - Update raptor test descriptions r=sparky,octavian_negru,perftest-reviewers
This patch adds the initial set of documentation that will be added to the source-docs website.

Differential Revision: https://phabricator.services.mozilla.com/D60241

--HG--
extra : moz-landing-system : lando
2020-02-12 07:00:43 +00:00
alexandru.ionescu
5b6ff5a709 Bug 1599099 - Generate in-tree perfdocs using ./mach lint --fix. r=octavian_negru,sparky
This patch adds the PerfDocs Generator. Some modificiations to `run_perfdocs` were also made because verification always has to occur before the documentation generation, and generation always has to be run after verification (if verification passes).

Differential Revision: https://phabricator.services.mozilla.com/D57111

--HG--
extra : moz-landing-system : lando
2020-02-12 07:00:15 +00:00
Florin Strugariu
6c2ef85237 Bug 1614282 - [Mozproxy] Add retry and custom error message to download_file_from_url r=whimboo,tarek
Differential Revision: https://phabricator.services.mozilla.com/D62201

--HG--
extra : moz-landing-system : lando
2020-02-11 21:04:04 +00:00
Barret Rennie
d181041431 Bug 1613743 - Merge application.json and jobs.json r=tarek,perftest-reviewers,sparky
Raptor was previously generating two artifacts for the `run-visual-metrics.py`
script to consume: `jobs.json` and `application.json`. These artifacts have
been merged.

Differential Revision: https://phabricator.services.mozilla.com/D62363

--HG--
extra : moz-landing-system : lando
2020-02-11 20:03:30 +00:00
Barret Rennie
d9766a8b08 Bug 1612005 - Do not specifiy individual videos in jobs.json r=tarek,perftest-reviewers,sparky
The `run-visual-metrics.py` script was intended to consume a `jobs.json` file
containing one `browsertime.json` per video. However it was not being used as
such and was continuously re-processing the first video specified in the
`browsertime.json` file. If a job were submitted with a `browsertime.json`
containing 15 videos and 15 different videos, only the first would be
processed. This leads to us having incorrect metrics because over all runs all
the metrics will be identical.

Now we only specify the `browsertime.json` in the `jobs.json` file and extract
the paths to videos from there. Also because we are never downloading inputs
this way, we get to remove some dead code and our dependency on `requests`.

Differential Revision: https://phabricator.services.mozilla.com/D62320

--HG--
extra : moz-landing-system : lando
2020-02-11 19:59:58 +00:00
Gregory Mierzwinski
b5685da087 Bug 1614504 - Remove blank lines from profile before passing it to browsertime. r=perftest-reviewers,octavian_negru
With this patch, we now remove all blank lines from the profiles we produce which cause Browsertime 8 to break.

Differential Revision: https://phabricator.services.mozilla.com/D62370

--HG--
extra : moz-landing-system : lando
2020-02-11 13:18:33 +00:00
Florin Strugariu
90a395b2ab Bug 1585022 - Add support to raptor-browsertime for running the ares6 benchmark on Firefox desktop r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D61414

--HG--
extra : moz-landing-system : lando
2020-02-10 08:13:19 +00:00
Florin Strugariu
f4def64e94 Bug 1613938 - Raptor tests hang when downloading conditioned profiles r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D62053

--HG--
extra : moz-landing-system : lando
2020-02-07 16:38:05 +00:00
Geoff Brown
3161788df7 Bug 1603499 - Report java exceptions from minidump .extra files; r=bc
Stop using mozcrash.check_for_java_exception(), which scans logcat for uncaught
exceptions. Instead, check for java_stack in the extra dump information during
normal crash reporting; if java_stack is present, display the java exception and
stack instead of dumping a full native crash report.

Differential Revision: https://phabricator.services.mozilla.com/D61836

--HG--
extra : moz-landing-system : lando
2020-02-07 16:05:52 +00:00
Marian Raiciof
6d2d6bf95e Bug 1612375 - Run raptor youtube playback tests on fennec68 r=perftest-reviewers,alexandru.irimovici,AlexandruIonescu
Differential Revision: https://phabricator.services.mozilla.com/D61313

--HG--
extra : moz-landing-system : lando
2020-02-06 13:58:11 +00:00
Florin Strugariu
3f74e6d794 Bug 1613332 - Add log messages around raptor conditioned profiles r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D61663

--HG--
extra : moz-landing-system : lando
2020-02-05 14:01:50 +00:00
Florin Strugariu
917f420412 Bug 1601009 - Add support to raptor-browsertime for running the speedometer benchmark on Firefox android r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D61305

--HG--
extra : moz-landing-system : lando
2020-02-05 08:59:19 +00:00
Cosmin Sabou
16a28d6c06 Backed out changeset c1b764e0ddb3 (bug 1601009) for massive raptor failures.
CLOSED TREE
2020-02-04 12:17:59 +02:00
Florin Strugariu
049e26b384 Bug 1601009 - Add support to raptor-browsertime for running the speedometer benchmark on Firefox android r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D61305

--HG--
extra : moz-landing-system : lando
2020-02-03 16:26:21 +00:00
Florin Strugariu
cb537c4f9f Bug 1611924 - raptor remove_raptor_webext does not reset the webext value after is closed r=perftest-reviewers,octavian_negru
Differential Revision: https://phabricator.services.mozilla.com/D61170

--HG--
extra : moz-landing-system : lando
2020-01-28 13:45:28 +00:00
Florin Strugariu
f1633f6bb4 Bug 1611781 - Raptor tests not showing mozproxy error messages r=perftest-reviewers,Marian,octavian_negru
Differential Revision: https://phabricator.services.mozilla.com/D61112

--HG--
extra : moz-landing-system : lando
2020-01-27 12:14:35 +00:00
Florin Strugariu
427a899017 Bug 1611110 - [Raptor] Do not check if runnier is started in dektop run_test_teardown r=perftest-reviewers,tarek,AlexandruIonescu
Differential Revision: https://phabricator.services.mozilla.com/D60816

--HG--
extra : moz-landing-system : lando
2020-01-27 10:21:34 +00:00
Marian Raiciof
f72ab8b7fe Bug 1553131 - Run raptor-youtube-playback jobs with profiler enabled in CI r=perftest-reviewers,sparky,tarek
Differential Revision: https://phabricator.services.mozilla.com/D42516

--HG--
extra : moz-landing-system : lando
2020-01-21 16:54:29 +00:00
Florin Strugariu
007caefd70 Bug 1609392 - Split Raptor and Browsertime classes from raptor.py r=tarek,AlexandruIonescu,octavian_negru,sparky,perftest-reviewers,alexandru.irimovici
Differential Revision: https://phabricator.services.mozilla.com/D60028

--HG--
rename : testing/raptor/raptor/raptor.py => testing/raptor/raptor/webextension/base.py
extra : moz-landing-system : lando
2020-01-21 09:07:19 +00:00
Tarek Ziadé
600d034817 Bug 1568092 - don't ignore tooltool_download() exit code and add a retry r=AlexandruIonescu,Bebe
This patch will check the exit code of the ProcessHandler() that
calls the tooltool script, and will also retry when it fails.

Differential Revision: https://phabricator.services.mozilla.com/D60397

--HG--
extra : moz-landing-system : lando
2020-01-21 07:17:05 +00:00
Rob Wood
583e5855cb Bug 1609029 - Fix perma [tier2] ytp raptor-manifest Error r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D59875

--HG--
extra : moz-landing-system : lando
2020-01-14 16:04:49 +00:00
Florin Strugariu
46a00cab18 Bug 1608374 - Split perftest from raptor.py r=AlexandruIonescu,rwood
Differential Revision: https://phabricator.services.mozilla.com/D59489

--HG--
extra : moz-landing-system : lando
2020-01-14 09:48:21 +00:00
Rob Wood
73c92a684c Bug 1607891 - Add fennec (68) support to raptor-browsertime r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D59737

--HG--
extra : moz-landing-system : lando
2020-01-13 20:08:07 +00:00
Tarek Ziadé
15cf5c2afd Bug 1607722 - Permanent speedometer KeyError: 'measure' r=sparky
If we don't have any provided 'measure' value, defaults to all.

Differential Revision: https://phabricator.services.mozilla.com/D59277

--HG--
extra : moz-landing-system : lando
2020-01-09 19:23:40 +00:00
Brindusan Cristian
ec66c3e408 Backed out changeset 80f5c59d5df4 (bug 1602657) as requested by sdonner on slack. CLOSED TREE 2020-01-10 01:01:15 +02:00
Florin Strugariu
ea4c73f19d Bug 1608038 - Split signal_handler from raptor.py r=AlexandruIonescu,perftest-reviewers,octavian_negru,rwood
Differential Revision: https://phabricator.services.mozilla.com/D59287

--HG--
extra : moz-landing-system : lando
2020-01-09 13:52:29 +00:00
Tarek Ziadé
961808bced Bug 1592746 - filter measures in perfherder_data r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D58963

--HG--
extra : moz-landing-system : lando
2020-01-07 16:47:00 +00:00
Marian Raiciof
2a56033519 Bug 1573200 - Add regex support for alert_on field from raptor tests r=perftest-reviewers,rwood
Differential Revision: https://phabricator.services.mozilla.com/D41781

--HG--
extra : moz-landing-system : lando
2020-01-06 18:20:22 +00:00
Stephen Donner
a9bbb507bf Bug 1606767 - Fix permafail on reference browser by using non-conditioned profile. r=rwood,perftest-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D58592

--HG--
extra : moz-landing-system : lando
2020-01-03 19:25:53 +00:00
Rob Wood
78ac064d46 Bug 1606826 - Browsertime android visual metrics job failing to get application name r=perftest-reviewers,stephendonner
Differential Revision: https://phabricator.services.mozilla.com/D58627

--HG--
extra : moz-landing-system : lando
2020-01-03 18:05:46 +00:00
David Major
dcb41e6ebd Bug 1602801 - Mark StyleBench as subtest_lower_is_better r=perftest-reviewers,rwood
Stylebench is patterned after Speedometer and calculates overall score from subtests in the same way. Stylebench should be marked as subtest_lower_is_better just like Speedometer.

Differential Revision: https://phabricator.services.mozilla.com/D56557

--HG--
extra : moz-landing-system : lando
2020-01-02 20:03:24 +00:00
Rob Wood
a1780842dc Bug 1601414 - Add application info json file to browsertime results for vismet jobs r=perftest-reviewers,stephendonner
Differential Revision: https://phabricator.services.mozilla.com/D58404

--HG--
extra : moz-landing-system : lando
2020-01-02 19:53:48 +00:00