mirror of
https://github.com/prowler-cloud/prowler.git
synced 2026-07-23 20:42:02 +00:00
fix(redshift): validation error for Cluster.multi_az (#7381)
This commit is contained in:
committed by
GitHub
parent
f2f41c9c44
commit
ee27636f32
+1
-1
@@ -11,7 +11,7 @@ class redshift_cluster_multi_az_enabled(Check):
|
||||
report.status_extended = (
|
||||
f"Redshift Cluster {cluster.id} does not have Multi-AZ enabled."
|
||||
)
|
||||
if cluster.multi_az:
|
||||
if cluster.multi_az == "Enabled":
|
||||
report.status = "PASS"
|
||||
report.status_extended = (
|
||||
f"Redshift Cluster {cluster.id} has Multi-AZ enabled."
|
||||
|
||||
@@ -45,7 +45,7 @@ class Redshift(AWSService):
|
||||
"AllowVersionUpgrade", False
|
||||
),
|
||||
encrypted=cluster.get("Encrypted", False),
|
||||
multi_az=cluster.get("MultiAZ", False),
|
||||
multi_az=cluster.get("MultiAZ", ""),
|
||||
region=regional_client.region,
|
||||
tags=cluster.get("Tags"),
|
||||
master_username=cluster.get("MasterUsername", ""),
|
||||
@@ -145,7 +145,7 @@ class Cluster(BaseModel):
|
||||
vpc_security_groups: list = []
|
||||
public_access: bool = False
|
||||
encrypted: bool = False
|
||||
multi_az: bool = False
|
||||
multi_az: str = None
|
||||
master_username: str = None
|
||||
database_name: str = None
|
||||
endpoint_address: str = None
|
||||
|
||||
+1
-1
@@ -149,7 +149,7 @@ class Test_redshift_cluster_multi_az_enabled:
|
||||
)
|
||||
|
||||
# Moto does not pass the multi_az parameter back.
|
||||
service_client.clusters[0].multi_az = 1
|
||||
service_client.clusters[0].multi_az = "Enabled"
|
||||
check = redshift_cluster_multi_az_enabled()
|
||||
result = check.execute()
|
||||
|
||||
|
||||
@@ -140,7 +140,8 @@ class Test_Redshift_Service:
|
||||
]
|
||||
assert redshift.clusters[0].parameter_group_name == "default.redshift-1.0"
|
||||
assert redshift.clusters[0].encrypted
|
||||
assert redshift.clusters[0].multi_az is False
|
||||
# Moto does not pass the multi_az parameter back.
|
||||
assert redshift.clusters[0].multi_az == ""
|
||||
assert redshift.clusters[0].master_username == "user"
|
||||
assert redshift.clusters[0].enhanced_vpc_routing
|
||||
assert redshift.clusters[0].database_name == "test"
|
||||
|
||||
Reference in New Issue
Block a user