Skip to content

Conversation

@targos
Copy link
Member

@github-actionsgithub-actionsbot added needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Jun 29, 2021
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@targostargos added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 30, 2021
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 30, 2021
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jun 30, 2021

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jul 9, 2021

@targostargos added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. dont-land-on-v12.x labels Jul 9, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@targostargos added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 10, 2021
@github-actionsgithub-actionsbot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 10, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/39196 ✔ Done loading data for nodejs/node/pull/39196 ----------------------------------- PR info ------------------------------------ Title deps: patch V8 to 9.1.269.38 (#39196) Author Michaël Zasso (@targos) Branch targos:patch-v8-91 -> nodejs:master Labels author ready, backport-open-v14.x, dont-land-on-v12.x, needs-ci, v8 engine Commits 1 - deps: patch V8 to 9.1.269.38 Committers 1 - Michaël Zasso PR-URL: https://github.com/nodejs/node/pull/39196 Fixes: https://github.com/nodejs/node/issues/37553 Refs: https://github.com/v8/v8/compare/9.1.269.36...9.1.269.38 Reviewed-By: Richard Lau Reviewed-By: Gireesh Punathil Reviewed-By: Jiawen Geng Reviewed-By: Matteo Collina Reviewed-By: Tobias Nießen Reviewed-By: Colin Ihrig ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/39196 Fixes: https://github.com/nodejs/node/issues/37553 Refs: https://github.com/v8/v8/compare/9.1.269.36...9.1.269.38 Reviewed-By: Richard Lau Reviewed-By: Gireesh Punathil Reviewed-By: Jiawen Geng Reviewed-By: Matteo Collina Reviewed-By: Tobias Nießen Reviewed-By: Colin Ihrig -------------------------------------------------------------------------------- ⚠ Commits were pushed since the last review: ⚠ - deps: patch V8 to 9.1.269.38 ℹ This PR was created on Tue, 29 Jun 2021 18:42:34 GMT ✔ Approvals: 6 ✔ - Richard Lau (@richardlau): https://github.com/nodejs/node/pull/39196#pullrequestreview-695388471 ✔ - Gireesh Punathil (@gireeshpunathil) (TSC): https://github.com/nodejs/node/pull/39196#pullrequestreview-695638866 ✔ - Jiawen Geng (@gengjiawen): https://github.com/nodejs/node/pull/39196#pullrequestreview-695639089 ✔ - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/39196#pullrequestreview-695711864 ✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/39196#pullrequestreview-701145607 ✔ - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/39196#pullrequestreview-701545529 ✔ Last GitHub Actions successful ℹ Last Full PR CI on 2021-07-10T06:33:31Z: https://ci.nodejs.org/job/node-test-pull-request/38934/ ℹ Last V8 CI on 2021-07-09T11:58:35Z: https://ci.nodejs.org/job/node-test-commit-v8-linux/4127/ - Querying data for job/node-test-pull-request/38934/ ✔ Build data downloaded ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1017514995

@targos
Copy link
MemberAuthor

Landed in e2148d7

@targostargos closed this Jul 10, 2021
@targostargos deleted the patch-v8-91 branch July 10, 2021 07:38
targos added a commit that referenced this pull request Jul 10, 2021
Refs: v8/v8@9.1.269.36...9.1.269.38Fixes: #37553 PR-URL: #39196 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
targos added a commit that referenced this pull request Jul 11, 2021
Refs: v8/v8@9.1.269.36...9.1.269.38Fixes: #37553 PR-URL: #39196 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@targostargos mentioned this pull request Jul 13, 2021
@targostargos added backported-to-v14.x and removed backport-open-v14.x commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Sep 5, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.needs-ciPRs that need a full CI run.v8 engineIssues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Node.js v14.15.5 segfault in v8::internal::ConcurrentMarking::Run

8 participants

@targos@nodejs-github-bot@mcollina@cjihrig@tniessen@gengjiawen@richardlau@gireeshpunathil