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

Add path to log messages. 

It is useful to see the path that failed to mount in the log messages.
pull/6/head
Jan Safranek 2016-04-08 09:41:09 +02:00
parent df055333ef
commit be2ca8dec3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/util/mount/nsenter_mount.go
View File

@ -118,7 +118,7 @@ func (n *NsenterMounter) doNsenterMount(source, target, fstype string, options [
exec := exec.New() exec := exec.New()
outputBytes, err := exec.Command(nsenterPath, args...).CombinedOutput() outputBytes, err := exec.Command(nsenterPath, args...).CombinedOutput()
if len(outputBytes) != 0 { if len(outputBytes) != 0 {
glog.V(5).Infof("Output from mount command: %v", string(outputBytes)) glog.V(5).Infof("Output of mounting %s to %s: %v", source, target, string(outputBytes))
} }
return err return err
@ -151,7 +151,7 @@ func (n *NsenterMounter) Unmount(target string) error {
exec := exec.New() exec := exec.New()
outputBytes, err := exec.Command(nsenterPath, args...).CombinedOutput() outputBytes, err := exec.Command(nsenterPath, args...).CombinedOutput()
if len(outputBytes) != 0 { if len(outputBytes) != 0 {
glog.V(5).Infof("Output from mount command: %v", string(outputBytes)) glog.V(5).Infof("Output of unmounting %s: %v", target, string(outputBytes))
} }
return err return err
@ -176,7 +176,7 @@ func (n *NsenterMounter) IsLikelyNotMountPoint(file string) (bool, error) {
exec := exec.New() exec := exec.New()
out, err := exec.Command(nsenterPath, args...).CombinedOutput() out, err := exec.Command(nsenterPath, args...).CombinedOutput()
if err != nil { if err != nil {
glog.V(2).Infof("Failed findmnt command: %v", err) glog.V(2).Infof("Failed findmnt command for path %s: %v", file, err)
// Different operating systems behave differently for paths which are not mount points. // Different operating systems behave differently for paths which are not mount points.
// On older versions (e.g. 2.20.1) we'd get error, on newer ones (e.g. 2.26.2) we'd get "/". // On older versions (e.g. 2.20.1) we'd get error, on newer ones (e.g. 2.26.2) we'd get "/".
// It's safer to assume that it's not a mount point. // It's safer to assume that it's not a mount point.
@ -184,7 +184,7 @@ func (n *NsenterMounter) IsLikelyNotMountPoint(file string) (bool, error) {
} }
strOut := strings.TrimSuffix(string(out), "\n") strOut := strings.TrimSuffix(string(out), "\n")
glog.V(5).Infof("IsLikelyNotMountPoint findmnt output: %v", strOut) glog.V(5).Infof("IsLikelyNotMountPoint findmnt output for path %s: %v", file, strOut)
if strOut == file { if strOut == file {
return false, nil return false, nil
} }