weizhouapache commented on PR #9849: URL: https://github.com/apache/cloudstack/pull/9849#issuecomment-2434814585
> @vishesh92 maybe we can keep the default triggering policy to be time based but document how size based policy could be used? So the PR is essentially changes in replace.properties & a separate doc PR to given examples on configuration of logging. actually we could add both policies in the packages and install on mgmt server and agent, and let user to choose. The old time-based policy should be default. as @rohityadavcloud and @harikrishna-patnala said I also agree with @NuxRo that it would be better to add the timestamp to the filename, instead of `%i%. @vishesh92 is it possible ? `LOGMAXINDEX=10` is too small in my opinion. the compression size of a 100MB log could be smaller than 10MB. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
