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

snuba events_analytics_ platform requires clickhouse 23.3 #1474

Open
1 task done
ntcong opened this issue Sep 24, 2024 · 6 comments
Open
1 task done

snuba events_analytics_ platform requires clickhouse 23.3 #1474

ntcong opened this issue Sep 24, 2024 · 6 comments
Labels

Comments

@ntcong
Copy link
Contributor

ntcong commented Sep 24, 2024

Issue submitter TODO list

  • I've searched for an already existing issues here

Describe the bug (actual behavior)

The upgrading process failed with snuba-migrate because events_analytics_ platform can't be migrated
Executing snuba migrations migrate manually and it fails

  File "/usr/local/lib/python3.11/site-packages/clickhouse_driver/client.py", line 237, in receive_packet
    raise packet.exception
clickhouse_driver.errors.ServerException: Code: 36.
DB::Exception: Codec DoubleDelta does not accept any arguments. Stack trace:

0. DB::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<
char> > const&, int, bool) @ 0xa40325a in /usr/bin/clickhouse
1. ? @ 0x140fec52 in /usr/bin/clickhouse
2. ? @ 0x14e1cf6e in /usr/bin/clickhouse
3. DB::ISerialization::enumerateStreams(DB::ISerialization::SubstreamPath&, std::__1::function<void (DB::I
Serialization::SubstreamPath const&)> const&, DB::ISerialization::SubstreamData const&) const @ 0x14238771
 in /usr/bin/clickhouse

Expected behavior

migrations successful

values.yaml

.

Helm chart version

25.9.0

Steps to reproduce

official self-hosted image uses 23.8 and automatically upgrade clickhouse to 23.8
https://github.com/getsentry/self-hosted/blob/master/install/upgrade-clickhouse.sh

I tried setting clickhouse image to 23.3.19.32 and the migrations went through.

Screenshots

No response

Logs

No response

Additional context

No response

@patsevanton
Copy link
Contributor

Please write your values.yaml

@shcherbak
Copy link
Contributor

shcherbak commented Sep 27, 2024

we facing similar problem with release 24.8
getsentry/snuba#6355
there is a syntax error in migration query

events_analytics_platform (readiness_state: limited)
[-]  0001_spans (IN PROGRESS)

@swade1987
Copy link

swade1987 commented Oct 4, 2024

I also raised the same issue #1508 but mine is for a total fresh installation.

When you read this error: getsentry/snuba#6355, it matches the exact point at which my failure occurred.

2024-10-04 11:19:58,663 Query: SELECT version FROM migrations_dist FINAL WHERE group = 'events_analytics_platform' AND migration_id = '0001_spans’; ⁠

Then I have a failure straight after ⁠ 0001_spans ⁠

@patsevanton
Copy link
Contributor

patsevanton commented Oct 14, 2024

Try tag 25.16.0

@patsevanton
Copy link
Contributor

@Mokto
Copy link
Contributor

Mokto commented Nov 23, 2024

This issue is stale because it has been open for 30 days with no activity.

@Mokto Mokto added the stale label Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants