Skip to content

Conversation

@dmanc
Copy link
Contributor

@dmanc dmanc commented Apr 8, 2025

No description provided.

@dmanc dmanc requested a review from ian-shim April 8, 2025 22:25
- "${NODE_CACHE_PATH_HOST}:/app/cache:rw"
- "${NODE_LOG_PATH_HOST}:/app/logs:rw"
- "${NODE_DB_PATH_HOST}:/data/operator/db:rw"
# - "${NODE_BLS_SIGNER_CERT_FILE_HOST}:/app/cerberus/cerberus.crt:readonly"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this intentional? Should we add a description

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yea, it's ugly, but Docker compose will fail if uncommented and the cert doesn't exist. I'm not opposed to removing it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Was just matching what testnet setup had. Added comment


### 2. Update `MAIN_SERVICE_IMAGE`
```
MAIN_SERVICE_IMAGE=ghcr.io/layr-labs/eigenda/opr-node:0.9.0-rc.4
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be ghcr.io/layr-labs/eigenda/opr-node:0.9.0

@ian-shim
Copy link
Contributor

ian-shim commented Jun 2, 2025

@cody-littley do we need to update the default cache memory + nginx setting here?

@dmanc dmanc merged commit 0ed1c62 into master Jun 5, 2025
@dmanc dmanc deleted the release/0.9.0 branch June 5, 2025 00:40
@dmanc dmanc restored the release/0.9.0 branch June 5, 2025 00:40
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.

3 participants