Skip to content

Conversation

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot commented Aug 21, 2022

This is an automated update of undici to 5.10.0

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Aug 21, 2022
Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

Rubber-stamp LGTM

@Trott Trott added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Aug 21, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2022
@nodejs-github-bot
Copy link
Collaborator Author

Copy link
Contributor

@aduh95 aduh95 left a comment

Choose a reason for hiding this comment

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

RSLGTM

@aduh95 aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Aug 21, 2022
@github-actions
Copy link
Contributor

Fast-track has been requested by @aduh95. Please 👍 to approve.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Collaborator Author

@MoLow
Copy link
Member

MoLow commented Aug 21, 2022

@nodejs/undici this seems to break with a container/machine without OpenSSL:
https://ci.nodejs.org/job/node-test-commit-linux-containered/33340/nodes=ubuntu1804_sharedlibs_withoutssl_x64/

17:44:45     [UNEXPECTED_FAILURE][FAIL] Importing an i32 global with an incorrectly-typed value: ""
17:44:45     Node.js is not compiled with OpenSSL crypto support
17:44:45     Error [ERR_NO_CRYPTO]: Node.js is not compiled with OpenSSL crypto support

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@aduh95 aduh95 removed fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Aug 21, 2022
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina changed the title deps: update undici to 5.9.1 deps: update undici to 5.10.0 Aug 23, 2022
@mcollina mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 23, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 23, 2022
@nodejs-github-bot
Copy link
Collaborator Author

@nodejs-github-bot
Copy link
Collaborator Author

@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 24, 2022
@nodejs-github-bot
Copy link
Collaborator Author

Commit Queue failed
- Loading data for nodejs/node/pull/44319
✔  Done loading data for nodejs/node/pull/44319
----------------------------------- PR info ------------------------------------
Title      deps: update undici to 5.10.0 (#44319)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     nodejs-github-bot:actions/tools-update-undici -> nodejs:main
Labels     dependencies
Commits    2
 - deps: update undici to 5.9.1
 - deps: update undici to 5.10.0
Committers 1
 - Node.js GitHub Bot 
PR-URL: https://github.com/nodejs/node/pull/44319
Reviewed-By: Tobias Nießen 
Reviewed-By: Rich Trott 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Matteo Collina 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/44319
Reviewed-By: Tobias Nießen 
Reviewed-By: Rich Trott 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Matteo Collina 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 21 Aug 2022 00:25:37 GMT
   ✔  Approvals: 5
   ✔  - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/44319#pullrequestreview-1079603475
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/44319#pullrequestreview-1079613718
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/44319#pullrequestreview-1079649039
   ✔  - Mohammed Keyvanzadeh (@VoltrexMaster): https://github.com/nodejs/node/pull/44319#pullrequestreview-1079681960
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/44319#pullrequestreview-1082864768
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2022-08-24T06:29:58Z: https://ci.nodejs.org/job/node-test-pull-request/46206/
- Querying data for job/node-test-pull-request/46206/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 44319
From https://github.com/nodejs/node
 * branch                  refs/pull/44319/merge -> FETCH_HEAD
✔  Fetched commits as 7900f6540ea8..2c4f9d3c69cc
--------------------------------------------------------------------------------
[main dc75916a9f] deps: update undici to 5.9.1
 Date: Sun Aug 21 00:25:34 2022 +0000
 9 files changed, 132 insertions(+), 37 deletions(-)
[main 5573e8557f] deps: update undici to 5.10.0
 Date: Tue Aug 23 21:26:28 2022 +0000
 8 files changed, 214 insertions(+), 36 deletions(-)
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
deps: update undici to 5.9.1

PR-URL: #44319
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Rich Trott rtrott@gmail.com
Reviewed-By: Antoine du Hamel duhamelantoine1995@gmail.com
Reviewed-By: Mohammed Keyvanzadeh mohammadkeyvanzade94@gmail.com
Reviewed-By: Matteo Collina matteo.collina@gmail.com

[detached HEAD c61af061c5] deps: update undici to 5.9.1
Date: Sun Aug 21 00:25:34 2022 +0000
9 files changed, 132 insertions(+), 37 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
deps: update undici to 5.10.0

PR-URL: #44319
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Rich Trott rtrott@gmail.com
Reviewed-By: Antoine du Hamel duhamelantoine1995@gmail.com
Reviewed-By: Mohammed Keyvanzadeh mohammadkeyvanzade94@gmail.com
Reviewed-By: Matteo Collina matteo.collina@gmail.com

[detached HEAD 1f5ff09b51] deps: update undici to 5.10.0
Date: Tue Aug 23 21:26:28 2022 +0000
8 files changed, 214 insertions(+), 36 deletions(-)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/2917091493

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Aug 24, 2022
@MoLow MoLow added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Aug 24, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 24, 2022
@nodejs-github-bot nodejs-github-bot merged commit cfdc713 into main Aug 24, 2022
@nodejs-github-bot nodejs-github-bot deleted the actions/tools-update-undici branch August 24, 2022 07:12
@nodejs-github-bot
Copy link
Collaborator Author

Landed in cfdc713

@kasicka
Copy link

kasicka commented Oct 9, 2022

The commit message and changelog entry are " deps: update undici to 5.9.1", but the PR is "deps: update undici to 5.10.0" and the PR contains actual change to 5.10.0.

@juanarbol juanarbol mentioned this pull request Oct 11, 2022
@mhdawson
Copy link
Member

@kasicka created #44982 to update the version in the changelog entries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. dependencies Pull requests that update a dependency file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants