upport log_max_age

[7ing]

LogMaxAge is the maximum # of days (exclusive) to retain old files based on the timestamp encoded in their filename.
log_max_age <= 0 means delete immediately, otherwise it retains the files that are created within logMaxAge days.

[7ing]

@thrawn01 @klizhentas @k-pom @anton-efimenko @pirogoeth
anyone maintain this repo? looks like the CI has very old version for GO, which makes the checks fails.

[thrawn01]

@7ing Thanks for the Pull Request!

In your pull request, remove the old golang versions from travis.yml and add 1.6 and 1.7 as the test targets

Then we can go from there.

[7ing]

@thrawn01 thanks for your reply. travis tries to install github.com/axw/gocov/gocov, but it does exist any more, can I remove those steps as well?

[thrawn01]

Yes, remove all the following

install:
 - export PATH=$HOME/gopath/bin:$PATH
 - go get -v github.com/axw/gocov
 - go install github.com/axw/gocov/gocov
 - go get -v github.com/golang/glog
 - go get -v launchpad.net/gocheck
 - go get -v github.com/mailgun/glogutils

and replace with

install:
  - go get -t .

go get -t . will retrieve all the dependencies needed by the code

[7ing]

@thrawn01 seems the test requires gocov

gocov test -exclude-goroot | gocov report
/home/travis/build.sh: line 57: gocov: command not found
/home/travis/build.sh: line 57: gocov: command not found

[thrawn01]

just remove

  - gocov test -exclude-goroot | gocov report

from travis for now. We can always setup coveralls.io later if we want

[7ing]

@thrawn01 sure it works. Thanks for your quick response on this PR.

[7ing]

any updates on this ?