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

[Infrastructure UI] Metrics Explorer Saved View flaky test fix #160504

Conversation

crespocarlos
Copy link
Contributor

@crespocarlos crespocarlos commented Jun 26, 2023

closes #156513

Summary

Fix flaky Metrics Explorer Saved Views functional test. Based on the screenshot, I suspect that the test tries to click the saved object button when it's still disabled.

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@crespocarlos crespocarlos added Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Feature:ObsHosts Hosts feature within Observability v8.10.0 release_note:skip Skip the PR/issue when compiling release notes labels Jun 26, 2023
@crespocarlos
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 14 16 +2
securitySolution 416 420 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 15 17 +2
securitySolution 497 501 +4
total +6

History

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

@crespocarlos crespocarlos marked this pull request as ready for review June 26, 2023 13:57
@crespocarlos crespocarlos requested a review from a team as a code owner June 26, 2023 13:57
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@mohamedhamed-ahmed mohamedhamed-ahmed left a comment

Choose a reason for hiding this comment

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

LGTM!

@crespocarlos crespocarlos merged commit 3d05f74 into elastic:main Jun 27, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 27, 2023
…ic#160504)

closes [elastic#156513](elastic#156513)

## Summary

Fix flaky Metrics Explorer Saved Views functional test. Based on the
[screenshot](https://buildkiteartifacts.com/e0f3970e-3a75-4621-919f-e6c773e2bb12/b1106efc-ba5e-4e90-a857-11c4a899c05d/0188e3a3-7b0a-4a2b-b305-41f088a36c7b/0188e3a5-5e7d-4cac-9ca2-ad049aa86341/x-pack/test/functional/screenshots/failure/InfraOps%20App%20Metrics%20UI%20Metrics%20Explorer%20Saved%20Views%20should%20update%20the%20current%20s-548c19241effa693af1bdaa2945c1bb8f7a0de0759993758d705b60a14e17c8d.png?response-content-type=image%2Fpng&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIAQPCP3C7LZDIHP4OQ%2F20230626%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230626T100738Z&X-Amz-Expires=600&X-Amz-Security-Token=IQoJb3JpZ2luX2VjENf%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaCXVzLWVhc3QtMSJHMEUCIQCWRh%2Bz%2BuUkCNazAWmBVuDmsKyUAOlJ%2F71ynwSjXcFzGwIgBwzs5UgxrUrkki2Bl7ucYrIa0ldQYroVGWCAyxTTGLMq8QMIQBAAGgwwMzIzNzk3MDUzMDMiDMeqhNGm7I4CeupKPyrOA4mOkzowrCIJ9q06o%2B0LNAadRLnRcoUyBFNpVdOOZ0Mz8PCIQv8PxnVFC9c9b0o7zzxXhhIssuWjMG4bjR9PVgAL1xIMwVp9qmsfGELZ4Ctc5Py18PZ6Nsp0IYU9dj58bxyronawXCypdqG%2BaAfcETMYhr7IwusqJQKtJr0XvuWHAABjQp8KLfzU04sKG5XjZtaj%2F9i8ma2ubesispDhEjdsZYr9cy5dLqkHHEBfIi0WHc6wsdlrFge5LVQSe0Cppf7FhKubL39v2FcbPA58ftR075YDIFfgTh4jMaC502WmUBDEI1HhBNPBtGKJQz7gN6iYIlIfW8q1rXPW5KiLU3D%2ByjU0gWAO6xyBb8ShIwxJ8fCRFrpT4487g5VFrJCDwAmhuCpKWu4GtgsWvMr0VNB4M%2BKJKR7%2FicDIEjo6HIDJen1%2FvEwAWRQQOumf5Fpqt3tYYaEtf5BcnFM2EU1q5KQvcaBLvNeXiOBD8qYqfkiDlOc8Zl8C%2F%2FVd%2Fs6LusnDjzrCR%2BiM504i0p6dcjdpIsWQ9HSZKtqAGzJ9%2B34vND8hha%2BXOcuGHTY6TSflpToAv60114tLLfCekIgGhf4397Cgun2HpEB3oHO2hQRuHjDX7%2BSkBjqlAbuJIlwQmjnxEgUXwKpASBJ7B8zgesS8cANKA4i%2FO7g0to4%2FUAhICFkVy1G%2BNLUUrVX2PAMYTAH7iMU8lWxfUcIBMmg%2BCWbL4O4CKUydbwfgojalAN8KksHf%2F3IAHJkH9OrQqG3jQxGlVDUv%2B%2B1WLYwECWczrombRpDVxBl4oX3%2Baq6pvEJdERYNgMU8zfEoOXkV8kfFx7R3utq5vt9lcnB2VDR32w%3D%3D&X-Amz-SignedHeaders=host&X-Amz-Signature=f8c04b64d57c9b317c4ccb68a7961d6aa166a61b38c52b457caf5a2f87b72dfb),
I suspect that the test tries to click the saved object button when it's
still disabled.

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 3d05f74)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.9

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 Jun 27, 2023
…160504) (#160588)

# Backport

This will backport the following commits from `main` to `8.9`:
- [[Infrastructure UI] Metrics Explorer Saved View flaky test fix
(#160504)](#160504)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Carlos
Crespo","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-27T07:54:20Z","message":"[Infrastructure
UI] Metrics Explorer Saved View flaky test fix (#160504)\n\ncloses
[#156513](https://github.com/elastic/kibana/issues/156513)\r\n\r\n##
Summary\r\n\r\nFix flaky Metrics Explorer Saved Views functional test.
Based on
the\r\n[screenshot](https://buildkiteartifacts.com/e0f3970e-3a75-4621-919f-e6c773e2bb12/b1106efc-ba5e-4e90-a857-11c4a899c05d/0188e3a3-7b0a-4a2b-b305-41f088a36c7b/0188e3a5-5e7d-4cac-9ca2-ad049aa86341/x-pack/test/functional/screenshots/failure/InfraOps%20App%20Metrics%20UI%20Metrics%20Explorer%20Saved%20Views%20should%20update%20the%20current%20s-548c19241effa693af1bdaa2945c1bb8f7a0de0759993758d705b60a14e17c8d.png?response-content-type=image%2Fpng&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIAQPCP3C7LZDIHP4OQ%2F20230626%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230626T100738Z&X-Amz-Expires=600&X-Amz-Security-Token=IQoJb3JpZ2luX2VjENf%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaCXVzLWVhc3QtMSJHMEUCIQCWRh%2Bz%2BuUkCNazAWmBVuDmsKyUAOlJ%2F71ynwSjXcFzGwIgBwzs5UgxrUrkki2Bl7ucYrIa0ldQYroVGWCAyxTTGLMq8QMIQBAAGgwwMzIzNzk3MDUzMDMiDMeqhNGm7I4CeupKPyrOA4mOkzowrCIJ9q06o%2B0LNAadRLnRcoUyBFNpVdOOZ0Mz8PCIQv8PxnVFC9c9b0o7zzxXhhIssuWjMG4bjR9PVgAL1xIMwVp9qmsfGELZ4Ctc5Py18PZ6Nsp0IYU9dj58bxyronawXCypdqG%2BaAfcETMYhr7IwusqJQKtJr0XvuWHAABjQp8KLfzU04sKG5XjZtaj%2F9i8ma2ubesispDhEjdsZYr9cy5dLqkHHEBfIi0WHc6wsdlrFge5LVQSe0Cppf7FhKubL39v2FcbPA58ftR075YDIFfgTh4jMaC502WmUBDEI1HhBNPBtGKJQz7gN6iYIlIfW8q1rXPW5KiLU3D%2ByjU0gWAO6xyBb8ShIwxJ8fCRFrpT4487g5VFrJCDwAmhuCpKWu4GtgsWvMr0VNB4M%2BKJKR7%2FicDIEjo6HIDJen1%2FvEwAWRQQOumf5Fpqt3tYYaEtf5BcnFM2EU1q5KQvcaBLvNeXiOBD8qYqfkiDlOc8Zl8C%2F%2FVd%2Fs6LusnDjzrCR%2BiM504i0p6dcjdpIsWQ9HSZKtqAGzJ9%2B34vND8hha%2BXOcuGHTY6TSflpToAv60114tLLfCekIgGhf4397Cgun2HpEB3oHO2hQRuHjDX7%2BSkBjqlAbuJIlwQmjnxEgUXwKpASBJ7B8zgesS8cANKA4i%2FO7g0to4%2FUAhICFkVy1G%2BNLUUrVX2PAMYTAH7iMU8lWxfUcIBMmg%2BCWbL4O4CKUydbwfgojalAN8KksHf%2F3IAHJkH9OrQqG3jQxGlVDUv%2B%2B1WLYwECWczrombRpDVxBl4oX3%2Baq6pvEJdERYNgMU8zfEoOXkV8kfFx7R3utq5vt9lcnB2VDR32w%3D%3D&X-Amz-SignedHeaders=host&X-Amz-Signature=f8c04b64d57c9b317c4ccb68a7961d6aa166a61b38c52b457caf5a2f87b72dfb),\r\nI
suspect that the test tries to click the saved object button when
it's\r\nstill disabled.\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"3d05f74cfdb1cf1083a90bbebf61094772fe3417","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Metrics
UI","Team:Infra Monitoring
UI","release_note:skip","backport:prev-minor","Feature:ObsHosts","v8.10.0"],"number":160504,"url":"https://github.com/elastic/kibana/pull/160504","mergeCommit":{"message":"[Infrastructure
UI] Metrics Explorer Saved View flaky test fix (#160504)\n\ncloses
[#156513](https://github.com/elastic/kibana/issues/156513)\r\n\r\n##
Summary\r\n\r\nFix flaky Metrics Explorer Saved Views functional test.
Based on
the\r\n[screenshot](https://buildkiteartifacts.com/e0f3970e-3a75-4621-919f-e6c773e2bb12/b1106efc-ba5e-4e90-a857-11c4a899c05d/0188e3a3-7b0a-4a2b-b305-41f088a36c7b/0188e3a5-5e7d-4cac-9ca2-ad049aa86341/x-pack/test/functional/screenshots/failure/InfraOps%20App%20Metrics%20UI%20Metrics%20Explorer%20Saved%20Views%20should%20update%20the%20current%20s-548c19241effa693af1bdaa2945c1bb8f7a0de0759993758d705b60a14e17c8d.png?response-content-type=image%2Fpng&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIAQPCP3C7LZDIHP4OQ%2F20230626%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230626T100738Z&X-Amz-Expires=600&X-Amz-Security-Token=IQoJb3JpZ2luX2VjENf%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaCXVzLWVhc3QtMSJHMEUCIQCWRh%2Bz%2BuUkCNazAWmBVuDmsKyUAOlJ%2F71ynwSjXcFzGwIgBwzs5UgxrUrkki2Bl7ucYrIa0ldQYroVGWCAyxTTGLMq8QMIQBAAGgwwMzIzNzk3MDUzMDMiDMeqhNGm7I4CeupKPyrOA4mOkzowrCIJ9q06o%2B0LNAadRLnRcoUyBFNpVdOOZ0Mz8PCIQv8PxnVFC9c9b0o7zzxXhhIssuWjMG4bjR9PVgAL1xIMwVp9qmsfGELZ4Ctc5Py18PZ6Nsp0IYU9dj58bxyronawXCypdqG%2BaAfcETMYhr7IwusqJQKtJr0XvuWHAABjQp8KLfzU04sKG5XjZtaj%2F9i8ma2ubesispDhEjdsZYr9cy5dLqkHHEBfIi0WHc6wsdlrFge5LVQSe0Cppf7FhKubL39v2FcbPA58ftR075YDIFfgTh4jMaC502WmUBDEI1HhBNPBtGKJQz7gN6iYIlIfW8q1rXPW5KiLU3D%2ByjU0gWAO6xyBb8ShIwxJ8fCRFrpT4487g5VFrJCDwAmhuCpKWu4GtgsWvMr0VNB4M%2BKJKR7%2FicDIEjo6HIDJen1%2FvEwAWRQQOumf5Fpqt3tYYaEtf5BcnFM2EU1q5KQvcaBLvNeXiOBD8qYqfkiDlOc8Zl8C%2F%2FVd%2Fs6LusnDjzrCR%2BiM504i0p6dcjdpIsWQ9HSZKtqAGzJ9%2B34vND8hha%2BXOcuGHTY6TSflpToAv60114tLLfCekIgGhf4397Cgun2HpEB3oHO2hQRuHjDX7%2BSkBjqlAbuJIlwQmjnxEgUXwKpASBJ7B8zgesS8cANKA4i%2FO7g0to4%2FUAhICFkVy1G%2BNLUUrVX2PAMYTAH7iMU8lWxfUcIBMmg%2BCWbL4O4CKUydbwfgojalAN8KksHf%2F3IAHJkH9OrQqG3jQxGlVDUv%2B%2B1WLYwECWczrombRpDVxBl4oX3%2Baq6pvEJdERYNgMU8zfEoOXkV8kfFx7R3utq5vt9lcnB2VDR32w%3D%3D&X-Amz-SignedHeaders=host&X-Amz-Signature=f8c04b64d57c9b317c4ccb68a7961d6aa166a61b38c52b457caf5a2f87b72dfb),\r\nI
suspect that the test tries to click the saved object button when
it's\r\nstill disabled.\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"3d05f74cfdb1cf1083a90bbebf61094772fe3417"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160504","number":160504,"mergeCommit":{"message":"[Infrastructure
UI] Metrics Explorer Saved View flaky test fix (#160504)\n\ncloses
[#156513](https://github.com/elastic/kibana/issues/156513)\r\n\r\n##
Summary\r\n\r\nFix flaky Metrics Explorer Saved Views functional test.
Based on
the\r\n[screenshot](https://buildkiteartifacts.com/e0f3970e-3a75-4621-919f-e6c773e2bb12/b1106efc-ba5e-4e90-a857-11c4a899c05d/0188e3a3-7b0a-4a2b-b305-41f088a36c7b/0188e3a5-5e7d-4cac-9ca2-ad049aa86341/x-pack/test/functional/screenshots/failure/InfraOps%20App%20Metrics%20UI%20Metrics%20Explorer%20Saved%20Views%20should%20update%20the%20current%20s-548c19241effa693af1bdaa2945c1bb8f7a0de0759993758d705b60a14e17c8d.png?response-content-type=image%2Fpng&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=ASIAQPCP3C7LZDIHP4OQ%2F20230626%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230626T100738Z&X-Amz-Expires=600&X-Amz-Security-Token=IQoJb3JpZ2luX2VjENf%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaCXVzLWVhc3QtMSJHMEUCIQCWRh%2Bz%2BuUkCNazAWmBVuDmsKyUAOlJ%2F71ynwSjXcFzGwIgBwzs5UgxrUrkki2Bl7ucYrIa0ldQYroVGWCAyxTTGLMq8QMIQBAAGgwwMzIzNzk3MDUzMDMiDMeqhNGm7I4CeupKPyrOA4mOkzowrCIJ9q06o%2B0LNAadRLnRcoUyBFNpVdOOZ0Mz8PCIQv8PxnVFC9c9b0o7zzxXhhIssuWjMG4bjR9PVgAL1xIMwVp9qmsfGELZ4Ctc5Py18PZ6Nsp0IYU9dj58bxyronawXCypdqG%2BaAfcETMYhr7IwusqJQKtJr0XvuWHAABjQp8KLfzU04sKG5XjZtaj%2F9i8ma2ubesispDhEjdsZYr9cy5dLqkHHEBfIi0WHc6wsdlrFge5LVQSe0Cppf7FhKubL39v2FcbPA58ftR075YDIFfgTh4jMaC502WmUBDEI1HhBNPBtGKJQz7gN6iYIlIfW8q1rXPW5KiLU3D%2ByjU0gWAO6xyBb8ShIwxJ8fCRFrpT4487g5VFrJCDwAmhuCpKWu4GtgsWvMr0VNB4M%2BKJKR7%2FicDIEjo6HIDJen1%2FvEwAWRQQOumf5Fpqt3tYYaEtf5BcnFM2EU1q5KQvcaBLvNeXiOBD8qYqfkiDlOc8Zl8C%2F%2FVd%2Fs6LusnDjzrCR%2BiM504i0p6dcjdpIsWQ9HSZKtqAGzJ9%2B34vND8hha%2BXOcuGHTY6TSflpToAv60114tLLfCekIgGhf4397Cgun2HpEB3oHO2hQRuHjDX7%2BSkBjqlAbuJIlwQmjnxEgUXwKpASBJ7B8zgesS8cANKA4i%2FO7g0to4%2FUAhICFkVy1G%2BNLUUrVX2PAMYTAH7iMU8lWxfUcIBMmg%2BCWbL4O4CKUydbwfgojalAN8KksHf%2F3IAHJkH9OrQqG3jQxGlVDUv%2B%2B1WLYwECWczrombRpDVxBl4oX3%2Baq6pvEJdERYNgMU8zfEoOXkV8kfFx7R3utq5vt9lcnB2VDR32w%3D%3D&X-Amz-SignedHeaders=host&X-Amz-Signature=f8c04b64d57c9b317c4ccb68a7961d6aa166a61b38c52b457caf5a2f87b72dfb),\r\nI
suspect that the test tries to click the saved object button when
it's\r\nstill disabled.\r\n\r\nCo-authored-by: Kibana Machine
<[email protected]>","sha":"3d05f74cfdb1cf1083a90bbebf61094772fe3417"}}]}]
BACKPORT-->

Co-authored-by: Carlos Crespo <[email protected]>
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:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability release_note:skip Skip the PR/issue when compiling release notes Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.9.0 v8.10.0
Projects
None yet
6 participants