-
Notifications
You must be signed in to change notification settings - Fork 382
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
cypress/factory with docker 4.36.0 error #1263
Comments
The file which your error log shows is missing:
is included in cypress-docker-images/factory/factory.Dockerfile Lines 121 to 130 in c61c638
The build log only shows the error, not the rest of the steps, so it's difficult to understand the cause, however it looks like the Which operating system are you running Docker Desktop Did you try re-installing your previous version I don't see anything in the release notes that I would expect to cause your issue due to upgrading. |
I'm having the same issue, running a build with factory:4.2.2, if I build the first image FROM cypres/factory AS whatever it runs to the end, but if I use whatever new image FROM whatever AS new_image, it will break as the script is not there anymore. I'v tried to copy it elsewhere and copy it back on the first stages but I don't know how to have it there before the image starts, as the first step is to call the install script. some references:
next stage image:
and the steps I can fetch from the cypress image:
so the question is, is it possible to use cypress in multistage builds? I have tried many workarounds copying the file away, copying it back to the new image... but no success |
Thank you for adding your experience with |
I can reproduce the issue in a multi-stage build when the # syntax = docker/dockerfile:1 Using Docker Desktop # syntax = docker/dockerfile:1.10.0 You may like to see if this could be a temporary workaround. @crlsgms |
well, for now I have tested 1.11, 1.4 , latest and this untagged one but may be some delay from harbor one thing I'm trying here is to copy the script before the ONBUILD chain finishes, and get it from a stage instead of dumping, copying and etc.. but until now now working, still need to test also the node-alpine versions do not come with bash, as this script do not work properly with only sh I had to add an apk add bash |
I'm not quite sure, but I think you were saying that the workaround of pinning to an earlier version is working for you.
Cypress isn't involved with |
The full log:
I've encountered the issue in both OSX and with Ubuntu in CI. I tried your suggestion of pinning the dockerfile syntax and this seems to have worked.
|
That's great! Thanks for confirming. I did some more research and found that there have been several changes in this area lately on the Docker side.
It looks to me like these changes could have introduced a bug in how |
This issue is reproduced and described in https://github.com/MikeMcC399/cy-docker-multistage I plan to report this as a Docker regression. |
I confirm that setting the #syntax fixed to :1.10.0 on Dockerfiles fixed the issue and I can again build child images, thanks @johnhok |
This issue is reproducible in multi-stage Docker builds in a parent-child-grandchild sequence without using a
|
I recently upgraded my docker desktop from
4.35.0
->4.36.0
and a build that was previously working usingcypress/factory
seems to be producing an error now.This seems to happen in multi staged builds.
An abridged version of the dockerfile where it fails on the codgen stage:
The text was updated successfully, but these errors were encountered: