From: Ondrej Lichtner olichtne@redhat.com
This is a followup to the previous patch that was posted by Artem. The same error he found also appears in the check_machine_compatibility method.
Signed-off-by: Ondrej Lichtner olichtne@redhat.com --- lnst/Controller/SlavePool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lnst/Controller/SlavePool.py b/lnst/Controller/SlavePool.py index 87a2b33..53f402f 100644 --- a/lnst/Controller/SlavePool.py +++ b/lnst/Controller/SlavePool.py @@ -525,7 +525,7 @@ class SetupMapper(object): def _check_machine_compatibility(self, req_id, pool_id): req_machine = self._mreqs[req_id] pool_machine = self._pool[pool_id] - for param, value in req_machine["params"]: + for param, value in req_machine["params"].iteritems(): if param in pool_machine["params"] and\ value != pool_machine["params"][param]: return False
Thu, Oct 23, 2014 at 01:32:55PM CEST, olichtne@redhat.com wrote:
From: Ondrej Lichtner olichtne@redhat.com
This is a followup to the previous patch that was posted by Artem. The same error he found also appears in the check_machine_compatibility method.
Signed-off-by: Ondrej Lichtner olichtne@redhat.com
applied, thanks
lnst-developers@lists.fedorahosted.org