fix(glance): donot umount s3fs on glance exit (#22037)

yunionio · Jan 26, 2025 · dd75e4d · dd75e4d
1 parent 10dd582
commit dd75e4d
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 7 deletions.
diff --git a/pkg/image/service/service.go b/pkg/image/service/service.go
@@ -176,9 +176,9 @@ func StartService() {
 		if options.Options.EnableTorrentService {
 			torrent.StopTorrents()
 		}
-		if options.Options.StorageDriver == api.IMAGE_STORAGE_DRIVER_S3 {
-			procutils.NewCommand("umount", options.Options.S3MountPoint).Run()
-		}
+		//if options.Options.StorageDriver == api.IMAGE_STORAGE_DRIVER_S3 {
+		//	procutils.NewCommand("umount", options.Options.S3MountPoint).Run()
+		//}
 	})
 }
 

diff --git a/pkg/util/procutils/cmdsetenv_others.go b/pkg/util/procutils/cmdsetenv_others.go
@@ -18,8 +18,10 @@
 package procutils
 
 import (
+	"fmt"
 	"os"
 	"os/exec"
+	"strings"
 
 	"yunion.io/x/executor/client"
 )
@@ -37,9 +39,19 @@ func cmdSetEnv(cmd *exec.Cmd) {
 	)
 }
 
+var paths = []string{
+	"/usr/bin",
+	"/usr/sbin",
+	"/usr/local/sbin",
+	"/usr/local/bin",
+	"/sbin",
+	"/bin",
+	"/opt/yunion/bin",
+}
+
 func remoteCmdSetEnv(cmd *client.Cmd) {
-	cmd.Env = append(
-		os.Environ(),
-		presetEnv...,
-	)
+	envs := []string{
+		fmt.Sprintf("PATH=%s", strings.Join(paths, ":")),
+	}
+	cmd.Env = append(envs, presetEnv...)
 }