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: Firefox UI Functional Tests.test/functional/apps/visualize/_tsvb_chart·ts - visualize app visual builder switch index patterns should be able to switch between index patterns #71979

Closed
kibanamachine opened this issue Jul 15, 2020 · 12 comments · Fixed by #72882, #74110, #74736 or #78754
Assignees
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Feature:TSVB TSVB (Time Series Visual Builder) skipped-test Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 15, 2020

A test failed on a tracked branch

Error: retry.try timeout: TimeoutError: Waiting for element to be located By(css selector, .euiFilterSelectItem)
Wait timed out after 10106ms
    at /dev/shm/workspace/kibana/node_modules/selenium-webdriver/lib/webdriver.js:842:17
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at onFailure (test/common/services/retry/retry_for_success.ts:28:9)
    at retryForSuccess (test/common/services/retry/retry_for_success.ts:68:13)

First failure: Jenkins Build

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Jul 15, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-test-triage (failed-test)

spalger added a commit that referenced this issue Jul 15, 2020
spalger added a commit that referenced this issue Jul 15, 2020
(cherry picked from commit 7787d1e)
spalger added a commit that referenced this issue Jul 15, 2020
(cherry picked from commit 7787d1e)
@spalger
Copy link
Contributor

spalger commented Jul 15, 2020

First failure out of PRs, but has failed 13 times in PRs in the last week

image

Skipped

master: 7787d1e
7.x/7.10: 8566491
7.9: c657cdb

@spalger spalger added blocker Feature:TSVB TSVB (Time Series Visual Builder) skipped-test Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.9.0 labels Jul 15, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

gmmorris added a commit to gmmorris/kibana that referenced this issue Jul 16, 2020
* master: (37 commits)
  [Lens] Handle failing existence check (elastic#70718)
  [Security Solution]Fix in-app links and popup window text (elastic#71403)
  [esArchiver] automatically retry if alias creation fails (elastic#71910)
  Move data stream index pattern creation test to xpack (elastic#71511)
  [Maps] Improve language for mvt card (elastic#71947)
  [Security][Detections] Unskip failing modal tests (elastic#71969)
  skip flaky suite (elastic#71987)
  skip flaky suite (elastic#71979)
  [Security Solution] [Detections] Revert "[Security Solution] [Detections] Fixes bug for determining when we hit max signals after filtering with lists (elastic#71768)" (elastic#71956)
  rename ilm policy to remove -default (elastic#71952)
  Adjust ordering of Management category apps to make Ingest Manager higher (elastic#71948)
  skip flaky suite (elastic#71971)
  skip flaky suite (elastic#71951)
  [kbn/optimizer] ignore compressed files when reporting stats (elastic#71940)
  skip flaky suite (elastic#71867)
  [ML] Fix new job with must_not saved search (elastic#71831)
  [Resolver] Fix bug where process detail panel doesn't show up (elastic#71754)
  Cleanup (elastic#71849)
  [Resolver] aria-level and aria-flowto support enhancements (elastic#71887)
  skip flaky suite (elastic#71304)
  ...
gmmorris added a commit to gmmorris/kibana that referenced this issue Jul 16, 2020
…feature-privileges

* alerting/consumer-based-rbac: (491 commits)
  [Lens] Handle failing existence check (elastic#70718)
  [Security Solution]Fix in-app links and popup window text (elastic#71403)
  [esArchiver] automatically retry if alias creation fails (elastic#71910)
  Move data stream index pattern creation test to xpack (elastic#71511)
  [Maps] Improve language for mvt card (elastic#71947)
  [Security][Detections] Unskip failing modal tests (elastic#71969)
  skip flaky suite (elastic#71987)
  skip flaky suite (elastic#71979)
  [Security Solution] [Detections] Revert "[Security Solution] [Detections] Fixes bug for determining when we hit max signals after filtering with lists (elastic#71768)" (elastic#71956)
  rename ilm policy to remove -default (elastic#71952)
  Adjust ordering of Management category apps to make Ingest Manager higher (elastic#71948)
  skip flaky suite (elastic#71971)
  skip flaky suite (elastic#71951)
  [kbn/optimizer] ignore compressed files when reporting stats (elastic#71940)
  skip flaky suite (elastic#71867)
  [ML] Fix new job with must_not saved search (elastic#71831)
  [Resolver] Fix bug where process detail panel doesn't show up (elastic#71754)
  Cleanup (elastic#71849)
  [Resolver] aria-level and aria-flowto support enhancements (elastic#71887)
  skip flaky suite (elastic#71304)
  ...
@dej611 dej611 self-assigned this Jul 16, 2020
@dej611
Copy link
Contributor

dej611 commented Jul 17, 2020

I managed to reproduce this issue semi-consistently on my local machine with the following setup:

  • Force a retry for the setAbsoluteRange action:
     await retry.try(async () => {
       await PageObjects.timePicker.setAbsoluteRange(fromTime, toTime);
     });
    Without this retry the EUI time picker does not behave correctly for some reason
  • make the runner start with the Firefox configuration
  • once the automated Firefox instance starts, move it beneath another application window so that it will be completely covered by it
    • every other run the bug appears

What I tried so far:

  • Force a retry of the combobox action
     await retry.try(async () => {
       await PageObjects.visualBuilder.selectIndexPatternTimeField('timestamp');
     });
    Note that the selectIndexPatternTimeField already has a retry internally, so here it's really just forcing it even more
  • Force a wait (500 and 2000 ms) before the test:
     before(async () => {
     	await PageObjects.common.sleep(2000);
     });
    Surprisingly this kicks in a strange behaviour in the EUI combobox where the dropdown is rendered in the wrong position, leading to an occlusion:
     ElementClickInterceptedError: Element <button id="i7cccf750-c814-11ea-be31-25ee38153d22__option-1" class="euiFilterSelectItem" type="button"> is not clickable at point (201,40) because another element <nav class="euiBreadcrumbs euiHeaderBreadcrumbs euiBreadcrumbs--truncate"> obscures it
    
  • Force a wait (500 and 2000 ms) before the combobox action leads to the same result as wait before the test

Some notes before explaining the attempts made:

  • In Chrome the bug cannot be reproduced - either regular or headless mode
  • In Firefox the bug cannot be reproduced when the window is in the front or when in headless mode
  • Shadowing the window as explained above showed other similar issues with other tests of the TSVB suite, still related to the EUI combobox dropdown.
  • The setup above works with the suite isolated from the rest of the group or even when all group is executed.

Why shadowing the browser window works to reproduce the bug?

I can only make assumptions here, but I guess the windows shadowing leads to a lazy rendering which may simulate some sort of simulation of a slow environment.

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine kibanamachine reopened this Aug 8, 2020
@kibanamachine
Copy link
Contributor Author

kibanamachine commented Aug 9, 2020

New failure: Jenkins Build

jobs:elastic+kibana+master:7264:ci-worker:parallel

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@spalger
Copy link
Contributor

spalger commented Sep 21, 2020

Opened backport of fix to 7.9 #78098

@flash1293
Copy link
Contributor

Thanks for taking care of that @spalger Can we close the issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment