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

virtcontainers: Add missing API release calls#830

Merged
bergwolf merged 1 commit into
kata-containers:masterfrom
caoruidong:add-release
Oct 18, 2018
Merged

virtcontainers: Add missing API release calls#830
bergwolf merged 1 commit into
kata-containers:masterfrom
caoruidong:add-release

Conversation

@caoruidong
Copy link
Copy Markdown
Member

Add missing release sandbox calls to network related functions in
virtcontainers API.

Fixes #732.

Signed-off-by: Ruidong Cao caoruidong@huawei.com

@caoruidong
Copy link
Copy Markdown
Member Author

/test

Copy link
Copy Markdown

@devimc devimc left a comment

Choose a reason for hiding this comment

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

lgtm

@jodh-intel
Copy link
Copy Markdown

OOI, how did you notice this issue? I guess we can't create any tests for this?

@caoruidong
Copy link
Copy Markdown
Member Author

@jodh-intel Recently I'm making some changes to network. What happens without release?Will they be released after runtime exit?

@jodh-intel
Copy link
Copy Markdown

Yes - Release() removes the sandbox from an internal hash, shuts down the QMP connection to the qemu and closes the gRPC connection to the agent.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 17, 2018

Codecov Report

Merging #830 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #830      +/-   ##
=========================================
+ Coverage   66.09%   66.1%   +0.01%     
=========================================
  Files          87      87              
  Lines       10505   10509       +4     
=========================================
+ Hits         6943    6947       +4     
  Misses       2863    2863              
  Partials      699     699

Add missing release sandbox calls to network related functions in
virtcontainers API.

Fixes kata-containers#732.

Signed-off-by: Ruidong Cao <caoruidong@huawei.com>
@jshachm
Copy link
Copy Markdown
Member

jshachm commented Oct 18, 2018

LGTM. nice catch~

Approved with PullApprove

@bergwolf
Copy link
Copy Markdown
Member

bergwolf commented Oct 18, 2018

LGTM!

Approved with PullApprove

@bergwolf
Copy link
Copy Markdown
Member

boot time and memory footprint metrics failures are known issues. Let's ignore it here.

@bergwolf bergwolf merged commit 708007e into kata-containers:master Oct 18, 2018
@caoruidong caoruidong deleted the add-release branch October 24, 2018 06:03
egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
network: Fix Could not create destination mount point: /etc/resolv.conf
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.

5 participants