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

version: update nemu with latest virtiofs patches#1639

Closed
chavafg wants to merge 1 commit intokata-containers:masterfrom
chavafg:topic/nemu-virtiofs-fix
Closed

version: update nemu with latest virtiofs patches#1639
chavafg wants to merge 1 commit intokata-containers:masterfrom
chavafg:topic/nemu-virtiofs-fix

Conversation

@chavafg
Copy link
Copy Markdown
Contributor

@chavafg chavafg commented May 8, 2019

This nemu release includes the latest virtiofsd patches
(changing the daemon message output).

Fixes: #1638.

Signed-off-by: Salvador Fuentes salvador.fuentes@intel.com

chavafg added a commit to chavafg/tests-1 that referenced this pull request May 8, 2019
Add configuration option to use virtio-fs.
We will currently use nemu for testing the
virtio-fs support.

Depends-on: github.com/kata-containers/runtime#1639

Fixes: kata-containers#1536.

Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
Copy link
Copy Markdown

@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.

Change looks fine but I think you'll need to update the uscan-url line above too to allow for the "pre-" prefix.

This nemu release includes the latest virtiofsd patches
(changing the daemon message output).

Fixes: kata-containers#1638.

Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
@chavafg chavafg force-pushed the topic/nemu-virtiofs-fix branch from a5cb3cd to b257135 Compare May 8, 2019 14:27
@chavafg
Copy link
Copy Markdown
Contributor Author

chavafg commented May 8, 2019

Thanks @jodh-intel, updated the uscan-url

Copy link
Copy Markdown

@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.

Thanks @chavafg.

lgtm

Copy link
Copy Markdown
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
hmm, a 'pre' release - normally I'd say that is not something we want in our version files... but, in this case I trust we know what we are doing...

@grahamwhaley
Copy link
Copy Markdown
Contributor

/test

@chavafg
Copy link
Copy Markdown
Contributor Author

chavafg commented May 9, 2019

closing in favor of #1645

@chavafg chavafg closed this May 9, 2019
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.

update nemu version with latest virtiofs patches

3 participants