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

Report only one DeleteDir step failure#1006

Merged
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/reduce-dleetedir-errors
Feb 24, 2021
Merged

Report only one DeleteDir step failure#1006
v1v merged 1 commit intoelastic:masterfrom
v1v:feature/reduce-dleetedir-errors

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Feb 23, 2021

What does this PR do?

Report only one deleteDir step failure

Why is it important?

deleteDir step might not be as important as some other steps and might cause some misleading, so if multiple deleteDir failures then let's report only one.

Related issues

Closes #1005

@botelastic botelastic Bot added the groovy label Feb 23, 2021
@v1v v1v requested a review from a team February 23, 2021 11:24
@v1v v1v self-assigned this Feb 23, 2021
@v1v v1v added backport-to-7.x Automatically backport PRs from master enhancement New feature or request ready-to-merge labels Feb 23, 2021
@ghost
Copy link
Copy Markdown

ghost commented Feb 23, 2021

💚 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

Expand to view the summary

Build stats

  • Build Cause: Pull request #1006 opened

  • Start Time: 2021-02-23T11:24:25.909+0000

  • Duration: 32 min 38 sec

  • Commit: 610d251

Test stats 🧪

Test Results
Failed 0
Passed 1048
Skipped 2
Total 1050

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1048
Skipped 2
Total 1050

@v1v v1v merged commit 219f748 into elastic:master Feb 24, 2021
@v1v v1v deleted the feature/reduce-dleetedir-errors branch February 24, 2021 11:09
mergify Bot pushed a commit that referenced this pull request Feb 24, 2021
mergify Bot added a commit that referenced this pull request Feb 24, 2021
(cherry picked from commit 219f748)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Mar 18, 2021
* upstream/7.x:
  Fix the getCause and getCauses for the AbortException (#1041) (#1042)
  gsutil for windows with python (#1008) (#1009)
  Report only one DeleteDir step failure (#1006) (#1007)
  [cosmetic] Replace multiline chars in the GitHub PR comment (#998) (#1003)
  [cosmetic] add the author (#999) (#1002)
  chore: use opentelemetry release (#1000) (#1001)
  Avoid cache gsutils temporary location (#994) (#995)
  [JCasC] google cloud engine local setup (#979) (#992)
  Support curl in addition to wget for gsutil (#988) (#989)
  Support windows for gsutil (#985) (#987)
  Add googleStorageUploadExt step (#867) (#982)
  Add releaseNotification step to simplify agent release pipelines (#976) (#981)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-to-7.x Automatically backport PRs from master enhancement New feature or request groovy ready-to-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Report only one delete workspace step failure

2 participants