Skip to content

Commit

Permalink
Make Github pipeline more robust by increasing HTTP ttl and adding a …
Browse files Browse the repository at this point in the history
…retry (#5164)

* Make Github pipeline more robust by increasing HTTP ttl and adding a retry.

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>

* Attempt to move maven HTTP args to pipeline file.

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>

* Removed curly braces.

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>

Signed-off-by: Santiago Pericasgeertsen <santiago.pericasgeertsen@oracle.com>
  • Loading branch information
spericas authored Oct 12, 2022
1 parent f030bf1 commit cf3e2fe
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/validate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ env:
JAVA_VERSION: '19'
JAVA_DISTRO: 'temurin'
HELIDON_PIPELINES: 'true'
MAVEN_HTTP_ARGS: '-Dmaven.wagon.httpconnectionManager.ttlSeconds=60 -Dmaven.wagon.http.retryHandler.count=3'

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
Expand Down Expand Up @@ -102,7 +103,7 @@ jobs:
cache: maven
- name: Maven build
run: |
mvn -B -e "-Dmaven.test.skip=true" -DskipTests -Ppipeline install
mvn -B -e "-Dmaven.test.skip=true" $MAVEN_HTTP_ARGS -DskipTests -Ppipeline install
cd examples
mvn -B verify
mp-tck:
Expand Down
2 changes: 1 addition & 1 deletion etc/scripts/includes/pipeline-env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ if [ -z "${__PIPELINE_ENV_INCLUDED__}" ]; then

if [ -n "${HELIDON_PIPELINES}" ] ; then
export PIPELINE="true"
MAVEN_ARGS="${MAVEN_ARGS} -B"
MAVEN_ARGS="${MAVEN_ARGS} -B ${MAVEN_HTTP_ARGS}"
export MAVEN_ARGS
# temporary fix for copyright plugin (currently fails on big renaming action for Nima)
git config diff.renameLimit 32768
Expand Down

0 comments on commit cf3e2fe

Please sign in to comment.