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

Skip flaky retry upgrade test. #3156

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

cmacknz
Copy link
Member

@cmacknz cmacknz commented Jul 31, 2023

This will fix the build on main and avoid wasting time while we investigate the root cause.

See #3155

This test fails often enough for me that we just need to disable it while we investigate.

@cmacknz cmacknz added the Team:Elastic-Agent Label for the Agent team label Jul 31, 2023
@cmacknz cmacknz self-assigned this Jul 31, 2023
@cmacknz cmacknz requested a review from a team as a code owner July 31, 2023 18:07
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

@mergify
Copy link
Contributor

mergify bot commented Jul 31, 2023

This pull request does not have a backport label. Could you fix it @cmacknz? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@cmacknz cmacknz added the backport-v8.9.0 Automated backport with mergify label Jul 31, 2023
@mergify mergify bot removed the backport-skip label Jul 31, 2023
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 31, 2023

💚 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: 2023-07-31T18:17:47.160+0000

  • Duration: 26 min 3 sec

Test stats 🧪

Test Results
Failed 0
Passed 6153
Skipped 31
Total 6184

💚 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.

  • /package : Generate the packages.

  • run integration tests : Run the Elastic Agent Integration tests.

  • run end-to-end tests : Generate the packages and run the E2E Tests.

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

This will fix the build on main and avoid wasting time while we
investigate the root cause.
@cmacknz cmacknz force-pushed the skip-flaky-upgrade-retry-test branch from 11610f8 to d5fad3e Compare July 31, 2023 18:17
@elasticmachine
Copy link
Contributor

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 98.718% (77/78) 👍
Files 68.75% (187/272) 👍
Classes 67.659% (341/504) 👍
Methods 54.036% (1071/1982) 👍
Lines 40.208% (12280/30541) 👎 -0.02
Conditionals 100.0% (0/0) 💚

Copy link
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it needs to be disabled.

@blakerouse blakerouse merged commit b60b8b0 into elastic:main Jul 31, 2023
mergify bot pushed a commit that referenced this pull request Jul 31, 2023
This will fix the build on main and avoid wasting time while we
investigate the root cause.

See #3155

This test fails often enough for me that we just need to disable it
while we investigate.

(cherry picked from commit b60b8b0)
@cmacknz cmacknz deleted the skip-flaky-upgrade-retry-test branch July 31, 2023 19:17
cmacknz added a commit that referenced this pull request Jul 31, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
This is an automatic backport of pull request #3156 done by
[Mergify](https://mergify.com).


---


<details>
<summary>Mergify commands and options</summary>

<br />

More conditions and actions can be found in the
[documentation](https://docs.mergify.com/).

You can also trigger Mergify actions by commenting on this pull request:

- `@Mergifyio refresh` will re-evaluate the rules
- `@Mergifyio rebase` will rebase this PR on its base branch
- `@Mergifyio update` will merge the base branch into this PR
- `@Mergifyio backport <destination>` will backport this PR on
`<destination>` branch

Additionally, on Mergify [dashboard](https://dashboard.mergify.com) you
can:

- look at your merge queues
- generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com
</details>

Co-authored-by: Craig MacKenzie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v8.9.0 Automated backport with mergify skip-changelog Team:Elastic-Agent Label for the Agent team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants