Closed Bug 1263922 Opened 4 years ago Closed 4 years ago

Adjust buildbot configs for mozilla-release release promotion builds

Categories

(Release Engineering :: Release Automation: Other, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(1 file, 2 obsolete files)

No description provided.
Attached patch bb-configs.diff (obsolete) — Splinter Review
To be landed before merge
Attachment #8740427 - Flags: review?(jlund)
Comment on attachment 8740427 [details] [diff] [review]
bb-configs.diff

Review of attachment 8740427 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with one line addition (see in line comments). 

looks about right from glancing mozilla/config.py[mozilla-beta] and mozilla/project_branches.py[jamun] :)

maybe we should enable this at the end of the week (a few days before merge) like we did with beta

::: mozilla/config.py
@@ +2535,5 @@
>  BRANCHES['mozilla-release']['enable_blocklist_update'] = True
>  BRANCHES['mozilla-release']['enable_valgrind'] = False
>  BRANCHES['mozilla-release']['enabled_products'] = ['firefox', 'mobile']
>  BRANCHES['mozilla-release']['platforms']['macosx64']['platform_objdir'] = "%s/x86_64" % OBJDIR
> +### Release Promotion

I think we will also need https://dxr.mozilla.org/build-central/source/buildbot-configs/mozilla/project_branches.py#305
Attachment #8740427 - Flags: review?(jlund) → review+
Attached patch bb-configs.diff (obsolete) — Splinter Review
Good catch! carry over r+
Attachment #8740427 - Attachment is obsolete: true
Attachment #8740477 - Flags: review+
Need to update the patch to include

BRANCHES['mozilla-release']['updates_builder_enabled'] = True
BRANCHES['mozilla-release']['update_verify_enabled'] = True

from https://reviewboard.mozilla.org/r/45821/diff/1#index_header
Attached patch bb-configs.diffSplinter Review
updated
Attachment #8740926 - Flags: review+
Attachment #8740477 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.