You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by zr...@apache.org on 2022/01/31 21:44:02 UTC
[trafficcontrol] branch 6.1.x updated (d93e9ad -> f8041b3)
This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a change to branch 6.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git.
from d93e9ad Use XML syntax for Ansible push warning (#6521)
new acb60c3 Remove now-unnecessary libraries from CDN-in-a-Box's Traffic Ops image (#6460)
new 4aad56f Adding a maxConnections value to TR (#6515)
new 19bea59 Ansible TR: Keep hourly archives of `access.log` (#6517)
new f8041b3 Ansible TR: Corrections to `access.log` archives file pattern (#6519)
The 4 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
CHANGELOG.md | 1 +
.../ansible/roles/traffic-router/defaults/main.yml | 9 +++++++--
infrastructure/cdn-in-a-box/traffic_ops/Dockerfile | 13 ++-----------
traffic_router/core/src/main/conf/server.xml | 2 +-
4 files changed, 11 insertions(+), 14 deletions(-)
[trafficcontrol] 03/04: Ansible TR: Keep hourly archives of `access.log` (#6517)
Posted by zr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a commit to branch 6.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
commit 19bea598e83957e06203c156cd8b7ebce57b1dfa
Author: Zach Hoffman <zr...@apache.org>
AuthorDate: Wed Jan 19 15:48:39 2022 -0800
Ansible TR: Keep hourly archives of `access.log` (#6517)
* Keep a maximum of 50 access.log archives
* Trigger an access.log rollover every hour
* Add date and an integer counter to access.log archives
* Limit each access.log archive to 200MB
Co-authored-by: jpappa200 <jo...@cable.comcast.com>
(cherry picked from commit cda63c97bba9d5fbc6f04a848a842c79c0815b96)
---
infrastructure/ansible/roles/traffic-router/defaults/main.yml | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/infrastructure/ansible/roles/traffic-router/defaults/main.yml b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
index d5d2be3..5c6463e 100644
--- a/infrastructure/ansible/roles/traffic-router/defaults/main.yml
+++ b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
@@ -139,12 +139,13 @@ tr_log4j2_opts: |
</Console>
<RollingFile name="traffic_router_access"
fileName="{{tr_log_dir}}/access.log"
- filePattern="{{tr_log_dir}}/access.log.%i" >
+ filePattern="{{tr_log_dir}}/access.log.%d{MM-dd-yyyy}-%2i" >
<PatternLayout pattern="%m%n" />
<Policies>
<SizeBasedTriggeringPolicy size="200MB" />
+ <CronTriggeringPolicy schedule="0 0 * * * ?" />
</Policies>
- <DefaultRolloverStrategy max="10" />
+ <DefaultRolloverStrategy max="50" />
<ThresholdFilter level="INFO" />
</RollingFile>
<RollingFile name="traffic_router"
[trafficcontrol] 01/04: Remove now-unnecessary libraries from CDN-in-a-Box's Traffic Ops image (#6460)
Posted by zr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a commit to branch 6.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
commit acb60c34ad960af0f5f028aa49ff2b136307d05e
Author: ocket8888 <oc...@apache.org>
AuthorDate: Wed Jan 19 09:36:00 2022 -0700
Remove now-unnecessary libraries from CDN-in-a-Box's Traffic Ops image (#6460)
(cherry picked from commit 2eec1ae16fdf6e97d925cb6ea381af773cda5ef5)
---
infrastructure/cdn-in-a-box/traffic_ops/Dockerfile | 13 ++-----------
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/infrastructure/cdn-in-a-box/traffic_ops/Dockerfile b/infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
index d58c916..342d9d8 100644
--- a/infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
+++ b/infrastructure/cdn-in-a-box/traffic_ops/Dockerfile
@@ -37,22 +37,13 @@ RUN set -o nounset -o errexit && \
if [[ "${RHEL_VERSION%%.*}" -eq 7 ]]; then \
use_repo=''; \
enable_repo=''; \
- llvm_version=5.0; \
- # needed for llvm-toolset-7-clang, which is needed for postgresql13-devel-13.2-1PGDG, required by TO rpm
- dnf -y install gcc centos-release-scl-rh; \
else \
use_repo='--repo=pgdg13'; \
enable_repo='--enablerepo=powertools'; \
- llvm_version=''; \
fi; \
dnf -y install "https://download.postgresql.org/pub/repos/yum/reporpms/EL-${RHEL_VERSION%%.*}-x86_64/pgdg-redhat-repo-latest.noarch.rpm"; \
- dnf -y install epel-release; \
- dnf -y install \
- # libicu is required by postgresql13
- libicu \
- # libicu-devel, clang-devel, and llvm-devel are required by postgresql13-devel
- libicu-devel clang-devel llvm${llvm_version}-devel; \
- dnf -y $use_repo -- install postgresql13 postgresql13-devel; \
+ dnf -y install epel-release libicu; \
+ dnf -y $use_repo -- install postgresql13; \
dnf -y $enable_repo install \
bind-utils \
gettext \
[trafficcontrol] 02/04: Adding a maxConnections value to TR (#6515)
Posted by zr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a commit to branch 6.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
commit 4aad56f855e3913c093ee3c9c2073c000d8a89b9
Author: Srijeet Chatterjee <30...@users.noreply.github.com>
AuthorDate: Wed Jan 19 15:03:35 2022 -0700
Adding a maxConnections value to TR (#6515)
* adding a maxConnections value to TR
* adding changelog
(cherry picked from commit 0b26836a5f8f410bfa4f2599fbe290a88dfa173f)
---
CHANGELOG.md | 1 +
infrastructure/ansible/roles/traffic-router/defaults/main.yml | 4 ++++
traffic_router/core/src/main/conf/server.xml | 2 +-
3 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0ea095d..719ffaa 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -48,6 +48,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
- [#6179](https://github.com/apache/trafficcontrol/issues/6179) Updated the Traffic Ops rpm to include the `ToDnssecRefresh` binary and make the `trafops_dnssec_refresh` cron job use it
- Traffic Portal no longer uses `ruby compass` to compile sass and now uses `dart-sass`.
- Changed Invalidation Jobs throughout (TO, TP, T3C, etc.) to account for the ability to do both REFRESH and REFETCH requests for resources.
+- Changed the `maxConnections` value on Traffic Router, to prevent the thundering herd problem (TR).
- The `admin` Role is now always guaranteed to exist, and can't be deleted or modified.
- [#6376](https://github.com/apache/trafficcontrol/issues/6376) Updated TO/TM so that TM doesn't overwrite monitoring snapshot data with CR config snapshot data.
- Updated `t3c-apply` to reduce mutable state in `TrafficOpsReq` struct.
diff --git a/infrastructure/ansible/roles/traffic-router/defaults/main.yml b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
index 0fa2313..d5d2be3 100644
--- a/infrastructure/ansible/roles/traffic-router/defaults/main.yml
+++ b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
@@ -66,6 +66,9 @@ tr_java_opts:
- "-Djava.security.egd=file:/dev/./urandom"
# server.xml
+tr_connector_opts:
+ maxConnections: 4000
+
tr_connector_default:
protocol: 'org.apache.traffic_control.traffic_router.protocol.LanguidNioProtocol'
maxThreads: 10000
@@ -92,6 +95,7 @@ tr_connectors:
portAttribute: 'ApiPort'
- port: 443
protocol: "{{tr_connector_default.protocol}}"
+ maxConnections: "{{tr_connector_opts.maxConnections}}"
maxThreads: "{{tr_connector_default.maxThreads}}"
connectionTimeout: "{{tr_connector_default.connectionTimeout}}"
mbeanPath: "{{tr_connector_default.mbeanPath}}"
diff --git a/traffic_router/core/src/main/conf/server.xml b/traffic_router/core/src/main/conf/server.xml
index 9635a31..8f09983 100644
--- a/traffic_router/core/src/main/conf/server.xml
+++ b/traffic_router/core/src/main/conf/server.xml
@@ -43,7 +43,7 @@
scheme="https" secure="true" SSLEnabled="true" clientAuth="false" sslProtocol="TLS" protocols="+TLSv1.1,+TLSv1.2,+TLSv1.3" connectionTimeout="10000"
mbeanPath="traffic-router:name=languidState" readyAttribute="Ready" portAttribute="SecureApiPort" sendReasonPhrase="true"
sslImplementationName="org.apache.traffic_control.traffic_router.protocol.RouterSslImplementation"> </Connector>
- <Connector port="443" protocol="org.apache.traffic_control.traffic_router.protocol.LanguidNioProtocol" maxThreads="10000"
+ <Connector port="443" protocol="org.apache.traffic_control.traffic_router.protocol.LanguidNioProtocol" maxConnections="4000" maxThreads="10000"
scheme="https" secure="true" SSLEnabled="true" clientAuth="false" sslProtocol="TLS" protocols="+TLSv1.1,+TLSv1.2,+TLSv1.3" connectionTimeout="10000"
mbeanPath="traffic-router:name=languidState" readyAttribute="Ready" portAttribute="SecurePort" sendReasonPhrase="true"
sslImplementationName="org.apache.traffic_control.traffic_router.protocol.RouterSslImplementation">
[trafficcontrol] 04/04: Ansible TR: Corrections to `access.log` archives file pattern (#6519)
Posted by zr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
zrhoffman pushed a commit to branch 6.1.x
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git
commit f8041b3b5d616003b32d7c0dc160f5f9621e5366
Author: Zach Hoffman <zr...@apache.org>
AuthorDate: Thu Jan 20 14:56:06 2022 -0800
Ansible TR: Corrections to `access.log` archives file pattern (#6519)
* Use date format yyyy-MM-dd
* Pad integer counter with '0'
(cherry picked from commit 31beba513f83949d7e4d0860b20e5a7fd567a3cd)
---
infrastructure/ansible/roles/traffic-router/defaults/main.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/infrastructure/ansible/roles/traffic-router/defaults/main.yml b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
index 5c6463e..b96082f 100644
--- a/infrastructure/ansible/roles/traffic-router/defaults/main.yml
+++ b/infrastructure/ansible/roles/traffic-router/defaults/main.yml
@@ -139,7 +139,7 @@ tr_log4j2_opts: |
</Console>
<RollingFile name="traffic_router_access"
fileName="{{tr_log_dir}}/access.log"
- filePattern="{{tr_log_dir}}/access.log.%d{MM-dd-yyyy}-%2i" >
+ filePattern="{{tr_log_dir}}/access.log.%d{yyyy-MM-dd}-%02i" >
<PatternLayout pattern="%m%n" />
<Policies>
<SizeBasedTriggeringPolicy size="200MB" />