Merge pull request #166 from deviceinsight/feature/random-pod-name

Kubernetes pod name now always includes random suffix
deviceinsight · Sep 25, 2023 · 15d5fe3 · 15d5fe3
2 parents 35144d3 + 4ab29e0
commit 15d5fe3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- [#165](https://github.com/deviceinsight/kafkactl/issues/165) Kubernetes pod name now always includes random suffix 
+
 ## 3.3.0 - 2023-09-05
 ### Fixed
 - [#162](https://github.com/deviceinsight/kafkactl/pull/162) Fix `consumer-group` crashes when a group member has no assignment

diff --git a/internal/k8s/executor.go b/internal/k8s/executor.go
@@ -119,11 +119,7 @@ func (kubectl *executor) Run(dockerImageType, entryPoint string, kafkactlArgs []
 		return err
 	}
 
-	podName := "kafkactl-" + randomString(10)
-
-	if kubectl.clientID != "" {
-		podName = "kafkactl-" + strings.ToLower(kubectl.clientID)
-	}
+	podName := fmt.Sprintf("kafkactl-%s-%s", strings.ToLower(kubectl.clientID), randomString(4))
 
 	kubectlArgs := []string{
 		"run", "--rm", "-i", "--tty", "--restart=Never", podName,