FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

build: fail early in test-macos.yml by Trott · Pull Request #41035 · nodejs/node · GitHub

/ node Public

build: fail early in test-macos.yml - #41035

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:macos-ci
Closed

build: fail early in test-macos.yml#41035
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:macos-ci

Conversation

Trott commented Nov 30, 2021

Copy link
Copy Markdown
Member

No description provided.

nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Nov 30, 2021

targos commented Nov 30, 2021

Copy link
Copy Markdown
Member

Why does npm ci fail?

Trott commented Nov 30, 2021
edited
Loading

Copy link
Copy Markdown
Member Author

Why does npm ci fail?

It is reporting corrupted tar files from the npm registry.

Here are some links to some failures:

Here's what a failure looks like:

Possibilities I've considered but haven't been able to reproduce:

  1. Perhaps using -j2 in the workflow or something else is causing a race condition where two processes are running npm at the same time and trying to write the same tar files at about the same time, resulting in corrupted tar files.
  2. Perhaps our workflows in particular or GitHub's macOS instances in general are hitting the registry hard enough that the IP or IP range is being rate-limited, resulting in failed tar downlodas.
  3. Perhaps there is some bug that only occurs on macOS and/or only when npm is invoked the way it is in our tools/doc/node_modules task in the Makefile.

targos commented Nov 30, 2021

Copy link
Copy Markdown
Member

I think it would be nice to get input from someone at @nodejs/npm

Trott commented Nov 30, 2021

Copy link
Copy Markdown
Member Author

It occurs to me that npm ci removes node_modules unconditionally so the failure may still happen anyway after the Build step. (Or maybe not because the relevant modules will be cached somewhere that npm-cli.js will find them? I was going to run the workflow a dozen times or whatever anyway to make sure it worked, so we'll probably find out if someone doesn't figure out a better way to address this issue.)

Comment thread .github/workflows/test-macos.yml Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe this would be better?

Suggested change
run: (cd tools/doc && npm ci)
run: make tools/doc/node_modules

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This might address the concern in #41035 (comment)?

Trott force-pushed the macos-ci branch 5 times, most recently from d780625 to 4d99903 Compare December 3, 2021 12:53

Trott commented Dec 3, 2021

Copy link
Copy Markdown
Member Author

Five successful GitHub Actions runs in a row for this PR:

Not exactly conclusive, as only 1 of the 5 runs before that failed with the npm/ZDATA error. But promising so far.

Trott force-pushed the macos-ci branch 6 times, most recently from c628380 to f68edc4 Compare December 3, 2021 17:55

Copy link
Copy Markdown
Member

Why does npm ci fail?

It is reporting corrupted tar files from the npm registry.

Here are some links to some failures:

* https://github.com/nodejs/node/runs/4364074731?check_suite_focus=true

* https://github.com/nodejs/node/runs/4360537144?check_suite_focus=true

Here's what a failure looks like:

Maybe the macOS runners have some sort of proxy in front of the registry and we're seeing npm/cli#3884? Possibly fix isaacs/minipass#28.

Trott commented Dec 4, 2021

Copy link
Copy Markdown
Member Author

Five more successful runs in a row:

PR-URL: nodejs#41035
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

Trott commented Dec 4, 2021

Copy link
Copy Markdown
Member Author

Landed in 781408f

Trott closed this Dec 4, 2021
Trott deleted the macos-ci branch December 4, 2021 05:30
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
PR-URL: #41035
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
PR-URL: #41035
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
PR-URL: #41035
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41035
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams mentioned this pull request Feb 1, 2022

mmomtchev commented Jul 2, 2022
edited
Loading

Copy link
Copy Markdown
Contributor

@Trott I am getting this failure when building a Debian package inside of a Launchpad container - and only inside the Launchpad container - so I guess macOS is not to blame

PS No, in fact, for me npm is failing because it cannot access the registry - it is still the same error code, sorry for the noise

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL