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

[CI] Report error in the catch section #22297

Merged
merged 1 commit into from
Oct 30, 2020
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Oct 30, 2020

What does this PR do?

#22220 caused a regression in the error reporting, therefore no GH checks are failing but the main one.

Why is it important?

Restore previous behaviour

@v1v v1v added automation ci Team:Automation Label for the Observability productivity team labels Oct 30, 2020
@v1v v1v requested a review from a team October 30, 2020 10:28
@v1v v1v self-assigned this Oct 30, 2020
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Oct 30, 2020
@elasticmachine
Copy link
Collaborator

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 16380
Skipped 1344
Total 17724

@elasticmachine
Copy link
Collaborator

💚 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 #22297 opened]

  • Start Time: 2020-10-30T10:28:45.512+0000

  • Duration: 64 min 58 sec

Test stats 🧪

Test Results
Failed 0
Passed 16380
Skipped 1344
Total 17724

@v1v v1v merged commit ed576aa into elastic:master Oct 30, 2020
@v1v v1v deleted the feature/report-error branch October 30, 2020 13:24
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  [JJBB] Add 6.8+ branches (elastic#22321)
  [CI] Support Windows-8 (elastic#22307)
  Change cloud.provider from googlecloud to gcp in billing metricset (elastic#22287)
  [packaging][beats-tester] use commit id binaries (elastic#22027)
  [CI] Report error in the catch section (elastic#22297)
v1v added a commit to v1v/beats that referenced this pull request Oct 30, 2020
* upstream/master:
  [JJBB] Add 6.8+ branches (elastic#22321)
  [CI] Support Windows-8 (elastic#22307)
  Change cloud.provider from googlecloud to gcp in billing metricset (elastic#22287)
  [packaging][beats-tester] use commit id binaries (elastic#22027)
  [CI] Report error in the catch section (elastic#22297)
v1v added a commit to v1v/beats that referenced this pull request Nov 3, 2020
v1v added a commit to v1v/beats that referenced this pull request Nov 3, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation ci Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants