This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
The following commit(s) were added to refs/heads/master by this push:
new 35ed1ba Set root ownership of TM logrotate file (#5111)
35ed1ba is described below
commit 35ed1ba7a8b959f78b11f061befb6293bce862a8
Author: Rawlin Peters <[email protected]>
AuthorDate: Tue Oct 6 11:53:22 2020 -0600
Set root ownership of TM logrotate file (#5111)
* Set root ownership of TM logrotate file
The TM logrotate config file needs to be owned by root in order for
logrotate to rotate the TM logs.
* Add changelog entry
---
CHANGELOG.md | 1 +
traffic_monitor/build/traffic_monitor.spec | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index a6e4bb1..cef9bd2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -60,6 +60,7 @@ The format is based on [Keep a
Changelog](http://keepachangelog.com/en/1.0.0/).
- Fixed #4981 - Cannot create routing regular expression with a blank pattern
param in Delivery Service [Related github
issues](https://github.com/apache/trafficcontrol/issues/4981)
- Fixed #4979 - Returns a Bad Request error during server creation with
missing profileId [Related github
issue](https://github.com/apache/trafficcontrol/issues/4979)
- Fixed #4237 - Do not return an internal server error when delivery service's
capacity is zero. [Related github
issue](https://github.com/apache/trafficcontrol/issues/4237)
+- Fixed #2712 - Invalid TM logrotate configuration permissions causing TM logs
to be ignored by logrotate. [Related github
issue](https://github.com/apache/trafficcontrol/issues/2712)
- Fixed #3400 - Allow "0" as a TTL value for Static DNS entries [Related
github issue](https://github.com/apache/trafficcontrol/issues/3400)
- Fixed #5050 - Allows the TP administrator to name a TP instance (production,
staging, etc) and flag whether it is production or not in
traffic_portal_properties.json [Related github
issue](https://github.com/apache/trafficcontrol/issues/5050)
- Fixed #4743 - Validate absolute DNS name requirement on Static DNS entry for
CNAME type [Related github
issue](https://github.com/apache/trafficcontrol/issues/4743)
diff --git a/traffic_monitor/build/traffic_monitor.spec
b/traffic_monitor/build/traffic_monitor.spec
index 4d89e54..5d022dd 100644
--- a/traffic_monitor/build/traffic_monitor.spec
+++ b/traffic_monitor/build/traffic_monitor.spec
@@ -101,7 +101,7 @@ fi
%defattr(644, traffic_monitor, traffic_monitor, 755)
%config(noreplace) %attr(600, traffic_monitor, traffic_monitor)
/opt/traffic_monitor/conf/traffic_monitor.cfg
%config(noreplace) %attr(600, traffic_monitor, traffic_monitor)
/opt/traffic_monitor/conf/traffic_ops.cfg
-%config(noreplace) /etc/logrotate.d/traffic_monitor
+%config(noreplace) %attr(644, root, root) /etc/logrotate.d/traffic_monitor
%dir /opt/traffic_monitor
%dir /opt/traffic_monitor/bin
