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

Unskip Dashboard to TSVB to Lens should convert a by reference #190230

Merged
merged 2 commits into from
Aug 12, 2024

Conversation

mbondyra
Copy link
Contributor

@mbondyra mbondyra commented Aug 9, 2024

Summary

Fixes #179307

What I suspect happening is that some 'ghost' click is somehow happening outside of the panel settings area (that should be open the point where tests breaks, but the screenshot shows it's closed) and that's why it is closed. Adding a retry around the whole panel settings actions should help, but let's keep an eye on it.

Screenshot 2024-08-09 at 13 44 20

The flakiness is rare so not sure if running the flaky test runner makes sense, but I'd do it anyway: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6733

@mbondyra mbondyra added 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 Feature:Lens backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) v8.16.0 v8.15.1 labels Aug 9, 2024
@mbondyra mbondyra force-pushed the lens/fix_tsvb_to_lens branch from ca7db42 to 25a5bac Compare August 9, 2024 11:47
@mbondyra mbondyra marked this pull request as ready for review August 9, 2024 11:48
@mbondyra mbondyra requested a review from a team as a code owner August 9, 2024 11:49
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

@kibanamachine
Copy link
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#6733

[✅] x-pack/test/functional/apps/lens/open_in_lens/tsvb/config.ts: 200/200 tests passed.

see run history

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] x-pack/test_serverless/functional/test_suites/observability/common_configs/config.group6.ts / discover discover field visualize button visualizes field to Lens and loads fields to the dimesion editor

Metrics [docs]

✅ unchanged

History

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

@mbondyra mbondyra merged commit bb73fc4 into elastic:main Aug 12, 2024
19 checks passed
@mbondyra mbondyra deleted the lens/fix_tsvb_to_lens branch August 12, 2024 18:46
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 12, 2024
…ic#190230)

## Summary

Fixes elastic#179307

What I suspect happening is that some 'ghost' click is somehow happening
outside of the panel settings area (that should be open the point where
tests breaks, but the screenshot shows it's closed) and that's why it is
closed. Adding a retry around the whole panel settings actions should
help, but let's keep an eye on it.

<img width="1100" alt="Screenshot 2024-08-09 at 13 44 20"
src="https://github.com/user-attachments/assets/17d61852-c371-453d-9f41-46fd52f9e384">

The flakiness is rare so not sure if running the flaky test runner makes
sense, but I'd do it anyway:
https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6733

(cherry picked from commit bb73fc4)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Aug 12, 2024
…#190230) (#190353)

# Backport

This will backport the following commits from `main` to `8.15`:
- [Unskip Dashboard to TSVB to Lens should convert a by reference
(#190230)](#190230)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Marta
Bondyra","email":"4283304+mbondyra@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-08-12T18:46:20Z","message":"Unskip
Dashboard to TSVB to Lens should convert a by reference (#190230)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/179307\r\n\r\nWhat I suspect
happening is that some 'ghost' click is somehow happening\r\noutside of
the panel settings area (that should be open the point where\r\ntests
breaks, but the screenshot shows it's closed) and that's why it
is\r\nclosed. Adding a retry around the whole panel settings actions
should\r\nhelp, but let's keep an eye on it.\r\n\r\n\r\n<img
width=\"1100\" alt=\"Screenshot 2024-08-09 at 13 44
20\"\r\nsrc=\"https://github.com/user-attachments/assets/17d61852-c371-453d-9f41-46fd52f9e384\">\r\n\r\nThe
flakiness is rare so not sure if running the flaky test runner
makes\r\nsense, but I'd do it
anyway:\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6733","sha":"bb73fc41bd5cc2abb99479442e62148546c37fbb","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:TSVB","Team:Visualizations","release_note:skip","Feature:Lens","backport:prev-minor","v8.16.0","v8.15.1"],"title":"Unskip
Dashboard to TSVB to Lens should convert a by
reference","number":190230,"url":"https://github.com/elastic/kibana/pull/190230","mergeCommit":{"message":"Unskip
Dashboard to TSVB to Lens should convert a by reference (#190230)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/179307\r\n\r\nWhat I suspect
happening is that some 'ghost' click is somehow happening\r\noutside of
the panel settings area (that should be open the point where\r\ntests
breaks, but the screenshot shows it's closed) and that's why it
is\r\nclosed. Adding a retry around the whole panel settings actions
should\r\nhelp, but let's keep an eye on it.\r\n\r\n\r\n<img
width=\"1100\" alt=\"Screenshot 2024-08-09 at 13 44
20\"\r\nsrc=\"https://github.com/user-attachments/assets/17d61852-c371-453d-9f41-46fd52f9e384\">\r\n\r\nThe
flakiness is rare so not sure if running the flaky test runner
makes\r\nsense, but I'd do it
anyway:\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6733","sha":"bb73fc41bd5cc2abb99479442e62148546c37fbb"}},"sourceBranch":"main","suggestedTargetBranches":["8.15"],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/190230","number":190230,"mergeCommit":{"message":"Unskip
Dashboard to TSVB to Lens should convert a by reference (#190230)\n\n##
Summary\r\n\r\nFixes
https://github.com/elastic/kibana/issues/179307\r\n\r\nWhat I suspect
happening is that some 'ghost' click is somehow happening\r\noutside of
the panel settings area (that should be open the point where\r\ntests
breaks, but the screenshot shows it's closed) and that's why it
is\r\nclosed. Adding a retry around the whole panel settings actions
should\r\nhelp, but let's keep an eye on it.\r\n\r\n\r\n<img
width=\"1100\" alt=\"Screenshot 2024-08-09 at 13 44
20\"\r\nsrc=\"https://github.com/user-attachments/assets/17d61852-c371-453d-9f41-46fd52f9e384\">\r\n\r\nThe
flakiness is rare so not sure if running the flaky test runner
makes\r\nsense, but I'd do it
anyway:\r\nhttps://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/6733","sha":"bb73fc41bd5cc2abb99479442e62148546c37fbb"}},{"branch":"8.15","label":"v8.15.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Marta Bondyra <4283304+mbondyra@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Feature:Lens Feature:TSVB TSVB (Time Series Visual Builder) release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.15.1 v8.16.0
Projects
None yet
5 participants