Bug 1163998 - Balrog rule locking doesn't work in merge scripts. r=jlund DONTBUILD

This commit is contained in:
Rail Aliiev 2015-09-24 17:10:11 -04:00
parent ecae29f061
commit 8e26becb4b
2 changed files with 14 additions and 24 deletions

View File

@ -116,7 +116,8 @@ class BalrogMixin(object):
c = self.config
dirs = self.query_abs_dirs()
submitter_script = os.path.join(
dirs["abs_tools_dir"], "scripts", "updates", "balrog-nightly-locker.py"
dirs["abs_tools_dir"], "scripts", "updates",
"balrog-nightly-locker.py"
)
credentials_file = os.path.join(
dirs["base_work_dir"], c["balrog_credentials_file"]
@ -126,6 +127,8 @@ class BalrogMixin(object):
self.query_exe("python"),
submitter_script,
"--credentials-file", credentials_file,
"--api-root", c["balrog_api_root"],
"--username", c["balrog_username"],
]
for r in rule_ids:
cmd.extend(["-r", str(r)])
@ -133,26 +136,8 @@ class BalrogMixin(object):
if self._log_level_at_least(INFO):
cmd.append("--verbose")
return_codes = []
for server in self.config["balrog_servers"]:
cmd.append("lock")
server_args = [
"--api-root", server["balrog_api_root"],
"--username", self._query_balrog_username(server)
]
cmd.append("lock")
self.info("Calling Balrog rule locking script.")
return_code = self.retry(
self.run_command, attempts=5,
args=(cmd + server_args + ['lock'],),
)
if server["ignore_failures"]:
self.info("Ignoring result, ignore_failures set to True")
else:
return_codes.append(return_code)
# use the worst (max) code
if max(return_codes) != 0:
self.return_code = 1
self.info("Calling Balrog rule locking script.")
self.retry(self.run_command, attempts=5, args=cmd,
kwargs={"halt_on_failure": True})

View File

@ -47,11 +47,16 @@ class GeckoMigration(MercurialScript, BalrogMixin, VirtualenvMixin, SelfServeMix
"default": "ffxbld <release@mozilla.com>",
"help": "Specify what user to use to commit to hg.",
}],
[['--balrog-api-root', ], {
"action": "store",
"dest": "balrog_api_root",
"type": "string",
"help": "Specify Balrog API root URL.",
}],
[['--balrog-username', ], {
"action": "store",
"dest": "balrog_username",
"type": "string",
"default": "ffxbld",
"help": "Specify what user to connect to Balrog with.",
}],
[['--balrog-credentials-file', ], {