Closed Bug 756995 Opened 13 years ago Closed 13 years ago

Linux and Windows Opt unit tests not running on comm-esr10 branch

Categories

(Release Engineering :: General, defect)

defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The opt unit tests aren't running on the comm-esr10 branch. I took a look through the configs but couldn't see anything obvious. From what I can tell: - sendchange gets sent from opt builds - there are opt unit test builders specifically for comm-esr10 - no pending builds get queued or listed - The config for comm-esr10 looks the same as Firefox. I couldn't see any specific overrides. coop, can you take a look? We're going to need this for the next esr release, i.e. we'll want this set up before John gets back.
Blocks: 698843
It appears I was slightly wrong. We do get opt unit tests on Mac, but not on Linux or Windows.
Summary: Opt unit tests not running on comm-esr10 branch → Linux and Windows Opt unit tests not running on comm-esr10 branch
Attached patch The fixSplinter Review
I believe this should fix it - once I realised Mac was the only one working, that then leads to something different about Linux & Windows. I re-checked the sendchange that the opt builders were generating and it had "pgo" in the branch name. As we're not doing pgo on any of our branches, a bit of tracing lead to pgo_strategy being the issue. There's still some things about enable_pgo = True, but they are on branches that are currently working. So I'll just file a follow-up bug on those and removal can be tested on staging before we change those.
Assignee: nobody → mbanner
Status: NEW → ASSIGNED
Attachment #625971 - Flags: review?(coop)
Attachment #625971 - Flags: review?(coop) → review+
This landed on production yesterday and opt tests are being trigged just fine now.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: