From 45229d6a40a403d4f290bba03be99578697a4060 Mon Sep 17 00:00:00 2001 From: Madhu Rajanna Date: Mon, 30 Mar 2020 13:55:37 +0530 Subject: [PATCH] Log error message for cephfs If the loading of cephfs fuse or the mount.ceph command fails log it. Signed-off-by: Madhu Rajanna (cherry picked from commit 3c3a624d1a9bed8a4a4cba2bc5fd062cc3c58a18) --- pkg/cephfs/volumemounter.go | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/pkg/cephfs/volumemounter.go b/pkg/cephfs/volumemounter.go index 3f86375fa..681ac661a 100644 --- a/pkg/cephfs/volumemounter.go +++ b/pkg/cephfs/volumemounter.go @@ -147,10 +147,12 @@ func loadAvailableMounters(conf *util.Config) error { kernelMounterProbe := exec.Command("mount.ceph") err := kernelMounterProbe.Run() - if err == nil { + if err != nil { + klog.Errorf("failed to run mount.ceph %v", err) + } else { // fetch the current running kernel info utsname := unix.Utsname{} - err := unix.Uname(&utsname) + err = unix.Uname(&utsname) if err != nil { return err } @@ -164,8 +166,11 @@ func loadAvailableMounters(conf *util.Config) error { } } - if fuseMounterProbe.Run() == nil { - klog.Infof("loaded mounter: %s", volumeMounterFuse) + err = fuseMounterProbe.Run() + if err != nil { + klog.Errorf("failed to run ceph-fuse %v", err) + } else { + klog.V(1).Infof("loaded mounter: %s", volumeMounterFuse) availableMounters = append(availableMounters, volumeMounterFuse) }