Closed
Bug 1129550
Opened 9 years ago
Closed 9 years ago
Stop generating aus2 snippets
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rail, Assigned: rail)
References
Details
Attachments
(7 files, 2 obsolete files)
14.06 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
22.44 KB,
patch
|
rail
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
44.46 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
67.18 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
43.10 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
29.54 KB,
patch
|
jlund
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
4.15 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Extracted from win64 related staging work
Attachment #8559285 -
Flags: review?(bhearsum)
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8559285 [details] [diff] [review] buildbotcustom-no-snippets.diff nm this, it's better to get rid of the code
Attachment #8559285 -
Attachment is obsolete: true
Attachment #8559285 -
Flags: review?(bhearsum)
Comment 3•9 years ago
|
||
See also bug 933426.
Updated•9 years ago
|
Blocks: diesnippets
Comment 4•9 years ago
|
||
Attachment #8559957 -
Flags: review?(rail)
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8559957 [details] [diff] [review] switch update server url + switch off snippet uploading everywhere I don't think this will work. We have a bunch of places in factory.py using this variable. Don't worry about it, this variable will be gone with the patch I'm testing
Attachment #8559957 -
Flags: review?(rail) → review-
Comment 6•9 years ago
|
||
(In reply to Rail Aliiev [:rail] from comment #5) > Comment on attachment 8559957 [details] [diff] [review] > switch update server url + switch off snippet uploading everywhere > > I don't think this will work. We have a bunch of places in factory.py using > this variable. > > Don't worry about it, this variable will be gone with the patch I'm testing I don't think this is true for releases. It's used a lot in nightlies, but only in ReleaseUpdatesFactory for releases AFAICT.
Comment 7•9 years ago
|
||
I verified this with checkconfig, it has some risk, but not a ton...
Attachment #8560520 -
Flags: review?(rail)
Assignee | ||
Updated•9 years ago
|
Attachment #8560520 -
Flags: review?(rail) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Comment on attachment 8559957 [details] [diff] [review] switch update server url + switch off snippet uploading everywhere these 2 patches will work together
Attachment #8559957 -
Flags: review- → review+
Comment 9•9 years ago
|
||
Comment on attachment 8560520 [details] [diff] [review] don't upload snippets + remove dead code Oops, wrong patch.
Attachment #8560520 -
Attachment is obsolete: true
Comment 10•9 years ago
|
||
Attachment #8560522 -
Flags: review?(rail)
Assignee | ||
Comment 11•9 years ago
|
||
Comment on attachment 8560522 [details] [diff] [review] more comprehensive buildbotcustom patch Review of attachment 8560522 [details] [diff] [review]: ----------------------------------------------------------------- looks good to me.
Attachment #8560522 -
Flags: review?(rail) → review+
Updated•9 years ago
|
Attachment #8559957 -
Flags: checked-in+
Updated•9 years ago
|
Attachment #8560522 -
Flags: checked-in+
Comment 12•9 years ago
|
||
Should be all done here. No more snippets for future releases!
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•9 years ago
|
||
I'm going to keep this open to delete moaar code! Not today though :)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 14•9 years ago
|
||
* passed test-masters.sh * the list of builders is the same * dump_masters.sh shows the following deletions for a build master: https://gist.github.com/rail/36005eda2408e130771b
Attachment #8561573 -
Flags: review?(bhearsum)
Assignee | ||
Comment 15•9 years ago
|
||
Attachment #8561575 -
Flags: review?(bhearsum)
Assignee | ||
Comment 16•9 years ago
|
||
Attachment #8561577 -
Flags: review?(bhearsum)
Assignee | ||
Comment 17•9 years ago
|
||
Attachment #8561578 -
Flags: review?(jlund)
Comment 18•9 years ago
|
||
Comment on attachment 8561573 [details] [diff] [review] no_aus-buildbotcustom-3.diff Review of attachment 8561573 [details] [diff] [review]: ----------------------------------------------------------------- I'm r+'ing these patches based on visual inspection + dump master output. I'd really really like to see this tested in staging though - these are pretty risky changes, and they're only validated by nightly builds.
Attachment #8561573 -
Flags: review?(bhearsum) → review+
Updated•9 years ago
|
Attachment #8561575 -
Flags: review?(bhearsum) → review+
Updated•9 years ago
|
Attachment #8561577 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 19•9 years ago
|
||
This is a follow up for attachment 8561573 [details] [diff] [review] to fix the android issue. It turned out that the script I deleted sets some properties in http://hg.mozilla.org/build/tools/file/a7c2695e5ed7/scripts/android/android_snippet.py#l249 calculated by http://hg.mozilla.org/build/tools/file/a7c2695e5ed7/scripts/android/android_snippet.py#l155 I replaced them with in-line and renamed the method to match the other one in factory.py. Android builds look fine now, waiting for windows...
Comment 20•9 years ago
|
||
Comment on attachment 8561578 [details] [diff] [review] no_aus-mozharness.diff Review of attachment 8561578 [details] [diff] [review]: ----------------------------------------------------------------- \o/\o/ long live balrog
Attachment #8561578 -
Flags: review?(jlund) → review+
Assignee | ||
Comment 21•9 years ago
|
||
Comment on attachment 8562448 [details] [diff] [review] no_aus-buildbotcustom-android.diff See comment 19, windows worked fine, both win32 (mozharness based) and win64 (MBF based). I additionally ran second win32 build to make sure it works fine with partials. L10N jobs worked fine too. The only build type I had troubles to test was Android nightly L10N. For some reason it was trying to upload files to stage.m.o.
Attachment #8562448 -
Flags: review?(bhearsum)
Assignee | ||
Comment 22•9 years ago
|
||
If you want to take a look at the staging results, they are still at http://dev-master1.srv.releng.scl3.mozilla.com:8033/builders
Updated•9 years ago
|
Attachment #8562448 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 23•9 years ago
|
||
Comment on attachment 8561578 [details] [diff] [review] no_aus-mozharness.diff https://hg.mozilla.org/build/mozharness/rev/44e2e5f6a0ad
Attachment #8561578 -
Flags: checked-in+
Assignee | ||
Comment 24•9 years ago
|
||
Comment on attachment 8561575 [details] [diff] [review] no_aus-buildbot-configs-1.diff https://hg.mozilla.org/build/buildbot-configs/rev/69ffb581b00c
Attachment #8561575 -
Flags: checked-in+
Assignee | ||
Comment 25•9 years ago
|
||
Comment on attachment 8561573 [details] [diff] [review] no_aus-buildbotcustom-3.diff https://hg.mozilla.org/build/buildbotcustom/rev/0ebc46469595
Attachment #8561573 -
Flags: checked-in+
Assignee | ||
Comment 26•9 years ago
|
||
Comment on attachment 8562448 [details] [diff] [review] no_aus-buildbotcustom-android.diff combined with the first patch https://hg.mozilla.org/build/buildbotcustom/rev/0ebc46469595
Attachment #8562448 -
Flags: checked-in+
Assignee | ||
Comment 27•9 years ago
|
||
The tools patch will be landed after a reconfig (it deletes some files used in production)
Assignee | ||
Comment 28•9 years ago
|
||
Comment on attachment 8561577 [details] [diff] [review] no_aus-tools.diff https://hg.mozilla.org/build/tools/rev/a0ac6a4b2646
Attachment #8561577 -
Flags: checked-in+
Assignee | ||
Comment 29•9 years ago
|
||
No more tears^W snippets!
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•