Skip to content

Commit

Permalink
Fixed ZooKeeper service usage to delete /controller znode on migratio…
Browse files Browse the repository at this point in the history
…n rollback (#10373)

Signed-off-by: Paolo Patierno <[email protected]>
  • Loading branch information
ppatierno authored Jul 24, 2024
1 parent 10e0c52 commit 36283d7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -898,7 +898,7 @@ protected Future<Void> maybeDeleteControllerZnode() {
*/
protected Future<Void> deleteControllerZnode() {
// migration rollback process ongoing
String zkConnectionString = KafkaResources.zookeeperServiceName(reconciliation.name()) + ":" + ZookeeperCluster.CLIENT_TLS_PORT;
String zkConnectionString = DnsNameGenerator.serviceDnsNameWithoutClusterDomain(reconciliation.namespace(), KafkaResources.zookeeperServiceName(reconciliation.name())) + ":" + ZookeeperCluster.CLIENT_TLS_PORT;
return KRaftMigrationUtils.deleteZooKeeperControllerZnode(
reconciliation,
vertx,
Expand Down

0 comments on commit 36283d7

Please sign in to comment.