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

Update sentry to 24.1.1 #1126

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

farodin91
Copy link
Contributor

No description provided.

@farodin91 farodin91 force-pushed the update-sentry-to-24.1.1 branch from 2745ff0 to 7ddf821 Compare January 31, 2024 14:23
@farodin91
Copy link
Contributor Author

@Mokto Would you like to review?

Copy link
Contributor

@Mokto Mokto left a comment

Choose a reason for hiding this comment

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

LGTM but I'm waiting for a few days to see if everything work as intended on my cluster.

@Mokto
Copy link
Contributor

Mokto commented Jan 31, 2024

Screenshot 2024-01-31 at 20 16 12

@farodin91 farodin91 force-pushed the update-sentry-to-24.1.1 branch from 7ddf821 to 72199b0 Compare February 2, 2024 07:12
@farodin91
Copy link
Contributor Author

@Mokto It should be fixed now.

@hd-deman
Copy link

hd-deman commented Feb 2, 2024

faced with same problem (getsentry/self-hosted#2728) after upgrade to 24.1.1

so we also need to support
this:
https://github.com/getsentry/self-hosted/pull/2722/files
here:
https://github.com/sentry-kubernetes/charts/blob/develop/sentry/templates/configmap-sentry.yaml#L146

It worked for me, but I don't want to create additional PR

Signed-off-by: Jan Jansen <[email protected]>
@farodin91 farodin91 force-pushed the update-sentry-to-24.1.1 branch from 72199b0 to fc40258 Compare February 5, 2024 07:32
@farodin91
Copy link
Contributor Author

@hd-deman updated.

@Mokto Mokto merged commit 101502a into sentry-kubernetes:develop Feb 5, 2024
1 check passed
@Mokto
Copy link
Contributor

Mokto commented Feb 5, 2024

Thanks!

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.

3 participants