Skip to content

Prepare for release of Microsoft.Azure.Batch Data Plane 9.0#4689

Merged
dsgouda merged 11 commits intoAzure:psSdkJson6from
matthchr:feature/batch-sdk-9.0
Aug 23, 2018
Merged

Prepare for release of Microsoft.Azure.Batch Data Plane 9.0#4689
dsgouda merged 11 commits intoAzure:psSdkJson6from
matthchr:feature/batch-sdk-9.0

Conversation

@matthchr
Copy link
Copy Markdown
Member

Description

Update the Azure Batch C# SDK with support for REST API version 2018-08-01.7.0
REST API Spec PR: Azure/azure-rest-api-specs#3657
Note that this is a breaking change.

In addition to various other improvements, this PR fixes: #4602, #4475, #2538


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

  - Requests whose default timeouts are 2m should have a client side timeout
    longer than 2m.
  - The underlying server bug/behavior which caused us to need to do this has been fixed
    in the latest REST API version.
  - This allows callers to bypass a superfluous GetFileProperties call
    if they're really interested in performance when downloading files.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is the REST spec merged? We cannot accept this PR until the spec is merged to Azure master

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's already merged I just generated from a private branch from before and haven't updated this file.

I can regenerate it if you want.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That would be much appreciated. If you can pull down latest changes and run msbuild build.proj before regenerating that'd be even better

  - Update SDK version to 9.0.
  - Update changelog.
@matthchr
Copy link
Copy Markdown
Member Author

@dsgouda - I did as you asked and regenerated -- can you merge this?

Copy link
Copy Markdown
Contributor

@dsgouda dsgouda left a comment

Choose a reason for hiding this comment

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

Please run msbuild build.proj /t:build /p:Scope=SDKs\Batch\DataPlane\Azure.Batch and commit the changes generated to .props file here
If you do not see any changes, simply replace _2018-03-01.6.1; with _2018-08-01.7.0; in the file

@dsgouda
Copy link
Copy Markdown
Contributor

dsgouda commented Aug 23, 2018

Looks great apart from a minor comment

Copy link
Copy Markdown
Contributor

@dsgouda dsgouda left a comment

Choose a reason for hiding this comment

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

LGTM will merge when CIs pass

@matthchr
Copy link
Copy Markdown
Member Author

@dsgouda - CI passed

@dsgouda dsgouda merged commit 404b1c0 into Azure:psSdkJson6 Aug 23, 2018
@matthchr matthchr deleted the feature/batch-sdk-9.0 branch August 23, 2018 20:03
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.

3 participants