Skip to content

AzureApiManagement breaking changes for api version 2019-12-01 #11618

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

Merged
merged 11 commits into from
May 2, 2020

Conversation

vfedonkin
Copy link
Contributor

Description

Azure Api Management breaking changes for may build
Implementation of the changes described here: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/545
Breaking change notification was included into april rc:
#11602

Checklist

  • [*] I have read the Submitting Changes section of CONTRIBUTING.md
  • [* ] The title of the PR is clear and informative
  • [* ] The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • [* ] If applicable, the changes made in the PR have proper test coverage
  • [* ] For public API changes to cmdlets:
    • [* ] a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • [* ] the markdown help files have been regenerated using the commands listed here

@vfedonkin vfedonkin requested a review from solankisamir April 17, 2020 03:36
@adxsdkps
Copy link
Collaborator

Can one of the admins verify this patch?

@wyunchi-ms
Copy link
Contributor

Please rerecord the test cases. @vfedonkin

@vfedonkin
Copy link
Contributor Author

@wyunchi-ms , tests rerecorded.

@vfedonkin vfedonkin changed the base branch from master to generation April 23, 2020 02:52
@vfedonkin vfedonkin changed the base branch from generation to master April 23, 2020 02:53
@vfedonkin
Copy link
Contributor Author

I will retarget this PR to "generation" branch, as told in https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/545

@vfedonkin
Copy link
Contributor Author

@msJinLei , please advise, should we merge this PR into "generation" branch, or should we continue with the master? Generation branch doesn't have our ApiManagement code at all.

@vfedonkin
Copy link
Contributor Author

@wyunchi-ms , tests rerecorded. Anything else required in this PR?
We must complete this before may build to include breaking changes in it

@solankisamir
Copy link
Member

solankisamir commented Apr 27, 2020

public class PsApiManagementSubscriptionKey //: PsApiManagementArmResource

remove this. #Resolved


Refers to: src/ApiManagement/ApiManagement.ServiceManagement/Models/PsApiManagementSubscriptionKey.cs:20 in 6015a7d. [](commit_id = 6015a7d, deletion_comment = False)

solankisamir
solankisamir previously approved these changes Apr 27, 2020
Copy link
Member

@solankisamir solankisamir left a comment

Choose a reason for hiding this comment

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

:shipit:

solankisamir
solankisamir previously approved these changes Apr 27, 2020
Copy link
Member

@solankisamir solankisamir left a comment

Choose a reason for hiding this comment

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

:shipit:

@wyunchi-ms
Copy link
Contributor

Hi @vfedonkin , @solankisamir there are still 33 case failed in ApiManagement. Could you please rerecord these test cases?

@vfedonkin
Copy link
Contributor Author

@wyunchi-ms , found it. Tests rerecorded.
I see some complains about help files: "Online version in the header of the file is incorrect"
But these help files are new and not published yet. Should I add online version anyway?

@solankisamir
Copy link
Member

online version:

add online version here like

https://docs.microsoft.com/en-us/powershell/module/az.apimanagement/get-azapimanagementauthorizationserver

In this and other files

Get-AzApiManagementAuthorizationServerClientSecret.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementIdentityProviderClientSecret.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementNamedValue.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementNamedValueSecretValue.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementOpenIdConnectProviderClientSecret.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementSubscriptionKey.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementTenantAccessSecret.md Online version in the header of the file is incorrect. The cor.
Get-AzApiManagementTenantGitAccessSecret.md Online version in the header of the file is incorrect. The cor.
New-AzApiManagementNamedValue.md Online version in the header of the file is incorrect. The cor.
Remove-AzApiManagementNamedValue.md Online version in the header of the file is incorrect. The cor.
Set-AzApiManagementNamedValue.md


Refers to: src/ApiManagement/ApiManagement/help/Get-AzApiManagementAuthorizationServerClientSecret.md:4 in 0c4a985. [](commit_id = 0c4a985, deletion_comment = False)

@vfedonkin
Copy link
Contributor Author

@wyunchi-ms , @solankisamir , tests rerecorded, help fixed

@vfedonkin
Copy link
Contributor Author

@solankisamir , @wyunchi-ms , all checks are green now.

Copy link
Member

@solankisamir solankisamir left a comment

Choose a reason for hiding this comment

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

:shipit:

@wyunchi-ms wyunchi-ms merged commit 842bcc0 into Azure:master May 2, 2020
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.

4 participants