-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Fix flaky functional test in saved objects managements -> delete SO #115291
Fix flaky functional test in saved objects managements -> delete SO #115291
Conversation
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
test/functional/apps/saved_objects_management/inspect_saved_objects.ts
Outdated
Show resolved
Hide resolved
Flaky test runner ref: https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/2057 |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
This comment has been minimized.
This comment has been minimized.
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-core (Team:Core) |
💔 Backport failed
To backport manually run: |
…lastic#115291) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Friendly reminder: Looks like this PR hasn’t been backported yet. |
1 similar comment
Friendly reminder: Looks like this PR hasn’t been backported yet. |
resolves #68400
Increases retry time to attempt to mitigate the race condition that might be making this test flaky.
Introduces a new helper to wait for the Saved Object to render in the Inspect view before deleting it.
The original work for changing the edit view to a read-only json view only targeted 8.0.0.
To fix the failing test on 7.16, we'll need to cut a new PR to 7.16 directly.
Checklist
Risk Matrix
Delete this section if it is not applicable to this PR.
Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.
When forming the risk matrix, consider some of the following examples and how they may potentially impact the change: