-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Discover] Unskip ES|QL Inspector tests #183894
Conversation
jughosta
commented
May 21, 2024
•
edited by kibanamachine
Loading
edited by kibanamachine
- Closes Failing test: Chrome UI Functional Tests.test/functional/apps/discover/group4/_esql_view·ts - discover/group4 discover esql view inspector shows Discover and Lens requests in Inspector #183847
This comment was marked as outdated.
This comment was marked as outdated.
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
@@ -257,7 +257,6 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) { | |||
}); | |||
}); | |||
|
|||
// FLAKY: https://github.com/elastic/kibana/issues/183847 | |||
describe.skip('inspector', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The flaky comment was removed, but it looks like the tests are still skipped in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦♀️
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
/ci |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6070[✅] test/functional/apps/discover/group4/config.ts: 25/25 tests passed. |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @jughosta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for fixing it!
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
- Closes elastic#183847 (cherry picked from commit d0d6075) # Conflicts: # x-pack/test_serverless/functional/test_suites/common/discover/esql/_esql_view.ts
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
2 similar comments
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
# Backport This will backport the following commits from `main` to `8.14`: - [[Discover] Unskip ES|QL Inspector tests (#183894)](#183894) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Julia Rechkunova","email":"julia.rechkunova@elastic.co"},"sourceCommit":{"committedDate":"2024-05-23T06:40:34Z","message":"[Discover] Unskip ES|QL Inspector tests (#183894)\n\n- Closes #183847 Unskip ES|QL Inspector tests (#183894)\n\n- Closes #183847 Unskip ES|QL Inspector tests (#183894)\n\n- Closes #183847"}}]}] BACKPORT--> Co-authored-by: Davis McPhee <davis.mcphee@elastic.co>