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

config: honor DEFSHAREDFS_QEMU_VIRTIOFS and CONFIG_QEMU_VIRTIOFS_IN #2080

Merged
merged 3 commits into from
Sep 23, 2019

Conversation

chavafg
Copy link
Contributor

@chavafg chavafg commented Sep 20, 2019

replace DEFSHAREDFS_QEMU_VIRTIOFS and CONFIG_QEMU_VIRTIOFS_IN
with their values on the generated configuration-qemu-virtiofs.toml

In addition, add configuration-qemu-virtio-fs.toml to .gitignore
and fix the virtiofsd name, as the binary built using our packaging scripts
is named virtiofsd and not virtiofsd-x86_64

Fixes: #2079

replace `DEFSHAREDFS_QEMU_VIRTIOFS` and `CONFIG_QEMU_VIRTIOFS_IN`
with their values on the generated `configuration-qemu-virtiofs.toml`

Fixes: kata-containers#2079

Signed-off-by: Salvador Fuentes <[email protected]>
add generated configuration file to gitignore.

Signed-off-by: Salvador Fuentes <[email protected]>
virtiofsd binary built using our packaging scripts
is named `virtiofsd` and not `virtiofsd-x86_64`

Signed-off-by: Salvador Fuentes <[email protected]>
@GabyCT
Copy link
Contributor

GabyCT commented Sep 20, 2019

/test

@codecov
Copy link

codecov bot commented Sep 20, 2019

Codecov Report

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

@@           Coverage Diff            @@
##             master   #2080   +/-   ##
========================================
  Coverage          ?   51.6%           
========================================
  Files             ?     108           
  Lines             ?   14623           
  Branches          ?       0           
========================================
  Hits              ?    7546           
  Misses            ?    6173           
  Partials          ?     904

@GabyCT GabyCT merged commit d3f480d into kata-containers:master Sep 23, 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.

config: some variables in configuration-qemu-virtiofs.toml are not replaced with actual values
3 participants