Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[incubator-kie-issues-1444-weekly] drools.weekly-deploy jobs frequent… #6056

Merged

Conversation

tkobayas
Copy link
Contributor

…ly fail with Request Timeout (408)

  • Disable parallel put

Issue:

…ly fail with Request Timeout (408)

- Disable parallel put
@@ -92,7 +92,7 @@ pipeline {
def installOrDeploy
if (shouldDeployToRepository()) {
installOrDeploy = "deploy -DdeployAtEnd -Dapache.repository.username=${REPOSITORY_USER} -Dapache.repository.password=${REPOSITORY_TOKEN} -DretryFailedDeploymentCount=5" +
" -Dhttp.socket.timeout=300000 -Dhttp.connection.timeout=300000"
" -Daether.connector.basic.parallelPut=false"
Copy link
Contributor Author

@tkobayas tkobayas Aug 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I wrote in apache/incubator-kie-issues#1444 (comment) , http.socket.timeout and http.connection.timeout don't seem to be effective.

"Request Timeout (408)" is generally caused by a slow client request. While testing locally, I found that maven 3.9.6 (= maven-resolver-1.9.18) uses 5 threads for parallel uploading by default. aether.connector.basic.parallelPut=false makes uploading sequential with single thread. I hope that it makes uploading stable. The entire job may take longer time, but I think it's worth trying.

https://maven.apache.org/components/resolver-archives/resolver-1.9.18/configuration.html

@tkobayas tkobayas merged commit 95b3532 into apache:main Aug 30, 2024
2 checks passed
@tkobayas
Copy link
Contributor Author

tkobayas commented Sep 2, 2024

09-01: SUCCESS

3 WARNINGs in the 1st attempt. 2nd attempt successful.

(Note: Failed to upload checksum doesn't stop the whole task. Could not transfer artifact stops the task and triggers retrying)

[2024-09-01T04:17:31.407Z] [INFO] --- deploy:3.1.1:deploy (default-deploy) @ drools-distribution ---
[2024-09-01T04:26:48.086Z] [WARNING] Failed to upload checksum to org/drools/drools-examples/999-20240901-SNAPSHOT/drools-examples-999-20240901-20240901.030335-1-javadoc.jar.md5
[2024-09-01T04:26:48.086Z] org.apache.http.client.HttpResponseException: status code: 408, reason phrase: Request Timeout (408)
[2024-09-01T04:26:48.086Z]     ...
[2024-09-01T04:28:36.241Z] [WARNING] Failed to upload checksum to org/drools/kiebase-inclusion/999-20240901-SNAPSHOT/kiebase-inclusion-999-20240901-20240901.030335-1-tests.jar.md5
[2024-09-01T04:28:36.241Z] org.apache.http.client.HttpResponseException: status code: 408, reason phrase: Request Timeout (408)
[2024-09-01T04:28:36.241Z]     ...
[2024-09-01T04:46:57.694Z] [WARNING] Encountered issue during deployment: Failed to deploy artifacts: Could not transfer artifact org.drools:drools-canonical-model:jar:999-20240901-20240901.030335-1 from/to apache.snapshots.https (https://repository.apache.org/content/repositories/snapshots): status code: 408, reason phrase: Request Timeout (408)
[2024-09-01T04:46:57.694Z] [INFO] Retrying deployment attempt 2 of 5
[2024-09-01T05:20:53.254Z] [INFO] ------------------------------------------------------------------------
[2024-09-01T05:20:53.254Z] [INFO] Reactor Summary for Drools :: Parent 999-20240901-SNAPSHOT:

This change aether.connector.basic.parallelPut=fals seemed to be effective, but let's see next week.

tkobayas added a commit to tkobayas/drools that referenced this pull request Sep 9, 2024
…ly fail with Request Timeout (408) (apache#6056)

- Disable deployAtEnd
rodrigonull pushed a commit to rodrigonull/incubator-kie-drools that referenced this pull request Oct 7, 2024
…ly fail with Request Timeout (408) (apache#6056)

- Disable parallel put
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants