Skip to content

Supporting LRO Patch with 201#3625

Merged
shahabhijeet merged 5 commits intoAzure:psSdkJson6from
shahabhijeet:Patch201IS3492
Sep 4, 2017
Merged

Supporting LRO Patch with 201#3625
shahabhijeet merged 5 commits intoAzure:psSdkJson6from
shahabhijeet:Patch201IS3492

Conversation

@shahabhijeet
Copy link
Copy Markdown
Contributor

Fix for issue #3592

  • Supports scenario for LRO Patch operation
  • Added Tests

shahabhijeet added 2 commits September 1, 2017 13:54
…d status and v1 ARM spec allows it to be empty, but V2.2 ARM spec require failed/cancelled status shoudl accompany error information. Added appropriate tests for the scenarios
@shahabhijeet
Copy link
Copy Markdown
Contributor Author

@azuresdkci test this please

@shahabhijeet
Copy link
Copy Markdown
Contributor Author

@shahabhijeet shahabhijeet merged commit aa6276d into Azure:psSdkJson6 Sep 4, 2017
@shahabhijeet
Copy link
Copy Markdown
Contributor Author

@shahabhijeet shahabhijeet deleted the Patch201IS3492 branch September 5, 2017 03:01
JasonYang-MSFT pushed a commit to JasonYang-MSFT/azure-sdk-for-net that referenced this pull request Dec 8, 2017
* Supporting LRO Patch operation. Initial set of tests. Fix for issue#3492

* Fixing a case where LRO PUT will fail during polling. Added test to cover the scenario

* Fixed another scenario where body during polling is empty for a failed status and v1 ARM spec allows it to be empty, but V2.2 ARM spec require failed/cancelled status shoudl accompany error information. Added appropriate tests for the scenarios

* Adding Release notes
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.

2 participants