Skip to content

[Reporting] update puppeteer to version 23.7.0#199304

Merged
sebelga merged 4 commits intoelastic:mainfrom
sebelga:update-puppeteer-chromium
Nov 7, 2024
Merged

[Reporting] update puppeteer to version 23.7.0#199304
sebelga merged 4 commits intoelastic:mainfrom
sebelga:update-puppeteer-chromium

Conversation

@sebelga
Copy link
Contributor

@sebelga sebelga commented Nov 7, 2024

Summary

Update for puppeteer, the following changeset updates puppeteer to version 23.7.0.

The chromium version required for this version of puppeteer is 130.0.6723.91 from revision 1356013, as such the chromium binary included for windows and darwin platforms either match or were the closest revision to the expectation. The linux headless binary was built from commit 53ac076783696778ecc8f360ea31765c29c240ad of the same revision.

pdfmake

I also updated the version of pdfmake to the latest 0.2.15

Manual tests

I successfully managed to create pdf report on my Mac machine and on Linux (following the instructions here)

Fixes #194494

@sebelga sebelga self-assigned this Nov 7, 2024
@sebelga sebelga added release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// backport:prev-major labels Nov 7, 2024
@sebelga
Copy link
Contributor Author

sebelga commented Nov 7, 2024

/ci

@sebelga
Copy link
Contributor Author

sebelga commented Nov 7, 2024

/ci

@sebelga
Copy link
Contributor Author

sebelga commented Nov 7, 2024

/ci

@tsullivan tsullivan added the backport:version Backport to applied version labels label Nov 7, 2024
Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Tested Linux in Docker

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #20 / console app console app with comments with multiline comments should allow in request body, using /* */

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
maps 3.0MB 3.0MB +394.0B

History

cc @sebelga

@sebelga sebelga marked this pull request as ready for review November 7, 2024 17:48
@sebelga sebelga requested a review from a team as a code owner November 7, 2024 17:48
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@sebelga sebelga merged commit 1761047 into elastic:main Nov 7, 2024
@sebelga sebelga deleted the update-puppeteer-chromium branch November 7, 2024 17:49
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 7.17, 8.15, 8.16, 8.x

https://github.com/elastic/kibana/actions/runs/11728488859

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 7, 2024
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts
8.15 Backport failed because of merge conflicts
8.16 Backport failed because of merge conflicts

You might need to backport the following PRs to 8.16:
- [Fleet] [Cloud Security] Add Testing Library ESLint for handling waitFor (#198735)
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 199304

Questions ?

Please refer to the Backport tool documentation

@tsullivan
Copy link
Member

@sebelga hopefully the merge conflicts in backporting are only due to the yarn.lock changes 🤞🏻

kibanamachine added a commit that referenced this pull request Nov 7, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Reporting] update puppeteer to version 23.7.0
(#199304)](#199304)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sébastien
Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2024-11-07T17:49:12Z","message":"[Reporting]
update puppeteer to version 23.7.0
(#199304)","sha":"1761047b27515c4348b742d9c45892b78ab51b49","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-major","backport:version","v7.17.26"],"title":"[Reporting]
update puppeteer to version
23.7.0","number":199304,"url":"https://github.com/elastic/kibana/pull/199304","mergeCommit":{"message":"[Reporting]
update puppeteer to version 23.7.0
(#199304)","sha":"1761047b27515c4348b742d9c45892b78ab51b49"}},"sourceBranch":"main","suggestedTargetBranches":["7.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199304","number":199304,"mergeCommit":{"message":"[Reporting]
update puppeteer to version 23.7.0
(#199304)","sha":"1761047b27515c4348b742d9c45892b78ab51b49"}},{"branch":"7.17","label":"v7.17.26","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
sebelga added a commit to sebelga/kibana that referenced this pull request Nov 8, 2024
(cherry picked from commit 1761047)

# Conflicts:
#	yarn.lock
sebelga added a commit to sebelga/kibana that referenced this pull request Nov 8, 2024
(cherry picked from commit 1761047)

# Conflicts:
#	package.json
#	yarn.lock
@sebelga
Copy link
Contributor Author

sebelga commented Nov 8, 2024

💚 All backports created successfully

Status Branch Result
8.16
8.15
7.17

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

sebelga added a commit to sebelga/kibana that referenced this pull request Nov 8, 2024
(cherry picked from commit 1761047)

# Conflicts:
#	package.json
#	yarn.lock
@sebelga
Copy link
Contributor Author

sebelga commented Nov 8, 2024

hopefully the merge conflicts in backporting are only due to the yarn.lock changes

Yes it was mainly that, and some changes to package.json too

sebelga added a commit that referenced this pull request Nov 8, 2024
)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Reporting] update puppeteer to version 23.7.0
(#199304)](#199304)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)
sebelga added a commit that referenced this pull request Nov 8, 2024
)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[Reporting] update puppeteer to version 23.7.0
(#199304)](#199304)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)
mbondyra pushed a commit to mbondyra/kibana that referenced this pull request Nov 8, 2024
sebelga added a commit that referenced this pull request Nov 8, 2024
)

# Backport

This will backport the following commits from `main` to `7.17`:
- [[Reporting] update puppeteer to version 23.7.0
(#199304)](#199304)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// v7.17.26 v8.15.5 v8.16.1 v8.17.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Screenshotting/PDF] Update pdfmake dependency

6 participants