Feat: Change in Tenderdash Role to investigate the chain halt issue.

vivekgsharma commented 4 months ago

Issue Being Fixed or Feature Implemented

Configured log rotation for Tenderdash logs
Added a Logstash filter to exclude TRACE logs from processing to optimize log
Updated the tenderdash_log_level to "trace" in the Ansible variables

Changes Made

Added a logrotate configuration for Tenderdash logs to ensure logs are rotated daily, compressed, and old logs are removed after three days.
Implemented a Logstash configuration to drop TRACE level logs to prevent them from being processed and stored
Set the tenderdash_log_level variable to "trace" in the ansible Varibales

How Has This Been Tested?

Breaking Changes

No breaking changes introduced.

Checklist:

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

vivekgsharma commented 4 months ago

Test is failing due to Risky file permission due to this line create 0640 tenderdash adm in the new added task , this permission might be necessary to access the log files. I can ignore this check after the review adding ignore lines in .ansible-lint

ktechmidas commented 4 months ago

@vivekgsharma what is adm? can't it use 600?

vivekgsharma commented 4 months ago

@vivekgsharma what is adm? can't it use 600?

adm is a system group, members of this group can read logs in various directories (/var/log). We can try 600

Reference for using 0640 https://serverfault.com/questions/694707/what-should-be-the-right-logs-permissions-for-nginx-on-centos

vivekgsharma commented 4 months ago

@ktechmidas changed to 0644 because most of the tasks are using this mode in the current setup. Ansible-lint checks have passed

ktechmidas commented 1 day ago

Closed - role no longer in use

dashpay / dash-network-deploy