8000 Improve clickhouse-keeper manifests by Slach · Pull Request #1234 · Altinity/clickhouse-operator · GitHub
[go: up one dir, main page]
Skip to content

Improve clickhouse-keeper manifests - #1234

Merged
Slach merged 17 commits into
0.23.0from
improve_keeper
Nov 23, 2023
Merged

Improve clickhouse-keeper manifests#1234
Slach merged 17 commits into
0.23.0from
improve_keeper

Conversation

@Slach
@Slach Slach commented Sep 4, 2023 •
edited
Loading
Copy link
Copy Markdown
Collaborator

Depends on ClickHouse/ClickHouse#53481

This will help a lot for maintainers to adopt your Pull Request.

Important items to consider before making a Pull Request

Please check items PR complies to:

  • All commits in the PR are squashed. More info
  • The PR is made into dedicated next-release branch, not into master branch1. More info
  • The PR is signed. More info

--

1 If you feel your PR does not affect any Go-code or any testable functionality (for example, PR contains docs only or supplementary materials), PR can be made into master branch, but it has to be confirmed by project's maintainer.

Slach added 3 commits August 30, 2023 18:25
@Slach
prepare clickhouse-keeper improvements
69ba9d9 
Signed-off-by: Slach <bloodjazman@gmail.com>
@Slach
Merge branch '0.22.0' of github.com:Altinity/clickhouse-operator into…
6230cef 
… improve_keeper
@Slach
clickhouse-keeper startup works, but test_keeper.py still not pass (s…
ea177b9 
…omething wrong with tearDown.sh, wrong NuRaft quorum state after replicas: 0 -> replicas: 3)

Signed-off-by: Slach <bloodjazman@gmail.com>
@Slach
Slach changed the base branch from master to 0.22.0 September 4, 2023 18:16
Slach added 2 commits September 5, 2023 12:13
@Slach
scale replicas 0 -> replicas 3 still not work, test_keeper.py failed
0b28e52 
Signed-off-by: Slach <bloodjazman@gmail.com>
@Slach
test_keeper.py passed for clickhouse/clickhouse-keeper:latest-alpine …
aee6be1 
…(23.8)

Signed-off-by: Slach <bloodjazman@gmail.com>
@Slach Slach changed the title [WIP] Improve clickhouse-keeper manifests Improve clickhouse-keeper manifests Sep 5, 2023
@Slach Slach changed the title Improve clickhouse-keeper manifests [wip] Improve clickhouse-keeper manifests Sep 5, 2023
@Slach
Slach commented Sep 5, 2023
Copy link
Copy Markdown
Collaborator Author

Wait when resolve ClickHouse/ClickHouse#53481 and ClickHouse/ClickHouse#54129

@bputt-e
bputt-e commented Sep 13, 2023
Copy link
Copy Markdown

does this fix the issue where force-recovery was always set to the first pod if I'm running a 3 clickkeeper cluster? It seems odd that force-recovery was always set to the first pod as if that pod were restarted or offline for a period of time, it effectively tells the rest of the clickkeeper instances that they must use the first pods state

@Slach Slach mentioned this pull request Sep 13, 2023
Closed
@Slach
Slach commented Sep 13, 2023 •
edited
Loading
Copy link
Copy Markdown
Collaborator Author

@bputt-e
using --force-recovery in old manifest required for prefer XML configs more priority then internal NuRaft state which stored in /var/lib/clickhouse-keeper, without this option even when you change XML config <raft_configuration> it will have no effect

in the new approach, we are trying to use command reconfig, which also stay not fully operable

@Slach Slach mentioned this pull request Nov 2, 2023
Closed
@Slach
Slach changed the base branch from 0.22.0 to 0.23.0 November 10, 2023 14:48
@Slach Slach mentioned this pull request Nov 20, 2023
Closed
@Slach Slach changed the title [wip] Improve clickhouse-keeper manifests Improve clickhouse-keeper manifests Nov 23, 2023
@Slach
Slach merged commit cd9b7fa into 0.23.0 Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunsingerus sunsingerus Awaiting requested review from sunsingerus
@alex-zaitsev alex-zaitsev Awaiting requested review from alex-zaitsev

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Slach @bputt-e
0