From e63c0009e46720e227672f8175890e5969980b95 Mon Sep 17 00:00:00 2001 From: The Magician Date: Wed, 5 Dec 2018 13:24:45 -0800 Subject: [PATCH] Ansible: `false`s being lost in network requests (#141) --- plugins/modules/gcp_compute_address.py | 2 +- plugins/modules/gcp_compute_backend_bucket.py | 2 +- plugins/modules/gcp_compute_backend_service.py | 2 +- plugins/modules/gcp_compute_disk.py | 2 +- plugins/modules/gcp_compute_firewall.py | 2 +- plugins/modules/gcp_compute_forwarding_rule.py | 2 +- plugins/modules/gcp_compute_global_address.py | 2 +- plugins/modules/gcp_compute_global_forwarding_rule.py | 2 +- plugins/modules/gcp_compute_health_check.py | 2 +- plugins/modules/gcp_compute_http_health_check.py | 2 +- plugins/modules/gcp_compute_https_health_check.py | 2 +- plugins/modules/gcp_compute_image.py | 2 +- plugins/modules/gcp_compute_instance.py | 2 +- plugins/modules/gcp_compute_instance_group.py | 2 +- plugins/modules/gcp_compute_instance_group_manager.py | 2 +- plugins/modules/gcp_compute_instance_template.py | 2 +- plugins/modules/gcp_compute_interconnect_attachment.py | 2 +- plugins/modules/gcp_compute_network.py | 2 +- plugins/modules/gcp_compute_region_disk.py | 2 +- plugins/modules/gcp_compute_route.py | 2 +- plugins/modules/gcp_compute_router.py | 2 +- plugins/modules/gcp_compute_ssl_certificate.py | 2 +- plugins/modules/gcp_compute_ssl_policy.py | 2 +- plugins/modules/gcp_compute_subnetwork.py | 2 +- plugins/modules/gcp_compute_target_http_proxy.py | 2 +- plugins/modules/gcp_compute_target_https_proxy.py | 2 +- plugins/modules/gcp_compute_target_pool.py | 2 +- plugins/modules/gcp_compute_target_ssl_proxy.py | 2 +- plugins/modules/gcp_compute_target_tcp_proxy.py | 2 +- plugins/modules/gcp_compute_target_vpn_gateway.py | 2 +- plugins/modules/gcp_compute_url_map.py | 2 +- plugins/modules/gcp_compute_vpn_tunnel.py | 2 +- plugins/modules/gcp_container_cluster.py | 2 +- plugins/modules/gcp_container_node_pool.py | 2 +- plugins/modules/gcp_dns_managed_zone.py | 2 +- plugins/modules/gcp_dns_resource_record_set.py | 2 +- plugins/modules/gcp_iam_service_account.py | 2 +- plugins/modules/gcp_pubsub_subscription.py | 2 +- plugins/modules/gcp_pubsub_topic.py | 2 +- plugins/modules/gcp_resourcemanager_project.py | 2 +- plugins/modules/gcp_spanner_database.py | 2 +- plugins/modules/gcp_spanner_instance.py | 2 +- plugins/modules/gcp_sql_database.py | 2 +- plugins/modules/gcp_sql_instance.py | 2 +- plugins/modules/gcp_sql_user.py | 2 +- plugins/modules/gcp_storage_bucket.py | 2 +- plugins/modules/gcp_storage_bucket_access_control.py | 2 +- 47 files changed, 47 insertions(+), 47 deletions(-) diff --git a/plugins/modules/gcp_compute_address.py b/plugins/modules/gcp_compute_address.py index 73f6e31..9f6489c 100644 --- a/plugins/modules/gcp_compute_address.py +++ b/plugins/modules/gcp_compute_address.py @@ -282,7 +282,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_backend_bucket.py b/plugins/modules/gcp_compute_backend_bucket.py index 9d9d5b7..2a3374a 100644 --- a/plugins/modules/gcp_compute_backend_bucket.py +++ b/plugins/modules/gcp_compute_backend_bucket.py @@ -225,7 +225,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_backend_service.py b/plugins/modules/gcp_compute_backend_service.py index a6f75da..e467219 100644 --- a/plugins/modules/gcp_compute_backend_service.py +++ b/plugins/modules/gcp_compute_backend_service.py @@ -753,7 +753,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_disk.py b/plugins/modules/gcp_compute_disk.py index 46fc55f..f407b17 100644 --- a/plugins/modules/gcp_compute_disk.py +++ b/plugins/modules/gcp_compute_disk.py @@ -515,7 +515,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_firewall.py b/plugins/modules/gcp_compute_firewall.py index 02bafb1..7f62d9b 100644 --- a/plugins/modules/gcp_compute_firewall.py +++ b/plugins/modules/gcp_compute_firewall.py @@ -524,7 +524,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_forwarding_rule.py b/plugins/modules/gcp_compute_forwarding_rule.py index 069acd0..a2d59e5 100644 --- a/plugins/modules/gcp_compute_forwarding_rule.py +++ b/plugins/modules/gcp_compute_forwarding_rule.py @@ -512,7 +512,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_global_address.py b/plugins/modules/gcp_compute_global_address.py index 36f06ec..fd6f62d 100644 --- a/plugins/modules/gcp_compute_global_address.py +++ b/plugins/modules/gcp_compute_global_address.py @@ -233,7 +233,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_global_forwarding_rule.py b/plugins/modules/gcp_compute_global_forwarding_rule.py index ff08edf..48ae255 100644 --- a/plugins/modules/gcp_compute_global_forwarding_rule.py +++ b/plugins/modules/gcp_compute_global_forwarding_rule.py @@ -507,7 +507,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_health_check.py b/plugins/modules/gcp_compute_health_check.py index f68ed01..5659da6 100644 --- a/plugins/modules/gcp_compute_health_check.py +++ b/plugins/modules/gcp_compute_health_check.py @@ -633,7 +633,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_http_health_check.py b/plugins/modules/gcp_compute_http_health_check.py index dd2ae03..b3231f0 100644 --- a/plugins/modules/gcp_compute_http_health_check.py +++ b/plugins/modules/gcp_compute_http_health_check.py @@ -287,7 +287,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_https_health_check.py b/plugins/modules/gcp_compute_https_health_check.py index 72e9832..157ab8a 100644 --- a/plugins/modules/gcp_compute_https_health_check.py +++ b/plugins/modules/gcp_compute_https_health_check.py @@ -284,7 +284,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_image.py b/plugins/modules/gcp_compute_image.py index 5dd4af4..6cf4918 100644 --- a/plugins/modules/gcp_compute_image.py +++ b/plugins/modules/gcp_compute_image.py @@ -528,7 +528,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_instance.py b/plugins/modules/gcp_compute_instance.py index 1d2bcb3..37beeba 100644 --- a/plugins/modules/gcp_compute_instance.py +++ b/plugins/modules/gcp_compute_instance.py @@ -1046,7 +1046,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_instance_group.py b/plugins/modules/gcp_compute_instance_group.py index e31aef2..0e8a38e 100644 --- a/plugins/modules/gcp_compute_instance_group.py +++ b/plugins/modules/gcp_compute_instance_group.py @@ -310,7 +310,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_instance_group_manager.py b/plugins/modules/gcp_compute_instance_group_manager.py index 04fecfa..a926b3b 100644 --- a/plugins/modules/gcp_compute_instance_group_manager.py +++ b/plugins/modules/gcp_compute_instance_group_manager.py @@ -407,7 +407,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_instance_template.py b/plugins/modules/gcp_compute_instance_template.py index 9bdf427..d60fdc3 100644 --- a/plugins/modules/gcp_compute_instance_template.py +++ b/plugins/modules/gcp_compute_instance_template.py @@ -990,7 +990,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_interconnect_attachment.py b/plugins/modules/gcp_compute_interconnect_attachment.py index ef86aa2..4ea397a 100644 --- a/plugins/modules/gcp_compute_interconnect_attachment.py +++ b/plugins/modules/gcp_compute_interconnect_attachment.py @@ -261,7 +261,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_network.py b/plugins/modules/gcp_compute_network.py index ac39070..86555c2 100644 --- a/plugins/modules/gcp_compute_network.py +++ b/plugins/modules/gcp_compute_network.py @@ -279,7 +279,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_region_disk.py b/plugins/modules/gcp_compute_region_disk.py index d13b3f7..914ace9 100644 --- a/plugins/modules/gcp_compute_region_disk.py +++ b/plugins/modules/gcp_compute_region_disk.py @@ -455,7 +455,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_route.py b/plugins/modules/gcp_compute_route.py index cb9a900..3df7616 100644 --- a/plugins/modules/gcp_compute_route.py +++ b/plugins/modules/gcp_compute_route.py @@ -326,7 +326,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_router.py b/plugins/modules/gcp_compute_router.py index e0d5888..9b2d908 100644 --- a/plugins/modules/gcp_compute_router.py +++ b/plugins/modules/gcp_compute_router.py @@ -330,7 +330,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_ssl_certificate.py b/plugins/modules/gcp_compute_ssl_certificate.py index 0272734..892c0aa 100644 --- a/plugins/modules/gcp_compute_ssl_certificate.py +++ b/plugins/modules/gcp_compute_ssl_certificate.py @@ -234,7 +234,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_ssl_policy.py b/plugins/modules/gcp_compute_ssl_policy.py index ef86a6e..cd513ec 100644 --- a/plugins/modules/gcp_compute_ssl_policy.py +++ b/plugins/modules/gcp_compute_ssl_policy.py @@ -272,7 +272,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_subnetwork.py b/plugins/modules/gcp_compute_subnetwork.py index e3d8921..ed18a8f 100644 --- a/plugins/modules/gcp_compute_subnetwork.py +++ b/plugins/modules/gcp_compute_subnetwork.py @@ -399,7 +399,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_http_proxy.py b/plugins/modules/gcp_compute_target_http_proxy.py index 0e39917..4a00fa0 100644 --- a/plugins/modules/gcp_compute_target_http_proxy.py +++ b/plugins/modules/gcp_compute_target_http_proxy.py @@ -267,7 +267,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_https_proxy.py b/plugins/modules/gcp_compute_target_https_proxy.py index 5e05f2b..9f4ff82 100644 --- a/plugins/modules/gcp_compute_target_https_proxy.py +++ b/plugins/modules/gcp_compute_target_https_proxy.py @@ -395,7 +395,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_pool.py b/plugins/modules/gcp_compute_target_pool.py index 778926d..604d3e6 100644 --- a/plugins/modules/gcp_compute_target_pool.py +++ b/plugins/modules/gcp_compute_target_pool.py @@ -321,7 +321,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_ssl_proxy.py b/plugins/modules/gcp_compute_target_ssl_proxy.py index aef1f15..97ff051 100644 --- a/plugins/modules/gcp_compute_target_ssl_proxy.py +++ b/plugins/modules/gcp_compute_target_ssl_proxy.py @@ -380,7 +380,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_tcp_proxy.py b/plugins/modules/gcp_compute_target_tcp_proxy.py index 73931e1..c13b431 100644 --- a/plugins/modules/gcp_compute_target_tcp_proxy.py +++ b/plugins/modules/gcp_compute_target_tcp_proxy.py @@ -292,7 +292,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_target_vpn_gateway.py b/plugins/modules/gcp_compute_target_vpn_gateway.py index ae98e41..cdc6fcd 100644 --- a/plugins/modules/gcp_compute_target_vpn_gateway.py +++ b/plugins/modules/gcp_compute_target_vpn_gateway.py @@ -238,7 +238,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_url_map.py b/plugins/modules/gcp_compute_url_map.py index 2cff11a..3cb7564 100644 --- a/plugins/modules/gcp_compute_url_map.py +++ b/plugins/modules/gcp_compute_url_map.py @@ -454,7 +454,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_compute_vpn_tunnel.py b/plugins/modules/gcp_compute_vpn_tunnel.py index 9806a27..ad84884 100644 --- a/plugins/modules/gcp_compute_vpn_tunnel.py +++ b/plugins/modules/gcp_compute_vpn_tunnel.py @@ -374,7 +374,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_container_cluster.py b/plugins/modules/gcp_container_cluster.py index 5e25cb8..895c021 100644 --- a/plugins/modules/gcp_container_cluster.py +++ b/plugins/modules/gcp_container_cluster.py @@ -699,7 +699,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_container_node_pool.py b/plugins/modules/gcp_container_node_pool.py index 9d0a93d..7964da6 100644 --- a/plugins/modules/gcp_container_node_pool.py +++ b/plugins/modules/gcp_container_node_pool.py @@ -526,7 +526,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_dns_managed_zone.py b/plugins/modules/gcp_dns_managed_zone.py index 4a67c6b..05f2ae7 100644 --- a/plugins/modules/gcp_dns_managed_zone.py +++ b/plugins/modules/gcp_dns_managed_zone.py @@ -243,7 +243,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_dns_resource_record_set.py b/plugins/modules/gcp_dns_resource_record_set.py index 20dc2e7..24c89d3 100644 --- a/plugins/modules/gcp_dns_resource_record_set.py +++ b/plugins/modules/gcp_dns_resource_record_set.py @@ -243,7 +243,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_iam_service_account.py b/plugins/modules/gcp_iam_service_account.py index 71267ae..06f10ec 100644 --- a/plugins/modules/gcp_iam_service_account.py +++ b/plugins/modules/gcp_iam_service_account.py @@ -179,7 +179,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_pubsub_subscription.py b/plugins/modules/gcp_pubsub_subscription.py index 31732b8..ee356d2 100644 --- a/plugins/modules/gcp_pubsub_subscription.py +++ b/plugins/modules/gcp_pubsub_subscription.py @@ -265,7 +265,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_pubsub_topic.py b/plugins/modules/gcp_pubsub_topic.py index 137bee7..83198c4 100644 --- a/plugins/modules/gcp_pubsub_topic.py +++ b/plugins/modules/gcp_pubsub_topic.py @@ -146,7 +146,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_resourcemanager_project.py b/plugins/modules/gcp_resourcemanager_project.py index fe46412..925bfd4 100644 --- a/plugins/modules/gcp_resourcemanager_project.py +++ b/plugins/modules/gcp_resourcemanager_project.py @@ -244,7 +244,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_spanner_database.py b/plugins/modules/gcp_spanner_database.py index 10e7758..e812f96 100644 --- a/plugins/modules/gcp_spanner_database.py +++ b/plugins/modules/gcp_spanner_database.py @@ -201,7 +201,7 @@ def resource_to_request(module): request = encode_request(request, module) return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_spanner_instance.py b/plugins/modules/gcp_spanner_instance.py index 41d2c00..b7ca710 100644 --- a/plugins/modules/gcp_spanner_instance.py +++ b/plugins/modules/gcp_spanner_instance.py @@ -238,7 +238,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_sql_database.py b/plugins/modules/gcp_sql_database.py index 46b41d6..f694a67 100644 --- a/plugins/modules/gcp_sql_database.py +++ b/plugins/modules/gcp_sql_database.py @@ -206,7 +206,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_sql_instance.py b/plugins/modules/gcp_sql_instance.py index d2b45bf..1b648d7 100644 --- a/plugins/modules/gcp_sql_instance.py +++ b/plugins/modules/gcp_sql_instance.py @@ -674,7 +674,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_sql_user.py b/plugins/modules/gcp_sql_user.py index 21df4cc..f790ade 100644 --- a/plugins/modules/gcp_sql_user.py +++ b/plugins/modules/gcp_sql_user.py @@ -203,7 +203,7 @@ def resource_to_request(module): request = {u'kind': 'sql#user', u'password': module.params.get('password'), u'host': module.params.get('host'), u'name': module.params.get('name')} return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_storage_bucket.py b/plugins/modules/gcp_storage_bucket.py index a98302c..09803f4 100644 --- a/plugins/modules/gcp_storage_bucket.py +++ b/plugins/modules/gcp_storage_bucket.py @@ -921,7 +921,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals diff --git a/plugins/modules/gcp_storage_bucket_access_control.py b/plugins/modules/gcp_storage_bucket_access_control.py index cb6a0b1..b766d58 100644 --- a/plugins/modules/gcp_storage_bucket_access_control.py +++ b/plugins/modules/gcp_storage_bucket_access_control.py @@ -272,7 +272,7 @@ def resource_to_request(module): } return_vals = {} for k, v in request.items(): - if v or v is False: + if v is not None: return_vals[k] = v return return_vals