913452 - Switch 'Thunderbird-Release' on TBPL to use {mozilla,comm}-esr24 branches rather than {mozilla,comm}-release

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect, P2)

Description

[Rail Aliiev [:rail]]

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

Comment 1

[Ed Morley [:emorley]]

mozilla-esr24 was just added by bug 913297

Comment 2

[Rail Aliiev [:rail]]

Hmm, I'm not even sure if we need this for Thunderbird, since there will be no TB 24ESR release (even though TB 24 will be built off comm-esr24 branch)

Comment 3

[Rail Aliiev [:rail]]

Actually we need to update Thunderbird-Release to point to esr24 since we switched releng automation to use that branch.

Comment 4

[Rail Aliiev [:rail]]

Attachment: TB release should use esr24

Comment 5

[Rail Aliiev [:rail]]

Per https://bugzilla.mozilla.org/show_bug.cgi?id=913197#c8 I have a carte blanche! :)

Comment 6

[Rail Aliiev [:rail]]

Comment on attachment 801118 [details] [diff] [review]
TB release should use esr24

http://hg.mozilla.org/webtools/tbpl/rev/6a3543bc4c47

Comment 7

[Ed Morley [:emorley]]

In production :-)

Comment 8

[Rail Aliiev [:rail]]

buildbotBranch should point to comm-release... Patch incoming.

Comment 9

[Rail Aliiev [:rail]]

Attachment: buildbotBranch: "comm-release"

Comment 10

[Ed Morley [:emorley]]

Comment on attachment 801548 [details] [diff] [review]
buildbotBranch: "comm-release"

Surely this should be changed buildbot-side instead?

We're using the esr24 repos but buildbot is treating it as -release ?
I'm confused! :-)

Comment 11

[Rail Aliiev [:rail]]

Heh, yeah, it's confusing.

It's all based on the fact that TB 24.0 *release* (comm-release buildbot branch) uses the *releases/comm-esr24* repo.

It's possible to change the buildbot part, but to me it sounds weird to have release builds named comm-esr24.

Mark, do you have any preferences?

Comment 12

[Mark Banner (:standard8)]

I'm generally happy with what works for everyone. Using comm-esr24 for buildbot would potentially be easier to remember, but I don't really have a preference. The last year's release builds I've only really seen from comm-esr17 anyway.

Comment 13

[Rail Aliiev [:rail]]

Comment on attachment 801548 [details] [diff] [review]
buildbotBranch: "comm-release"

No need for this

Comment 14

[Rail Aliiev [:rail]]

Attachment: kill Thunderbird-Release, add Thunderbird-Esr24

In the parent bug we're killing comm-release in favour of comm-esr24. Let's do the same here!

Comment 15

[Ed Morley [:emorley]]

Comment on attachment 802350 [details] [diff] [review]
kill Thunderbird-Release, add Thunderbird-Esr24

I'd maybe swap the ordering of es24 and esr17, since the rest of the Thunderbird repos go from new to old, but not too fussed either way :-)

Comment 16

[Rail Aliiev [:rail]]

Comment on attachment 802350 [details] [diff] [review]
kill Thunderbird-Release, add Thunderbird-Esr24

When I changed the order, the patch turned into a one-liner :) 

http://hg.mozilla.org/webtools/tbpl/rev/60ce1f27d055

Comment 17

[Ed Morley [:emorley]]

Second patch in production :-)