Closed Bug 1311199 Opened 3 years ago Closed 3 years ago

Windows builds fail on incubator/stylo repo with 'stage_server' KeyError in query_mach_build_env

Categories

(Firefox Build System :: General, defect, P1)

Unspecified
Windows
defect

Tracking

(firefox52 fixed)

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: cpeterson, Assigned: manishearth)

References

Details

Attachments

(1 file)

The incubator/stylo repo [1] is unable to build Firefox on Windows due to a build script error about where `stage_server` is not set [2]. incubator/stylo's Linux, Mac, and Android builds work just fine. I see comments in the build scripts that "stage_server is dictated from build_pool_specifics.py".

[1] https://treeherder.mozilla.org/#/jobs?repo=stylo&selectedJob=16240
[2] https://treeherder.mozilla.org/logviewer.html#?job_id=16267&repo=stylo#L576

18:14:23    FATAL - Uncaught exception: Traceback (most recent call last):
18:14:23    FATAL -   File "Z:\task_1476293750\build\src\testing\mozharness\mozharness\base\script.py", line 2033, in run
18:14:23    FATAL -     self.run_action(action)
18:14:23    FATAL -   File "Z:\task_1476293750\build\src\testing\mozharness\mozharness\base\script.py", line 1972, in run_action
18:14:23    FATAL -     self._possibly_run_method(method_name, error_if_missing=True)
18:14:23    FATAL -   File "Z:\task_1476293750\build\src\testing\mozharness\mozharness\base\script.py", line 1912, in _possibly_run_method
18:14:23    FATAL -     return getattr(self, method_name)()
18:14:23    FATAL -   File "Z:\task_1476293750\build\src\testing\mozharness\mozharness\mozilla\building\buildbase.py", line 1618, in build
18:14:23    FATAL -     env.update(self.query_mach_build_env())
18:14:23    FATAL -   File "Z:\task_1476293750\build\src\testing\mozharness\mozharness\mozilla\building\buildbase.py", line 918, in query_mach_build_env
18:14:23    FATAL -     'stage_server': c['stage_server']
18:14:23    FATAL - KeyError: 'stage_server'
18:14:23    FATAL - Running post_fatal callback...
18:14:23 ERROR - setting return code to 2 because fatal was called
The taskcluster windows build config is missing 'stage_server'; I'm not sure why. catlee, garndt, any ideas?
Flags: needinfo?(garndt)
Flags: needinfo?(catlee)
I believe this is failing because the stylo branch isn't listed in branch_specifics.py (https://dxr.mozilla.org/mozilla-central/source/testing/mozharness/configs/builds/branch_specifics.py)

That code doesn't make sense in taskcluster though, so the proper fix is probably to modify buildbase.py so that it stops assuming these config values exist
https://dxr.mozilla.org/mozilla-central/source/testing/mozharness/mozharness/mozilla/building/buildbase.py?q=buildbase.py&redirect_type=direct#908
Flags: needinfo?(catlee)
removing ni? looks like catlee has pointed things in the right direction
Flags: needinfo?(garndt)
Cameron, can you take a look at this bug? catlee described the problem and fix (to the mozharness script) in comment 2. IIUC, mozharness should be fixed in mozilla-central, not just patched in the incubator/stylo repo.
Flags: needinfo?(cam)
Manish, catlee described this Windows build problem and fix (to the mozharness script) in comment 2. IIUC, mozharness should be fixed in mozilla-central, not just patched in the incubator/stylo repo.
Flags: needinfo?(cam) → needinfo?(manishearth)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef1ce8b7619407049aea8b84a6cb69c7791c3039&selectedJob=30466826 with current patch. Build isn't finished yet, but it's gotten past the step where it was stuck and it's now running the actual build.
Assignee: nobody → manishearth
Flags: needinfo?(manishearth)
Comment on attachment 8807655 [details]
Bug 1311199 - Don't assume `stage_server` config properties exist when creating the mach build environment;

https://reviewboard.mozilla.org/r/90750/#review90434
Attachment #8807655 - Flags: review?(catlee) → review+
Pushed by manishearth@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5c253ee2e5bc
Don't assume `stage_server` config properties exist when creating the mach build environment; r=catlee
OS: Unspecified → Windows
Priority: -- → P1
https://hg.mozilla.org/mozilla-central/rev/5c253ee2e5bc
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.