675286 - stop running debsign on non en-US, multi maemo-gtk releases

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Lukas Blakk [:lsblakk] use ?needinfo]

since we only do en-US and multi now, make debsign more efficient to match.

Comment 1

[Aki Sasaki (not active)]

Attachment: change debsign masters to use mozilla-{beta,release}

Comment 2

[Aki Sasaki (not active)]

Attachment: change mozharness to use mozilla-{beta,release} and only do en-US and multi

also switches the config file type to .py, and fixes a bug with that (fh.close()).

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 550107 [details] [diff] [review]
change mozharness to use mozilla-{beta,release} and only do en-US and multi

Review of attachment 550107 [details] [diff] [review]:
-----------------------------------------------------------------

right on, yay for cleaning up and fixing bugs at the same time

Comment 4

[Aki Sasaki (not active)]

Comment on attachment 550107 [details] [diff] [review]
change mozharness to use mozilla-{beta,release} and only do en-US and multi

http://hg.mozilla.org/build/mozharness/rev/78d7231266f0

Comment 5

[Aki Sasaki (not active)]

Comment on attachment 550106 [details] [diff] [review]
change debsign masters to use mozilla-{beta,release}

http://hg.mozilla.org/build/buildbot-configs/rev/7d22e76577b4

Ran a staging deb sign job which only did en-US and multi, and went green.

Comment 6

[Aki Sasaki (not active)]

Also reconfiged pmm:8011.

Comment 8

[Rail Aliiev [:rail]]

Fennec 6.0b5 failed:
ERROR: Can't find deb_repos/release_mozilla-beta.json in ['.', '/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/scripts/../configs', '/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/scripts/../../configs']!
Traceback (most recent call last):
  File "scripts/signdebs.py", line 298, in <module>
    debSigner = MaemoDebSigner()
  File "scripts/signdebs.py", line 59, in __init__
    require_config_file=require_config_file)
  File "/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/mozharness/base/script.py", line 589, in __init__
    super(MercurialScript, self).__init__(**kwargs)
  File "/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/mozharness/base/script.py", line 49, in __init__
    **kwargs)
  File "/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/mozharness/base/config.py", line 198, in __init__
    self.parse_args()
  File "/builds/scratchbox/users/cltbld/home/cltbld/deb-mozilla-beta/mozharness/base/config.py", line 336, in parse_args
    if key in defaults and value == defaults[key] and key in self._config:
TypeError: argument of type 'NoneType' is not iterable

Patch incoming

Comment 9

[Rail Aliiev [:rail]]

Attachment: Use *.py configs

Comment 10

[Rail Aliiev [:rail]]

Comment on attachment 550675 [details] [diff] [review]
Use *.py configs

http://hg.mozilla.org/build/buildbot-configs/rev/2c19b83f7881

Comment 11

[Rail Aliiev [:rail]]

worked fine.