Skip to content

[Dataset Quality] Fix flaky test for degraded fields flyout#256076

Closed
couvq wants to merge 2 commits intoelastic:mainfrom
couvq:fix_dataset_quality_degraded_field_flyout_flaky_test
Closed

[Dataset Quality] Fix flaky test for degraded fields flyout#256076
couvq wants to merge 2 commits intoelastic:mainfrom
couvq:fix_dataset_quality_degraded_field_flyout_flaky_test

Conversation

@couvq
Copy link
Copy Markdown
Contributor

@couvq couvq commented Mar 4, 2026

Summary

This PR should fix a flaky test that was discovered after a recent serverless build. The fix is similar to this PR where assertions of UI element presence were occurring without a proper await for all the neccessary data to be loaded.

69 passing (8.0m)
--
1 failing
 
1)    serverless observability UI
Dataset Quality
Degraded fields flyout
detecting root cause for ignored fields
current field limit issues
should display increase field limit as a possible mitigation for special packages like apm app:
 
Error: expected testSubject(datasetQualityDetailsDegradedFieldFlyoutFieldLimitMitigationAccordion) to exist
at TestSubjects.existOrFail (test_subjects.ts:76:13)
at Context.<anonymous> (degraded_field_flyout.ts:967:11)
at Object.apply (wrap_function.js:74:16)

@couvq couvq added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:obs-onboarding Observability Onboarding Team labels Mar 4, 2026
@couvq couvq marked this pull request as ready for review March 4, 2026 18:46
@couvq couvq requested a review from a team as a code owner March 4, 2026 18:46
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-onboarding-team (Team:obs-onboarding)

@elastic-vault-github-plugin-prod elastic-vault-github-plugin-prod bot requested a review from a team as a code owner March 4, 2026 19:01
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@couvq
Copy link
Copy Markdown
Contributor Author

couvq commented Mar 5, 2026

Closing as we decided to skip the test in order to unblock serverless releases. Issue likely stems from how apm index mappings are handled, our test is not set up to handle this in its current state.

@couvq couvq closed this Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-onboarding Observability Onboarding Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants