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: Add Redis dependency to resolve missing service error #290

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wantaekchoi
Copy link

Overview

This pull request addresses the issue of missing Redis dependency in the docker-compose.yml file, which was causing errors. The following changes have been made:

  1. Redis Service Addition:

    • Added a new service definition for Redis under x-chainlensredis-ref.
    • Defined the Redis container using the redis image with the container name chainlensredis.
  2. Chainlens API Updates:

    • Included environment variables REDIS_HOST and REDIS_PORT in x-chainlensapi-ref for Redis connectivity.
    • Updated the depends_on section in x-chainlensapi-ref to include chainlensredis.
  3. Chainlens Web Updates:

    • Added WS_API_URL environment variable in x-chainlensweb-ref.
  4. Service Configuration:

    • Added chainlensredis service to the main services section with an assigned IP address in the quorum-dev-quickstart network.

Motivation

This fix ensures that the Chainlens API functions correctly by resolving the error caused by the missing Redis service.

Testing

  • Verified that the docker-compose setup runs successfully with the new Redis service.
  • Ensured that the Chainlens API can connect to Redis and operate without errors.

- Added Redis service definition under `x-chainlensredis-ref`.
- Updated `x-chainlensapi-ref` to include Redis environment variables (`REDIS_HOST` and `REDIS_PORT`).
- Added `chainlensredis` to the `depends_on` section of `x-chainlensapi-ref`.
- Updated `x-chainlensweb-ref` to include `WS_API_URL` environment variable.
- Added `chainlensredis` service to the main `services` section with specific network configurations.

This update resolves the error caused by the missing Redis service, ensuring the Chainlens API functions correctly.
@CLAassistant
Copy link

CLAassistant commented Jul 4, 2024

CLA assistant check
All committers have signed the CLA.

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