-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Upgrade screenshotting Chromium and Puppeteer to v15 #140072
Conversation
buildkite test this |
Pinging @elastic/kibana-app-services (Team:AppServicesUx) |
@elasticmachine merge upstream |
x-pack/plugins/screenshotting/server/browsers/chromium/driver.ts
Outdated
Show resolved
Hide resolved
I pulled some error message text from a debug file by doing the following:
My guess is the build is missing lib dependencies for Vulkan. I plan to update the build script with something similar to lines 227-241 here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving this as the .zip files have been updated with the vulkan libs they need for software rasterization without GPU to work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before merging, let's investigate the cloud issue (timing out). I'll take a look probably tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should not merge before resolving timeouts on cloud. I'll take a look tomorrow.
@elasticmachine merge upstream |
I've tested it on Ubuntu 20, and can verify that:
|
@elasticmachine merge upstream |
Do you think cloud timeouts are related to Chromium upgrade, or PDF generation in general? I thought it generates PNG reports consistently, my guess is maybe there is some problem specific to PDFs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know where the extra 20mb added to our distribution size is coming from? I don't want to block merging, but I hope we can circle back on this. Chromium + fonts + pdf + img libs account for about half of Kibana's download size. If we need it all that's fine, I'm just hoping we can double check if there's room for improvement.
Hi @jbudz for reference, here are the sizes of the bundled binary files for linux.
The previous sizes were:
The reason for the replacement of swiftshader with libvulkan appears related to |
💔 Build FailedFailed CI StepsMetrics [docs]
History
To update your PR or re-run it, just comment with: |
All of the changes related to build-improvement are handled in this PR: #143022 |
Summary
Mozilla/5.0 (X11; Linux aarch64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/105.0.5173.0 Safari/537.36
Checklist
Delete any items that are not applicable to this PR.
For maintainers