Skip to content

Commit 6d80acd

Browse files
committed
lint and style
1 parent b444dbc commit 6d80acd

File tree

2 files changed

+15
-9
lines changed

2 files changed

+15
-9
lines changed

src/aks-preview/azext_aks_preview/managed_cluster_decorator.py

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -924,17 +924,23 @@ def get_container_network_logs(self, mc: ManagedCluster) -> Union[bool, None]:
924924
"""Get the enablement of container network logs
925925
926926
:return: bool or None"""
927-
enable_container_network_logs = self.raw_param.get("enable_container_network_logs") or self.raw_param.get("enable_retina_flow_logs")
928-
disable_container_network_logs = self.raw_param.get("disable_container_network_logs") or self.raw_param.get("disable_retina_flow_logs")
929-
if enable_container_network_logs is None and disable_container_network_logs is None:
927+
enable_cnl = (
928+
self.raw_param.get("enable_container_network_logs") or
929+
self.raw_param.get("enable_retina_flow_logs")
930+
)
931+
disable_cnl = (
932+
self.raw_param.get("disable_container_network_logs") or
933+
self.raw_param.get("disable_retina_flow_logs")
934+
)
935+
if enable_cnl is None and disable_cnl is None:
930936
return None
931-
if enable_container_network_logs and disable_container_network_logs:
937+
if enable_cnl and disable_cnl:
932938
raise MutuallyExclusiveArgumentError(
933939
"Cannot specify --enable-container-network-logs and "
934940
"--disable-container-network-logs at the same time."
935941
)
936942
if (
937-
enable_container_network_logs and
943+
enable_cnl and
938944
(not self.raw_param.get("enable_acns", False) and
939945
not (mc.network_profile and mc.network_profile.advanced_networking and
940946
mc.network_profile.advanced_networking.enabled)) or
@@ -944,9 +950,9 @@ def get_container_network_logs(self, mc: ManagedCluster) -> Union[bool, None]:
944950
"Container network logs requires '--enable-acns', advanced networking "
945951
"to be enabled, and the monitoring addon to be enabled."
946952
)
947-
enable_container_network_logs = bool(enable_container_network_logs) if enable_container_network_logs is not None else False
948-
disable_container_network_logs = bool(disable_container_network_logs) if disable_container_network_logs is not None else False
949-
return enable_container_network_logs or not disable_container_network_logs
953+
enable_cnl = bool(enable_cnl) if enable_cnl is not None else False
954+
disable_cnl = bool(disable_cnl) if disable_cnl is not None else False
955+
return enable_cnl or not disable_cnl
950956

951957
def get_load_balancer_managed_outbound_ip_count(self) -> Union[int, None]:
952958
"""Obtain the value of load_balancer_managed_outbound_ip_count.

src/aks-preview/linter_exclusions.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ aks create:
132132
enable_retina_flow_logs:
133133
rule_exclusions:
134134
- option_length_too_long
135-
disable_container_network_logs:
135+
enable_container_network_logs:
136136
rule_exclusions:
137137
- option_length_too_long
138138
node_provisioning_default_pools:

0 commit comments

Comments
 (0)