Merge pull request #6747 from ciupicri/cloud-bugfix

Bugfix for gc_storage and s3

cloud/gc_storage

@@ -152,11 +152,12 @@ def key_check(module, gs, bucket, obj):
 def keysum(module, gs, bucket, obj):
     bucket = gs.lookup(bucket)
     key_check = bucket.get_key(obj)
-    if key_check:
+    if not key_check:
-        md5_remote = key_check.etag[1:-1]
+        return None
-        etag_multipart = '-' in md5_remote # Check for multipart, etag is not md5
+    md5_remote = key_check.etag[1:-1]
-        if etag_multipart is True:
+    etag_multipart = '-' in md5_remote # Check for multipart, etag is not md5
-            module.fail_json(msg="Files uploaded with multipart of gs are not supported with checksum, unable to compute checksum.")
+    if etag_multipart is True:
+        module.fail_json(msg="Files uploaded with multipart of gs are not supported with checksum, unable to compute checksum.")
     return md5_remote
 
 def bucket_check(module, gs, bucket):

cloud/s3

@@ -145,11 +145,12 @@ def key_check(module, s3, bucket, obj):
 def keysum(module, s3, bucket, obj):
     bucket = s3.lookup(bucket)
     key_check = bucket.get_key(obj)
-    if key_check:
+    if not key_check:
-        md5_remote = key_check.etag[1:-1]
+        return None
-        etag_multipart = '-' in md5_remote # Check for multipart, etag is not md5
+    md5_remote = key_check.etag[1:-1]
-        if etag_multipart is True:
+    etag_multipart = '-' in md5_remote # Check for multipart, etag is not md5
-            module.fail_json(msg="Files uploaded with multipart of s3 are not supported with checksum, unable to compute checksum.")
+    if etag_multipart is True:
+        module.fail_json(msg="Files uploaded with multipart of s3 are not supported with checksum, unable to compute checksum.")
     return md5_remote
 
 def bucket_check(module, s3, bucket):