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

Bug 1661667 - Disable conditioned profiles in the fenix branch. #17078

Merged
merged 1 commit into from Dec 16, 2020

Conversation

gmierz
Copy link
Contributor

@gmierz gmierz commented Dec 16, 2020

This patch is for disabling conditioned profiles on the Fenix branch to try to fix some of the intermittents we are seeing here. They are disabled in mozilla-central at the moment as well.

@gmierz gmierz requested review from a team as code owners December 16, 2020 17:32
@gmierz
Copy link
Contributor Author

gmierz commented Dec 16, 2020

Try run with condprof disabled: https://treeherder.mozilla.org/jobs?repo=try&tier=1%2C2%2C3&revision=4a27bdbf8013406bdeb3f4a8761afbd03034a18e

It seems to help the failure rate.

@gabrielluong gabrielluong added the needs:review PRs that need to be reviewed label Dec 16, 2020
Copy link
Contributor

@jmaher jmaher left a comment

Choose a reason for hiding this comment

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

thanks for the try push link and this puts things in parity with m-c.

@gabrielluong gabrielluong removed the needs:review PRs that need to be reviewed label Dec 16, 2020
Copy link
Member

@gabrielluong gabrielluong left a comment

Choose a reason for hiding this comment

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

Rubberstamping for Fenix.

@gabrielluong
Copy link
Member

Assuming this is ready to land.

@gabrielluong gabrielluong merged commit 401dd92 into mozilla-mobile:master Dec 16, 2020
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this pull request Aug 4, 2021
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.

None yet

3 participants