Bugfix for gc_storage and s3

Make keysum return None if not key_check (this case wasn't covered).
reviewable/pr18780/r1
Cristian Ciupitu 11 years ago
parent 5fc8a6bbd9
commit 03eb65f25d

@ -152,11 +152,12 @@ def key_check(module, gs, bucket, obj):
def keysum(module, gs, bucket, obj): def keysum(module, gs, bucket, obj):
bucket = gs.lookup(bucket) bucket = gs.lookup(bucket)
key_check = bucket.get_key(obj) 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 return md5_remote
def bucket_check(module, gs, bucket): def bucket_check(module, gs, bucket):

@ -145,11 +145,12 @@ def key_check(module, s3, bucket, obj):
def keysum(module, s3, bucket, obj): def keysum(module, s3, bucket, obj):
bucket = s3.lookup(bucket) bucket = s3.lookup(bucket)
key_check = bucket.get_key(obj) 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 return md5_remote
def bucket_check(module, s3, bucket): def bucket_check(module, s3, bucket):

Loading…
Cancel
Save