This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 03f71d9 Limit maven http connection pool TTL in Github flows (#8386)
03f71d9 is described below
commit 03f71d97436ec1d16edd08108cb0e93580438898
Author: Lari Hotari <[email protected]>
AuthorDate: Tue Oct 27 23:41:08 2020 +0200
Limit maven http connection pool TTL in Github flows (#8386)
Fixes #6231
Maven's wagon-http documentation:
https://maven.apache.org/wagon/wagon-providers/wagon-http/
Set maven.wagon.httpconnectionManager.ttlSeconds to
25 seconds.
maven.wagon.httpconnectionManager.ttlSeconds documentation in source code:
https://github.com/apache/maven-wagon/blob/a7c8e3470dd968961e87a7cb9a3829d3bec77383/wagon-providers/wagon-http-shared/src/main/java/org/apache/maven/wagon/shared/http/AbstractHttpClientWagon.java#L297-L305
Also add -Dmaven.wagon.http.retryHandler.count=3 which
will retry http calls 3 times. However this has no impact on
retrying downloads. (documentatation: "Any retry handler can
only react to exceptions when executing the request and receiving
the response head. It will not salvage in-flight failures of
ongoing response body streams.")
---
.github/workflows/ci-cpp.yaml | 3 +++
.github/workflows/ci-go-functions-style.yaml | 3 +++
.github/workflows/ci-go-functions-test.yaml | 3 +++
.github/workflows/ci-integration-backwards-compatibility.yaml | 3 +++
.github/workflows/ci-integration-cli.yaml | 3 +++
.github/workflows/ci-integration-function-state.yaml | 3 +++
.github/workflows/ci-integration-messaging.yaml | 3 +++
.github/workflows/ci-integration-process.yaml | 3 +++
.github/workflows/ci-integration-schema.yaml | 3 +++
.github/workflows/ci-integration-sql.yaml | 3 +++
.github/workflows/ci-integration-standalone.yaml | 3 +++
.github/workflows/ci-integration-thread.yaml | 3 +++
.github/workflows/ci-integration-tiered-filesystem.yaml | 3 +++
.github/workflows/ci-integration-tiered-jcloud.yaml | 3 +++
.github/workflows/ci-license.yaml | 3 +++
.github/workflows/ci-pulsar-website-build.yaml | 3 +++
.github/workflows/ci-pulsarbot.yaml | 3 +++
.github/workflows/ci-unit-broker-broker-gp1.yaml | 3 +++
.github/workflows/ci-unit-broker-broker-gp2.yaml | 3 +++
.github/workflows/ci-unit-broker-client-api.yaml | 3 +++
.github/workflows/ci-unit-broker-client-impl.yaml | 3 +++
.github/workflows/ci-unit-broker-other.yaml | 3 +++
.github/workflows/ci-unit-proxy.yaml | 5 ++++-
.github/workflows/ci-unit.yaml | 3 +++
24 files changed, 73 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/ci-cpp.yaml b/.github/workflows/ci-cpp.yaml
index 5aff322..76bdb25 100644
--- a/.github/workflows/ci-cpp.yaml
+++ b/.github/workflows/ci-cpp.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
cpp-tests:
diff --git a/.github/workflows/ci-go-functions-style.yaml
b/.github/workflows/ci-go-functions-style.yaml
index 55c0482..81b3adb 100644
--- a/.github/workflows/ci-go-functions-style.yaml
+++ b/.github/workflows/ci-go-functions-style.yaml
@@ -32,6 +32,9 @@ on:
- '.github/workflows/**'
- 'pulsar-function-go/**'
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
check-style:
diff --git a/.github/workflows/ci-go-functions-test.yaml
b/.github/workflows/ci-go-functions-test.yaml
index 5cd4785..1e24560 100644
--- a/.github/workflows/ci-go-functions-test.yaml
+++ b/.github/workflows/ci-go-functions-test.yaml
@@ -32,6 +32,9 @@ on:
- '.github/workflows/**'
- 'pulsar-function-go/**'
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
go-functions-tests:
diff --git a/.github/workflows/ci-integration-backwards-compatibility.yaml
b/.github/workflows/ci-integration-backwards-compatibility.yaml
index 84e00fd..eeeec83 100644
--- a/.github/workflows/ci-integration-backwards-compatibility.yaml
+++ b/.github/workflows/ci-integration-backwards-compatibility.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
backwards-compatibility:
diff --git a/.github/workflows/ci-integration-cli.yaml
b/.github/workflows/ci-integration-cli.yaml
index 9183919..11ed495 100644
--- a/.github/workflows/ci-integration-cli.yaml
+++ b/.github/workflows/ci-integration-cli.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
cli:
diff --git a/.github/workflows/ci-integration-function-state.yaml
b/.github/workflows/ci-integration-function-state.yaml
index e842ed94..6f963f3 100644
--- a/.github/workflows/ci-integration-function-state.yaml
+++ b/.github/workflows/ci-integration-function-state.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
function-state:
diff --git a/.github/workflows/ci-integration-messaging.yaml
b/.github/workflows/ci-integration-messaging.yaml
index fb17d38..0b6e39a 100644
--- a/.github/workflows/ci-integration-messaging.yaml
+++ b/.github/workflows/ci-integration-messaging.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
messaging:
diff --git a/.github/workflows/ci-integration-process.yaml
b/.github/workflows/ci-integration-process.yaml
index 17e03ed..f7a56ac 100644
--- a/.github/workflows/ci-integration-process.yaml
+++ b/.github/workflows/ci-integration-process.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
process:
diff --git a/.github/workflows/ci-integration-schema.yaml
b/.github/workflows/ci-integration-schema.yaml
index e608af7..b32be4c 100644
--- a/.github/workflows/ci-integration-schema.yaml
+++ b/.github/workflows/ci-integration-schema.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
schema:
diff --git a/.github/workflows/ci-integration-sql.yaml
b/.github/workflows/ci-integration-sql.yaml
index 3f8f523..56d7fd6 100644
--- a/.github/workflows/ci-integration-sql.yaml
+++ b/.github/workflows/ci-integration-sql.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
sql:
diff --git a/.github/workflows/ci-integration-standalone.yaml
b/.github/workflows/ci-integration-standalone.yaml
index 39aff11..10fbd5d 100644
--- a/.github/workflows/ci-integration-standalone.yaml
+++ b/.github/workflows/ci-integration-standalone.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
standalone:
diff --git a/.github/workflows/ci-integration-thread.yaml
b/.github/workflows/ci-integration-thread.yaml
index 337880c..7e0fcc0 100644
--- a/.github/workflows/ci-integration-thread.yaml
+++ b/.github/workflows/ci-integration-thread.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
thread:
diff --git a/.github/workflows/ci-integration-tiered-filesystem.yaml
b/.github/workflows/ci-integration-tiered-filesystem.yaml
index db0aab3..668dca1 100644
--- a/.github/workflows/ci-integration-tiered-filesystem.yaml
+++ b/.github/workflows/ci-integration-tiered-filesystem.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
tiered-filesystem:
diff --git a/.github/workflows/ci-integration-tiered-jcloud.yaml
b/.github/workflows/ci-integration-tiered-jcloud.yaml
index 60f272e..1919ff6 100644
--- a/.github/workflows/ci-integration-tiered-jcloud.yaml
+++ b/.github/workflows/ci-integration-tiered-jcloud.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
tiered-jcloud:
diff --git a/.github/workflows/ci-license.yaml
b/.github/workflows/ci-license.yaml
index 7167dd3..36c73d6 100644
--- a/.github/workflows/ci-license.yaml
+++ b/.github/workflows/ci-license.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
license-check:
diff --git a/.github/workflows/ci-pulsar-website-build.yaml
b/.github/workflows/ci-pulsar-website-build.yaml
index f057e40..9d0319d 100644
--- a/.github/workflows/ci-pulsar-website-build.yaml
+++ b/.github/workflows/ci-pulsar-website-build.yaml
@@ -22,6 +22,9 @@ on:
schedule:
- cron: '0 */6 * * *'
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
build-website:
name: Build and publish pulsar website
diff --git a/.github/workflows/ci-pulsarbot.yaml
b/.github/workflows/ci-pulsarbot.yaml
index 848f3f2..aa342d9 100644
--- a/.github/workflows/ci-pulsarbot.yaml
+++ b/.github/workflows/ci-pulsarbot.yaml
@@ -22,6 +22,9 @@ on:
issue_comment:
types: [created]
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
action-runner:
diff --git a/.github/workflows/ci-unit-broker-broker-gp1.yaml
b/.github/workflows/ci-unit-broker-broker-gp1.yaml
index 57e461b..dc4f1e9 100644
--- a/.github/workflows/ci-unit-broker-broker-gp1.yaml
+++ b/.github/workflows/ci-unit-broker-broker-gp1.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
diff --git a/.github/workflows/ci-unit-broker-broker-gp2.yaml
b/.github/workflows/ci-unit-broker-broker-gp2.yaml
index 42882b1..2bee605 100644
--- a/.github/workflows/ci-unit-broker-broker-gp2.yaml
+++ b/.github/workflows/ci-unit-broker-broker-gp2.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
diff --git a/.github/workflows/ci-unit-broker-client-api.yaml
b/.github/workflows/ci-unit-broker-client-api.yaml
index 5b221c4..78c4bc7 100644
--- a/.github/workflows/ci-unit-broker-client-api.yaml
+++ b/.github/workflows/ci-unit-broker-client-api.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
diff --git a/.github/workflows/ci-unit-broker-client-impl.yaml
b/.github/workflows/ci-unit-broker-client-impl.yaml
index 16d6038..b77e7b8 100644
--- a/.github/workflows/ci-unit-broker-client-impl.yaml
+++ b/.github/workflows/ci-unit-broker-client-impl.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
diff --git a/.github/workflows/ci-unit-broker-other.yaml
b/.github/workflows/ci-unit-broker-other.yaml
index 76febb3..f0e2347 100644
--- a/.github/workflows/ci-unit-broker-other.yaml
+++ b/.github/workflows/ci-unit-broker-other.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
diff --git a/.github/workflows/ci-unit-proxy.yaml
b/.github/workflows/ci-unit-proxy.yaml
index 64b6e92..6fed7de 100644
--- a/.github/workflows/ci-unit-proxy.yaml
+++ b/.github/workflows/ci-unit-proxy.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
@@ -93,4 +96,4 @@ jobs:
if: failure()
with:
name: surefire-artifacts
- path: artifacts.zip
\ No newline at end of file
+ path: artifacts.zip
diff --git a/.github/workflows/ci-unit.yaml b/.github/workflows/ci-unit.yaml
index 8eb023b..093b7c5 100644
--- a/.github/workflows/ci-unit.yaml
+++ b/.github/workflows/ci-unit.yaml
@@ -26,6 +26,9 @@ on:
branches:
- branch-*
+env:
+ MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25
-Dmaven.wagon.http.retryHandler.count=3
+
jobs:
unit-tests:
