From b8222953704a80ad3d6384e5eefb4598dc5e9a94 Mon Sep 17 00:00:00 2001 From: Praveen Kuttappan Date: Fri, 29 Mar 2024 10:35:22 -0400 Subject: [PATCH 1/4] Skip API review package name approval for management plane packages --- eng/common/scripts/Create-APIReview.ps1 | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/eng/common/scripts/Create-APIReview.ps1 b/eng/common/scripts/Create-APIReview.ps1 index cdf467180269..f185580a3188 100644 --- a/eng/common/scripts/Create-APIReview.ps1 +++ b/eng/common/scripts/Create-APIReview.ps1 @@ -153,6 +153,14 @@ function Submit-APIReview($packageInfo, $packagePath) } } +function IsApiviewStatusCheckRequired($packageInfo) +{ + # This function can be modified to control this check by individual language function + if (($packageInfo.SdkType -eq "client" -or $packageInfo.SdkType -eq "spring") -and $packageInfo.IsNewSdk) { + return $true + } + return $false +} function ProcessPackage($packageName) { @@ -234,8 +242,14 @@ function ProcessPackage($packageName) # Check if package name is approved. Preview version cannot be released without package name approval if (!$pkgNameStatus.IsApproved) { - Write-Error $($pkgNameStatus.Details) - return 1 + if (IsApiviewStatusCheckRequired($pkgInfo)) + { + Write-Error $($pkgNameStatus.Details) + return 1 + } + else{ + Write-Host "Package name is not approved for package $($packageName), however it is not required for this package type so it can still be released without API review approval." + } } # Ignore API review status for prerelease version Write-Host "Package version is not GA. Ignoring API view approval status" @@ -245,7 +259,7 @@ function ProcessPackage($packageName) # Return error code if status code is 201 for new data plane package # Temporarily enable API review for spring SDK types. Ideally this should be done be using 'IsReviewRequired' method in language side # to override default check of SDK type client - if (($pkgInfo.SdkType -eq "client" -or $pkgInfo.SdkType -eq "spring") -and $pkgInfo.IsNewSdk) + if (IsApiviewStatusCheckRequired($pkgInfo)) { if (!$apiStatus.IsApproved) { From 4a8d1e1519623f179f0f963ec08476cad4354c5d Mon Sep 17 00:00:00 2001 From: Praveen Kuttappan Date: Fri, 29 Mar 2024 10:40:49 -0400 Subject: [PATCH 2/4] Remove the comment on function --- eng/common/scripts/Create-APIReview.ps1 | 1 - 1 file changed, 1 deletion(-) diff --git a/eng/common/scripts/Create-APIReview.ps1 b/eng/common/scripts/Create-APIReview.ps1 index f185580a3188..f2ee285a3433 100644 --- a/eng/common/scripts/Create-APIReview.ps1 +++ b/eng/common/scripts/Create-APIReview.ps1 @@ -155,7 +155,6 @@ function Submit-APIReview($packageInfo, $packagePath) function IsApiviewStatusCheckRequired($packageInfo) { - # This function can be modified to control this check by individual language function if (($packageInfo.SdkType -eq "client" -or $packageInfo.SdkType -eq "spring") -and $packageInfo.IsNewSdk) { return $true } From 49ad51f85923c8ad1de8f6064a604690dd461461 Mon Sep 17 00:00:00 2001 From: Praven Kuttappan <55455725+praveenkuttappan@users.noreply.github.com> Date: Fri, 29 Mar 2024 11:45:03 -0400 Subject: [PATCH 3/4] Update eng/common/scripts/Create-APIReview.ps1 Co-authored-by: Wes Haggard --- eng/common/scripts/Create-APIReview.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/scripts/Create-APIReview.ps1 b/eng/common/scripts/Create-APIReview.ps1 index f2ee285a3433..0c030716919f 100644 --- a/eng/common/scripts/Create-APIReview.ps1 +++ b/eng/common/scripts/Create-APIReview.ps1 @@ -241,7 +241,7 @@ function ProcessPackage($packageName) # Check if package name is approved. Preview version cannot be released without package name approval if (!$pkgNameStatus.IsApproved) { - if (IsApiviewStatusCheckRequired($pkgInfo)) + if (IsApiviewStatusCheckRequired $pkgInfo) { Write-Error $($pkgNameStatus.Details) return 1 From bf1505680100c7630bd6043231919137a20c2857 Mon Sep 17 00:00:00 2001 From: Praven Kuttappan <55455725+praveenkuttappan@users.noreply.github.com> Date: Fri, 29 Mar 2024 11:45:15 -0400 Subject: [PATCH 4/4] Update eng/common/scripts/Create-APIReview.ps1 Co-authored-by: Wes Haggard --- eng/common/scripts/Create-APIReview.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/scripts/Create-APIReview.ps1 b/eng/common/scripts/Create-APIReview.ps1 index 0c030716919f..246a87be6fd3 100644 --- a/eng/common/scripts/Create-APIReview.ps1 +++ b/eng/common/scripts/Create-APIReview.ps1 @@ -258,7 +258,7 @@ function ProcessPackage($packageName) # Return error code if status code is 201 for new data plane package # Temporarily enable API review for spring SDK types. Ideally this should be done be using 'IsReviewRequired' method in language side # to override default check of SDK type client - if (IsApiviewStatusCheckRequired($pkgInfo)) + if (IsApiviewStatusCheckRequired $pkgInfo) { if (!$apiStatus.IsApproved) {