Dan Fuller

@komal-saini This is ready to go from my point of view, just want to make sure that this isn't called from anywhere until https://github.com/getsentry/sentry/pull/37378 merges?

Thanks for the suggestion @lucas-zimerman. We've added this to our backlog for consideration in the future.

We can run this against getsentry via https://develop.sentry.dev/workflow/

@ethanhs This sort of alert is used pretty widely on sentry.io and is working. This makes me think that this is specific to either self-hosted, or this instance. Likely something...

I feel like we'd be better off figuring out the specific problem rather than hacking around this with a new table

It looks like `SENTRY_CACHE` should be set to `sentry.cache.redis.RedisCache`, and `SENTRY_OPTIONS` should have an entry like `SENTRY_OPTIONS["redis.clusters"]["default"] = {}` There's an example of this in our docker config file here:...

> @getsentry/crons There are a few failing tests in these results from the monitor details endpoint. > > https://github.com/getsentry/sentry/blob/793c62cb6cf3f99dc8fe6d13de5945a333bf09c0/src/sentry/monitors/endpoints/base_monitor_details.py#L181-L247 > > This passage of code currently makes a cross database...

> @wedamija It does, i can add tests for it if you think it's worth it As long as that functionality is already tested I think it's ok. If the...

If we want, we could just make this api per project as well, instead of having two separate ones

> > If we want, we could just make this api per project as well, instead of having two separate ones > > could add a query parameter to filter...