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

[RCA Repair Item] Add exclude-container flag #2504

Merged
merged 10 commits into from Jan 16, 2024

Conversation

siminsavani-msft
Copy link
Member

@siminsavani-msft siminsavani-msft commented Dec 15, 2023

Customers are unable to exclude containers when doing an Account to Account transfer. This PR introduces a new flag, --exclude-path that allows the user to add as many container names as they want to be excluded during Account to Account transfer.

AzCopy sync does not support service level transfers and we spit out this error: Failed with error service level URLs (https://xxx.blob.core.windows.net/) are not supported in sync:, therefore, exclude-containers is not added to sync.

AzCopy remove does also does not support deleting at service level. No error message is spit out but containers and blobs are not deleted after a simple remove https://xxx.blob.core.windows.net/$[SAS] so exclude-container is not added to remove. Specific containers can be removed by using the container URL.

@siminsavani-msft siminsavani-msft added this to the 10.23 milestone Dec 15, 2023
cmd/zc_filter.go Outdated Show resolved Hide resolved
vibhansa-msft
vibhansa-msft previously approved these changes Jan 12, 2024
Copy link
Member

@vibhansa-msft vibhansa-msft left a comment

Choose a reason for hiding this comment

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

Minor comment to add logs whenever we skip a container.

@siminsavani-msft siminsavani-msft merged commit 837492e into dev Jan 16, 2024
17 checks passed
@siminsavani-msft siminsavani-msft deleted the siminsavani/exclude-container branch January 16, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants