fix set_tags so it doesn't return anything

Signed-off-by: Sylvain Hellegouarch <sh@defuze.org>

Author: Lawouach

CHANGELOG.md

@@ -2,7 +2,16 @@
 
 ## [Unreleased][]
 
-[Unreleased]: https://github.com/chaostoolkit-incubator/chaostoolkit-google-cloud-platform/compare/0.36.0...HEAD
+[Unreleased]: https://github.com/chaostoolkit-incubator/chaostoolkit-google-cloud-platform/compare/0.36.1...HEAD
+
+## [0.36.1][] - 2024-07-02
+
+[0.36.0]: https://github.com/chaostoolkit-incubator/chaostoolkit-google-cloud-platform/compare/0.36.0...0.36.1
+
+### Fixed
+
+- Remove the return statement from `set_tags` since the extended operation
+  payload is set to nothing anyway
 
 ## [0.36.0][] - 2024-07-01
 

chaosgcp/__init__.py

@@ -130,6 +130,8 @@ def wait_on_operation(
 def wait_on_extended_operation(
     operation: ExtendedOperation, frequency: int = 1, timeout: int = 60
 ) -> None:
+    # note that extended operations return nothing and set theyr result payload
+    # to None
     start = time.time()
 
     while True:

chaosgcp/compute/actions.py

@@ -19,7 +19,7 @@
 from google.cloud import compute_v1
 from google.cloud.compute_v1.types import Tags
 
-from chaosgcp import load_credentials, wait_on_extended_operation, to_dict
+from chaosgcp import load_credentials, wait_on_extended_operation
 
 __all__ = ["set_instance_tags"]
 logger = logging.getLogger("chaostoolkit")
@@ -32,14 +32,14 @@ def set_instance_tags(
     tags_list: list,
     configuration: Configuration = None,
     secrets: Secrets = None,
-) -> Dict[str, Any]:
+) -> None:
     """Set a Network Tags to a GCE VM instance
 
     :param project_id : the project ID in which the DNS record is present
     :param ip_address: the IP address for the A record that needs to be changed
     :param zone: the name of the zone where the GCE VM is provisioned
     :param tags_list : list of network tags to be set to the GCE VM instance
-    :return JSON Response which is in form of dictionary
+    :return nothing
     """
     credentials = load_credentials(secrets)
 
@@ -69,12 +69,9 @@ def set_instance_tags(
     operation = client.set_tags(request=request)
 
     # Handle the response
+    # do not return anything as extended operations do not carry a payload
     wait_on_extended_operation(operation)
 
-    response = operation.result()
-
-    return to_dict(response)
-
 
 def suspend_vm_instance(
     project_id: str, zone: str, instance_name: str, secrets: Secrets = None