Closed
Bug 1119997
Opened 11 years ago
Closed 11 years ago
Stop using mozilla-centos-i386 mock
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files)
6.76 KB,
patch
|
catlee
:
review+
|
Details | Diff | Splinter Review |
901 bytes,
patch
|
massimo
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #947462 +++
The last places to use it are:
in m-c:
b2g/config/hamachi/config.json
b2g/config/hamachi/limited-memory-config.json
b2g/config/helix/config.json
b2g/config/wasabi/config.json
in mozharness:
configs/single_locale/linux.py
Assignee | ||
Comment 1•11 years ago
|
||
Presumably, this should work.
For whoever comes first.
Assignee: nobody → mh+mozilla
Attachment #8546910 -
Flags: review?(rail)
Attachment #8546910 -
Flags: review?(mshal)
Attachment #8546910 -
Flags: review?(catlee)
Attachment #8546910 -
Flags: review?(bhearsum)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8546911 -
Flags: review?(catlee)
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #2)
> Created attachment 8546911 [details] [diff] [review]
> mozharness part
Note that the package list already has the i686 packages listed, interestingly.
Comment 4•11 years ago
|
||
Comment on attachment 8546910 [details] [diff] [review]
0001-Bug-1119997-Stop-using-mozilla-centos-i386-mock.patch
Review of attachment 8546910 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm
Attachment #8546910 -
Flags: review?(rail)
Attachment #8546910 -
Flags: review?(mshal)
Attachment #8546910 -
Flags: review?(catlee)
Attachment #8546910 -
Flags: review?(bhearsum)
Attachment #8546910 -
Flags: review+
Assignee | ||
Comment 5•11 years ago
|
||
Keywords: leave-open
Assignee | ||
Comment 6•11 years ago
|
||
Comment 8•11 years ago
|
||
Comment on attachment 8546911 [details] [diff] [review]
mozharness part
I'll defer to Massimo for this one, since he's working on the repacks.
Attachment #8546911 -
Flags: review?(catlee) → review?(mgervasini)
Updated•11 years ago
|
Attachment #8546911 -
Flags: review?(mgervasini) → review+
Assignee | ||
Comment 9•11 years ago
|
||
Can this be tested before landing, or should I land and we'll see what happens?
Flags: needinfo?(mgervasini)
Comment 10•11 years ago
|
||
I have tested the mozharness changes on the staging environment, the mock setup step was successful.
Flags: needinfo?(mgervasini)
Assignee | ||
Comment 11•11 years ago
|
||
(In reply to Massimo Gervasini [:mgerva] from comment #10)
> I have tested the mozharness changes on the staging environment, the mock
> setup step was successful.
The more interesting thing is whether what happens in that mock environment works.
Flags: needinfo?(mgervasini)
Comment 12•11 years ago
|
||
Yes, I can confirm that the mock environment works too.
Flags: needinfo?(mgervasini)
Assignee | ||
Comment 13•11 years ago
|
||
(In reply to Massimo Gervasini [:mgerva] from comment #12)
> Yes, I can confirm that the mock environment works too.
Let me rephrase: do those l10n jobs that run with that particular mozharness setup still work with this change?
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(mgervasini)
Assignee | ||
Comment 15•11 years ago
|
||
https://hg.mozilla.org/build/mozharness/rev/fdcf9076afec
Leaving open until this is merged on the production branch.
Keywords: leave-open
Comment 16•11 years ago
|
||
In production: https://hg.mozilla.org/build/mozharness/rev/fdcf9076afec
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•