github
/
k3s
mirror of https://github.com/k3s-io/k3s
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

nfs: check if /sbin/mount.nfs* is present in CanSupport() 

Signed-off-by: Huamin Chen <hchen@redhat.com>
pull/6/head
Huamin Chen 2015-10-21 11:43:46 -04:00
parent 0888c2ef38
commit 1b3a88dc87
2 changed files with 34 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
pkg/volume/nfs/nfs.go
View File

@ -19,10 +19,12 @@ package nfs
import (
"fmt"
"os"
"runtime"
"k8s.io/kubernetes/pkg/api"
"k8s.io/kubernetes/pkg/types"
"k8s.io/kubernetes/pkg/util"
"k8s.io/kubernetes/pkg/util/exec"
"k8s.io/kubernetes/pkg/util/mount"
"k8s.io/kubernetes/pkg/volume"
@ -66,9 +68,29 @@ func (plugin *nfsPlugin) Name() string {
return nfsPluginName
}
func hasNFSMount() bool {
exe := exec.New()
switch runtime.GOOS {
case "linux":
cmd1 := exe.Command("/bin/ls", "/sbin/mount.nfs")
_, err1 := cmd1.CombinedOutput()
cmd2 := exe.Command("/bin/ls", "/sbin/mount.nfs4")
_, err2 := cmd2.CombinedOutput()
return (err1 == nil || err2 == nil)
case "darwin":
cmd := exe.Command("/bin/ls", "/sbin/mount_nfs")
_, err := cmd.CombinedOutput()
return err == nil
}
return false
}
func (plugin *nfsPlugin) CanSupport(spec *volume.Spec) bool {
return (spec.PersistentVolume != nil && spec.PersistentVolume.Spec.NFS != nil) ||
(spec.Volume != nil && spec.Volume.NFS != nil)
if (spec.Volume != nil && spec.Volume.NFS == nil) || (spec.PersistentVolume != nil && spec.PersistentVolume.Spec.NFS == nil) {
return false
}
// see if /sbin/mount.nfs* is there
return hasNFSMount()
}
func (plugin *nfsPlugin) GetAccessModes() []api.PersistentVolumeAccessMode {

12
pkg/volume/nfs/nfs_test.go
View File

@ -38,12 +38,16 @@ func TestCanSupport(t *testing.T) {
if plug.Name() != "kubernetes.io/nfs" {
t.Errorf("Wrong name: %s", plug.Name())
}
if !plug.CanSupport(&volume.Spec{Volume: &api.Volume{VolumeSource: api.VolumeSource{NFS: &api.NFSVolumeSource{}}}}) {
foundMount := hasNFSMount()
if plug.CanSupport(&volume.Spec{Volume: &api.Volume{VolumeSource: api.VolumeSource{NFS: &api.NFSVolumeSource{}}}}) != foundMount {
t.Errorf("Expected true")
}
if !plug.CanSupport(&volume.Spec{PersistentVolume: &api.PersistentVolume{Spec: api.PersistentVolumeSpec{PersistentVolumeSource: api.PersistentVolumeSource{NFS: &api.NFSVolumeSource{}}}}}) {
if plug.CanSupport(&volume.Spec{PersistentVolume: &api.PersistentVolume{Spec: api.PersistentVolumeSpec{PersistentVolumeSource: api.PersistentVolumeSource{NFS: &api.NFSVolumeSource{}}}}}) != foundMount {
t.Errorf("Expected true")
}
if plug.CanSupport(&volume.Spec{PersistentVolume: &api.PersistentVolume{Spec: api.PersistentVolumeSpec{PersistentVolumeSource: api.PersistentVolumeSource{}}}}) {
t.Errorf("Expected false")
}
if plug.CanSupport(&volume.Spec{Volume: &api.Volume{VolumeSource: api.VolumeSource{}}}) {
t.Errorf("Expected false")
}
@ -63,6 +67,10 @@ func TestGetAccessModes(t *testing.T) {
}
func TestRecycler(t *testing.T) {
if foundMount := hasNFSMount(); !foundMount {
// FindRecyclablePluginBySpec will test CanSupport() but mount helper is absent
return
}
plugMgr := volume.VolumePluginMgr{}
plugMgr.InitPlugins([]volume.VolumePlugin{&nfsPlugin{nil, newMockRecycler, volume.VolumeConfig{}}}, volume.NewFakeVolumeHost("/tmp/fake", nil, nil))