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

[Logs UI][Metrics UI] Reduce infra bundle size #89025

Closed
Tracked by #88678
afgomez opened this issue Jan 21, 2021 · 3 comments
Closed
Tracked by #88678

[Logs UI][Metrics UI] Reduce infra bundle size #89025

afgomez opened this issue Jan 21, 2021 · 3 comments
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature performance R&D Research and development ticket (not meant to produce code, but to make a decision) Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@afgomez
Copy link
Contributor

afgomez commented Jan 21, 2021

Our bundle limit size is currently at 200KB. We would like to reduce that to increase the page load time. As a challenge we would like to cut that limit in half.

Questions

  • What are the outliers in infra.bundle.js?
  • Can they be loaded asyncronously?
  • Can they be replaced with a light weight alternative?
  • Is there any dead weight?

Related: what can we do to reduce the overall footprint of the infra plugin? Even if the entry point is around 200K, some of the other chunks are well above that.

@afgomez afgomez added R&D Research and development ticket (not meant to produce code, but to make a decision) Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.0.0 labels Jan 21, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@afgomez afgomez added this to the Logs UI 7.12 milestone Jan 21, 2021
@afgomez afgomez changed the title [Logs UI][Metrics UI][R&D] Reduce the infra bundle size [Logs UI][Metrics UI][R&D] How to reduce the infra bundle size? Jan 26, 2021
@afgomez afgomez self-assigned this Feb 16, 2021
@afgomez
Copy link
Contributor Author

afgomez commented Feb 16, 2021

I have noticed some things in our bundle:

Screenshot 2021-02-16 at 13 18 08

Potentially unnecessary modules

  • Inside node_modules there are two big boxes: URL and mustache. We can see where they are being used and potentially replace them with native alternatives or remove them completely.

Alerting components

  • Inside public/alerting and common/alerting there are modules for log_threshold and metric_threshold. These can potentially be moved so the log modules are loaded only in logs, and the metrics only in metrics.

@jasonrhodes jasonrhodes removed this from the Logs UI 7.12 milestone Jul 12, 2021
@lizozom lizozom changed the title [Logs UI][Metrics UI][R&D] How to reduce the infra bundle size? [Logs UI][Metrics UI] Reduce infra bundle size Apr 18, 2022
@smith
Copy link
Contributor

smith commented Jun 14, 2023

Not planning on making any specific changes here at this time.

@smith smith closed this as not planned Won't fix, can't repro, duplicate, stale Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature performance R&D Research and development ticket (not meant to produce code, but to make a decision) Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

4 participants