Conversation
mariacarmina
requested review from
alexcos20,
kremalicious and
mihaisc
as code owners
|
Output: aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | default log level: 20, env var LOG_LEVEL INFO
aquarius_1 | 2022-09-20 18:59:41,195 - aquarius.events.events_monitor - INFO - Added 8996 to chains list
aquarius_1 | 2022-09-20 18:59:41 26fe708664e0 root[25] INFO Stable connection to ES.
aquarius_1 | 2022-09-20 18:59:41,269 - aquarius.events.events_monitor - INFO - PURGATORY: Enabling purgatory
aquarius_1 | 2022-09-20 18:59:41,269 - aquarius.events.events_monitor - INFO - Starting the events monitor.
aquarius_1 | 2022-09-20 18:59:41 26fe708664e0 __main__[25] INFO EventsMonitor: started
aquarius_1 | 2022-09-20 18:59:41 26fe708664e0 root[25] INFO Stable connection to ES.When I start barge with aquarius, there are only |
compose-files/thegraph.yml
Outdated
| ipfs: '172.15.0.16:5001' | ||
| ethereum: 'development:${NETWORK_RPC_URL}' | ||
| RUST_LOG: info | ||
| RUST_LOG: ${RUST_LOG} |
There was a problem hiding this comment.
can we name it SUBGRAPH_LOG_LEVEL ?
start_ocean.sh
Outdated
| printf $COLOR_Y'Starting without IPFS...\n\n'$COLOR_RESET | ||
| ;; | ||
| --with-thegraph) | ||
| export RUST_LOG=${RUST_LOG:-INFO} |
There was a problem hiding this comment.
can we move this above, where all other _LOG_LEVELS statements are located ?
There was a problem hiding this comment.
I declared the log levels for each component and where they were separated with #. Should I define the log levels at the top of the script?
There was a problem hiding this comment.
yes, have a section with all envs related to logging. maybe around line 43 in start_ocean.sh
compose-files/thegraph.yml
Outdated
| ipfs: '172.15.0.16:5001' | ||
| ethereum: 'development:${NETWORK_RPC_URL}' | ||
| RUST_LOG: info | ||
| SUBGRAPH_LOG_LEVEL: ${SUBGRAPH_LOG_LEVEL} |
There was a problem hiding this comment.
| SUBGRAPH_LOG_LEVEL: ${SUBGRAPH_LOG_LEVEL} | |
| RUST_LOG: ${SUBGRAPH_LOG_LEVEL} |
There was a problem hiding this comment.
because graph_node has no idea what SUBGRAPH_LOG_LEVEL is :)
start_ocean.sh
Outdated
| # Export LOG LEVEL | ||
| export AQUARIUS_LOG_LEVEL=${AQUARIUS_LOG_LEVEL:-INFO} | ||
| export PROVIDER_LOG_LEVEL=${PROVIDER_LOG_LEVEL:-INFO} | ||
| export SUBGRAPH_LOG_LEVEL=${SUBGRAPH_LOG_LEVEL:-INFO} |
There was a problem hiding this comment.
| export SUBGRAPH_LOG_LEVEL=${SUBGRAPH_LOG_LEVEL:-INFO} | |
| export SUBGRAPH_LOG_LEVEL=${SUBGRAPH_LOG_LEVEL:-info} |
There was a problem hiding this comment.
not sure if graph_node debug level is case sensitive or not, so better keep what we had
There was a problem hiding this comment.
Ok sure, it was previously with lowercase, I will keep as it was.
|
Can you also update the README, to reflect the changes? |
2 participants
Description
Removed Aquarius debug logs. Set env var to establish log level
Is this PR related with an open issue?
Related to Issue #314
Types of changes