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

cacheutil: upgrade rueidis to 0.0.93 to fix https://github.com/rueian/rueidis/pull/170 #6098

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

rueian
Copy link
Contributor

@rueian rueian commented Feb 2, 2023

Hi @GiedriusS, @douglascamata, @yeya24,

A user reported a panic bug when using Thanos v0.30.1 with an old Redis here redis/rueidis#55 (comment).

The bug is only possible to be triggered when the client-side caching is disabled and the first request to a Redis connection is timed out. So, if the client-side caching is enabled, then the bug will not be triggered.

A fix has been addressed in redis/rueidis#170 and this PR upgrades rueidis to v0.0.93 to apply it.

@rueian rueian force-pushed the upgrade-rueidis-0.0.93 branch 2 times, most recently from a548546 to f0bcf69 Compare February 2, 2023 13:51
@pull-request-size pull-request-size bot added size/L and removed size/M labels Feb 2, 2023
@rueian rueian force-pushed the upgrade-rueidis-0.0.93 branch 2 times, most recently from 3adf9c4 to 782ef5c Compare February 2, 2023 14:10
Copy link
Contributor

@douglascamata douglascamata left a comment

Choose a reason for hiding this comment

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

LGTM and thanks again for being so helpful, @rueian!

Copy link
Contributor

@douglascamata douglascamata left a comment

Choose a reason for hiding this comment

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

Could you add a changelog entry, please? 🙏

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Yeah as mentioned by @douglascamata let's have a changelog entry! Otherwise LGTM.

@rueian rueian force-pushed the upgrade-rueidis-0.0.93 branch from 782ef5c to 99bb689 Compare February 8, 2023 15:12
@rueian rueian force-pushed the upgrade-rueidis-0.0.93 branch from 99bb689 to 0121edc Compare February 8, 2023 15:32
@rueian
Copy link
Contributor Author

rueian commented Feb 8, 2023

Thanks @douglascamata, @yeya24! I have updated the CHANGELOG.md.

@yeya24 yeya24 enabled auto-merge February 8, 2023 16:43
@yeya24 yeya24 merged commit 6073b82 into thanos-io:main Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants