----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5907/#review9420 -----------------------------------------------------------
Ship it! Ship It! - disheng su On July 11, 2012, 10:23 a.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5907/ > ----------------------------------------------------------- > > (Updated July 11, 2012, 10:23 a.m.) > > > Review request for cloudstack. > > > Description > ------- > > Changes done: Modified log4j.xml to restrict the growing of file size > depending upon the total size.Defaulting it to 10MB for each log file and > keeping a maximum backup of 4 files. > (Path for the log files and rotation policy was not configured correctly). > > > Diffs > ----- > > build/deploy/production/server/conf/log4j-cloud.xml 1b56396 > > Diff: https://reviews.apache.org/r/5907/diff/ > > > Testing > ------- > > Tested the same changes on log4j-cloud.xml config file present in the system > vms.Verified the 3 files /var/log/cloud.log, /var/log/cloud/cloud.out and > /usr/local/cloud/systemvm/cloud.log > were rotated correctly. > > > Thanks, > > Saksham Srivastava > >