Closed Bug 1186679 Opened 4 years ago Closed 4 years ago

thunderbird release builds should use mozilla- repo path not comm- repo path

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jlund, Unassigned)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1186678 +++

e.g. http://ftp.mozilla.org/pub/mozilla.org/thunderbird/candidates/40.0b1-candidates/build1/logs/release-comm-beta-macosx64_build-bm86-build1-build3.txt.gz

bash -c 'python archiver_client.py mozharness --repo releases/comm-beta --tag THUNDERBIRD_40_0b1_RELEASE --debug'

the above releases/comm-beta should be releases/mozilla-beta
looks like I patched all the ScriptFactory release based factories but again I neglected to address the TB case for ReleaseBuildFactory (MercurialBuildFactory).
Attachment #8637713 - Flags: review?(rail)
(In reply to Jordan Lund (:jlund) from comment #1)
> Created attachment 8637713 [details] [diff] [review]
> 150722_bug_1186679_use_mozilla_repo_path_for_tb_release.patch
> 
> looks like I patched all the ScriptFactory release based factories but again
> I neglected to address the TB case for ReleaseBuildFactory
> (MercurialBuildFactory).

dump_master diff: http://people.mozilla.org/~jlund/tb_release_mozilla_repo_path-dump_master.diff

(granted I only had 'comm-beta' in my master_config.json's 'thunderbird_release_branches' and all ff branches in the rest of master_config.json lists)
Comment on attachment 8637713 [details] [diff] [review]
150722_bug_1186679_use_mozilla_repo_path_for_tb_release.patch

lgtm
Attachment #8637713 - Flags: review?(rail) → review+
Comment on attachment 8637713 [details] [diff] [review]
150722_bug_1186679_use_mozilla_repo_path_for_tb_release.patch

thanks

remote:   https://hg.mozilla.org/build/buildbotcustom/rev/bfe0b68213ee
Attachment #8637713 - Flags: checked-in+
This patch appears to have worked; retriggered builds have gotten past the mozharness step.
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.