Skip to content

Handling CS Start if Redis not present #529

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

Merged
merged 4 commits into from
May 22, 2024
Merged

Conversation

farhanW3
Copy link
Contributor

@farhanW3 farhanW3 commented May 17, 2024

Issue:

If you have CS setup but no redis, it will not run on startup, but if the configuration table values are updated, we catch the update & restart all cron. Thus, we missed this check

Resolution:
Updated the place where we check if redis is present in the chainIndexerListener flow.


PR-Codex overview

This PR removes a conditional check for Redis availability before starting the Chain Indexer Listener in the worker module.

Detailed summary

  • Removed conditional check for Redis availability before starting Chain Indexer Listener
  • Updated chainIndexerListener.ts to always check for Redis before starting the listener

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@farhanW3 farhanW3 requested a review from arcoraven May 17, 2024 18:55
@farhanW3 farhanW3 merged commit 0825932 into main May 22, 2024
4 checks passed
@farhanW3 farhanW3 deleted the fk/update-redis-check branch May 22, 2024 23:08
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