fix: print debug logs when lima disk command fails

Signed-off-by: Hsing-Yu (David) Chen <[email protected]>
runfinch · Mar 7, 2023 · 82cc0ce · 82cc0ce
1 parent 674b379
commit 82cc0ce
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/pkg/disk/disk.go b/pkg/disk/disk.go
@@ -116,7 +116,10 @@ func (m *userDataDiskManager) limaDiskExists() bool {
 
 func (m *userDataDiskManager) createLimaDisk() error {
 	cmd := m.lcc.CreateWithoutStdio("disk", "create", diskName, "--size", diskSize)
-	return cmd.Run()
+	if logs, err := cmd.CombinedOutput(); err != nil {
+		return fmt.Errorf("failed to create disk, debug logs: %s", logs)
+	}
+	return nil
 }
 
 func (m *userDataDiskManager) attachPersistentDiskToLimaDisk() error {
@@ -165,5 +168,8 @@ func (m *userDataDiskManager) limaDiskIsLocked() bool {
 
 func (m *userDataDiskManager) unlockLimaDisk() error {
 	cmd := m.lcc.CreateWithoutStdio("disk", "unlock", diskName)
-	return cmd.Run()
+	if logs, err := cmd.CombinedOutput(); err != nil {
+		return fmt.Errorf("failed to unlock disk, debug logs: %s", logs)
+	}
+	return nil
 }
diff --git a/pkg/disk/disk_test.go b/pkg/disk/disk_test.go
@@ -55,7 +55,7 @@ func TestUserDataDiskManager_InitializeUserDataDisk(t *testing.T) {
 				cmd.EXPECT().Output().Return([]byte(""), nil)
 
 				lcc.EXPECT().CreateWithoutStdio(mockCreateArgs).Return(cmd)
-				cmd.EXPECT().Run().Return(nil)
+				cmd.EXPECT().CombinedOutput().Return(nil, nil)
 
 				dfs.EXPECT().Stat(finch.UserDataDiskPath(homeDir)).Return(nil, fs.ErrNotExist)
 				dfs.EXPECT().Stat(path.Dir(finch.UserDataDiskPath(homeDir))).Return(nil, nil)
@@ -107,7 +107,7 @@ func TestUserDataDiskManager_InitializeUserDataDisk(t *testing.T) {
 				cmd.EXPECT().Output().Return([]byte(""), nil)
 
 				lcc.EXPECT().CreateWithoutStdio(mockCreateArgs).Return(cmd)
-				cmd.EXPECT().Run().Return(nil)
+				cmd.EXPECT().CombinedOutput().Return(nil, nil)
 
 				dfs.EXPECT().Stat(finch.UserDataDiskPath(homeDir)).Return(nil, nil)
 
@@ -130,7 +130,7 @@ func TestUserDataDiskManager_InitializeUserDataDisk(t *testing.T) {
 
 				dfs.EXPECT().Stat(lockPath).Return(nil, nil)
 				lcc.EXPECT().CreateWithoutStdio(mockUnlockArgs).Return(cmd)
-				cmd.EXPECT().Run().Return(nil)
+				cmd.EXPECT().CombinedOutput().Return(nil, nil)
 			},
 		},
 	}