Skip to content
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

cache add: transferring cached image: Process exited with status 137 from signal KILL #4097

Closed
perrywang opened this issue Apr 15, 2019 · 6 comments · Fixed by #5032
Closed
Assignees
Labels
cmd/cache Issues with the "cache" command help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. r/2019q2 Issue was last reviewed 2019q2
Milestone

Comments

@perrywang
Copy link

perrywang commented Apr 15, 2019

problem
I am adding offline ~2G image to minikube with cache add command but it always failed.
When I ssh login VM I found a ~400M image file in /tmp folder. I try a lot of times it seems it is always failed at ~400M data transfered.
Small size image has no such problems.
The strange is when I put this image file in files folder it is always transferred to VM successfully. And also I can run docker load -i xxx successfully in VM.
logs
! Failed to cache and load images: loading cached images: loading image {MINI_HOME}\cache\images\xxxx-web_2.89.27:
transferring cached image: Process exited with status 137 from signal KILL
environment:
windows 10 + virtualbox
minikube 1.0.0

@afbjorklund
Copy link
Collaborator

afbjorklund commented Apr 15, 2019

Seems to indicate an issue with transferring the image, like a scp timeout ?

@perrywang
Copy link
Author

Seems to indicate an issue with transferring the image, like a scp timeout ?

I am also doubt timeout.
The strange is put image in files folder it can be transferred to VM successfully. Confusing.

@afbjorklund
Copy link
Collaborator

If you scp the image yourself, does it transfer OK to the VM ? 2G is slighly bigger than what CI tests :-)

@perrywang
Copy link
Author

If you scp the image yourself, does it transfer OK to the VM ? 2G is slighly bigger than what CI tests :-)

I doubt it is relating with default target /tmp it is different partition only with ~2G size.
Could team consider expose this arg to user?

@afbjorklund
Copy link
Collaborator

Ah, right. What you are saying is that "minikube cache add" copies to /tmp which is on tmpfs (RAM)!
So that 400M limit is probably how much RAM you have available (free -m), after starting kubernetes.

We could definitely use something on the data disk instead, 20G instead of 2G... Already do, for DVP:

/dev/sda1 on /tmp/hostpath_pv type ext4 (rw,relatime,data=ordered)
/dev/sda1 on /tmp/hostpath-provisioner type ext4 (rw,relatime,data=ordered)

Since we delete the images after loading them into the container runtime, it normally doesn't matter.

@afbjorklund afbjorklund added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 19, 2019
@perrywang
Copy link
Author

Ah, right. What you are saying is that "minikube cache add" copies to /tmp which is on tmpfs (RAM)!
So that 400M limit is probably how much RAM you have available (free -m), after starting kubernetes.

We could definitely use something on the data disk instead, 20G instead of 2G... Already do, for DVP:

/dev/sda1 on /tmp/hostpath_pv type ext4 (rw,relatime,data=ordered)
/dev/sda1 on /tmp/hostpath-provisioner type ext4 (rw,relatime,data=ordered)

Since we delete the images after loading them into the container runtime, it normally doesn't matter.

Yes. @afbjorklund findmnt show such mounts. The problem is even I have more memory df -h still show /tmp only 2G available. It seems /tmp mounted with such options. Currently as workaround I use files directory to transfer to /data which was mounted from /dev/sda1/data.

@tstromberg tstromberg changed the title minikube cache add command failed cache add: transferring cached image: Process exited with status 137 from signal KILL Apr 22, 2019
@tstromberg tstromberg added kind/bug Categorizes issue or PR as related to a bug. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. and removed kind/feature Categorizes issue or PR as related to a new feature. labels Apr 22, 2019
@tstromberg tstromberg added priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. and removed priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. labels May 6, 2019
@tstromberg tstromberg added cmd/cache Issues with the "cache" command r/2019q2 Issue was last reviewed 2019q2 priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. and removed priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels May 23, 2019
@tstromberg tstromberg added this to the v1.4.0 Candidate milestone Aug 7, 2019
@tstromberg tstromberg self-assigned this Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd/cache Issues with the "cache" command help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. r/2019q2 Issue was last reviewed 2019q2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants