Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

ci: reprovision worker if it's flaky#3137

Merged
v1v merged 8 commits intoelastic:mainfrom
v1v:feature/use-retry-if-node-died
Oct 20, 2022
Merged

ci: reprovision worker if it's flaky#3137
v1v merged 8 commits intoelastic:mainfrom
v1v:feature/use-retry-if-node-died

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Oct 19, 2022

What does this PR do?

Re-provision if the worker is flaky and retry again in a new CI worker.

Why is it important?

If the worker is unstable, then the stage will fail and hence the build.

It assumes the worker is unstable if it cannot reach the dir in the finally section.

Issues

A similar approach was planed in elastic/beats#22662 but it never got merged.

@v1v v1v added automation Team:Automation Label for the Observability productivity team backport-v7.17.0 Automated backport with mergify backport-v8.4.0 Automated backport with mergify backport-v8.5.0 Automated backport with mergify labels Oct 19, 2022
@v1v v1v requested review from a team October 19, 2022 14:01
@v1v v1v self-assigned this Oct 19, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 19, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-10-20T08:46:07.486+0000

  • Duration: 61 min 25 sec

Test stats 🧪

Test Results
Failed 0
Passed 216
Skipped 0
Total 216

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

Comment thread .ci/Jenkinsfile Outdated
Comment thread .ci/Jenkinsfile Outdated
Comment thread .ci/Jenkinsfile Outdated
Comment thread .ci/Jenkinsfile Outdated
@v1v v1v marked this pull request as ready for review October 19, 2022 15:11
@v1v v1v merged commit e1fa72f into elastic:main Oct 20, 2022
mergify Bot pushed a commit that referenced this pull request Oct 20, 2022
mergify Bot pushed a commit that referenced this pull request Oct 20, 2022
mergify Bot pushed a commit that referenced this pull request Oct 20, 2022
v1v added a commit that referenced this pull request Oct 31, 2022
(cherry picked from commit e1fa72f)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Oct 31, 2022
(cherry picked from commit e1fa72f)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Nov 14, 2022
(cherry picked from commit e1fa72f)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automation backport-v7.17.0 Automated backport with mergify backport-v8.4.0 Automated backport with mergify backport-v8.5.0 Automated backport with mergify Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants