search

flexera-public / rightscale_cookbooks

Other
76 stars 103 forks source link

Sprint12 05 a46724 reduce ntp logging verbosity #66

Closed arangamani closed 12 years ago

arangamani commented 12 years ago

Changed the template ntp.conf.erb to include only major events in log. It was checked by running the server for few hours and by changing the time manually which wrote a single line in the /var/log/messages. {{{ Jun 7 14:07:03 ip-10-36-10-212 ntpd[3831]: time reset -610.032977 s }}}

ekargard commented 12 years ago

Go ahead and merge

arangamani commented 12 years ago

Is it a command to me?

On Thu, Jun 7, 2012 at 6:31 PM, ekargard < reply@reply.github.com

wrote:

Go ahead and merge

Reply to this email directly or view it on GitHub:

https://github.com/rightscale/rightscale_cookbooks/pull/66#issuecomment-6192507

ekargard commented 12 years ago

Command?? More like a request. There may be multiple reviewers so each of one needs to give the go ahead to merge. Once everyone agrees the developer can do the merge. In changes like this one I think the reviewer should just go ahead and do the merge.

arangamani commented 12 years ago

I didn't mean to stress on that word 'command'. I wanted to make sure if you asked me to do it or someone else to. I'll go ahead and merge it.

On Thu, Jun 7, 2012 at 8:03 PM, ekargard < reply@reply.github.com

wrote:

Command?? More like a request. There may be multiple reviewers so each of one needs to give the go ahead to merge. Once everyone agrees the developer can do the merge. In changes like this one I think the reviewer should just go ahead and do the merge.

Reply to this email directly or view it on GitHub:

https://github.com/rightscale/rightscale_cookbooks/pull/66#issuecomment-6193508