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

chore: add retry(5) to the Jenkinsfile #1576

Closed
wants to merge 2 commits into from
Closed

Conversation

lemeurherve
Copy link
Member

@lemeurherve lemeurherve commented Dec 22, 2023

I removed it in #1565, putting it back trying to get builds passing.

@zbynek
Copy link
Contributor

zbynek commented Dec 22, 2023

LGTM, but why is the build failing?

@dduportal
Copy link
Contributor

LGTM, but why is the build failing?

Not sure why to be honest :| There have been error when the build fetches the plugin contents since months. There used to be HTTP/500 from the backend which has been fixed 2 months ago (OOM in the JVM).

Now we see TCP/TLS errors like the one described in #1575 (comment) but only in infra.ci.jenkins.io, not on ci.jenkins.io which is weird.

@dduportal
Copy link
Contributor

I removed it in #1565, putting it back trying to get builds passing.

Problem is that the initial Build stage fails, so infra.ci.jenkins.io never reaches the "Deploy" phase (which is retried 5 times), so the build fails without retrying alas.

@lemeurherve
Copy link
Member Author

Superseded by #1575

@lemeurherve lemeurherve deleted the lemeurherve-patch-1 branch December 22, 2023 12:12
@zbynek
Copy link
Contributor

zbynek commented Dec 22, 2023

But... API failing for a single plugin should result in a single docs page missing, not in whole build failing.

@dduportal
Copy link
Contributor

But... API failing for a single plugin should result in a single docs page missing, not in whole build failing.

I wholeheartedly agree with you! But i have no idea how to do it with this toolchain (out of my depth here)

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