Merge "autohold: prevent using ',' in change argument"

This commit is contained in:
Zuul 2018-04-10 23:05:28 +00:00 committed by Gerrit Code Review
commit 1b78f02c91
1 changed files with 3 additions and 0 deletions

View File

@ -187,6 +187,9 @@ class Client(zuul.cmd.ZuulApp):
if self.args.change and self.args.ref:
print("Change and ref can't be both used for the same request")
return False
if "," in self.args.change:
print("Error: change argument can not contain any ','")
return False
node_hold_expiration = self.args.node_hold_expiration
r = client.autohold(tenant=self.args.tenant,