Skip to content

Commit 123da77

Browse files
authored
Change 'tls' parameter to 'auto_tls' for cluster module (#285)
* Change 'tls' parameter to 'auto_tls' due to collision with existing TLS verification parameters * Update API documentation for Auto-TLS parameter Signed-off-by: Webster Mudge <wmudge@cloudera.com>
1 parent 79e7c8a commit 123da77

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

plugins/modules/cluster.py

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -400,7 +400,7 @@
400400
required: no
401401
aliases:
402402
- data_contexts
403-
tls:
403+
auto_tls:
404404
description:
405405
- Flag enabling TLS for the cluster.
406406
type: bool
@@ -838,7 +838,7 @@ def __init__(self, module):
838838
self.contexts = self.get_param("contexts")
839839
self.auto_assign = self.get_param("auto_assign")
840840
self.control_plane = self.get_param("control_plane")
841-
self.tls = self.get_param("tls")
841+
self.auto_tls = self.get_param("auto_tls")
842842
self.force = self.get_param("force")
843843

844844
self.changed = False
@@ -926,8 +926,8 @@ def process(self):
926926
self.create_cluster_from_parameters()
927927

928928
# Toggle AutoTLS
929-
if self.tls is not None:
930-
if self.tls:
929+
if self.auto_tls is not None:
930+
if self.auto_tls:
931931
enable_tls_cmd = (
932932
self.cluster_api.configure_auto_tls_services_command(
933933
cluster_name=self.name
@@ -973,8 +973,8 @@ def process(self):
973973
self.create_cluster_from_parameters()
974974

975975
# Toggle AutoTLS
976-
if self.tls is not None:
977-
if self.tls:
976+
if self.auto_tls is not None:
977+
if self.auto_tls:
978978
enable_tls_cmd = (
979979
self.cluster_api.configure_auto_tls_services_command(
980980
cluster_name=self.name
@@ -1023,8 +1023,8 @@ def process(self):
10231023
self.create_cluster_from_parameters()
10241024

10251025
# Toggle AutoTLS
1026-
if self.tls is not None:
1027-
if self.tls:
1026+
if self.auto_tls is not None:
1027+
if self.auto_tls:
10281028
enable_tls_cmd = (
10291029
self.cluster_api.configure_auto_tls_services_command(
10301030
cluster_name=self.name
@@ -1045,8 +1045,8 @@ def process(self):
10451045
else:
10461046
self.changed = True
10471047
# Toggle AutoTLS
1048-
if self.tls is not None:
1049-
if self.tls:
1048+
if self.auto_tls is not None:
1049+
if self.auto_tls:
10501050
enable_tls_cmd = (
10511051
self.cluster_api.configure_auto_tls_services_command(
10521052
cluster_name=self.name
@@ -1081,8 +1081,8 @@ def process(self):
10811081
self.create_cluster_from_parameters()
10821082

10831083
# Toggle AutoTLS
1084-
if self.tls is not None:
1085-
if self.tls:
1084+
if self.auto_tls is not None:
1085+
if self.auto_tls:
10861086
enable_tls_cmd = (
10871087
self.cluster_api.configure_auto_tls_services_command(
10881088
cluster_name=self.name
@@ -1687,7 +1687,7 @@ def main():
16871687
# Optional data contexts, required for compute-type clusters
16881688
contexts=dict(type="list", elements="str", aliases=["data_contexts"]),
16891689
# Optional enable/disable TLS for the cluster
1690-
tls=dict(type="bool", aliases=["tls_enabled", "cluster_tls"]),
1690+
auto_tls=dict(type="bool", aliases=["tls_enabled", "cluster_tls"]),
16911691
# Optional force first run services initialization
16921692
force=dict(type="bool", aliases=["forced_init"]),
16931693
# Optional auto-assign roles on cluster (honors existing assignments)

0 commit comments

Comments
 (0)