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

[FLAKY ON MKI] x-pack/test_serverless/functional/test_suites/observability/dataset_quality/degraded_field_flyout.ts #202641

Closed
wayneseymour opened this issue Dec 3, 2024 · 2 comments · Fixed by #203165
Assignees
Labels
failed-test A test failure on a tracked branch, potentially flaky-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:obs-ux-logs Observability Logs User Experience Team

Comments

@wayneseymour
Copy link
Member

wayneseymour commented Dec 3, 2024

Summary

This test started to fail consistently (in 3 of the last 3 runs)

Test

serverless observability UI Dataset Quality Degraded fields flyout testing 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 message

Error: expected testSubject(datasetQualityDetailsDegradedFieldFlyoutFieldLimitMitigationAccordion) to exist
    at TestSubjects.existOrFail (test_subjects.ts:76:13)
    at Context. (degraded_field_flyout.ts:822:11)
    at Object.apply (wrap_function.js:74:16)

Screenshot

Image

Notes

Here's the Kibana commit diff between the last successful and the first failing run: 08c3bc2...6c0cf0e
This PR from the list looks related: #198692

@wayneseymour wayneseymour added failed-test A test failure on a tracked branch, potentially flaky-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. labels Dec 3, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Dec 3, 2024
@gbamparop gbamparop added the Team:obs-ux-logs Observability Logs User Experience Team label Dec 3, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Dec 3, 2024
wayneseymour added a commit that referenced this issue Dec 3, 2024
…ility/dataset_quality/degraded_field_flyout.ts` (#202655)

## Summary

See details: #202641
@weltenwort weltenwort self-assigned this Dec 4, 2024
hop-dev pushed a commit to hop-dev/kibana that referenced this issue Dec 5, 2024
…ility/dataset_quality/degraded_field_flyout.ts` (elastic#202655)

## Summary

See details: elastic#202641
@weltenwort
Copy link
Member

Looks like I finally found the difference between the environments that causes this: Locally and in CI the geoip ingest pipeline processor adds a field: tags: "_geoip_database_unavailable_GeoLite2-City.mmdb" to every document ingested into the APM app log data streams, but on MKI it does not. Since the test is intentionally trying to push the number of fields over the limit to trigger the ignore_dynamic_beyond_limit index setting, this breaks the test.

To me it sounds like the proper solution would be to make sure the Elasticsearch geoip processor works correctly in the local and CI serverless test environment as well and to align the data quality tests with that expectation.

CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 9, 2024
…ility/dataset_quality/degraded_field_flyout.ts` (elastic#202655)

## Summary

See details: elastic#202641
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
…ility/dataset_quality/degraded_field_flyout.ts` (elastic#202655)

## Summary

See details: elastic#202641
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
failed-test A test failure on a tracked branch, potentially flaky-test impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants