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

[Reporting] Stabilize ILM test for Cloud #117003

Merged

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Nov 1, 2021

Closes #109456

Summary

ESTF-Cloud testing runs our test suites in a different order than the PR-build job. This means tests could fail if they are not stable enough to run in an unexpected order.

A Reporting test seems to fail in the ESTF-Cloud runner because the test expects the cluster to have the Reporting ILM policy installed by default. This expectation is broken if a previous test deleted the ILM policies in the cluster, which is what seems to be happening.

This fixes the problem by explicitly setting up the Reporting ILM policy that is expected.

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] Default CI Group #11 / detection engine api security and spaces enabled Detection exceptions data types and operators Rule exception operators for data type integer "is in list" operator working against string values in the data set will return 1 result if we have a list which contains the integer range of 1-3

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tsullivan tsullivan added the SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week label Nov 1, 2021
Copy link
Contributor

@jloleysens jloleysens left a comment

Choose a reason for hiding this comment

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

LGTM!

@tsullivan tsullivan merged commit 2a1eaad into elastic:main Nov 2, 2021
@tsullivan tsullivan deleted the reporting/stabilize-ilm-test-cloud branch November 2, 2021 13:57
tsullivan added a commit to tsullivan/kibana that referenced this pull request Nov 2, 2021
tsullivan added a commit to tsullivan/kibana that referenced this pull request Nov 2, 2021
tsullivan added a commit to tsullivan/kibana that referenced this pull request Nov 2, 2021
@sophiec20 sophiec20 added the (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead release_note:skip Skip the PR/issue when compiling release notes SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week v7.15.2 v7.16.0 v8.0.0 v8.1.0
Projects
None yet
5 participants