Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Dec 11, 2020

Motivation

PR #8386 configured the http connection pooling timeout to 25 seconds.
However the "connection reset" issue with maven dependency downloads has persisted.
he mitigation used in other projects is to disable maven's http connection pooling completely.
The solution is proposed here:

Modifications

Disable maven's http connection pool by passing -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false in MAVEN_OPTS environment variable.

…" issues

- previously the http connection pooling timeout was reduced to 25 seconds in
  PR apache#8386. However the issue has persisted. The mitigation used in other
  projects is to disable maven's http connection pooling completely.
  The solution is proposed here:
  - actions/runner-images#1499 (comment)
@lhotari lhotari force-pushed the lh-disable-maven-http-connection-pool branch from 4b7530d to d27d6ac Compare December 12, 2020 15:14
@lhotari
Copy link
Member Author

lhotari commented Dec 12, 2020

/pulsarbot rerun-failure-checks

1 similar comment
@lhotari
Copy link
Member Author

lhotari commented Dec 12, 2020

/pulsarbot rerun-failure-checks

@sijie sijie added this to the 2.8.0 milestone Dec 13, 2020
@sijie sijie merged commit 9405e6b into apache:master Dec 13, 2020
bjhargrave added a commit to bndtools/bnd that referenced this pull request Feb 16, 2021
We set retries to 3 and disable maven connection pooling which sometimes
interacts badly with the Azure backend of GitHub Actions vms.

See apache/pulsar#8921

Signed-off-by: BJ Hargrave <bj@hargrave.dev>
lhotari added a commit to lhotari/pulsar that referenced this pull request Apr 28, 2023
- Maven http connection pooling was originally disabled with apache#8921
- Maven is switching to "Maven Resolver" ("aether") in newer versions
  instead of using Wagon.
  - Disable Maven http connection pooling also for aether
  - Set request timeout to 60000ms also for aether
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants