Closed Bug 656459 Opened 11 years ago Closed 11 years ago

use build/tools for staging buildbot-configs

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aki, Assigned: aki)

Details

Attachments

(3 files)

I'd like to also 

    tools_repo = '%s%s' % (branchConfig['hgurl'],
                           releaseConfig.get('tools_repo_path', 
                               branchConfig['build_tools_repo_path']))

in buildbotcustom.process.release.generateReleaseBranchObjects, so we can override the tools_repo_path back to stage-ffxbld/tools for staging releases.
The upside to this is that both "normal" staging nightly/depend builds and "normal" staging release builds work correctly out of the box.

The downside might be if someone wants to change the build/tools repo to their own user repo, and does a staging release and forgets that it's set to stage-ffxbld/tools in the staging release config.
Attachment #531777 - Flags: review?(rail)
Attachment #531776 - Flags: review?(rail) → review+
Attachment #531777 - Flags: review?(rail) → review+
(In reply to comment #5)
> Thanks for doing this Aki! Could you update
> https://wiki.mozilla.org/Release:Release_Automation_on_Mercurial:
> Documentation#Buildbot_configs when this lands, too?

Done.

Though if we're changing repo paths to use user repos on every staging release, suddenly the staging release build_tools_repo_path becomes an extra step rather than a convenience.

We can remove those if people don't want it ?
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
(In reply to comment #6)
> (In reply to comment #5)
> > Thanks for doing this Aki! Could you update
> > https://wiki.mozilla.org/Release:Release_Automation_on_Mercurial:
> > Documentation#Buildbot_configs when this lands, too?
> 
> Done.
> 
> Though if we're changing repo paths to use user repos on every staging
> release, suddenly the staging release build_tools_repo_path becomes an extra
> step rather than a convenience.
> 
> We can remove those if people don't want it ?

As it stands now, the staging configs are set-up to work OOB in stage-ffxbld, so I think this is good. If we decide to change them to be more OOB friendly to personal user repositories, we can change this at that point.

re: your documentation edit, I still see build_tools_repo_path being modified in the staging_config.py diff. Isn't that unnecessary now that this bug has landed? (Honest question.)
(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > Thanks for doing this Aki! Could you update
> > > https://wiki.mozilla.org/Release:Release_Automation_on_Mercurial:
> > > Documentation#Buildbot_configs when this lands, too?
> > 
> > Done.
> > 
> > Though if we're changing repo paths to use user repos on every staging
> > release, suddenly the staging release build_tools_repo_path becomes an extra
> > step rather than a convenience.
> > 
> > We can remove those if people don't want it ?
> 
> As it stands now, the staging configs are set-up to work OOB in
> stage-ffxbld, so I think this is good. If we decide to change them to be
> more OOB friendly to personal user repositories, we can change this at that
> point.

Cool.

> 
> re: your documentation edit, I still see build_tools_repo_path being
> modified in the staging_config.py diff. Isn't that unnecessary now that this
> bug has landed? (Honest question.)

I'm not sure.
I only did that for completeness, since I don't want people to set up an entire set of user repos for themselves and forget about the tools repo during a staging release.

Is that helpful or harmful?
(In reply to comment #8)
> (In reply to comment #7)
> > 
> > re: your documentation edit, I still see build_tools_repo_path being
> > modified in the staging_config.py diff. Isn't that unnecessary now that this
> > bug has landed? (Honest question.)
> 
> I'm not sure.
> I only did that for completeness, since I don't want people to set up an
> entire set of user repos for themselves and forget about the tools repo
> during a staging release.
> 
> Is that helpful or harmful?

Sorry, I'm having trouble connecting your response to my question...what I'm not clear about is whether or not the build_tools_repo_path in staging_config.py still affects releases or not. If it does, the docs seem fine; if it doesn't, we can remove that part from the docs?
(In reply to comment #9) 
> Sorry, I'm having trouble connecting your response to my question...what I'm
> not clear about is whether or not the build_tools_repo_path in
> staging_config.py still affects releases or not. If it does, the docs seem
> fine; if it doesn't, we can remove that part from the docs?

Oh, I think I was completely misunderstanding your question.
I blame morning.
Edited.
I think a few things were missed here; patch incoming.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 536890 [details] [diff] [review]
use releaseConfig['build_tools_repo_path'] everywhere, when it exists

Oops.
Attachment #536890 - Flags: review?(aki) → review+
Attachment #536890 - Flags: checked-in+
This hit production today.
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.