Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ClickHouse 22.3.3.44 #85

Merged
merged 2 commits into from
Apr 15, 2022
Merged

Upgrade ClickHouse 22.3.3.44 #85

merged 2 commits into from
Apr 15, 2022

Conversation

pan3793
Copy link
Collaborator

@pan3793 pan3793 commented Apr 15, 2022

Docker image supports ARM64

@pan3793
Copy link
Collaborator Author

pan3793 commented Apr 15, 2022

v22.3.3.44 may has issue on ARM64
ClickHouse/ClickHouse#36286

@pan3793
Copy link
Collaborator Author

pan3793 commented Apr 15, 2022

@martin-g The ClickHouse 22.3.3.44 passed integration test on x86_64, would you please check if the integration test works on ARM64?

@pan3793 pan3793 merged commit 004e3fd into master Apr 15, 2022
@pan3793 pan3793 deleted the ck branch April 15, 2022 09:37
@martin-g
Copy link
Contributor

The problem seems to be in the ARM64 image itself.
On x86_64 I can do:

  1. docker run --rm -it clickhouse/clickhouse-server:22.3.3.44 bash
  2. root@5401f1d253f9:/# su-exec test
    Usage: su-exec user-spec command [args]

But on aarch64 it fails with segfault:

docker run --rm -it clickhouse/clickhouse-server:22.3.3.44 bash
/entrypoint.sh: line 39:    10 Segmentation fault      $gosu test -f "$CLICKHOUSE_CONFIG" -a -r "$CLICKHOUSE_CONFIG"
Configuration file '/etc/clickhouse-server/config.xml' isn't readable by user with id '101'

To enter in the docker container I need to override the entrypoint:

docker run --rm -it -v --entrypoint bash  clickhouse/clickhouse-server:22.3.3.44

and then trying to use su-exec segfaults:

 su-exec test
Segmentation fault

@pan3793
Copy link
Collaborator Author

pan3793 commented Apr 15, 2022

Sounds like another issue than ClickHouse/ClickHouse#36286 of ClickHouse, would you mind report it to the ClickHouse project? We can upgrade once the upstream releases the patched version.

@martin-g
Copy link
Contributor

@pan3793 ClickHouse/ClickHouse#36293

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