Skip to content

Conversation

@stratoula
Copy link
Contributor

@stratoula stratoula commented Oct 14, 2020

Summary

Closes #80246.
I skip these tests for cloud as Discover visualizes a field on the legacy visualizations for OSS and on Lens for Basic+. So these tests are testing the OSS navigation so they should be skipped on cloud as the Basic+ navigation is already tested on the x-pack functional tests.

@stratoula stratoula added release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.11.0 v8.0.0 Feature:Discover Discover Application failed-test A test failure on a tracked branch, potentially flaky-test labels Oct 14, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@stratoula stratoula marked this pull request as ready for review October 14, 2020 10:29
@elasticmachine
Copy link
Contributor

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

@stratoula stratoula requested a review from kertal October 14, 2020 10:29
@stratoula stratoula added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Oct 14, 2020
@elasticmachine
Copy link
Contributor

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

Copy link
Member

@kertal kertal left a comment

Choose a reason for hiding this comment

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

Code LGTM 👍 , thx for the comment giving context, didn't pull and test

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 Feature:Discover Discover Application release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.10.0 v7.11.0 v8.0.0

Projects

None yet

4 participants