fix: talosctl cluster create not to enforce kubeprism always

The command should be able to deploy old versions of Talos as well, even before KubePrism. The version contract correctly enables/disables KubePrism by default, so take default flag value as "don't change defaults". Signed-off-by: Andrey Smirnov <[email protected]> (cherry picked from commit e128d3c)
siderolabs · Dec 8, 2023 · fe66611 · fe66611
1 parent 41fc054
commit fe66611
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.conform.yaml b/.conform.yaml
@@ -8,7 +8,7 @@ policies:
           gitHubOrganization: siderolabs
       spellcheck:
         locale: US
-      maximumOfOneCommit: true
+      maximumOfOneCommit: false
       header:
         length: 89
         imperative: true

diff --git a/cmd/talosctl/cmd/mgmt/cluster/create.go b/cmd/talosctl/cmd/mgmt/cluster/create.go
@@ -523,9 +523,11 @@ func create(ctx context.Context, flags *pflag.FlagSet) (err error) {
 			)
 		}
 
-		genOptions = append(genOptions,
-			generate.WithKubePrismPort(kubePrismPort),
-		)
+		if kubePrismPort != constants.DefaultKubePrismPort {
+			genOptions = append(genOptions,
+				generate.WithKubePrismPort(kubePrismPort),
+			)
+		}
 
 		defaultInternalLB, defaultEndpoint := provisioner.GetLoadBalancers(request.Network)