Skip to content

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.

@wantaekchoi wantaekchoi closed this Aug 5, 2025
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