You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by oc...@apache.org on 2021/03/19 18:01:52 UTC

[trafficcontrol] 04/08: Fix logrotate permissions for Traffic Stats (#5650)

This is an automated email from the ASF dual-hosted git repository.

ocket8888 pushed a commit to branch 5.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git

commit ed89519f9228be29368af5fe18b94f3170e156a7
Author: Hank Beatty <hb...@users.noreply.github.com>
AuthorDate: Wed Mar 17 15:48:03 2021 -0400

    Fix logrotate permissions for Traffic Stats (#5650)
    
    (cherry picked from commit 15b574a705a04d6db1ef3605a319947de5205b03)
---
 CHANGELOG.md                           | 3 ++-
 traffic_stats/build/traffic_stats.spec | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 230a1b0..9510d72 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,7 +10,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 ### Fixed
 - [#5565](https://github.com/apache/trafficcontrol/issues/5565) - TO GET /caches/stats panic converting string to uint64
 - [#5558](https://github.com/apache/trafficcontrol/issues/5558) - Fixed `TM UI` and `/api/cache-statuses` to report aggregate `bandwidth_kbps` correctly.
-- - Fix for config gen missing max_origin_connections on mids in certain scenarios
+- Fix for config gen missing max_origin_connections on mids in certain scenarios
+- Fixed Invalid TS logrotate configuration permissions causing TS logs to be ignored by logrotate.
 
 ## [5.1.0] - 2021-02-21
 ### Added
diff --git a/traffic_stats/build/traffic_stats.spec b/traffic_stats/build/traffic_stats.spec
index dae4e02..f498960 100644
--- a/traffic_stats/build/traffic_stats.spec
+++ b/traffic_stats/build/traffic_stats.spec
@@ -116,7 +116,7 @@ fi
 
 %config(noreplace) %attr(600, traffic_stats, traffic_stats) /opt/traffic_stats/conf/traffic_stats.cfg
 %config(noreplace) %attr(600, traffic_stats, traffic_stats) /opt/traffic_stats/conf/traffic_stats_seelog.xml
-%config(noreplace) /etc/logrotate.d/traffic_stats
+%config(noreplace) %attr(644, root, root) /etc/logrotate.d/traffic_stats
 
 %dir /opt/traffic_stats
 %dir /opt/traffic_stats/bin