From af54349a5ef245bdf54bb85dd818be12a4faf426 Mon Sep 17 00:00:00 2001 From: Travis Nielsen Date: Fri, 8 May 2020 11:19:42 -0600 Subject: [PATCH] ceph: remove invalid verbose params from lv activate The activation of the lvs for OSDs had an invalid argument to vgchange. The invalid argument is now reverted to enable the OSDs in this scenario again. Signed-off-by: Travis Nielsen --- pkg/daemon/ceph/osd/daemon.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/daemon/ceph/osd/daemon.go b/pkg/daemon/ceph/osd/daemon.go index 04b2f5d2a..c1fe0fc01 100644 --- a/pkg/daemon/ceph/osd/daemon.go +++ b/pkg/daemon/ceph/osd/daemon.go @@ -68,12 +68,12 @@ func StartOSD(context *clusterd.Context, osdType, osdID, osdUUID, lvPath string, go handleTerminate(context, lvPath, volumeGroupName) // It's fine to continue if deactivate fails since we will return error if activate fails - if op, err := context.Executor.ExecuteCommandWithCombinedOutput("vgchange", "-anvv", volumeGroupName); err != nil { + if op, err := context.Executor.ExecuteCommandWithCombinedOutput("vgchange", "-an", volumeGroupName); err != nil { logger.Errorf("failed to deactivate volume group for lv %q. output: %s. %v", lvPath, op, err) return nil } - if op, err := context.Executor.ExecuteCommandWithCombinedOutput("vgchange", "-ayvv", volumeGroupName); err != nil { + if op, err := context.Executor.ExecuteCommandWithCombinedOutput("vgchange", "-ay", volumeGroupName); err != nil { return errors.Wrapf(err, "failed to activate volume group for lv %q. output: %s", lvPath, op) } }