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

fc: config: vhost-net not supported #2194

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

egernst
Copy link
Member

@egernst egernst commented Nov 13, 2019

vhost-net backend is not supported by the Firecracker VMM. It doesn't
make sense to have this in the configuration, and we should explicitly
disable it.

Fixes: #2192

Signed-off-by: Eric Ernst [email protected]

vhost-net backend is not supported by the Firecracker VMM. It doesn't
make sense to have this in the configuration, and we should explicitly
disable it.

Fixes: kata-containers#2192

Signed-off-by: Eric Ernst <[email protected]>
@egernst
Copy link
Member Author

egernst commented Nov 13, 2019

/test

@egernst egernst requested a review from amshinde November 13, 2019 22:17
@codecov
Copy link

codecov bot commented Nov 13, 2019

Codecov Report

Merging #2194 into master will increase coverage by 0.66%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2194      +/-   ##
==========================================
+ Coverage   50.98%   51.65%   +0.66%     
==========================================
  Files         110      110              
  Lines       15224    15574     +350     
==========================================
+ Hits         7762     8044     +282     
- Misses       6506     6531      +25     
- Partials      956      999      +43

@GabyCT GabyCT merged commit 35f54fd into kata-containers:master Nov 14, 2019
@egernst egernst deleted the fixup-vhost-fc branch June 26, 2020 05:33
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.

remove vhost-net configuration from FC configs
4 participants