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
[Cases] Unskip MKI tests #168924
[Cases] Unskip MKI tests #168924
Conversation
await retry.waitFor('update toast exist', async () => { | ||
return await testSubjects.exists('toastCloseButton'); | ||
}); | ||
|
||
await testSubjects.click('toastCloseButton'); | ||
|
||
await sync.click(); | ||
await header.waitUntilLoadingHasFinished(); |
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.
We need to wait here for the case to be refreshed. Otherwise we get an 403 conflict error because the test is too fast.
await dashboard.waitForRenderComplete(); | ||
}); | ||
|
||
after(async () => { | ||
await cases.api.deleteAllCases(); | ||
await svlCases.api.deleteAllCaseItems(); |
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.
Needed to avoid permission errors when deleting directly a document from the .kibana_alerting_cases
system index.
Pinging @elastic/response-ops (Team:ResponseOps) |
Pinging @elastic/response-ops-cases (Feature:Cases) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @cnasikas |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 2146a7e) # Conflicts: # x-pack/test_serverless/functional/test_suites/observability/cases/attachment_framework.ts # x-pack/test_serverless/functional/test_suites/observability/cases/list_view.ts # x-pack/test_serverless/functional/test_suites/observability/cases/view_case.ts # x-pack/test_serverless/functional/test_suites/security/ftr/cases/list_view.ts # x-pack/test_serverless/functional/test_suites/security/ftr/cases/view_case.ts # x-pack/test_serverless/functional/test_suites/security/index.ts
# Backport This will backport the following commits from `main` to `8.11`: - [[Cases] Unskip MKI tests (#168924)](#168924) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Christos Nasikas","email":"christos.nasikas@elastic.co"},"sourceCommit":{"committedDate":"2023-10-23T09:21:25Z","message":"[Cases] Unskip MKI tests (#168924)\n\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"2146a7ef1634a06cf6b4b70edc55dd22fd5b6a4c","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:skip","Team:ResponseOps","Feature:Cases","ci:skip-cypress-osquery","v8.11.0","v8.12.0"],"number":168924,"url":"#168924 Unskip MKI tests (#168924)\n\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"2146a7ef1634a06cf6b4b70edc55dd22fd5b6a4c"}},"sourceBranch":"main","suggestedTargetBranches":["8.11"],"targetPullRequestStates":[{"branch":"8.11","label":"v8.11.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"#168924 Unskip MKI tests (#168924)\n\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"2146a7ef1634a06cf6b4b70edc55dd22fd5b6a4c"}}]}] BACKPORT-->
Summary
This PR unskips all Cases' serverless tests for MKI. I run all tests without any issues to a Serverless instance deployed on the cloud.
Notable changes:
deleteAllCaseItems
from the Serverless API service and not the ESS API service. Not using the Serverless API service was causing issues when trying to delete documents from the.kibana_alerting_cases
system index.Fixes: #165135, #168534, #169005
Flaky test runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3579, https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3639
Checklist
Delete any items that are not applicable to this PR.
For maintainers