Closed Bug 1409170 Opened 7 years ago Closed 7 years ago

use official balrog stage environment for staging releases

Categories

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

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

References

Details

Attachments

(3 files, 1 obsolete file)

We'll need to switch a few things:
- Any references to the public endpoint (should be aus4.stage.mozaws.net)
- Any references to the admin endpoint (should be balrog-admin.stage.mozaws.net)
- Any references to balrogVPNProxy (should be balrogStageVPNProxy)

Any tasks that use the proxy should continue to go through "http://balrog/api" to get at the stage admin host.
Attachment #8919037 - Flags: review?(rail)
Comment on attachment 8919037 [details] [diff] [review]
pass along balrog_vpn_proxy in release runner

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

This looks good but is not enough. The `releasetasks_graph_gen.py` only takes care of:

* second graphs releases (like graphs we submit in the second week due to 5-days expiration)
* all Fennec releases (which are triggered by release-runner2.py which uses all these release-configs)

So we need to make more changes, to cover the "normal" releases. This happens because, once you amend the templates by introducing a new variable (e.g. `balrog_vpn_proxy`), you need to make sure it's present in bb-configs as well.

1) for production releases under https://hg.mozilla.org/build/buildbot-configs/file/default/mozilla/config.py (well, in this particular case, it's in fact https://dxr.mozilla.org/build-central/source/buildbot-configs/mozilla/production_config.py#50)

2) for staging releases under https://dxr.mozilla.org/build-central/source/buildbot-configs/mozilla/project_branches.py#416

Release runner consumes the info from bbconfigs and passes them to the jinja templates. So it needs to know of these beforehand. There are a few places to change:

a) add it here - see sample https://dxr.mozilla.org/build-central/source/tools/buildfarm/release/release-runner.py#481

b) (you already did) add it here - see sample https://dxr.mozilla.org/build-central/source/tools/buildfarm/release/releasetasks_graph_gen.py#52

c) add it here - both in the kwargs and in the dict within it - https://dxr.mozilla.org/build-central/source/tools/lib/python/kickoff/__init__.py#276
Attachment #8919037 - Flags: review?(rail) → review-
Comment on attachment 8919036 [details] [review]
parameterize balrog vpn proxy in releasetasks

Made some comments in the PR. Basically, we can't merge this unless we land first the other tools/bbconfigs patches.
Attachment #8919036 - Flags: review?(rail) → review+
Attachment #8919037 - Attachment is obsolete: true
I only ended up overriding this for jamun for now to avoid potentially messing up any work that's going on maple -- we should switch that too, once we're sure this works, and the change won't be disruptive to ongoing work there.
Attachment #8919330 - Flags: review?(mtabara)
Attachment #8919329 - Flags: review?(mtabara) → review+
Attachment #8919330 - Flags: review?(mtabara) → review+
(In reply to Ben Hearsum (:bhearsum) from comment #6)
> Created attachment 8919330 [details] [diff] [review]
> add balrog_vpn_proxy to buildbot-configs
> 
> I only ended up overriding this for jamun for now to avoid potentially
> messing up any work that's going on maple -- we should switch that too, once
> we're sure this works, and the change won't be disruptive to ongoing work
> there.

Sgtm.
Attachment #8919330 - Flags: checked-in+
Attachment #8919329 - Flags: checked-in+
Comment on attachment 8919036 [details] [review]
parameterize balrog vpn proxy in releasetasks

https://github.com/mozilla-releng/releasetasks/commit/3e6a09ae79d4dd8907cc449aba8309fd81b27113

+ deplyed in both bm83 and bm85.
Attachment #8919036 - Flags: checked-in+
As far as I know this is done now - we've had multiple staging releases succesfully use balrog stage.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: