Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[TEST] Raptor to Browsertime Migration #8780

Closed
wants to merge 143 commits into from

Conversation

gmierz
Copy link
Contributor

@gmierz gmierz commented Feb 26, 2020

This PR is for testing the migration from Raptor-webext to Raptor-browsertime.

@firefoxci-taskcluster
Copy link

No Taskcluster jobs started for this pull request
The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.

@gmierz
Copy link
Contributor Author

gmierz commented Feb 26, 2020

@mitchhentges, @JohanLorenzo do you know what I might be missing to have the raptor/browsertime tests run from this PR? It looks like I need to be added as a contributor.

@JohanLorenzo
Copy link
Contributor

Hey Gregory! I confirm being a contributor is what you're missing. In the meantime, I can help you out by opening a new PR so you get to see them running.

@gmierz
Copy link
Contributor Author

gmierz commented Feb 27, 2020

Hello @JohanLorenzo! That would be great, thank you. Would I be able to push updates to that PR or will a new PR need to be made for each update I make?

@mcarare mcarare changed the title [TEST] Raptor to Browsertime Migration [TEST] Raptor to Browsertime Migration Mar 2, 2020
@codecov-io
Copy link

codecov-io commented Mar 3, 2020

Codecov Report

Merging #8780 into master will increase coverage by 0.15%.
The diff coverage is 13%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8780      +/-   ##
============================================
+ Coverage     18.74%   18.89%   +0.15%     
- Complexity      470      502      +32     
============================================
  Files           319      330      +11     
  Lines         12831    13061     +230     
  Branches       1691     1727      +36     
============================================
+ Hits           2405     2468      +63     
- Misses        10225    10384     +159     
- Partials        201      209       +8
Impacted Files Coverage Δ Complexity Δ
...kmarks/selectfolder/SelectBookmarkFolderAdapter.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...y/history/viewholders/HistoryListItemViewHolder.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...components/searchengine/CustomSearchEngineStore.kt 36.36% <ø> (ø) 4 <0> (ø) ⬇️
.../mozilla/fenix/library/history/HistoryViewModel.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...pp/src/main/java/org/mozilla/fenix/ext/Fragment.kt 50% <ø> (ø) 0 <0> (ø) ⬇️
...rc/main/java/org/mozilla/fenix/perf/Performance.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...main/java/org/mozilla/fenix/share/ShareFragment.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...g/mozilla/fenix/components/AccountAbnormalities.kt 87.5% <ø> (ø) 9 <0> (ø) ⬇️
...enix/components/searchengine/SearchEngineWriter.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
...illa/fenix/settings/advanced/LocaleSettingsView.kt 0% <ø> (ø) 0 <0> (ø) ⬇️
... and 99 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 314bee2...ef61959. Read the comment docs.

tomprince added a commit to tomprince/fenix that referenced this pull request Mar 4, 2020
…tribute).

The mozilla-central ffmpeg toolchain (used in Bug 1613479/mozilla-mobile#8780) has an
artifact that does not start with `public/build/` (it starts with just
`public/`). To easily support that, we want to set the `toolchain-artfiact`
attribute for that task.

We switched `index_search` to be a job-type in
[taskgraph](https://hg.mozilla.org/ci/taskgraph/rev/932b4cf48cc785c31b03f0b9f2cba654bc67bc00)
which pass through `attributes` from the task definition (and is also more
future proof for adding fenix-specific toolchain tasks). This updates taskgraph
and the consumers of the current toolchain.
tomprince added a commit to tomprince/fenix that referenced this pull request Mar 4, 2020
…tribute).

The mozilla-central ffmpeg toolchain (used in Bug 1613479/mozilla-mobile#8780) has an
artifact that does not start with `public/build/` (it starts with just
`public/`). To easily support that, we want to set the `toolchain-artfiact`
attribute for that task.

We switched `index_search` to be a job-type in
[taskgraph](https://hg.mozilla.org/ci/taskgraph/rev/932b4cf48cc785c31b03f0b9f2cba654bc67bc00)
which pass through `attributes` from the task definition (and is also more
future proof for adding fenix-specific toolchain tasks). This updates taskgraph
and the consumers of the current toolchain.
tomprince added a commit that referenced this pull request Mar 4, 2020
…tribute).

The mozilla-central ffmpeg toolchain (used in Bug 1613479/#8780) has an
artifact that does not start with `public/build/` (it starts with just
`public/`). To easily support that, we want to set the `toolchain-artfiact`
attribute for that task.

We switched `index_search` to be a job-type in
[taskgraph](https://hg.mozilla.org/ci/taskgraph/rev/932b4cf48cc785c31b03f0b9f2cba654bc67bc00)
which pass through `attributes` from the task definition (and is also more
future proof for adding fenix-specific toolchain tasks). This updates taskgraph
and the consumers of the current toolchain.
@@ -18,6 +18,8 @@ only-for-abis:
- arm64-v8a

job-defaults:
attributes:
retrigger: true
dependencies:
geckoview-nightly: geckoview-nightly
linux64-minidump-stackwalk: toolchain-linux64-minidump-stackwalk
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dependency failure the prints seem to be trying to fix can be fixed by removing this line. I think this would be fixed if you rebase.

@gmierz
Copy link
Contributor Author

gmierz commented Mar 17, 2020

Merged in PR#9087.

@gmierz gmierz closed this Mar 17, 2020
severinrudie pushed a commit to severinrudie/fenix that referenced this pull request Mar 18, 2020
…tribute).

The mozilla-central ffmpeg toolchain (used in Bug 1613479/mozilla-mobile#8780) has an
artifact that does not start with `public/build/` (it starts with just
`public/`). To easily support that, we want to set the `toolchain-artfiact`
attribute for that task.

We switched `index_search` to be a job-type in
[taskgraph](https://hg.mozilla.org/ci/taskgraph/rev/932b4cf48cc785c31b03f0b9f2cba654bc67bc00)
which pass through `attributes` from the task definition (and is also more
future proof for adding fenix-specific toolchain tasks). This updates taskgraph
and the consumers of the current toolchain.
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.