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

Cleanup template properly #764

Merged
merged 2 commits into from
Sep 20, 2018
Merged

Conversation

bergwolf
Copy link
Member

As CI has found out by running the UTs, we are not cleaning up vm factory properly in error path. Also the UTs need to close factory properly so that we do not leave the template mountpoint behind.

So that we don't leave it behind.

Signed-off-by: Peng Tao <bergwolf@gmail.com>
Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm
thx!

@grahamwhaley
Copy link
Contributor

Related to kata-containers/tests#761

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 165675 KB
Proxy: 4207 KB
Shim: 8764 KB

Memory inside container:
Total Memory: 2043460 KB
Free Memory: 2006704 KB

So that we do not leave artifacts like template mountpoint
behind.

Fixes: kata-containers#763

Signed-off-by: Peng Tao <bergwolf@gmail.com>
@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 165696 KB
Proxy: 4107 KB
Shim: 8740 KB

Memory inside container:
Total Memory: 2043460 KB
Free Memory: 2006588 KB

@codecov
Copy link

codecov bot commented Sep 20, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@76b0c3c). Click here to learn what that means.
The diff coverage is 71.42%.

@@            Coverage Diff            @@
##             master     #764   +/-   ##
=========================================
  Coverage          ?   65.29%           
=========================================
  Files             ?       87           
  Lines             ?    10570           
  Branches          ?        0           
=========================================
  Hits              ?     6902           
  Misses            ?     2976           
  Partials          ?      692

@jodh-intel
Copy link
Contributor

jodh-intel commented Sep 20, 2018

lgtm

Approved with PullApprove

@grahamwhaley
Copy link
Contributor

Travis in snooze mode - I don't see a build over at https://travis-ci.org/kata-containers/runtime/builds I'll spend a few minutes to see if I can see a reason why maybe.

@bergwolf
Copy link
Member Author

It seems that travis decides I have sent to many build requests to them... I have contacted their support. Until they solve it, travis-ci will continue ignoring my PRs. sigh :(

The account bergwolf has been flagged for potential abuse. Please contact support@travis-ci.com.

@grahamwhaley
Copy link
Contributor

Wow @bergwolf - that is the strangest 'sign of success' yet - err, congratulations on the many PRs ;-)

@gnawux
Copy link
Member

gnawux commented Sep 20, 2018

congrats @bergwolf; lol

@jcvenegas
Copy link
Member

I think it is ok to merge this PR by now. @sboeuf @amshinde

@amshinde
Copy link
Member

Yup. Lets merge :)

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.

7 participants