From 0b284c1431592b8031a306488f97f8a41525f47b Mon Sep 17 00:00:00 2001 From: Joe Clarke Date: Mon, 26 Aug 2019 11:32:14 -0400 Subject: [PATCH] Cast booleans to strings. (#60544) This avoids an error when building the API request: TypeError: Argument must be bytes or unicode, got 'bool' This would occur when one explicitly sets async_bool or ignore_owners in a playbook/role. --- lib/ansible/modules/storage/netapp/na_ontap_snapshot.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/ansible/modules/storage/netapp/na_ontap_snapshot.py b/lib/ansible/modules/storage/netapp/na_ontap_snapshot.py index 2ae35fac35d..8adda71bab1 100644 --- a/lib/ansible/modules/storage/netapp/na_ontap_snapshot.py +++ b/lib/ansible/modules/storage/netapp/na_ontap_snapshot.py @@ -201,7 +201,7 @@ class NetAppOntapSnapshot(object): snapshot_obj.add_new_child("volume", self.parameters['volume']) # Set up optional variables to create a snapshot if self.parameters.get('async_bool'): - snapshot_obj.add_new_child("async", self.parameters['async_bool']) + snapshot_obj.add_new_child("async", str(self.parameters['async_bool'])) if self.parameters.get('comment'): snapshot_obj.add_new_child("comment", self.parameters['comment']) if self.parameters.get('snapmirror_label'): @@ -225,7 +225,7 @@ class NetAppOntapSnapshot(object): snapshot_obj.add_new_child("volume", self.parameters['volume']) # set up optional variables to delete a snapshot if self.parameters.get('ignore_owners'): - snapshot_obj.add_new_child("ignore-owners", self.parameters['ignore_owners']) + snapshot_obj.add_new_child("ignore-owners", str(self.parameters['ignore_owners'])) if self.parameters.get('snapshot_instance_uuid'): snapshot_obj.add_new_child("snapshot-instance-uuid", self.parameters['snapshot_instance_uuid']) try: