github/kata-containers
1
0
Fork 1
mirror of https://github.com/kata-containers/kata-containers.git synced 2025-04-28 19:54:35 +00:00
Code Issues Projects Releases Wiki Activity

runtime/virtiofsd: Drop all references to "--cache=none"

Browse Source 
This syntax belongs to the legacy C virtiofsd implementation that
we don't support anymore since kata-containers 3.1.3 because
of other API breaking changes.

People have been warned to switch from "none" to "never" since
kata-containers 2.5.2. Let's officially do that.

The compat code that would convert "none" to "never" isn't
needed anymore. Just drop it.

Fixes #7864

Signed-off-by: Greg Kurz <groug@kaod.org>
This commit is contained in:
Greg Kurz 2023-09-06 17:56:01 +02:00
parent 81536f21af
commit 72c510d057
2 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/runtime/virtcontainers/virtiofsd.go
View File

@ -39,7 +39,6 @@ var (
const ( const (
typeVirtioFSCacheModeNever = "never" typeVirtioFSCacheModeNever = "never"
typeVirtioFSCacheModeNone = "none"
typeVirtioFSCacheModeAlways = "always" typeVirtioFSCacheModeAlways = "always"
typeVirtioFSCacheModeAuto = "auto" typeVirtioFSCacheModeAuto = "auto"
) )
@ -219,9 +218,6 @@ func (v *virtiofsd) valid() error {
if v.cache == "" { if v.cache == "" {
v.cache = typeVirtioFSCacheModeAuto v.cache = typeVirtioFSCacheModeAuto
} else if v.cache == typeVirtioFSCacheModeNone {
v.Logger().Warn("virtio-fs cache mode `none` is deprecated since Kata Containers 2.5.0 and will be removed in the future release, please use `never` instead. For more details please refer to https://github.com/kata-containers/kata-containers/issues/4234.")
v.cache = typeVirtioFSCacheModeNever
} else if v.cache != typeVirtioFSCacheModeAuto && v.cache != typeVirtioFSCacheModeAlways && v.cache != typeVirtioFSCacheModeNever { } else if v.cache != typeVirtioFSCacheModeAuto && v.cache != typeVirtioFSCacheModeAlways && v.cache != typeVirtioFSCacheModeNever {
return errVirtiofsdInvalidVirtiofsCacheMode(v.cache) return errVirtiofsdInvalidVirtiofsCacheMode(v.cache)
} }

13
src/runtime/virtcontainers/virtiofsd_test.go
View File

@ -76,15 +76,15 @@ func TestVirtiofsdArgs(t *testing.T) {
v := &virtiofsd{ v := &virtiofsd{
path: "/usr/bin/virtiofsd", path: "/usr/bin/virtiofsd",
sourcePath: "/run/kata-shared/foo", sourcePath: "/run/kata-shared/foo",
cache: "none", cache: "never",
} }
expected := "--syslog --cache=none --shared-dir=/run/kata-shared/foo --fd=123" expected := "--syslog --cache=never --shared-dir=/run/kata-shared/foo --fd=123"
args, err := v.args(123) args, err := v.args(123)
assert.NoError(err) assert.NoError(err)
assert.Equal(expected, strings.Join(args, " ")) assert.Equal(expected, strings.Join(args, " "))
expected = "--syslog --cache=none --shared-dir=/run/kata-shared/foo --fd=456" expected = "--syslog --cache=never --shared-dir=/run/kata-shared/foo --fd=456"
args, err = v.args(456) args, err = v.args(456)
assert.NoError(err) assert.NoError(err)
assert.Equal(expected, strings.Join(args, " ")) assert.Equal(expected, strings.Join(args, " "))
@ -130,12 +130,7 @@ func TestValid(t *testing.T) {
{"source is not available", func(v *virtiofsd) { {"source is not available", func(v *virtiofsd) {
v.sourcePath = "/foo/bar" v.sourcePath = "/foo/bar"
}, errVirtiofsdSourceNotAvailable, nil}, }, errVirtiofsdSourceNotAvailable, nil},
{"replace cache mode none by never", func(v *virtiofsd) { {"invalid cache mode", func(v *virtiofsd) {
v.cache = "none"
}, nil, func(name string, a *assert.Assertions, v *virtiofsd) {
a.Equal("never", v.cache, "test case %+s, cache mode none should be replaced by never", name)
}},
{"invald cache mode: replace none by never", func(v *virtiofsd) {
v.cache = "foo" v.cache = "foo"
}, errVirtiofsdInvalidVirtiofsCacheMode("foo"), nil}, }, errVirtiofsdInvalidVirtiofsCacheMode("foo"), nil},
} }