Refactor integration test (step 3) - Azure blob mount robustness (#4894)

ts/nni_manager/training_service/reusable/shared_storages/azureblobStorageService.ts

@@ -155,16 +155,17 @@ export class AzureBlobSharedStorageService extends SharedStorageService {
             return Promise.reject(errorMessage);
         }
 
-        try {
-            this.log.debug(`Local mount command is: ${this.localMountCommand}`);
-            const result = await cpp.exec(this.localMountCommand);
-            if (result.stderr) {
-                throw new Error(result.stderr);
-            }
-        } catch (error) {
-            const errorMessage: string = `${this.storageType} Shared Storage: Mount ${this.storageAccountName}/${this.containerName} to ${this.localMountPoint} failed, error is ${error}`;
-            this.log.error(errorMessage);
-            return Promise.reject(errorMessage);
+        // FIXME: This has security risks. The command might contain secrets.
+        this.log.debug(`Local mount command is: ${this.localMountCommand}`);
+        const result = await cpp.exec(this.localMountCommand);
+        if (result.stderr) {
+            // FIXME: I don't know how to check whether it failed.
+            // But looking at stderr definitely isn't a good idea.
+            this.log.warning(
+                `${this.storageType} Shared Storage: Mount ${this.storageAccountName}/${this.containerName} to ${this.localMountPoint}. Stderr is not empty.`
+            );
+            this.log.warning(`Stdout: ${result.stdout}`);
+            this.log.warning(`Stderr: ${result.stderr}`);
         }
 
         return Promise.resolve();