This repository has been archived by the owner on Oct 10, 2022. It is now read-only.

WIP: Log rotation #31

Open

jsosulska wants to merge 4 commits into master from log_rotation

jsosulska commented Nov 4, 2020

This attempts to fix #22 .
Also, first commit here! 🥳

Updated README.
Variables.tf updated with new variables, and organized by alphabetical.
Updated Clients + Server(main.tf) variables.
Added logging.hcl to silo changes for logging.
Sized based rotation set as default.
Updated chmod/chown commands for path creations.

jsosulska added 3 commits

October 29, 2020 00:32


          Added Log rotation settings, strategies, and syslog

104540a


          Added log rotation options

04f39c9

fmt

743a404

jsosulska changed the title ~~Log rotation~~ WIP: Log rotation

ksatirli reviewed

View reviewed changes

modules/consul_cluster/scripts/install_hashitools_consul_client.sh.tpl Outdated Show resolved Hide resolved

ksatirli reviewed

View reviewed changes

modules/consul_cluster/scripts/install_hashitools_consul_server.sh.tpl Outdated Show resolved Hide resolved

ksatirli reviewed

View reviewed changes

modules/consul_cluster/variables.tf Show resolved Hide resolved

ksatirli reviewed

View reviewed changes

variables.tf Outdated Show resolved Hide resolved

ksatirli reviewed

View reviewed changes

variables.tf Outdated Show resolved Hide resolved

jsosulska commented

View reviewed changes

variables.tf Outdated Show resolved Hide resolved


          Apply suggestions from code review

ad150fc

ksatirli approved these changes

View reviewed changes

Member

ksatirli left a comment

This looks great to me!

hashicorp-cla commented Mar 12, 2022 •

edited

Loading

All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet