-
Notifications
You must be signed in to change notification settings - Fork 521
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
feat: Sentry 24.11.2 #1640
base: develop
Are you sure you want to change the base?
feat: Sentry 24.11.2 #1640
Conversation
ddb671c
to
6947220
Compare
@patsevanton My PR bumps sentry to fix security vulunability which was fixed in 24.11.1. |
I search topic for 24.9.0: |
The direct upgrade to 24.11 is fine. But yeah the tests need to pass. |
👋 Hi, @kimxogus, |
6947220
to
d46d8a8
Compare
After create new release #1648 i will create PR for update to https://github.com/getsentry/sentry/releases/tag/24.10.0 |
snuba-migrate job fails with this error.
|
hey @kimxogus Please let me know, if this branch is working? Thank You. |
7307063
to
2aa5d95
Compare
It works with current latest clickhouse chart. I'll update chart dependency when #1650 is released. |
Quickly tested as well (without changing values as presented in this PR), seems to run well (but I am not loaded yet, just deploying). |
Test succeeded :) |
👋 Hi, @kimxogus, |
@kimxogus Please remove topic snuba-metrics-summaries getsentry/snuba@24.10.0...24.11.2 |
All done |
@kimxogus
k get pod -n test | grep Error
|
git clone https://github.com/kimxogus/sentry-charts.git
|
@patsevanton Can you show the error logs? |
@kimxogus like:
|
@patsevanton Clickhouse chart |
@Mokto Can you review this? |
Approve |
getsentry/snuba@24.9.0...24.11.2#diff-fee8bd5753b6822df5d1569a8920c171f77c26f7ea7f087966666a0c50834de8L67
getsentry/snuba@6ea6caf
getsentry/sentry@24.9.0...24.11.2#diff-b64a93ab3e9c9979177910f15585257ed467776500fc3dc496b8ed434ec0ab02R29-R61
getsentry/snuba@1a9e9c9
getsentry/snuba@83f3d5a
getsentry/snuba@219fd2c
Closes #1655