Remove --output-level quiet from azcopy calls

Signed-off-by: adamrtalbot <12817534+adamrtalbot@users.noreply.github.com>

Author: adamrtalbot

plugins/nf-azure/src/main/nextflow/cloud/azure/file/AzBashLib.groovy

@@ -70,29 +70,29 @@ class AzBashLib extends BashFunLib<AzBashLib> {
                 azcopy cp "$name" "$target/$dir_name?$AZ_SAS" --recursive --block-blob-tier $AZCOPY_BLOCK_BLOB_TIER --block-size-mb $AZCOPY_BLOCK_SIZE_MB
               fi
             else
-              azcopy cp --output-level quiet "$name" "$target/$name?$AZ_SAS" --block-blob-tier $AZCOPY_BLOCK_BLOB_TIER --block-size-mb $AZCOPY_BLOCK_SIZE_MB
+              azcopy cp "$name" "$target/$name?$AZ_SAS" --block-blob-tier $AZCOPY_BLOCK_BLOB_TIER --block-size-mb $AZCOPY_BLOCK_SIZE_MB
             fi
         }
-        
+
         nxf_az_download() {
             local source=$1
             local target=$2
             local basedir=$(dirname $2)
             mkdir -p "$basedir"
-        
+
             # Try to download as file first, let azcopy handle the detection
-            if ! azcopy cp --output-level quiet "$source?$AZ_SAS" "$target"; then
+            if ! azcopy cp "$source?$AZ_SAS" "$target"; then
                 # If failed, remove any partial target and try as directory
                 rm -rf "$target"
                 mkdir -p "$target"
-                if ! azcopy cp --output-level quiet"$source/*?$AZ_SAS" "$target" --recursive; then
+                if ! azcopy cp "$source/*?$AZ_SAS" "$target" --recursive; then
                     rm -rf "$target"
                     >&2 echo "Unable to download path: $source"
                     exit 1
                 fi
             fi
         }
-        '''.stripIndent(true)
+        '''.stripIndent()
     }
 
     String render() {

plugins/nf-azure/src/test/nextflow/cloud/azure/batch/AzFileCopyStrategyTest.groovy

@@ -179,18 +179,19 @@ class AzFileCopyStrategyTest extends Specification {
                     local source=$1
                     local target=$2
                     local basedir=$(dirname $2)
-                    local ret
                     mkdir -p "$basedir"
                 
-                    ret=$(azcopy cp "$source?$AZ_SAS" "$target" 2>&1) || {
-                        ## if fails check if it was trying to download a directory
-                        mkdir -p $target
-                        azcopy cp "$source/*?$AZ_SAS" "$target" --recursive >/dev/null || {
-                            rm -rf $target
+                    # Try to download as file first, let azcopy handle the detection
+                    if ! azcopy cp "$source?$AZ_SAS" "$target"; then
+                        # If failed, remove any partial target and try as directory
+                        rm -rf "$target"
+                        mkdir -p "$target"
+                        if ! azcopy cp "$source/*?$AZ_SAS" "$target" --recursive; then
+                            rm -rf "$target"
                             >&2 echo "Unable to download path: $source"
                             exit 1
-                        }
-                    }
+                        fi
+                    fi
                 }
                 
                 '''.stripIndent(true)