[trafficserver] branch 9.0.x updated: Updated ChangeLog
This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/trafficserver.git The following commit(s) were added to refs/heads/9.0.x by this push: new 5ba803e Updated ChangeLog 5ba803e is described below commit 5ba803efa0ae958b922cbf53ffc397eaef2671b1 Author: Leif Hedstrom AuthorDate: Fri Apr 2 17:27:50 2021 -0600 Updated ChangeLog --- CHANGELOG-9.0.1 | 5 + 1 file changed, 5 insertions(+) diff --git a/CHANGELOG-9.0.1 b/CHANGELOG-9.0.1 index cb396df..3fcbae1 100644 --- a/CHANGELOG-9.0.1 +++ b/CHANGELOG-9.0.1 @@ -52,6 +52,11 @@ Changes with Apache Traffic Server 9.0.1 #7592 - Remove inline for detail::cache::CacheData::idAddr #7596 - Remove UnixNetVConnection::startEvent - not actually called. #7602 - Fix the connection limit crash while using parents + #7612 - Document http.default_buffer_water_mark #7616 - Increase Proxy Verifier caching delay. #7620 - Adding negative revalidating AuTests. + #7621 - Update server_response_body_bytes when background fill worked #7634 - statichit: misc. fixes + #7646 - Remove erroneous manager.log mesg with remap include file reload + #7665 - AuTest: New log and When condition for ATS initialized + #7672 - Add information about TSHttpArgIndexReserve name change to upgrade notes
[trafficserver] branch 9.0.x updated: Add information about TSHttpArgIndexReserve name change to upgrade notes (#7672)
This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/trafficserver.git The following commit(s) were added to refs/heads/9.0.x by this push: new a15f71e Add information about TSHttpArgIndexReserve name change to upgrade notes (#7672) a15f71e is described below commit a15f71ea919e89f254bf8b06cfd87ccf534d732a Author: Susan Hinrichs AuthorDate: Fri Apr 2 15:54:07 2021 -0500 Add information about TSHttpArgIndexReserve name change to upgrade notes (#7672) --- doc/release-notes/upgrading.en.rst | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/doc/release-notes/upgrading.en.rst b/doc/release-notes/upgrading.en.rst index b56dd8a..1677bf0 100644 --- a/doc/release-notes/upgrading.en.rst +++ b/doc/release-notes/upgrading.en.rst @@ -110,7 +110,7 @@ longer a special case and are handled the same as a response with a non-zero len * `proxy.config.ssl.client.verify.server` was deprecated for ATS 8.x and has been removed. Use -:ts:cv:`proxy.config.ssl.client.verify.server.properties` instead. + :ts:cv:`proxy.config.ssl.client.verify.server.properties` instead. Deprecated records.config Settings ~~ @@ -144,13 +144,13 @@ The default cipher list has changed: * :ts:cv:`proxy.config.ssl.server.cipher_suite` * :ts:cv:`proxy.config.exec_thread.autoconfig.scale` went from a value of 1.5 to 1. This controls the number of worker threads and -the ratio is now worker thread 1 to 1 processing thread on the CPU or CPUs. + the ratio is now worker thread 1 to 1 processing thread on the CPU or CPUs. Settings with new behavior ~~ * :ts:cv:`proxy.config.http.connect_attempts_max_retries_dead_server` specifies the exact number of times a dead server will be -retried for each new request. Before ATS tried 1 time more than this setting making it impossible to set the retry value to 0. + retried for each new request. Before ATS tried 1 time more than this setting making it impossible to set the retry value to 0. Metrics --- @@ -180,7 +180,7 @@ Command Line Options The following command line options were either renamed or removed. * `--with-max-api-stats` was replace with `--maxRecords` to specify the total number of metric instead of just the total API metrics -to use when running ATS. + to use when running ATS. * `--read_core` was removed and gdb should be used instead. Deprecated or Removed Features @@ -216,7 +216,7 @@ Renamed or Modified APIs * :func:`TSHttpTxnServerPush` now returns a :c:type:`TSReturnCode` * :func:`TSContSchedule` and :func:`TSContScheduleAPI` by default will run on the same thread from which they are called. * :func:`TSFetchUrl` and :func:`TSFetchCreate` now return a :c:type:`TSFetchSM` - +* :func:`TSHttpArgIndexReserve` is now renamed to be :func:`TSHttpTxnArgIndexReserve` Cache -
[trafficserver] branch 9.1.x updated: Do NOT kill tunnel if it has any consumer besides HT_HTTP_CLIENT (#7641)
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 9.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/9.1.x by this push:
new 31c3884 Do NOT kill tunnel if it has any consumer besides
HT_HTTP_CLIENT (#7641)
31c3884 is described below
commit 31c38848b863e426d26f8af64b2d64f42b8306e0
Author: Masaori Koshiba
AuthorDate: Fri Apr 2 07:56:13 2021 +0900
Do NOT kill tunnel if it has any consumer besides HT_HTTP_CLIENT (#7641)
(cherry picked from commit 270ca6e84eba758e44c9b2a15046eaa7b8af8d8a)
---
proxy/http/HttpSM.cc| 4 ++--
proxy/http/HttpTunnel.h | 26 ++
2 files changed, 28 insertions(+), 2 deletions(-)
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index fed6985..0a4c69d 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -981,9 +981,9 @@ HttpSM::state_watch_for_client_abort(int event, void *data)
* client.
*/
case VC_EVENT_EOS: {
-// We got an early EOS.
+// We got an early EOS. If the tunnal has cache writer, don't kill it for
background fill.
NetVConnection *netvc = ua_txn->get_netvc();
-if (ua_txn->allow_half_open()) {
+if (ua_txn->allow_half_open() || tunnel.has_consumer_besides_client()) {
if (netvc) {
netvc->do_io_shutdown(IO_SHUTDOWN_READ);
}
diff --git a/proxy/http/HttpTunnel.h b/proxy/http/HttpTunnel.h
index c6aa9fc..7774e2c 100644
--- a/proxy/http/HttpTunnel.h
+++ b/proxy/http/HttpTunnel.h
@@ -282,6 +282,7 @@ public:
}
bool is_tunnel_alive() const;
bool has_cache_writer() const;
+ bool has_consumer_besides_client() const;
HttpTunnelProducer *add_producer(VConnection *vc, int64_t nbytes,
IOBufferReader *reader_start, HttpProducerHandler sm_handler,
HttpTunnelType_t vc_type, const char *name);
@@ -514,6 +515,31 @@ HttpTunnel::has_cache_writer() const
return false;
}
+/**
+ Return false if there is only a consumer for client
+ */
+inline bool
+HttpTunnel::has_consumer_besides_client() const
+{
+ bool res = true;
+
+ for (const auto : consumers) {
+if (!consumer.alive) {
+ continue;
+}
+
+if (consumer.vc_type == HT_HTTP_CLIENT) {
+ res = false;
+ continue;
+} else {
+ res = true;
+ break;
+}
+ }
+
+ return res;
+}
+
inline bool
HttpTunnelConsumer::is_downstream_from(VConnection *vc)
{
[trafficserver] branch master updated (270ca6e -> f11cab6)
This is an automated email from the ASF dual-hosted git repository. masaori pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/trafficserver.git. from 270ca6e Do NOT kill tunnel if it has any consumer besides HT_HTTP_CLIENT (#7641) add f11cab6 Add AuTest for Background Fill (#7613) No new revisions were added by this update. Summary of changes: tests/gold_tests/cache/background_fill.test.py | 140 + .../cache/gold/background_fill_0_stderr.gold | 19 +++ .../cache/gold/background_fill_1_stderr.gold | 19 +++ .../cache/gold/background_fill_2_stderr.gold | 19 +++ 4 files changed, 197 insertions(+) create mode 100644 tests/gold_tests/cache/background_fill.test.py create mode 100644 tests/gold_tests/cache/gold/background_fill_0_stderr.gold create mode 100644 tests/gold_tests/cache/gold/background_fill_1_stderr.gold create mode 100644 tests/gold_tests/cache/gold/background_fill_2_stderr.gold
