1337645 - fennec l10n bumper script

Status: RESOLVED FIXED

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

Description

[Aki Sasaki (not active)]

This should take the beta / release changesets for Fennec and land them in-tree.

The first function should generate the patch. A second function should attempt to commit and push -- we can use hgtool or the like. In the future, we can use the first function and send the patch to vcspush scriptworker.

We'll need a looped push like https://hg.mozilla.org/build/mozharness/file/de9295df9085/scripts/bump_gaia_json.py#l189 or https://hg.mozilla.org/build/tools/file/tip/lib/python/util/hg.py#l536

Comment 1

[Chris AtLee [:catlee]]

b2g bumper was here: https://hg.mozilla.org/build/mozharness/file/a234552327b6/scripts/b2g_bumper.py#l584

Comment 2

[Aki Sasaki (not active)]

https://hg.mozilla.org/projects/date/rev/1e2f8292296f5926a6dd9da079676b7158e49176
bug 1337645 - initial l10n-changesets.json. r=me

Comment 3

[Aki Sasaki (not active)]

https://hg.mozilla.org/projects/jamun/rev/a6760f758c58ff79bc0f271de0c4d17d5c99a5cd
bug 1337645 - add initial l10n-changesets.json. r=testing

Comment 4

[Aki Sasaki (not active)]

Attachment: bug 1337645 - add an l10n bumper for fennec relpro.

Review commit: https://reviewboard.mozilla.org/r/110818/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/110818/

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8835122 [details]
bug 1337645 - add an l10n bumper for fennec relpro.

https://reviewboard.mozilla.org/r/110818/#review112414

::: testing/mozharness/scripts/l10n_bumper.py:74
(Diff revision 1)
> +            ),
> +        })
> +        self.abs_dirs = abs_dirs
> +        return self.abs_dirs
> +
> +    def hg_add(self, repo_path, path):

Is this used? It looks like we can remove it safely.

::: testing/mozharness/scripts/l10n_bumper.py:119
(Diff revision 1)
> +        return True
> +
> +    def _read_json(self, path):
> +        if not os.path.exists(path):
> +            self.error("%s doesn't exist!" % path)
> +            return

This check looks redundant to me. Feel free to remove this block.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a29bb4c502e
add an l10n bumper for fennec relpro. r=rail

Comment 7

[Aki Sasaki (not active)]

https://hg.mozilla.org/projects/jamun/rev/00cb3d474591906de7572d367cba2f7fe38e4ee1
bug 1337645 - add l10n-bumper script. r=rail

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/3a29bb4c502e