trafficcontrol icon indicating copy to clipboard operation
trafficcontrol copied to clipboard

Store Apache Traffic Server and grove logs in /var/log/trafficserver

Open zrhoffman opened this issue 10 months ago • 5 comments

This PR changes configuration to log Apache Traffic Server and Grove to /var/log/trafficserver instead of /opt/trafficserver/var/log/trafficserver so that the logs are on a different partition than the application itself.


Which Traffic Control components are affected by this PR?

  • Traffic Control Cache Config (t3c, formerly ORT) - tests
  • Apache Traffic Server
  • Grove
  • Transitive Source Build targeting ATS
  • Automation - Ansible roles configuration for grove log directory

What is the best way to verify this PR?

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

zrhoffman avatar Apr 03 '24 07:04 zrhoffman

Should we also upgrade ATS's version? https://github.com/apache/trafficcontrol/pull/7958

ntheanh201 avatar Apr 03 '24 18:04 ntheanh201

Should we also upgrade ATS's version? #7958

Definitely. I think the only reason that #7958 is not approved already is the realpath on macOS issue that the reviewer encounted. Hopefully #7981 will unblock that.

zrhoffman avatar Apr 03 '24 19:04 zrhoffman

I cannot test DS with your new ATS Edge, when I try to start ATS service, it shows that you still have to change to /var/log/trafficserver with traffic.out

P/s: I created a PR to your repo to fix this issue, can you take a look? @zrhoffman image

ntheanh201 avatar Apr 07 '24 09:04 ntheanh201

I cannot test DS with your new ATS Edge, when I try to start ATS service, it shows that you still have to change to /var/log/trafficserver with traffic.out

P/s: I created a PR to your repo to fix this issue, can you take a look? @zrhoffman image

Thanks @ntheanh201! zrhoffman/trafficcontrol#122 is merged.

zrhoffman avatar Apr 15 '24 15:04 zrhoffman

Is there anything else in this PR that you need to check? the CI failed? @zrhoffman

ntheanh201 avatar May 21 '24 16:05 ntheanh201