LGTM, thanks On Fri, Jun 26, 2015 at 10:38 PM, 'Lisa Velden' via ganeti-devel < [email protected]> wrote:
> As RestorePrivateValueWrapping() can also handle None, we do not have > to check here. > > Signed-off-by: Lisa Velden <[email protected]> > --- > lib/jqueue/exec.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/jqueue/exec.py b/lib/jqueue/exec.py > index 8da1d12..8e61805 100644 > --- a/lib/jqueue/exec.py > +++ b/lib/jqueue/exec.py > @@ -138,7 +138,7 @@ def main(): > > if secret_params: > for i in range(0, len(secret_params)): > - if hasattr(job.ops[i].input, "osparams_secret") and > secret_params[i]: > + if hasattr(job.ops[i].input, "osparams_secret"): > job.ops[i].input.osparams_secret = secret_params[i] > > execfun = mcpu.Processor(context, job_id, job_id).ExecOpCode > -- > 2.4.3.573.g4eafbef > > Hrvoje Ribicic Ganeti Engineering Google Germany GmbH Dienerstr. 12, 80331, München Geschäftsführer: Graham Law, Christine Elizabeth Flores Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg
