Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

virtcontainers: revert "fix shared dir resource remaining" #778

Merged
merged 1 commit into from
Sep 25, 2018

Conversation

devimc
Copy link

@devimc devimc commented Sep 24, 2018

This reverts commit 8a6d383.

Don't remove all directories in the shared directory because
docker cp re-mounts all the mount points specified in the
config.json causing serious problems in the host.

fixes #777

Signed-off-by: Julio Montes [email protected]

This reverts commit 8a6d383.

Don't remove all directories in the shared directory because
`docker cp` re-mounts all the mount points specified in the
config.json causing serious problems in the host.

fixes kata-containers#777

Signed-off-by: Julio Montes <[email protected]>
@devimc
Copy link
Author

devimc commented Sep 24, 2018

/test

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 167715 KB
Proxy: 4005 KB
Shim: 8760 KB

Memory inside container:
Total Memory: 2043448 KB
Free Memory: 2006572 KB

@codecov
Copy link

codecov bot commented Sep 24, 2018

Codecov Report

Merging #778 into master will increase coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #778      +/-   ##
==========================================
+ Coverage   65.31%   65.32%   +0.01%     
==========================================
  Files          87       87              
  Lines       10622    10611      -11     
==========================================
- Hits         6938     6932       -6     
+ Misses       2986     2982       -4     
+ Partials      698      697       -1

@devimc
Copy link
Author

devimc commented Sep 24, 2018

re-starting jenkins-ci-fedora-27

@jodh-intel
Copy link
Contributor

jodh-intel commented Sep 25, 2018

lgtm

Approved with PullApprove

@jodh-intel jodh-intel merged commit e509150 into kata-containers:master Sep 25, 2018
@jshachm
Copy link
Member

jshachm commented Sep 25, 2018

Need find another way to do the clean up. Will do it later~ @amshinde

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

issues with docker cp
5 participants