Skip to content

Storage code migration#3014

Merged
weshaggard merged 680 commits into
Azure:masterfrom
g2vinay:storage-code-migration
Mar 8, 2019
Merged

Storage code migration#3014
weshaggard merged 680 commits into
Azure:masterfrom
g2vinay:storage-code-migration

Conversation

@g2vinay

@g2vinay g2vinay commented Mar 4, 2019

Copy link
Copy Markdown
Member

Includes source code as it is from Storage Java SDK master branch with history preserved.
Code location is in storage/data-plane/
In this PR, Storage SDK source code is migrated without any changes.
SDK component is not configured in the build.
No code changes on our end are included in this PR.

Follow up PR will include:
Storage SDK build integration to the central java repo.
Live tests Integration to our Azure Dev Ops pipeline.
Code + Pom cleanup

…view-dev

New storage sdk v10 preview dev
…view-dev

New storage sdk v10 preview dev
…0-Preview-dev

Revert "New storage sdk v10 preview dev"
@azuresdkci

Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@g2vinay
g2vinay requested a review from amishra-dev March 5, 2019 00:13
@JonathanGiles JonathanGiles added the Client This issue points to a problem in the data-plane of the library. label Mar 5, 2019

@JonathanGiles JonathanGiles left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As this is a direct import from the storage repo into this repo, and doesn't include any changes, cleanups, or integration into the build system - I am ok with approving this and I look forward to the cleanup PR.

@amishra-dev

Copy link
Copy Markdown

@rickle-msft please look at this PR when you are back, we want to make sure that everything we have in our repo moved over here successfully.

@g2vinay please wait for Rick to approve before merging

@amishra-dev
amishra-dev requested a review from rickle-msft March 5, 2019 18:52
@scbedd

scbedd commented Mar 7, 2019

Copy link
Copy Markdown
Member

@g2vinay Hey mate make sure to hit me up when this merges. I'll need to get the ball rolling on documentation once that happens.

Do you also plan to migrate the preview branches? (if any?)

@weshaggard

Copy link
Copy Markdown
Member

@weshaggard

Copy link
Copy Markdown
Member

SDK component is not configured in the build.
No code changes on our end are included in this PR.

Is there any reason we don't go ahead and make those changes as part of this PR as well? There seems like there is a lot of unnecessary files and stuff coming in that we could eliminate completely if we just cleaned them up at the same time. Also as I mentioned in another similar PR we have analysis tools like component governance that will start flagging stuff even if this isn't part of the build.

@weshaggard weshaggard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

First step looks good lets be sure to file follow-up items for the remaining clean-up work.

@weshaggard
weshaggard merged commit e449983 into Azure:master Mar 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.