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

Upgrade maven-deploy-plugin to 3.1.2 #16987

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

wborn
Copy link
Member

@wborn wborn commented Jul 3, 2024

This may help with fixing or diagnosing read timouts during deployment.

This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
@wborn wborn requested a review from a team as a code owner July 3, 2024 11:02
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/builds-for-4-2-0-snapshot-failing/155962/13

Copy link
Member

@holgerfriedrich holgerfriedrich left a comment

Choose a reason for hiding this comment

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

Let's give it a try...
The Jenkins job config is stored in Jenkins, not in git, correct? Do you know how I could have a look without Jenkins credentials?

@jlaur jlaur added the infrastructure Build system and Karaf related issues and PRs label Jul 3, 2024
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

Thanks!

@jlaur jlaur merged commit 766981b into openhab:main Jul 3, 2024
5 checks passed
@jlaur jlaur added this to the 4.2 milestone Jul 3, 2024
@wborn wborn deleted the upgrade-maven-deploy-plugin branch July 4, 2024 06:04
@wborn
Copy link
Member Author

wborn commented Jul 4, 2024

The whole pipeline is defined in this Jenkinsfile. There isn't that much in the Jenkins job itself except for defining the parameters and some default values.

I already tried adding some connection related parameters to MAVEN_EXTRA_OPTS but it didn't help much so far.
E.g. parameters like:

-Dhttp.keepAlive=false
-Dmaven.wagon.http.pool=false
-Dmaven.wagon.http.retryHandler.count=3
-Daether.connector.http.retryHandler.count=5
-Daether.connector.http.reuseConnections=false
-Daether.connector.http.connectionMaxTtl=25
-Daether.connector.connectTimeout=120000

Reverting Maven and Java to some older versions to see if it is due to a recent regression also didn't help.

Next I might want to try if the pipeline has the same connection issues when it runs on agent1 instead of the built-in node.

psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Jul 12, 2024
This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
pgfeller pushed a commit to pgfeller/openhab-addons that referenced this pull request Sep 29, 2024
This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
cipianpascu pushed a commit to cipianpascu/openhab-addons that referenced this pull request Jan 2, 2025
This may help with fixing or diagnosing read timouts during deployment.

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Build system and Karaf related issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants