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

Fix Redis keys #201

Merged
merged 9 commits into from
Oct 8, 2024
Merged

Fix Redis keys #201

merged 9 commits into from
Oct 8, 2024

Conversation

catileptic
Copy link
Contributor

@catileptic catileptic commented Jul 17, 2024

The new RabbitMQ task queue implementation introduced new Redis keys. In the initial implementation, their structure was malformed due to a bug in how make_key was called.

Instead of having redis keys look like this: tq:qds:0000:analyze:finished they looked like this: tq#qds#0000#analyze:finished.

This PR fixed the problem in two stages:

First, it introduced code that creates and uses correct keys.

Then, it temporarily introduced code that delete the incorrect keys. This code was removed after all incorrect keys have been removed from the Aleph production environment.

@catileptic catileptic marked this pull request as draft July 17, 2024 17:01
@catileptic catileptic marked this pull request as ready for review September 25, 2024 16:49
@catileptic catileptic changed the title Fix Redis keys (temporary solution) Fix Redis keys Oct 8, 2024
@catileptic catileptic merged commit 59629f1 into release/1.23.0 Oct 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants