publish v0.5.31 and use new process group for child

Signed-off-by: Mayank Sachan <[email protected]>

Author: mssachan

.github/workflows/release.yml

@@ -3,7 +3,7 @@ name: Release
 on:
   push:
     branches:
-      - main
+      - detach-process
 
 jobs:
   release:
@@ -16,8 +16,8 @@ jobs:
           - cos-csi-mounter
 
     env:
-      IS_LATEST_RELEASE: 'true'
-      APP_VERSION: 1.0.4
+      IS_LATEST_RELEASE: 'false'
+      APP_VERSION: 0.5.31
 
     steps:
     - name: Checkout Code
@@ -63,15 +63,11 @@ jobs:
           /home/runner/work/ibm-object-csi-driver/ibm-object-csi-driver/cos-csi-mounter/cos-csi-mounter-${{ env.APP_VERSION }}.deb.tar.gz.sha256
           /home/runner/work/ibm-object-csi-driver/ibm-object-csi-driver/cos-csi-mounter/cos-csi-mounter-${{ env.APP_VERSION }}.rpm.tar.gz
           /home/runner/work/ibm-object-csi-driver/ibm-object-csi-driver/cos-csi-mounter/cos-csi-mounter-${{ env.APP_VERSION }}.rpm.tar.gz.sha256
-        tag_name: v1.0.4
-        name: v1.0.4
+        tag_name: v0.5.31
+        name: v0.5.31
         body: |
           ## 🚀 What’s New
-          - Enable --vfs-cache-mode flag for rclone mount to support random write
-          - Update golang dependencies
-          - Fetch IAM Endpoint Dynamically
-          - Add TLS cipher suites in the mount options
-          - Fix issue with s3fs mount randomly taking time using IAM credentials on Private VPC clusters
+          - Detach child process goup to fix rclone mount hang issue
         prerelease: ${{ env.IS_LATEST_RELEASE != 'true' }}
 
     - name: Perform CodeQL Analysis

cos-csi-mounter/Makefile

@@ -1,5 +1,5 @@
 NAME := cos-csi-mounter
-APP_VERSION := 1.0.4
+APP_VERSION := 0.5.31
 BUILD_DIR := $(NAME)-$(APP_VERSION)
 BIN_DIR := bin
 

pkg/mounter/utils/mounter_utils.go

@@ -34,7 +34,16 @@ type MounterOptsUtils struct {
 func (su *MounterOptsUtils) FuseMount(path string, comm string, args []string) error {
 	klog.Info("-FuseMount-")
 	klog.Infof("FuseMount params:\n\tpath: <%s>\n\tcommand: <%s>\n\targs: <%v>", path, comm, args)
-	out, err := command(comm, args...).CombinedOutput()
+
+	cmd := exec.Command(comm, args...)
+
+	// Set the system process attribute
+	// This is often used to prevent a child process from being killed
+	// when its parent exits.
+	cmd.SysProcAttr = &syscall.SysProcAttr{
+		Setpgid: true, // Set a new process group ID for the child
+	}
+	out, err := cmd.CombinedOutput()
 	if err != nil {
 		klog.Warningf("FuseMount: mount command failed: mounter=%s, args=%v, error=%v, output=%s", comm, args, err, string(out))
 		klog.Infof("FuseMount: checking if path already exists and is a mountpoint: path=%s", path)