Remove macosx-universal mozconfigs and mozharness configs

RESOLVED FIXED

Status

RESOLVED FIXED
2 years ago
6 months ago

People

(Reporter: mshal, Assigned: mshal)

Tracking

unspecified

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years 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

2 years 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

2 years 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

2 years 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

2 years 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

2 years 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

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

Comment 9

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