dash-network-deploy icon indicating copy to clipboard operation
dash-network-deploy copied to clipboard

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

Open vivekgsharma opened this issue 1 year ago • 4 comments

Issue Being Fixed or Feature Implemented

  1. Configured log rotation for Tenderdash logs
  2. Added a Logstash filter to exclude TRACE logs from processing to optimize log
  3. 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 avatar May 08 '24 15:05 vivekgsharma

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

vivekgsharma avatar May 08 '24 16:05 vivekgsharma

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

ktechmidas avatar May 09 '24 06:05 ktechmidas

@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 avatar May 09 '24 06:05 vivekgsharma

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

vivekgsharma avatar May 09 '24 07:05 vivekgsharma

Closed - role no longer in use

ktechmidas avatar Sep 23 '24 13:09 ktechmidas