Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added code to use the azcopy responsefile option to pass parameters #4704

Merged
merged 3 commits into from
Jul 3, 2017

Conversation

cltshivash
Copy link
Contributor

Added code to use the azcopy responsefile option to pass parameters

$azlog = ("{0}\..\azlog" -f $src)
$args = ("/Source:`"{0}`" /Dest:{1} /DestSAS:{2} /S /Z:`"{3}`"" -f $src, $dest, $sas, $azlog)

# Create a file with the arguments as the content
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Create [](start = 6, length = 6)

task.json

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated


$responseFileArgs = [String]::Format("/@:""{0}""", $responseFile)
try {
Write-Verbose "Calling AzCopy tool with file $responseFile" -Verbose
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to put the file name in verbose log.. People can easily look into the file and get the sas key..

Copy link
Member

@RanjanarMS RanjanarMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to put the file name in verbose log..? People can easily look into the file and get the sas key

@cltshivash cltshivash merged commit 9569b94 into master Jul 3, 2017
@cltshivash cltshivash deleted the users/shivash/azcopyparamfile branch July 3, 2017 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants