745968 - Pulse notifications for l10n beta candidate builds should include 'l10n' instead of 'repack'

Status: RESOLVED WONTFIX

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

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Recently I have setup our Mozmill CI system to run the BFT tests against beta candidate builds of Firefox. This works fine for en-US builds by watching the 'release-mozilla-beta' branch.

What I have noticed is that I don't get any notification for localized beta candidate builds. Those notification should be send out for the same branch like the en-US builds.

Comment 1

[Chris AtLee [:catlee]]

we are sending out the notifications for release repacks, they're just structured differently.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

As talked with Chris on IRC a change from 'repack' to 'l10n' should be possible. But it's probably not a good idea to make this change now. We should wait until separate notifications can be send out by the repack machines for each individual locale. I will file this bug shortly.

Comment 3

[[e-mail] (:bhearsum)]

This probably means renaming the builders. I suspect this is a low risk change that would involve:
* String changes in a few places:
** https://github.com/mozilla/buildbotcustom/blob/6901146c9d3349772a9f9b175a91974af4ebb102/process/release.py#L271
** https://github.com/mozilla/buildbotcustom/blob/6901146c9d3349772a9f9b175a91974af4ebb102/process/release.py#L714
** https://github.com/mozilla/buildbotcustom/blob/6901146c9d3349772a9f9b175a91974af4ebb102/process/release.py#L753
** https://github.com/mozilla/buildbotcustom/blob/6901146c9d3349772a9f9b175a91974af4ebb102/process/release.py#L774
** https://github.com/mozilla/buildbotcustom/blob/6901146c9d3349772a9f9b175a91974af4ebb102/process/release.py#L1652

I think that's it, but it would need a test in staging before we can land it.

In any case, this isn't high priority.

Comment 4

[Simone Bruno [:simone]]

The plan is to work on this next week (16-20 September).

Comment 5

[[e-mail] (:bhearsum)]

We might get this for free with release promotion.

Comment 6

[Rail Aliiev [:rail]]

Do we still need this?

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

I feel that we are stuck with the repack term also in release promotion. So lets keep it and don't worry about this minor thing.

Comment 8

[Rail Aliiev [:rail]]

Thank you!