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.

Sign up for GitHub

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

Jump to bottom

Remove the code used to render the logs stream app and the logs settings page in Observability #204005

Open
gbamparop opened this issue Dec 12, 2024 · 1 comment · May be fixed by #204115
Open

Remove the code used to render the logs stream app and the logs settings page in Observability #204005

gbamparop opened this issue Dec 12, 2024 · 1 comment · May be fixed by #204115
Labels
Team:obs-ux-logs Observability Logs User Experience Team technical debt Improvement of the software architecture and operational architecture

Comments

@gbamparop
Copy link
Contributor

gbamparop commented Dec 12, 2024 •
edited
Loading

📓 Summary

🛑 Blocked by #203996

The logs stream app and the logs settings page in Observability were disabled in #203996 and will not be available in 9.0.0. This issue is to remove the code that is used to render these pages from the infra plugin, which will not result in functional changes.

✔️ Acceptance Criteria

  • The code used to render the logs stream app and the logs settings page in Observability is removed
  • The functional tests targeting the log stream app and the logs settings pages are removed
  • The API tests targeting the log stream app and the logs settings pages are removed
  • CODEOWNERS will be updated to reflect these changes
@gbamparop gbamparop added Team:obs-ux-logs Observability Logs User Experience Team technical debt Improvement of the software architecture and operational architecture labels Dec 12, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

This was referenced Dec 12, 2024
Merged
Draft
gbamparop added a commit that referenced this issue Dec 19, 2024
@gbamparop @kibanamachine @elasticmachine
Disable log stream and settings pages (#203996)
90277c3 
## Release note
Logs Stream and the logs settings page in Observability are removed. Use
the [Discover](https://www.elastic.co/guide/en/kibana/current/discover.html) application,
which now offers a contextual experience for logs, to explore your logs.
The logs stream panel in dashboards is removed, use Discover sessions
instead.

## 📓 Summary
The Logs Stream app in Observability and the log stream panel available
in dashboards were hidden behind an advanced setting as part of
#194519 in `8.16.0`. At the same
time, a link was added to the left navigation for the logs settings
page.

This PR which targets `9.0.0` disables these pages as well as the
dashboard panel:
- Removes the navigation item for the logs settings page and the
corresponding route
- Removes the `observability:enableLogsStream` setting and keeps the
redirects to logs explorer. The locators will be updated to point to
Discover as part of #182229.

Removing the code that renders the logs stream and the settings pages
will be done in a [follow-up
issue](#204005).

### Left navigation

#### Classic
<img width="238" alt="image"
src="https://github.com/user-attachments/assets/bc72c5ce-ed32-472e-91c1-8bd691dd2420"
/>

#### Solution
<img width="275" alt="image"
src="https://github.com/user-attachments/assets/3b21a2ae-5e82-478e-97bb-e12303178a24"
/>

### Navigating to /app/logs/settings
<img width="1722" alt="image"
src="https://github.com/user-attachments/assets/07b4197c-6063-4a59-8194-a97ce2fa3cd7"
/>


Closes elastic/observability-dev#4156

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this issue Dec 19, 2024
@gbamparop @kibanamachine
@elasticmachine @JoseLuisGJ
Disable log stream and settings pages (elastic#203996)
05f21ea 
## Release note
Logs Stream and the logs settings page in Observability are removed. Use
the [Discover](https://www.elastic.co/guide/en/kibana/current/discover.html) application,
which now offers a contextual experience for logs, to explore your logs.
The logs stream panel in dashboards is removed, use Discover sessions
instead.

## 📓 Summary
The Logs Stream app in Observability and the log stream panel available
in dashboards were hidden behind an advanced setting as part of
elastic#194519 in `8.16.0`. At the same
time, a link was added to the left navigation for the logs settings
page.

This PR which targets `9.0.0` disables these pages as well as the
dashboard panel:
- Removes the navigation item for the logs settings page and the
corresponding route
- Removes the `observability:enableLogsStream` setting and keeps the
redirects to logs explorer. The locators will be updated to point to
Discover as part of elastic#182229.

Removing the code that renders the logs stream and the settings pages
will be done in a [follow-up
issue](elastic#204005).

### Left navigation

#### Classic
<img width="238" alt="image"
src="https://github.com/user-attachments/assets/bc72c5ce-ed32-472e-91c1-8bd691dd2420"
/>

#### Solution
<img width="275" alt="image"
src="https://github.com/user-attachments/assets/3b21a2ae-5e82-478e-97bb-e12303178a24"
/>

### Navigating to /app/logs/settings
<img width="1722" alt="image"
src="https://github.com/user-attachments/assets/07b4197c-6063-4a59-8194-a97ce2fa3cd7"
/>


Closes elastic/observability-dev#4156

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Team:obs-ux-logs Observability Logs User Experience Team technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove log stream and settings gbamparop/kibana
2 participants
@gbamparop @elasticmachine