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

Failing test: Serverless Observability Functional Tests.x-pack/test_serverless/functional/test_suites/observability/observability_log_explorer/header_menu·ts - serverless observability UI Observability Log Explorer Header menu Discover fallback link "before all" hook for "should render a button link " #167515

Closed
kibanamachine opened this issue Sep 28, 2023 · 5 comments · Fixed by #167482 or #173092
Assignees
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Project:Serverless Work as part of the Serverless project for its initial release Team:obs-ux-logs Observability Logs User Experience Team

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Sep 28, 2023

A test failed on a tracked branch

Error: retry.tryForTime timeout: Error: retry.try timeout: TimeoutError: Waiting for element to be located By(css selector, [data-test-subj="kibanaChrome"])
Wait timed out after 61536ms
    at /var/lib/buildkite-agent/builds/kb-n2-4-spot-92958a9bffd811e4/elastic/kibana-on-merge/kibana/node_modules/selenium-webdriver/lib/webdriver.js:929:17
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at onFailure (retry_for_success.ts:17:9)
    at retryForSuccess (retry_for_success.ts:59:13)
    at RetryService.try (retry.ts:31:12)
    at common_page.ts:273:21
    at runAttempt (retry_for_success.ts:29:15)
    at retryForSuccess (retry_for_success.ts:68:21)
    at RetryService.tryForTime (retry.ts:22:12)
    at CommonPageObject.navigateToApp (common_page.ts:272:5)
    at Object.navigateTo (observability_log_explorer.ts:176:14)
    at Context.<anonymous> (header_menu.ts:48:9)
    at Object.apply (wrap_function.js:73:16)
    at onFailure (retry_for_success.ts:17:9)
    at retryForSuccess (retry_for_success.ts:59:13)
    at RetryService.tryForTime (retry.ts:22:12)
    at CommonPageObject.navigateToApp (common_page.ts:272:5)
    at Object.navigateTo (observability_log_explorer.ts:176:14)
    at Context.<anonymous> (header_menu.ts:48:9)
    at Object.apply (wrap_function.js:73:16)

First failure: CI Build - main

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Sep 28, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label Sep 28, 2023
@kibanamachine kibanamachine added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Sep 28, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Sep 28, 2023
@tonyghiani tonyghiani self-assigned this Oct 2, 2023
@gbamparop gbamparop added the Project:Serverless Work as part of the Serverless project for its initial release label Oct 13, 2023
dej611 pushed a commit to dej611/kibana that referenced this issue Oct 17, 2023
## 📓 Summary

Update login logic, sorting and search algorithm to stabilize the flaky
tests for Log Explorer.

Closes elastic#165915
Closes elastic#165916 
Closes elastic#165975  
Closes elastic#166016 
Closes elastic#166672 
Closes elastic#166675 
Closes elastic#166817 
Closes elastic#167058
Closes elastic#167189 
Closes elastic#167499 
Closes elastic#167515  
Closes elastic#167562 
Closes elastic#167567 
Closes elastic#167621 
Closes elastic#167664 
Closes elastic#167773

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
@kibanamachine kibanamachine reopened this Dec 11, 2023
@kibanamachine
Copy link
Contributor Author

New failure: CI Build - main

@weltenwort weltenwort self-assigned this Dec 11, 2023
@weltenwort
Copy link
Member

The most recent fail seems to be caused by a timing problem of the common page object's navigate method. Its ensureCurrentUrl feature seems to be too slow sometimes so it misses the right moment before the URL is rewritten by the app.

I'm working on a "fix" that disables the check for the log explorer navigation helper.

@smith smith added Team:obs-ux-logs Observability Logs User Experience Team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Dec 11, 2023
@elasticmachine
Copy link
Contributor

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

@achyutjhunjhunwala
Copy link
Contributor

Similar test fails on Stateful stack also, linking it here - #171802

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 Project:Serverless Work as part of the Serverless project for its initial release Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
7 participants