Azure-Samples / azure-batch-samples

Azure Batch and HPC Code Samples
Other
260 stars 488 forks source link

Unnecessary check for management cert when using storage account key #125

Closed ffaghri1 closed 8 years ago

ffaghri1 commented 8 years ago

This check: if args.subscriptionid is not None and args.managementcert is None: raise ValueError( 'cannot specify subscription id without a management cert')

is unnecessary for the following usage: python $blobxfer_location/blobxfer.py --timeout 1800 --numworkers $NUM_WORKERS --subscriptionid $subscriptionid --storageaccountkey $storage_account --download --remoteresource

alfpark commented 8 years ago

A subscriptionid is not needed with storage account key. A subscriptionid should only be specified with a management cert for legacy ASM operation.