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

The Session Replay menu is not visible. #918

Closed
hyunkj42 opened this issue Jun 28, 2023 · 11 comments
Closed

The Session Replay menu is not visible. #918

hyunkj42 opened this issue Jun 28, 2023 · 11 comments
Labels

Comments

@hyunkj42
Copy link

Hello.
I upgraded to sentry 23.3.1. (18.0.0)
The relay, snuba-subscription-consumer-events, and snuba-subscription-consumer-transactions pods have errors, but the web is normally started.
However, the expected Session Replay menu is not visible.
image

expected Session Replay menu ㅜ
image

Can you tell me why?

@hiloyt
Copy link

hiloyt commented Jun 28, 2023

Because this is not a default option.

If you would like to see this menu, you need to add:

"organizations:session-replay-ui"

to the templates/configmap-sentry.yaml in the SENTRY_FEATURES.update( section.

@hyunkj42
Copy link
Author

@hiloyt
oh thank you :)
After adding the settings you mentioned, the menu will appear.
thank you.

@hyunkj42 hyunkj42 reopened this Jun 28, 2023
@hyunkj42
Copy link
Author

@hiloyt
But why is that value missing from configmap-sentry.yaml in the helm chart provided here?

I made the session replay menu appear, but does it work normally???
In fact, I haven't tested it yet because I don't know how to test it. ;;

@hiloyt
Copy link

hiloyt commented Jun 28, 2023

As I mentioned before, this option is set to false by default in the server.py for Sentry self-hosted. So I think the authors of this chart decide to do this as well (disabled by default). You still can find Replays in the projects, you don't need this sidebar link.

In my company, we are using this feature and works great. Of course, you need also to edit your apps to support this feature, not only enable it for the Sentry instance.

https://docs.sentry.io/product/session-replay/getting-started/?original_referrer=https%3A%2F%2Fwww.google.co.uk%2F

@bcarroll22
Copy link

@hiloyt I'm not sure this is true anymore.

https://github.com/getsentry/self-hosted/blob/master/sentry/sentry.conf.example.py#L253

https://github.com/sentry-kubernetes/charts/blob/develop/sentry/templates/configmap-sentry.yaml#L357

This repo doesn't seem to handle features the same way that self hosted does. This seems to overwrite the whole features array on line 357, whereas the self hosted only updates specific values. If you run a fresh install of the self-hosted version you'll have session replay and the menu using their features array, but not so with the helm version.

@hiloyt
Copy link

hiloyt commented Jun 28, 2023

@bcarroll22 They handling this exactly in the same way, but I didn't check server.py and they have this enabled from the default right now:

https://github.com/getsentry/sentry/blob/master/src/sentry/conf/server.py#L1539C17-L1539C17
getsentry/self-hosted#2010

So they removed this flag because is not anymore needed.

So something different is odd. Maybe the init process.

@bcarroll22
Copy link

Is there a reason you take the whole features array from the main Sentry repo instead of select values in the array like they do in the self-hosted repo? See here (note: link to self-hosted, not sentry):

https://github.com/getsentry/self-hosted/blob/master/sentry/sentry.conf.example.py#L253

I would think the approach the chart config takes currently would make it much harder to maintain, right?

@Mokto
Copy link
Contributor

Mokto commented Jul 29, 2023

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

@Mokto Mokto added the stale label Jul 29, 2023
@Mokto
Copy link
Contributor

Mokto commented Aug 13, 2023

This issue was closed because it has been inactive for 14 days since being marked as stale.

@Mokto Mokto closed this as completed Aug 13, 2023
@ohjongsung
Copy link

FYI,
it's work!

values.yaml

config:
  sentryConfPy: |
    # No Python Extension Config Given
    SENTRY_FEATURES["organizations:session-replay-ui"] = True

@hyunkj42
Copy link
Author

@ohjongsung
Hello.
Oh, there is such a way.
I solved it by adding it directly to the configmap-sentry.yaml config. ^^;

                "organizations:session-replay",
                "organizations:session-replay-ui",

thanks for the comment. :)

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