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

Attempt to leverage standalone headless chrome binary #207110

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

eokoneyo
Copy link
Contributor

@eokoneyo eokoneyo commented Jan 17, 2025

Summary

This PR relates to https://github.com/elastic/kibana-team/issues/655#issuecomment-2593568117

In this PR, we attempt to leverage the standalone headless chrome binary provided by the chrome team for reporting in Kibana, in place of using the entirety of chrome in darwin and win platforms, furthermore adopting this approach would imply that we don't need to build our own version of the headless chrome anymore.

Giving this a try it works for the darwin platform, see

Screenshot 2025-01-17 at 16 08 06

However for the linux platform the provided binary doesn't work for arm, see screenshot below
Screenshot 2025-01-17 at 17 24 07

This is a known issue, see GoogleChromeLabs/chrome-for-testing#1, which sort of puts a pin in this idea :( of not having to building our our own variant for headless chrome.

@eokoneyo eokoneyo added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels Jan 17, 2025
@eokoneyo eokoneyo self-assigned this Jan 17, 2025
@elasticmachine
Copy link
Contributor

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!

@eokoneyo
Copy link
Contributor Author

eokoneyo commented Jan 17, 2025

Given the progression of the conversation in the linked issue above, maybe it's not worth pursing this approach till the issue is resolved, alternatively we could stick to this approach for all other platforms and continue with building just for ARM 🤷🏾 , @tsullivan wdyt?

@eokoneyo eokoneyo added the Feature:Reporting:Framework Reporting issues pertaining to the overall framework label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Reporting:Framework Reporting issues pertaining to the overall framework release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants