Skip to content

Commit

Permalink
Machine: fix default value for Machine pool_params
Browse files Browse the repository at this point in the history
Should be {} instead of None since this is automatically passed to the
_from_dict() method of a Parameters instance. This affects recipes where
we dynamically connect to machines as part of the recipe itself - for
example guest vms created during the recipe.

Signed-off-by: Ondrej Lichtner <olichtne@redhat.com>
  • Loading branch information
olichtne committed Oct 7, 2020
1 parent 10dc741 commit 86c1ddd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lnst/Controller/Machine.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ class Machine(object):
"""

def __init__(self, m_id, hostname, msg_dispatcher, ctl_config,
libvirt_domain=None, rpcport=None, security=None, pool_params=None):
libvirt_domain=None, rpcport=None, security=None, pool_params={}):
self._id = m_id
self._hostname = hostname
self._mapped = False
Expand Down

0 comments on commit 86c1ddd

Please sign in to comment.