[FIX] cloud_platform_azure: Fix container name pattern check (#338)

* [FIX] cloud_platform_azure: Fix container name pattern check

* Fix linting
This commit is contained in:
Patrick Tombez
2022-02-01 16:06:20 +01:00
committed by GitHub
co-authored by GitHub
parent e33662cc1a
commit e2c90bdac1
@@ -92,24 +92,24 @@ class CloudPlatform(models.AbstractModel):
" 'ir_attachment.location' parameter." " 'ir_attachment.location' parameter."
) )
# A bucket name is defined under the following format # A bucket name is defined under the following format
# ^[a-z]+_[a-z]+_\d+$ # ^[a-z]+\-[a-z]+\-\d+$
# Anything other than prod bucket must be suffixed with env name # Anything other than prod bucket must be suffixed with env name
# #
# Use AZURE_STORAGE_NAME_UNSTRUCTURED to by-pass check # Use AZURE_STORAGE_NAME_UNSTRUCTURED to by-pass check
# on bucket name structure # on bucket name structure
if os.environ.get("AZURE_STORAGE_NAME_UNSTRUCTURED"): if os.environ.get("AZURE_STORAGE_NAME_UNSTRUCTURED"):
return return
prod_bucket = bool(re.match(r"^[a-z]+_[a-z]+_\d+$", storage_name)) prod_bucket = bool(re.match(r"^[a-z]+\-[a-z]+\-\d+$", storage_name))
if environment_name == "prod": if environment_name == "prod":
assert prod_bucket, ( assert prod_bucket, (
"AZURE_STORAGE_NAME should match '^[a-z]+_[a-z]+_\\d+$', " "AZURE_STORAGE_NAME should match '^[a-z]+\\-[a-z]+\\-\\d+$', "
"we got: '%s'" % (storage_name,) "we got: '%s'" % (storage_name,)
) )
else: else:
# if we are using the prod bucket on another instance # if we are using the prod bucket on another instance
# such as an integration, we must be sure to be in read only! # such as an integration, we must be sure to be in read only!
assert not prod_bucket, ( assert not prod_bucket, (
"AZURE_STORAGE_NAME should not match '^[a-z]+_[a-z]+_\\d+$', " "AZURE_STORAGE_NAME should not match '^[a-z]+\\-[a-z]+\\-\\d+$', "
"we got: '%s'" % (storage_name,) "we got: '%s'" % (storage_name,)
) )