hyperledger / indy-node-container

Providing Containers to run Indy Node
Apache License 2.0
21 stars 22 forks source link

Improve Logging #83

Closed solidnerd closed 1 year ago

solidnerd commented 2 years ago

This PR introducing a lot of changes to the handling of the Logging. Introducing new variables:

Closes #68 #74

solidnerd commented 2 years ago

Afterwards in the container it will be set

 docker exec -it $(docker ps -ql)  cat /etc/indy/indy_config.py
# Current network
NETWORK_NAME = "sandbox"

# Disable stdout logging
enableStdOutLogging = False

# Directory to store ledger.
LEDGER_DIR = '/var/lib/indy'

# Directory to store logs.
LOG_DIR = "/var/log/indy"

# Directory to store keys.
KEYS_DIR = '/var/lib/indy'

# Directory to store genesis transactions files.
GENESIS_DIR = '/var/lib/indy'

# Directory to store backups.
BACKUP_DIR = '/var/lib/indy/backup'

# Directory to store plugins.
PLUGINS_DIR = '/var/lib/indy/plugins'

# Directory to store node info.
NODE_INFO_DIR = '/var/lib/indy'

# Defines the LogLevel of IndyNode 
logLevel = 0
Echsecutor commented 2 years ago

Please add new env variables to .env

Echsecutor commented 2 years ago

@solidnerd do you have a few minutes to have another look at this PR? Or should I just do the suggested changes and have someone else review?

solidnerd commented 2 years ago

@Echsecutor I had some problems with my GitHub notifications in the past that was the reason why I didn't responded so fast. Everything you mentioned was done from my side. So lets meet next week and check it again.