-
Notifications
You must be signed in to change notification settings - Fork 16
chore: more docker improvements #2004
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
Changes from all commits
4bd7456
6934f56
3bd5099
afe2674
e41d950
826d54a
5864c96
f2e1c36
34ec5cc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,22 +1,22 @@ | ||
| # Minimal compose for CI integration tests. | ||
| # Provides only the infrastructure services needed by orchestrator.ts: | ||
| # devnet (local EVM) and postgres (database). | ||
| # devnet (local EVM) and ensdb (database). | ||
| services: | ||
| devnet: | ||
| extends: | ||
| file: services/devnet.yml | ||
| service: devnet | ||
|
|
||
| postgres: | ||
| ensdb: | ||
| extends: | ||
| file: services/postgres.yml | ||
| service: postgres | ||
| file: services/ensdb.yml | ||
| service: ensdb | ||
| env_file: | ||
| - path: ./envs/.env.docker.common | ||
| - path: envs/.env.docker.common | ||
| required: true | ||
|
|
||
| volumes: | ||
| # Docker Compose requires volumes used by services to be declared in each | ||
| # compose file that references them — they cannot be inherited via `extends`. | ||
| postgres_data: | ||
| ensdb_data: | ||
| driver: local |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,16 +1,15 @@ | ||
| # Shared Docker Compose environment variables | ||
| # Used by docker/docker-compose.yml and its variants | ||
|
|
||
|
|
||
| # Postgres | ||
|
sevenzing marked this conversation as resolved.
|
||
| POSTGRES_DB=postgres | ||
| POSTGRES_USER=postgres | ||
| POSTGRES_PASSWORD=password | ||
|
|
||
| # Internal service URLs (container-to-container) | ||
| ENSDB_URL=postgresql://postgres:password@postgres:5432/postgres | ||
| ENSDB_URL=postgresql://postgres:password@ensdb:5432/postgres | ||
| ENSRAINBOW_URL=http://ensrainbow:3223 | ||
|
|
||
| # ENS Admin | ||
| # ENSAdmin | ||
| ENSADMIN_PUBLIC_URL=http://localhost:4173 | ||
|
sevenzing marked this conversation as resolved.
|
||
| NEXT_PUBLIC_SERVER_CONNECTION_LIBRARY=http://localhost:4334 | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,18 @@ | ||
| # Default configuration for devnet docker-compose stack. | ||
| # These values work out of the box — override by creating .env.docker.local. | ||
|
|
||
| # ENSIndexer | ||
| PLUGINS=subgraph,ensv2 | ||
| # ENSIndexer and ENSApi | ||
| ENSINDEXER_SCHEMA_NAME=docker_devnet_v1 | ||
| # ENSIndexer and ENSApi | ||
| RPC_URL_1=http://devnet:8545 | ||
| # ENSIndexer and ENSRainbow | ||
| LABEL_SET_VERSION=0 | ||
| # ENSIndexer and ENSRainbow | ||
| LABEL_SET_ID=ens-test-env | ||
| # ENSRainbow | ||
| DB_SCHEMA_VERSION=3 | ||
|
sevenzing marked this conversation as resolved.
|
||
| # TODO: in future we might migrate devnet to different chain_id like 1337 or 31337 | ||
| # need to update this `RPC_URL_15658733` in that case | ||
| RPC_URL_15658733=http://devnet:8545 | ||
|
Comment on lines
+4
to
+18
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Reorder these env keys before merging.
🧰 Tools🪛 dotenv-linter (4.0.0)[warning] 7-7: [UnorderedKey] The ENSINDEXER_SCHEMA_NAME key should go before the PLUGINS key (UnorderedKey) [warning] 11-11: [UnorderedKey] The LABEL_SET_VERSION key should go before the PLUGINS key (UnorderedKey) [warning] 13-13: [UnorderedKey] The LABEL_SET_ID key should go before the LABEL_SET_VERSION key (UnorderedKey) [warning] 15-15: [UnorderedKey] The DB_SCHEMA_VERSION key should go before the ENSINDEXER_SCHEMA_NAME key (UnorderedKey) 🤖 Prompt for AI Agents |
||
Uh oh!
There was an error while loading. Please reload this page.