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

fix: never process elasticsearch CI artifacts#2246

Merged
mdelapenya merged 1 commit intoelastic:mainfrom
mdelapenya:fix-elasticsearch-artifacts
Mar 17, 2022
Merged

fix: never process elasticsearch CI artifacts#2246
mdelapenya merged 1 commit intoelastic:mainfrom
mdelapenya:fix-elasticsearch-artifacts

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

What does this PR do?

It excludes elasticsearch CI artifacts from being processed when a PR from beats/elastic-agent triggers the e2e tests.

Why is it important?

We detected that PRs from Beats/Elastic-Agent triggering the e2e AND running the kubernetes-autodiscover test suite, which is the only one affected by this regression.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the Unit tests (make unit-test), and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

Related issues

@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify bug Something isn't working labels Mar 17, 2022
@mdelapenya mdelapenya self-assigned this Mar 17, 2022
@mdelapenya mdelapenya requested review from a team and ChrsMark March 17, 2022 15:38
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 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-03-17T15:44:19.830+0000

  • Duration: 49 min 3 sec

Test stats 🧪

Test Results
Failed 0
Passed 276
Skipped 0
Total 276

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

@mdelapenya mdelapenya merged commit eb03afb into elastic:main Mar 17, 2022
mergify bot pushed a commit that referenced this pull request Mar 17, 2022
mergify bot pushed a commit that referenced this pull request Mar 17, 2022
mergify bot pushed a commit that referenced this pull request Mar 17, 2022
mdelapenya added a commit that referenced this pull request Mar 21, 2022
(cherry picked from commit eb03afb)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request Mar 21, 2022
(cherry picked from commit eb03afb)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit that referenced this pull request Mar 21, 2022
(cherry picked from commit eb03afb)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request Mar 30, 2022
* main: (72 commits)
  fix: always quote variables in shell scripts (elastic#2284)
  chore: bring back system integration scenarios (elastic#2233)
  chore: do not notify build green for fleet nightly builds (elastic#2277)
  chore: include suite, platform and tags in test reports name (elastic#2273)
  bump stack version 8.2.0-dcff22d7 (elastic#2270)
  bump stack version 8.2.0-4509f321 (elastic#2265)
  chore: bump Godog to v0.12.4 (elastic#2245)
  bump stack version 8.2.0-5cc993c1 (elastic#2254)
  bump stack version 8.2.0-82b803a1 (elastic#2250)
  fix: never process elasticsearch CI artifacts (elastic#2246)
  bump stack version 8.2.0-ff67d7b8 (elastic#2242)
  ci: periodic builds for the last two minor versions (elastic#2241)
  bump stack version 8.2.0-9bac538c (elastic#2240)
  fix: add end-of-file line in templates (elastic#2237)
  chore: add Christos to SSH users
  bump stack version 8.2.0-fee3b8d2 (elastic#2234)
  Separate ES workload from Agent spec and make it req for the scenarios (elastic#2203)
  bump stack version 8.2.0-d02c907a (elastic#2228)
  bump stack version 8.2.0-63265ec9 (elastic#2225)
  fix: wrong link (elastic#2220)
  ...
@mdelapenya mdelapenya deleted the fix-elasticsearch-artifacts branch July 13, 2022 09:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify bug Something isn't working Team:Automation Label for the Observability productivity team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants