Skip to content

[8.18] [Stack Monitoring] New FTR for "view logs" link (#208351)#214296

Merged
kibanamachine merged 4 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-208351
Mar 13, 2025
Merged

[8.18] [Stack Monitoring] New FTR for "view logs" link (#208351)#214296
kibanamachine merged 4 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-208351

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

## Summary

Closes elastic#202675

A [recent bug](elastic#199902) broke
the Stack Monitoring UI only if/when logs were present. Functional test
coverage was lacking to detect such a situation.

This PR adds some functional test coverage to make sure that the "View
logs" link in Stack Monitoring works whenever logs are available,
wherever logs can be visible, i.e. in the cluster overview page, the
node details page and the index details page.

### Checklist

- [X] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit c218f42)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 12, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 12, 2025 22:41
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #1 / PushButton renders the correct label when the connector has not been pushed

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
monitoring 507.8KB 507.9KB +69.0B

History

cc @consulthys

@kibanamachine kibanamachine merged commit a98a4c3 into elastic:8.18 Mar 13, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants