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

Remove nemu #2196

Merged
merged 1 commit into from
Nov 15, 2019
Merged

Remove nemu #2196

merged 1 commit into from
Nov 15, 2019

Conversation

egernst
Copy link
Member

@egernst egernst commented Nov 13, 2019

Deprecation was announced in 1.8, and we no longer support the NEMU VMM, as it is not actively maintained and the benefits of NEMU are now mostly realized by QEMU, Firecracker and Cloud Hypervisor VMMs.

This PR removes it from the runtime.

@egernst
Copy link
Member Author

egernst commented Nov 13, 2019

/test

@egernst
Copy link
Member Author

egernst commented Nov 13, 2019

/test

@codecov
Copy link

codecov bot commented Nov 13, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@35f54fd). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2196   +/-   ##
=========================================
  Coverage          ?   51.25%           
=========================================
  Files             ?      110           
  Lines             ?    15114           
  Branches          ?        0           
=========================================
  Hits              ?     7747           
  Misses            ?     6410           
  Partials          ?      957

@egernst
Copy link
Member Author

egernst commented Nov 14, 2019

/test ubuntu-18-04

Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Just for clarity, no children, kittens, 🐈s, 🐄s, 🦄s, 🐫s or 🪲s were hurt in the preperation of this PR folks!

lgtm

We do need to think about the user impact of this change though. It needs to be mentioned (prominently) in the next releases release notes as a minimum as it's a breaking change.

Copy link
Contributor

@chavafg chavafg left a comment

Choose a reason for hiding this comment

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

lgtm, although I see a conflict file error.

NEMU deprecation was announced in 1.8 of Kata. Removing from tree.

Thanks for all the fish!

Fixes: kata-containers#2195

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

egernst commented Nov 15, 2019

/test

@jodh-intel
Copy link
Contributor

Restarted Debian CI which failed with the following error:

07:45:31 time="2019-11-15T07:45:31.099785692Z" level=error msg="RemovePodSandbox for \"aae8b91b25ed05d536accc355c9cb5c05cfcd64ba003dfa23d13b3dd2a63cb62\" failed" error="failed to remove container \"17b964cfc0d1aa53ec8101987b2c46036144d853917ffdc3675e0748d55dc503\": failed to set removing state for container \"17b964cfc0d1aa53ec8101987b2c46036144d853917ffdc3675e0748d55dc503\": container is in starting state, can't be removed"
07:45:31 time="2019-11-15T07:45:31.115041189Z" level=debug msg="received signal" signal=terminated
07:45:31 time="2019-11-15T07:45:31.115155695Z" level=info msg="Stop CRI service"
07:45:31 time="2019-11-15T07:45:31.115257799Z" level=info msg="Stop CRI service"
07:45:31 -------------------------------------
07:45:31 make: *** [Makefile:132: shimv2] Error 2
07:45:31 Failed at 53: sudo -E PATH="$PATH" bash -c "make test"

@chavafg chavafg merged commit 3886dcf into kata-containers:master Nov 15, 2019
@egernst egernst deleted the remove-nemu branch June 26, 2020 05:34
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.

6 participants