If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove macosx-universal mozconfigs and mozharness configs

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
8 months ago
7 months ago

People

(Reporter: mshal, Assigned: mshal)

Tracking

unspecified

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

8 months ago
According to https://bugzilla.mozilla.org/show_bug.cgi?id=1339182#c9 it should be safe to remove the macosx-universal mozconfigs now. I imagine we can delete the related mozharness configs as well.
Comment hidden (mozreview-request)
(Assignee)

Comment 2

7 months ago
The only remaining file that references 'macosx-universal' in the tree is branch_specifics.py: https://dxr.mozilla.org/mozilla-central/rev/106a96755d3bcebe64bbbc3b521d65d262ba9c02/testing/mozharness/configs/builds/branch_specifics.py#204

Can we remove that too or does it need to stick around so long as esr52 is alive?

Comment 3

7 months ago
(In reply to Michael Shal [:mshal] from comment #2)
> The only remaining file that references 'macosx-universal' in the tree is
> branch_specifics.py:
> https://dxr.mozilla.org/mozilla-central/rev/
> 106a96755d3bcebe64bbbc3b521d65d262ba9c02/testing/mozharness/configs/builds/
> branch_specifics.py#204
> 
> Can we remove that too or does it need to stick around so long as esr52 is
> alive?

iiuc - that can be removed everywhere but mozilla-esr52. branch_specifics.py is read from in-tree so safe to remove from central

Comment 4

7 months ago
mozreview-review
Comment on attachment 8848588 [details]
Bug 1342911 - Remove macosx-universal mozconfigs & mozharness configs;

https://reviewboard.mozilla.org/r/121496/#review124678

I think this is fine to do. these files should be used by their respective trees/repos regardless if it's from bbot or taskcluster. So removing this on central is fine.
Attachment #8848588 - Flags: review?(jlund) → review+
Comment hidden (mozreview-request)
(Assignee)

Comment 6

7 months ago
Comment on attachment 8848588 [details]
Bug 1342911 - Remove macosx-universal mozconfigs & mozharness configs;

Asking for re-review on removing the mozilla-esr52 block.
Attachment #8848588 - Flags: review+ → review?(jlund)

Comment 7

7 months ago
mozreview-review
Comment on attachment 8848588 [details]
Bug 1342911 - Remove macosx-universal mozconfigs & mozharness configs;

https://reviewboard.mozilla.org/r/121496/#review125166
Attachment #8848588 - Flags: review?(jlund) → review+

Comment 8

7 months ago
Pushed by mshal@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/17050f0040ef
Remove macosx-universal mozconfigs & mozharness configs; r=jlund

Comment 9

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/17050f0040ef
Status: NEW → RESOLVED
Last Resolved: 7 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.