Граф коммитов

61 Коммитов

Автор SHA1 Сообщение Дата
xuanqi xu 33b538ca48 Bug 1636534 - Added test for notebook layer and perftestnotebook. r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D77386
2020-06-02 21:35:03 +00:00
xuanqi xu 2b9bf62319 Bug 1636534 - Separate perftestetl from perftestnotebook r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D75684
2020-06-02 21:34:49 +00:00
Tarek Ziadé 7c486e2ce0 Bug 1641834 - started a Sphinx doc r=sparky,acreskey
Initial Sphinx structure with minimal content.
This is mostly to get started in writing doc

Differential Revision: https://phabricator.services.mozilla.com/D77440
2020-06-03 08:17:06 +00:00
Tom Prince 581eba974b Bug 1641971: Pass `mach perftest` options explicitly, rather than inpsect parameters; r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D77548
2020-06-03 02:35:09 +00:00
Narcis Beleuzu 479c45bbea Backed out 7 changesets (bug 1641971) for breaking Gecko Decision Task opt . CLOSED TREE
Backed out changeset a039e60a26d2 (bug 1641971)
Backed out changeset e7d4c490c31a (bug 1641971)
Backed out changeset f2322981b294 (bug 1641971)
Backed out changeset d123b08d2c20 (bug 1641971)
Backed out changeset 57772c0b9b3e (bug 1641971)
Backed out changeset 11a887f73dbb (bug 1641971)
Backed out changeset e9c7a59d4bc5 (bug 1641971)
2020-06-03 05:30:51 +03:00
Tom Prince 11aedca9b2 Bug 1641971: Pass `mach perftest` options explicitly, rather than inpsect parameters; r=tarek
Differential Revision: https://phabricator.services.mozilla.com/D77548
2020-06-02 18:23:07 +00:00
Gregory Mierzwinski 55c95e297a Bug 1637994 - Add a transformer for extracting times/durations from logcat logs. r=acreskey,mozperftest-reviewers,tarek
This patch adds a transformer that is capable of extracting times/entries or durations for logcat logs. Here, we also implement the ability for users to pass options to the transformers so that they can be more generic (through transformer-options). The logcat transformer makes use of this, and also shows how a custom function can be used within transformers. A new test file is added for the transformer, as well as a test in test_perfherder.py so we can test the full pipeline.

Differential Revision: https://phabricator.services.mozilla.com/D77094
2020-05-29 12:40:40 +00:00
Gregory Mierzwinski 115d1b4294 Bug 1641566 - Update in-tree browsertime to latest. r=tarek,perftest-reviewers,Bebe
Differential Revision: https://phabricator.services.mozilla.com/D77257
2020-05-29 12:24:33 +00:00
Tarek Ziadé 3c6bbe84ba Bug 1641549 - Run the metrics layer separately r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D77292
2020-05-29 12:21:52 +00:00
Tarek Ziadé 77da205d14 Bug 1640627 - added android logs options r=sparky
--android-capture-logcat  - captures the logcat
--android-clear-logcat - clears the logcat on start
--android-capture-adb - captures the adb commands

Differential Revision: https://phabricator.services.mozilla.com/D77038
2020-05-28 16:42:37 +00:00
Tarek Ziadé c0c06d6f1f Bug 1641351 - raise on node error in BT r=acreskey
We forgot to check that the node command succeeded.
This patch fixes it.

Differential Revision: https://phabricator.services.mozilla.com/D77148
2020-05-27 21:43:38 +00:00
Tarek Ziadé bddb8551ed Bug 1640747 - Add test-level support for handling exceptions while running the tests r=acreskey
This patch adds the `on_exception` hook, that will let developers
drive the runner behavior when a layer fails.

Differential Revision: https://phabricator.services.mozilla.com/D76794
2020-05-26 13:35:00 +00:00
Tarek Ziadé 92a2c142f7 Bug 1640649 - add --android-timeout r=sparky
This patch surfaces the timeout value for ADBDevice.
It also adds the ability to run a single test with
mach perftest-test and a new --skip-lint option
to skip black/flake8

Differential Revision: https://phabricator.services.mozilla.com/D76791
2020-05-26 14:42:58 +00:00
Tarek Ziadé af3ea56177 Bug 1639321 - increase the coverage to 85 r=sparky
Increase the coverage to 85%

Differential Revision: https://phabricator.services.mozilla.com/D76048
2020-05-26 14:44:14 +00:00
Gregory Mierzwinski f94f60a9c2 Bug 1640171 - Make the test file a str before using it. r=mozperftest-reviewers,tarek
This patch fixes a bug where a PosixPath is attempted to be used rather than a string path. There's also a regression in the coverage test run in `./mach perftest-test` on some platforms that is fixed by this patch (the attrs module being used by pytest is too old, so we update during setup). Lastly, a regression test is added to the browsertime tests to ensure that the command only contains knowingly compatible types.

Depends on D76713

Differential Revision: https://phabricator.services.mozilla.com/D76599
2020-05-25 20:07:58 +00:00
Tarek Ziadé d634c1741b Bug 1639493 - improved README r=acreskey
Added some minimal doc

Differential Revision: https://phabricator.services.mozilla.com/D76107
2020-05-20 13:12:15 +00:00
Yue Zhang d8d83da843 Bug 1636535 - Remove subclass requirement for custom data transformers. r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D75394
2020-05-20 19:48:58 +00:00
Tarek Ziadé f8d5627e71 Bug 1639212 - allow URLs for hooks and scripts r=sparky
This patch allows using URLS for --hooks and scripts

Differential Revision: https://phabricator.services.mozilla.com/D75967
2020-05-19 20:27:27 +00:00
Gregory Mierzwinski 7788d4b576 Bug 1639211 - Implement --perfherder-app in mozperftest Perfherder layer. r=mozperftest-reviewers,tarek
This patch adds a new --perfherder-app that can be used to specify the shorthand name of the application being tested. The choices here are limited to what the perfherder schema can accept. We also add these settings to the existing taskcluster tasks to split the perfherder data. A couple tests are also added for this new flag.

Differential Revision: https://phabricator.services.mozilla.com/D76010
2020-05-19 19:07:27 +00:00
Gregory Mierzwinski 99953d32a0 Bug 1638001 - Add test name from metadata to perfherder data. r=acreskey,mozperftest-reviewers,tarek
This patch adds a `test_name` field to the test metadata that can be used to change the name of the test when output in the perfherder format. It also implements this for the app-link tests.

Differential Revision: https://phabricator.services.mozilla.com/D75544
2020-05-19 19:52:52 +00:00
Tarek Ziadé 7d6014b5a8 Bug 1635573 - vendor coverage r=ahal
Vendor coverage, and make sure we can run out of third_party as much as possible

Differential Revision: https://phabricator.services.mozilla.com/D74531
2020-05-19 15:04:28 +00:00
Tarek Ziadé ffc7bb2c00 Bug 1637959 - Added more test coverage r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D75309
2020-05-15 07:19:39 +00:00
Tarek Ziadé e5c6044cc3 Bug 1636179 - switch mpu to perftest-test r=sparky
This patch will use perftest-test on any code change in python/mozperftest

Differential Revision: https://phabricator.services.mozilla.com/D75080
2020-05-14 13:01:59 +00:00
Tarek Ziadé c84c8756b8 Bug 1637231 - add applink tests r=acreskey
Adds tests for applink for gve/fenix on p2/g5

Differential Revision: https://phabricator.services.mozilla.com/D75097
2020-05-14 13:01:59 +00:00
Yue Zhang 740c6c3ab7 Bug 1630610 - Add tests for transformers/single_json.py. r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D74554
2020-05-13 00:29:36 +00:00
Yue Zhang 44c0a36b80 Bug 1630610 - Add tests for perftestnotebook.py and transformer.py. r=sparky
Depends on D73926

Differential Revision: https://phabricator.services.mozilla.com/D73927
2020-05-13 12:08:40 +00:00
Yue Zhang a8fc57d316 Bug 1630610 - Add setup function for pytest. r=sparky
Depends on D73154

Differential Revision: https://phabricator.services.mozilla.com/D73926
2020-05-13 00:29:16 +00:00
Yue Zhang 118aa2368b Bug 1630610 - Fix broken code. r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D73154
2020-05-13 00:29:08 +00:00
Tarek Ziadé 734633e6c5 Bug 1637315 - Plug flake8 r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D74886
2020-05-12 16:57:27 +00:00
Tarek Ziadé 080870795e Bug 1635136 - add a --push-to-try option r=aerickson
This patch is adding an option to push a perftest run in the CI.

It's based on :
- sparse profiles
- push_to_try
- options passed through try_task_config.json

Differential Revision: https://phabricator.services.mozilla.com/D74115
2020-05-12 21:19:48 +00:00
Tarek Ziadé c1ea70b2d6 Bug 1636871 - allow URLs and aliases for --android-install-apk r=sparky
This patch let us use urls for that option

Differential Revision: https://phabricator.services.mozilla.com/D74629
2020-05-12 17:01:17 +00:00
Gregory Mierzwinski b563884d06 Bug 1637303 - Add perfherder data validation to mozperftest. r=mozperftest-reviewers,tarek
This patch enables perfherder data validation with jsonschema and also adds some extra tests for the Perfherder layer.

Differential Revision: https://phabricator.services.mozilla.com/D74877
2020-05-12 18:06:15 +00:00
Andrew Creskey 5276336dcf Bug 1634516 Extend the android applink tests to cover all variations of fenix r=tarek,sparky
Adds support for all variations of fenix (app name and activity passed by argument to mach perfest)
Fixes bug in mach perftest argument passing (android-activity was not being set)

Differential Revision: https://phabricator.services.mozilla.com/D73334
2020-05-11 14:30:01 +00:00
Tarek Ziadé fa240018c6 Bug 1635546 - deactivate test under macOS r=sparky
until we have Bug 1635573

Differential Revision: https://phabricator.services.mozilla.com/D73943
2020-05-05 21:29:00 +00:00
Tarek Ziadé 9a003b79ba Bug 1635546 - fixed dependencies r=sparky
We need to set up the notebook deps.
The train of patch re-activated some
tests that required the dependencies used
by the notebook, which were not added in the
path.

Differential Revision: https://phabricator.services.mozilla.com/D73936
2020-05-05 20:35:28 +00:00
Tarek Ziadé a44d97ff35 Bug 1635389 - clean the code r=sparky
We missed a pass on black, and python.ini was incomplete

Differential Revision: https://phabricator.services.mozilla.com/D73822
2020-05-05 14:47:58 +00:00
Tarek Ziadé 7517317c35 Bug 1635076 - add a custom test runner r=sparky
We're running black+coverage

Differential Revision: https://phabricator.services.mozilla.com/D73693
2020-05-05 15:00:27 +00:00
Tarek Ziadé bacdbf01c7 Bug 1635075 - add --browsertime-geckodriver r=sparky
Adds an option to point geckodriver

Differential Revision: https://phabricator.services.mozilla.com/D73617
2020-05-04 18:12:59 +00:00
Gregory Mierzwinski 2a62f49c9a Bug 1630665 - Implement new browser intermediate results standard. r=tarek
This patch implements the new intermediate results standard and adds the mechanisms required to handle it. Results validation is done with jsonschema and some manual validation (because of some unfortunate issues with jsonschema) and some tests were implemented to ensure that we fail/pass where expected. The metrics modules were modified to handle multiple suites.

One thing that is disabled in this patch is the subtest/single-metric specifications through the "results" field. We'll do one thing at a time here and we also have no use for subtests yet (although we definitely will).

Differential Revision: https://phabricator.services.mozilla.com/D72067
2020-05-04 16:28:27 +00:00
Tarek Ziadé 0e14df854f Bug 1634359 - improve coverage r=sparky
Adds more coverage

Differential Revision: https://phabricator.services.mozilla.com/D73253
2020-04-30 16:13:22 +00:00
Tarek Ziadé f738edb6cd Bug 1634349 - follow up patch r=sparky
Fix a regression on macOs@CI

Differential Revision: https://phabricator.services.mozilla.com/D73317
2020-04-30 18:07:58 +00:00
Tarek Ziadé 00444bef79 Bug 1634349 - Split a pure Python runner r=sparky
In order to be able to run outside mach, we want to split the runner so we can
call it directly from Python.

Differential Revision: https://phabricator.services.mozilla.com/D73252
2020-04-30 14:32:56 +00:00
Tarek Ziadé 56bca0b81a Bug 1633875 - pick the depencendy in third party r=Bebe
This patch will use the dependency that is in the third_party/python/ dir

Differential Revision: https://phabricator.services.mozilla.com/D73016
2020-04-29 09:26:34 +00:00
Tarek Ziadé 33f1eee4d5 Bug 1633437 - Support for test metadata r=acreskey
This patch adds support for tests metadata. A test script parser is added as
well as a new "doc" flavor that can be used to display the script info in the
command line. This parser will be the basis for building automated docs and
scripts verifications if we want to do this.

Differential Revision: https://phabricator.services.mozilla.com/D72800
2020-04-28 17:07:14 +00:00
Tarek Ziadé 89f357d5e0 Bug 1632780 - improve test coverage r=sparky
Added test coverage

Differential Revision: https://phabricator.services.mozilla.com/D72329
2020-04-27 15:40:58 +00:00
Tarek Ziadé b1ad9498a5 Bug 1632404 - added missing error logger r=sparky
The android layer calls layer.error(), let's make sure it's
really there :)

Differential Revision: https://phabricator.services.mozilla.com/D72102
2020-04-23 21:46:49 +00:00
Tarek Ziadé 92eb19eb28 Bug 1632402 - added --browsertime-iterations r=sparky
Added the --browsertime-iterations option

Differential Revision: https://phabricator.services.mozilla.com/D72101
2020-04-23 19:40:56 +00:00
Tarek Ziadé b6d9b10181 Bug 1632267 - Clean up the android layer r=acreskey
We're back to an explicit --android option and --android-app-name, the layer won't use the --browser-binary option as an implicit
fallback anymore. This is cleaner.

Differential Revision: https://phabricator.services.mozilla.com/D72031
2020-04-23 14:33:32 +00:00
Tarek Ziadé 37c5e7f665 Bug 1632192 - create a specific test job for mozperftest r=sparky
Differential Revision: https://phabricator.services.mozilla.com/D71987
2020-04-22 17:13:27 +00:00
Tarek Ziadé 43f8df442e Bug 1632075 - split the browsertime layer into manageable chunks r=sparky
Started to split the browsertime layer into logical modules

Differential Revision: https://phabricator.services.mozilla.com/D71906
2020-04-22 14:21:16 +00:00