Skip to content

ci: e2e-testing should fail the build#31135

Merged
v1v merged 1 commit intoelastic:mainfrom
v1v:feature/dra-e2e-test-fail-pipeline
Apr 4, 2022
Merged

ci: e2e-testing should fail the build#31135
v1v merged 1 commit intoelastic:mainfrom
v1v:feature/dra-e2e-test-fail-pipeline

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Apr 4, 2022 •
edited
Loading

What

Fail the build if the e2e-testing failed

Why

Otherwise the DRA stages are triggered regardless and therefore the staging or snapshot artifacts are uploaded. While the e2e-testing is the quality gate keeper

image

@v1v v1v added backport-v8.3.0 Automated backport with mergify backport-v8.2.0 Automated backport with mergify labels Apr 4, 2022
@v1v v1v requested a review from a team April 4, 2022 17:50
@v1v v1v requested a review from a team as a code owner April 4, 2022 17:50
@v1v v1v self-assigned this Apr 4, 2022
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label Team:Automation Label for the Observability productivity team and removed needs_team Indicates that the issue/PR needs a Team:* label labels Apr 4, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 4, 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-04-04T17:50:35.392+0000

  • Duration: 15 min 51 sec

Test stats 🧪

Test Results
Failed 0
Passed 3
Skipped 0
Total 3

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@v1v v1v merged commit 2241413 into elastic:main Apr 4, 2022
mergify bot pushed a commit that referenced this pull request Apr 4, 2022
@v1v
ci: e2e-testing should fail the build (#31135)
e8a557d 
(cherry picked from commit 2241413)
@mergify mergify bot mentioned this pull request Apr 4, 2022
Merged
v1v added a commit that referenced this pull request Apr 4, 2022
@mergify @v1v
ci: e2e-testing should fail the build (#31135) (#31137)
d5c7706 
(cherry picked from commit 2241413)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
emilioalvap pushed a commit to emilioalvap/beats that referenced this pull request Apr 6, 2022
@v1v @emilioalvap
ci: e2e-testing should fail the build (elastic#31135)
946f713
kush-elastic pushed a commit to kush-elastic/beats that referenced this pull request May 2, 2022
@v1v @kush-elastic
ci: e2e-testing should fail the build (elastic#31135)
aa03abb
This was referenced May 10, 2022
Closed
Merged
@mdelapenya mdelapenya added the backport-7.17 Automated backport to the 7.17 branch with mergify label Jun 14, 2022
@mdelapenya
Copy link
Copy Markdown
Contributor

mdelapenya commented Jun 14, 2022

@Mergifyio backport 7.17

@mergify mergify bot mentioned this pull request Jun 14, 2022
Merged
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jun 14, 2022

backport 7.17

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Jun 14, 2022
@v1v @mergify
ci: e2e-testing should fail the build (#31135)
f8b4a4e 
(cherry picked from commit 2241413)
mergify bot pushed a commit that referenced this pull request Jun 14, 2022
@v1v @mergify
ci: e2e-testing should fail the build (#31135)
7af6cb8 
(cherry picked from commit 2241413)

# Conflicts:
#	.ci/packaging.groovy
@mergify mergify bot mentioned this pull request Jun 14, 2022
Closed
mdelapenya pushed a commit that referenced this pull request Jun 14, 2022
@mergify @v1v
ci: e2e-testing should fail the build (#31135) (#31925)
638844d 
(cherry picked from commit 2241413)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
@cmacknz cmacknz mentioned this pull request Jun 23, 2022
Open
4 tasks
cmacknz added a commit to cmacknz/beats that referenced this pull request Jun 23, 2022
@cmacknz
Revert "ci: e2e-testing should fail the build (elastic#31135)"
74257ce 
This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
elastic#32076
cmacknz added a commit that referenced this pull request Jun 23, 2022
@cmacknz
Revert "ci: e2e-testing should fail the build (#31135)" (#32079)
ef6096f 
This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
#32076
mergify bot pushed a commit that referenced this pull request Jul 4, 2022
@cmacknz @mergify
Revert "ci: e2e-testing should fail the build (#31135)" (#32079)
9898329 
This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
#32076

(cherry picked from commit ef6096f)
v1v pushed a commit that referenced this pull request Jul 4, 2022
@mergify
[8.3](backport #32079) Revert "ci: e2e-testing should fail the build (#…
9304d1d 
…31135)" (#32187)
mergify bot pushed a commit that referenced this pull request Oct 13, 2022
@cmacknz @mergify
Revert "ci: e2e-testing should fail the build (#31135)" (#32079)
21a9114 
This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
#32076

(cherry picked from commit ef6096f)
rdner pushed a commit that referenced this pull request Oct 13, 2022
@mergify @cmacknz
Revert "ci: e2e-testing should fail the build (#31135)" (#32079) (#33343
c276e4b 
)

This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
#32076

(cherry picked from commit ef6096f)

Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co>
chrisberkhout pushed a commit that referenced this pull request Jun 1, 2023
@v1v @chrisberkhout
ci: e2e-testing should fail the build (#31135)
9798dfe
chrisberkhout pushed a commit that referenced this pull request Jun 1, 2023
@cmacknz @chrisberkhout
Revert "ci: e2e-testing should fail the build (#31135)" (#32079)
55b9f58 
This reverts commit 2241413.

Allow the E2E tests to keep running but not fail the build when they
fail while we work to stabilize them:
#32076
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cmacknz cmacknz cmacknz approved these changes

+1 more reviewer

@mdelapenya mdelapenya mdelapenya approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@v1v v1v

Labels

backport-7.17 Automated backport to the 7.17 branch with mergify backport-v8.2.0 Automated backport with mergify backport-v8.3.0 Automated backport with mergify Team:Automation Label for the Observability productivity team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@v1v @elasticmachine @mdelapenya @cmacknz