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.

Sign up for GitHub

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

Jump to bottom

Remove references to Redis as a transport #887

Closed

asachs01 opened this issue Nov 7, 2018 · 0 comments

Labels

1.x

Contributor

asachs01 commented Nov 7, 2018 •

edited by apaskulin

I propose we remove all references to Redis as a transport for Sensu:

Because it's not supported for production
Because of Redis transport duplicate checks sensu-transport#31
It's not an advised best practice
Having it as an option when it is obviously broken leads to a poor user experience and an influx of support cases where we have to say, "Sorry, but this isn't a best practice"

apaskulin changed the title ~~Ditch any reference to Redis as a transport~~ Remove references to Redis as a transport

apaskulin added the 1.x label

asachs01 mentioned this issue

Adding danger class #888

Closed

asachs01 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment