-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
dockercloud builds are failing #4808
Comments
I'm not sure what is up with the link from the PR check, but seems to be failing during dependency install due to Cypress (for some reason 🤔) Step 5/14 : COPY package.json package-lock.json /usr/src/app/
---> 43eaac92919c
Step 6/14 : COPY gh-badges /usr/src/app/gh-badges/
---> eb54dfede451
Step 7/14 : RUN NODE_ENV=development CYPRESS_INSTALL_BINARY=0 npm ci
---> Running in f6120315507e
�[91mnpm ERR! code ENOENT
npm ERR! syscall spawn git
npm ERR! path git
npm ERR! errno ENOENT
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t ssh://[email protected]/cypress-io/request.git
npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
�[0m
�[91m
npm ERR! A complete log of this run can be found in:
npm ERR! /root/.npm/_logs/2020-03-22T19_55_06_655Z-debug.log
�[0m
Removing intermediate container f6120315507e
The command '/bin/sh -c NODE_ENV=development CYPRESS_INSTALL_BINARY=0 npm ci' returned a non-zero code: 1 |
Right. I think the issue is that git it not installed. Although, the purpose of |
Exactly, that's the bit that's puzzling me.
I haven't seen anything and their docs still say that's the way to do it 🤷♂️ |
Looks like a regression in Cypress: cypress-io/cypress#6752 It's fixed, but not yet released. We could downgrade, or wait it out and upgrade. |
cypress-io/cypress#6777 (comment) The release is scheduled for Monday, and Cypress 4.1.0 doesn't have this issue. |
The Cypress 4.3.0 update was merged in #4858, and did not seem to fix this problem. |
Err, maybe that's wrong. Master looks good. Perhaps these PR's that are still failing just need updating. |
I think this is fixed. |
This has been working for a while, therefore closing. :) |
Looks like this has been consistently failing for the last couple of days.
I don't have a login for dockercloud to get any more info on this.
The text was updated successfully, but these errors were encountered: