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

AzureFileCopy support AzCopy (v10) sync command #9157

Open
eelkram opened this issue Dec 22, 2018 · 11 comments

Comments

@eelkram
Copy link

commented Dec 22, 2018

The AzureFileCopy task should be updated to support the new AzCopy v10 tool and the new sync command. This would be extremely useful in conjunction with the static website hosting on Azure Storage capabilities.

@rajatagrawal-dev

This comment has been minimized.

Copy link
Member

commented May 7, 2019

@eelkram Can you give more details on your scenario? Where do you see the need for the sync command and how will it complement the static website support?

@eelkram

This comment has been minimized.

Copy link
Author

commented May 7, 2019

@rajatagrawal-dev - I use AzCopy to deploy a React front end on Azure Storage that is served up via the static website support and fronted via an Azure CDN. During the build process the artifacts are dynamically named with a hash to eliminate caching issues. When using the current AzCopy commands it moves the new files up to Azure Storage but never removes the old files. I would like to be able to use the sync command to copy out the new files, update existing files and remove the ones that no longer exist without having to write custom code.

Let me know if you have any other questions.

@rajatagrawal-dev

This comment has been minimized.

Copy link
Member

commented May 9, 2019

@eelkram Can you work around for now by adding a task before Azure File Copy task? That task can be an Azure CLI task or Azure Powershell task. You can write a one line command in here to delete the container.

@mark-studer

This comment has been minimized.

Copy link

commented May 9, 2019

Yep, I can work around it for now. Once this is added I will switch to using the sync command.

@albernhagen

This comment has been minimized.

Copy link

commented May 16, 2019

Is there any timeline as to when this will be added? I'd prefer to not delete the storage container before deploy if I can help it

@oscarwest

This comment has been minimized.

Copy link

commented May 17, 2019

We would benefit greatly from this as well.
Ref to issue at their repo:
Azure/azure-storage-azcopy#75

@rajatagrawal-dev

This comment has been minimized.

Copy link
Member

commented May 17, 2019

After going through the documentation of AzCopy v10, I think there are a lot of changes we might have to do in the task in order to upgrade to that. It might change the current task design in order for us to support the new features of v10.

This is not something that we have planned for as of now. Hence, I can't comment on any timelines here.
I will update this issue once we have any plans for this. Sorry for the inconvenience.

@MaxMelcher

This comment has been minimized.

Copy link

commented Jun 9, 2019

Would you accept a PR for this @rajatagrawal-dev?

@rajatagrawal-dev

This comment has been minimized.

Copy link
Member

commented Jun 10, 2019

@MaxMelcher We won't update the current task (Azure File Copy v3) to support AzCopy v10 as it will most likely break existing users. For AzCopy v10 support, we will have to create a new major version of the task (Azure File Copy v4). Since it is a major version, the task will have to be developed internally.

But you are free to create a new task which supports AzCopy v10 and upload to your own Azure DevOps instance. Use tfx-cli for that purpose.

@rwerlang

This comment has been minimized.

Copy link

commented Jul 17, 2019

I'm having the same issue publishing a reactjs to a storage account.
The problem to workaround is that PowerShell does not support using a container named as $web, because $ is treated as invalid.

Any other workarounds to delete the blob before using the AzureFileCopy?

@MaxMelcher

This comment has been minimized.

Copy link

commented Jul 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
9 participants
You can’t perform that action at this time.