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

[Functional] Fixes cloud failures on TSVB markdown #77158

Merged
merged 6 commits into from
Sep 15, 2020

Conversation

stratoula
Copy link
Contributor

@stratoula stratoula commented Sep 10, 2020

Summary

Closes #76840. Adds a retry to function which locates the TSVB tabs.
Closes #76450
Closes #76186

All the markdown tests that have to do with entering a markdown and wait to be computed fail on Cloud. This was caused because the markdown was typed char by char but the PageObjects.visChart.waitForVisualizationRenderingStabilized() was computed and returned true before the typing had ended.
I have followed a simpler approach. I type the markdown to be computed immediately and then give some time in order to be computed. I have tested on cloud and it seems to not fail, same happens for the regular tet runner.

https://internal-ci.elastic.co/job/elastic+estf-cloud-kibana-flaky-test-runner/121/ - 10times
https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/832/ 100 times

Checklist

@stratoula stratoula changed the title [TSVB] Remove the char by char typing on markodown [TSVB] Remove the char by char typing on markdown Sep 10, 2020
@stratoula stratoula changed the title [TSVB] Remove the char by char typing on markdown [Functional] Remove the char by char typing on TSVB markdown Sep 10, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@stratoula stratoula changed the title [Functional] Remove the char by char typing on TSVB markdown [Functional] Fixes cloud failures on TSVB markdown Sep 15, 2020
@stratoula stratoula added test-cloud failed-test A test failure on a tracked branch, potentially flaky-test Feature:TSVB TSVB (Time Series Visual Builder) Team:Visualizations Visualization editors, elastic-charts and infrastructure release_note:skip Skip the PR/issue when compiling release notes v8.0.0 v7.10.0 labels Sep 15, 2020
@stratoula stratoula marked this pull request as ready for review September 15, 2020 12:47
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Code review only

@stratoula stratoula merged commit 98113ee into elastic:master Sep 15, 2020
stratoula added a commit to stratoula/kibana that referenced this pull request Sep 15, 2020
* [TSVB] Remove the char by char typing on markodown

* revert

* Add the markdown to be computed immediately and give some time to compute it

* Add retry to locate TSVB tabs
stratoula added a commit that referenced this pull request Sep 15, 2020
* [TSVB] Remove the char by char typing on markodown

* revert

* Add the markdown to be computed immediately and give some time to compute it

* Add retry to locate TSVB tabs
stratoula added a commit to stratoula/kibana that referenced this pull request Sep 30, 2020
* [TSVB] Remove the char by char typing on markodown

* revert

* Add the markdown to be computed immediately and give some time to compute it

* Add retry to locate TSVB tabs
stratoula added a commit that referenced this pull request Sep 30, 2020
* [TSVB] Remove the char by char typing on markodown

* revert

* Add the markdown to be computed immediately and give some time to compute it

* Add retry to locate TSVB tabs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment