8000 Add replica restart fix for operator when remote-sever configmap is propagated by oo007 · Pull Request #1 · oo007/clickhouse-operator · GitHub
[go: up one dir, main page]

Skip to content

Add replica restart fix for operator when remote-sever configmap is propagated - #1

Open
oo007 wants to merge 2 commits into
masterfrom
replica-restart-fix-operator
Open

oo007 wants to merge 2 commits into
masterfrom
replica-restart-fix-operator

Conversation

@oo007
@oo007 oo007 commented Jul 1, 2026
Copy link
Copy Markdown
Owner

Add replica restart fix for operator when remote-sever configmap is propagated

cr.GetRuntime().UnlockCommonConfig()

w.includeAllHostsIntoCluster(ctx, cr)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove these comments. Altinity maintainers avoid unnecessary comments as they look AI generated.

Comment thread pkg/controller/chi/worker-reconciler-chi.go Outdated
}

// restartCreatedHosts restarts, once, each host that was created during the current reconcile.
//
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leave the first comment, but remove rest of the comments. Altinity maintainers avoid unnecessary comments as they look AI generated.

w.a.V(1).M(host).F().
Info("Restart created host after final remote_servers propagation. Host: %s", host.GetName())

// Ensure the final common ConfigMap has propagated to this host before restarting,
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0