Fix: corrigido acesso a atributos de ContainerResource como objeto
This commit is contained in:
@@ -83,7 +83,7 @@ class ValidationService:
|
|||||||
validations.append(ResourceValidation(
|
validations.append(ResourceValidation(
|
||||||
pod_name=pod_name,
|
pod_name=pod_name,
|
||||||
namespace=namespace,
|
namespace=namespace,
|
||||||
container_name=container["name"],
|
container_name=container.name,
|
||||||
validation_type="missing_requests",
|
validation_type="missing_requests",
|
||||||
severity="error",
|
severity="error",
|
||||||
message="Container without defined requests",
|
message="Container without defined requests",
|
||||||
@@ -95,7 +95,7 @@ class ValidationService:
|
|||||||
validations.append(ResourceValidation(
|
validations.append(ResourceValidation(
|
||||||
pod_name=pod_name,
|
pod_name=pod_name,
|
||||||
namespace=namespace,
|
namespace=namespace,
|
||||||
container_name=container["name"],
|
container_name=container.name,
|
||||||
validation_type="missing_limits",
|
validation_type="missing_limits",
|
||||||
severity="warning",
|
severity="warning",
|
||||||
message="Container without defined limits",
|
message="Container without defined limits",
|
||||||
@@ -103,20 +103,20 @@ class ValidationService:
|
|||||||
))
|
))
|
||||||
|
|
||||||
# 3. QoS Class validation based on Red Hat recommendations
|
# 3. QoS Class validation based on Red Hat recommendations
|
||||||
qos_validation = self._validate_qos_class(pod_name, namespace, container["name"], qos_class, requests, limits)
|
qos_validation = self._validate_qos_class(pod_name, namespace, container.name, qos_class, requests, limits)
|
||||||
if qos_validation:
|
if qos_validation:
|
||||||
validations.append(qos_validation)
|
validations.append(qos_validation)
|
||||||
|
|
||||||
# 3. Validate limit:request ratio
|
# 3. Validate limit:request ratio
|
||||||
if requests and limits:
|
if requests and limits:
|
||||||
cpu_validation = self._validate_cpu_ratio(
|
cpu_validation = self._validate_cpu_ratio(
|
||||||
pod_name, namespace, container["name"], requests, limits
|
pod_name, namespace, container.name, requests, limits
|
||||||
)
|
)
|
||||||
if cpu_validation:
|
if cpu_validation:
|
||||||
validations.append(cpu_validation)
|
validations.append(cpu_validation)
|
||||||
|
|
||||||
memory_validation = self._validate_memory_ratio(
|
memory_validation = self._validate_memory_ratio(
|
||||||
pod_name, namespace, container["name"], requests, limits
|
pod_name, namespace, container.name, requests, limits
|
||||||
)
|
)
|
||||||
if memory_validation:
|
if memory_validation:
|
||||||
validations.append(memory_validation)
|
validations.append(memory_validation)
|
||||||
@@ -124,7 +124,7 @@ class ValidationService:
|
|||||||
# 4. Validate minimum values
|
# 4. Validate minimum values
|
||||||
if requests:
|
if requests:
|
||||||
min_validation = self._validate_minimum_values(
|
min_validation = self._validate_minimum_values(
|
||||||
pod_name, namespace, container["name"], requests
|
pod_name, namespace, container.name, requests
|
||||||
)
|
)
|
||||||
validations.extend(min_validation)
|
validations.extend(min_validation)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user