search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.14k stars 4.2k forks source link

Dragonfly Compatibility #80440

Open nadecancode opened 1 month ago

nadecancode commented 1 month ago

Environment

self-hosted (https://develop.sentry.dev/self-hosted/)

Steps to Reproduce

Use Dragonfly instead of Redis

Expected Result

EVALSHA runs successfully

Actual Result

EVALSHA fails with error bad argument getsentry/sentry#1 to randomseed

Product Area

Unknown

Link

No response

DSN

No response

Version

latest available

nadecancode commented 1 month ago

https://github.com/getsentry/sentry/pull/65825 This PR addresses the issue but it somehow got closed with "no activity". It would be nice if someone from Sentry can reopen that PR to not only support Dragonfly as well but also expand Lua version compatibility in case Redis wants to move to a higher version of Lua engine.

getsantry[bot] commented 1 month ago

Assigning to @getsentry/support for routing ⏲️

aldy505 commented 1 month ago

@InterstellarStella Hello, can you please transfer this back into getsentry/sentry as we (as in the self-hosted team) can't do anything about this, since the changes that needs to be made shall occur in the getsentry/sentry repository? Thank you.

getsantry[bot] commented 1 month ago

Assigning to @getsentry/support for routing ⏲️

aldy505 commented 1 month ago

@JLuse Hello, any specific reason why you decided to transfer this issue back into self-hosted?

getsantry[bot] commented 5 days ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 5 days ago

Routing to @getsentry/product-owners-processing for triage ⏲️

loewenheim commented 5 days ago

https://github.com/getsentry/sentry/pull/65825 has been merged.