From b177aff41bb3a8893baf8f0777dbc73fd5dc5529 Mon Sep 17 00:00:00 2001 From: Ondra Machacek Date: Fri, 19 Oct 2018 15:13:29 +0200 Subject: [PATCH] ovirt_vm: Fix initialization of cloud init (#47348) Fixes: https://github.com/ansible/ansible/issues/45900 This PR fixes the case when the cloud_init_persist was used, but we still sent use_cloud_init=False, which is in oVirt API evaluated as not to use cloud_init in first VM execution. This patch is changing it to send just None, instead of False. Signed-off-by: Ondra Machacek --- lib/ansible/modules/cloud/ovirt/ovirt_vm.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/ansible/modules/cloud/ovirt/ovirt_vm.py b/lib/ansible/modules/cloud/ovirt/ovirt_vm.py index c0de1edb704..5994d7b59b9 100644 --- a/lib/ansible/modules/cloud/ovirt/ovirt_vm.py +++ b/lib/ansible/modules/cloud/ovirt/ovirt_vm.py @@ -2101,8 +2101,8 @@ def main(): ), wait_condition=lambda vm: vm.status == otypes.VmStatus.UP, # Start action kwargs: - use_cloud_init=not module.params.get('cloud_init_persist') and module.params.get('cloud_init') is not None, - use_sysprep=not module.params.get('cloud_init_persist') and module.params.get('sysprep') is not None, + use_cloud_init=True if not module.params.get('cloud_init_persist') and module.params.get('cloud_init') is not None else None, + use_sysprep=True if not module.params.get('cloud_init_persist') and module.params.get('sysprep') is not None else None, vm=otypes.Vm( placement_policy=otypes.VmPlacementPolicy( hosts=[otypes.Host(name=module.params['host'])]