-
Notifications
You must be signed in to change notification settings - Fork 227
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
Create missing resources in sync, allow blob to overlap #2894
Create missing resources in sync, allow blob to overlap #2894
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. just had one question about the commented test
Creates parity with accidental functionality of blob, extends functionality on FNS
4fa0826
to
a0c692f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Looks good to me! However, it seems that some tests are failing. Could you please take a look?"
…ob-overlap' into adreed/create-resources-allow-blob-overlap
78e1676
to
b134210
Compare
Creates parity with accidental functionality of blob, extends functionality on FNS
Description
As it stands, when using a Blob destination, AzCopy is capable of creating a file if it is missing, as well as a directory if it is missing. However, if a directory stub exists, sync does not allow it to be overwritten, or overlapped, if using trailing slash directory stubs (e.g.
foo/
).In addition, this PR extends the resource creation functionality to containers, shares, and filesystems.
Feature / Bug Fix: Fleshing out the result of undefined behavior with Sync, creating parity with other destination resources.
Related Links:
Email Subject
Re: SciClone/AzCopy issues summary and resolution plan
ADO
#30057272
Type of Change
How Has This Been Tested?
Both sides of the feature, blob overlapping and resource creation, are exhaustively tested with two new E2E tests, covering their relevant scenarios.