Skip to content

Commit 76b1951

Browse files
YoYoJaebwintersrads-1996azure-sdkdanieljurek
authored
Jessli/convert (#43556) merge main
* add eval result converter * Add result converter * update converter params to optional * add eval meta data * fix type * remove useless file * get eval meta data as input * fix build errors * remove useless import * resolve comments * update * update comments * fix checker failure * add error msg and error code * Surface evaluator error msg * update UT * fix usage * make eval_meta_data optional * remove useless lines * update param name to add underscore * parse updated annotation results * update trace_id * expose sample data for sdk evaluators * update * Fix column mapping bug for AOAI evaluators with custom data mapping (#43429) * fix nesting bug for custom data mapping * address comments * remove extra code and fix test case * run formatter * use dumps * Modify logic for message body on Microsoft.ApplicationInsights.MessageData to include default message for messages with empty body and export logs (#43091) * Modify logic in PR (#43060) to include default message for messages with empty body and export logs * Update CHANGELOG * Update logic as per updated spec * Addressed comments * Set-VcpkgWriteModeCache -- add token timeout param for cmake generate's that exceed 1 hour (this can happen in C++ API View) (#43470) Co-authored-by: Daniel Jurek <[email protected]> * update * fix UT * fix tests * Added Tests and Samples for Paginated Queries (#43472) * added tests and samples for paginated queries * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> * added single partition pagination sample --------- Co-authored-by: Andrew Mathew <[email protected]> Co-authored-by: Copilot <[email protected]> * [Test Proxy] Support AARCH64 platform (#43428) * Delete doc/dev/how_to_request_a_feature_in_sdk.md (#43415) this doc is outdated * fix test * [AutoRelease] t2-iothub-2025-10-03-03336(can only be merged by SDK owner) (#43230) * code and test * update pyproject.toml --------- Co-authored-by: azure-sdk <PythonSdkPipelines> Co-authored-by: ChenxiJiang333 <[email protected]> * [AutoRelease] t2-redisenterprise-2025-10-17-18412(can only be merged by SDK owner) (#43476) * code and test * update changelog * update changelog * Update CHANGELOG.md --------- Co-authored-by: azure-sdk <PythonSdkPipelines> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: ChenxiJiang333 <[email protected]> * Extend basic test for "project_client.agents" to do more operations (#43516) * Sync eng/common directory with azure-sdk-tools for PR 12478 (#43457) * Updated validate pkg template to use packageInfo * Fixed typo * Fixed the right variable to use * output debug log * Fixed errors in expression evaluation * removed debug code * Fixed an issue in pipeline * Updated condition for variable setting step * Join paths of the script path * Use join-path * return from the function rather than exit --------- Co-authored-by: ray chen <[email protected]> * Reorder error and warning log line processing (#43456) Co-authored-by: Wes Haggard <[email protected]> * [App Configuration] - Release 1.7.2 (#43520) * release 1.7.2 * update change log * Modify CODEOWNERS for Azure SDK ownership changes (#43524) Updated CODEOWNERS to reflect new ownership for Azure SDK components. * Migrate Confidential Ledger library from swagger to typespec codegen (#42664) * regen * add default cert endpoint with tsp * remove refs to old namespace * update async operation patch * fix operations patch * fix header impl * more header fixes * revert receipt directory removal * cspell * regen certificates under correct namespace * regen ledger client * update namespace name * revert certificate change * update shared files after regen * updates * delete extra files * cspell * match return type to current behavior * cspell * mypy * pylint * update docs * regen * regen * fix patch * Revert "mypy" This reverts commit 6351ead. * add info in tsp_location.yaml * regen * update patch files * update patch files * fix patch * update patch files * regen * update tsp-location.yaml * generate certificate client * update patch files * fixes * regen clients * update pyproject.toml deps * update assets * regen * revert test change * nit * fix test input * regen with new model * update tests * update tests * apiview props * regen * update tests * update assets * apiview props * temp relative package updates * fix name * fix ledger ci (#43181) * remove swagger * remove extra configs * wip revert package dep temporarily * update readme * fix config files * Revert "wip revert package dep temporarily" This reverts commit db553c4. * move tests * add identity samples --------- Co-authored-by: catalinaperalta <[email protected]> * rm certificate files * update changelog * misc fixes * update shared reqs * test * pylint --------- Co-authored-by: catalinaperalta <[email protected]> * update scripts (#43527) Co-authored-by: helen229 <[email protected]> * [AutoPR azure-mgmt-mongocluster]-generated-from-SDK Generation - Python-5459673 (#43448) * Configurations: 'specification/mongocluster/resource-manager/Microsoft.DocumentDB/MongoCluster/tspconfig.yaml', API Version: 2025-09-01, SDK Release Type: stable, and CommitSHA: 'c5601446fc65494f18157aecbcc79cebcfbab1fb' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5459673 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release. * update changelog --------- Co-authored-by: ChenxiJiang333 <[email protected]> * App Configuration Provider - Key Vault Refresh (#41882) * Sync refresh changes * Key Vault Refresh * adding tests and fixing sync refresh * Updating Async * Fixed Async Tests * Updated tests and change log * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> * Fixing merge issue * Updating comments * Updating secret refresh * Update _azureappconfigurationproviderasync.py * Fixing Optional Endpoint * fix mypy issue * fixing async test * mixing merge * fixing test after merge * Update testcase.py * Secret Provider Base * removing unused imports * updating exception * updating resolve key vault references * Review comments * fixing tests * tox updates * Updating Tests * Updating Async to be the same as sync * Fixing formatting * fixing tox and unneeded "" * fixing tox items * fix cspell + tests recording * Update test_async_secret_provider.py * Post Merge updates * Move cache to shared code * removed unneeded disabled * Update Secret Provider * Updating usage * Update assets.json * Updated to make secret refresh update dictionary * removing _secret_version_cache * Update assets.json * Update _secret_provider_base.py --------- Co-authored-by: Copilot <[email protected]> * Increment package version after release of azure-appconfiguration (#43531) * Patch `azure-template` back to `green` (#43533) * Update sdk/template/azure-template/pyproject.toml to use `repository` instead of `source` * added brackets for sql query keyword value (#43525) Co-authored-by: Andrew Mathew <[email protected]> * update changelog (#43532) Co-authored-by: catalinaperalta <[email protected]> * App Config Provider - Provider Refactor (#43196) * Code Cleanup * Move validation to shared file * Updating Header Check * Update test_azureappconfigurationproviderbase.py * moved async tests to aio folder * post merge updates --------- Co-authored-by: Ethan Winters <[email protected]> Co-authored-by: rads-1996 <[email protected]> Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: Andrew Mathew <[email protected]> Co-authored-by: Andrew Mathew <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: McCoy Patiño <[email protected]> Co-authored-by: Yuchao Yan <[email protected]> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: Darren Cohen <[email protected]> Co-authored-by: ray chen <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Zhiyuan Liang <[email protected]> Co-authored-by: Matthew Metcalf <[email protected]> Co-authored-by: catalinaperalta <[email protected]> Co-authored-by: catalinaperalta <[email protected]> Co-authored-by: helen229 <[email protected]> Co-authored-by: Scott Beddall <[email protected]>
1 parent 6ac7f0e commit 76b1951

File tree

367 files changed

+11039
-10063
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

367 files changed

+11039
-10063
lines changed

.github/CODEOWNERS

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -62,18 +62,12 @@
6262
# PRLabel: %Storage
6363
/sdk/storage/ @annatisch @jalauzon-msft @vincenttran-msft @weirongw23-msft
6464

65-
# AzureSdkOwners: @xiangyan99
65+
# AzureSdkOwners: @mrm9084
6666
# ServiceLabel: %App Configuration
6767
# ServiceOwners: @albertofori @avanigupta @mrm9084
6868

6969
# PRLabel: %App Configuration
70-
/sdk/appconfiguration/ @xiangyan99
71-
72-
# ServiceOwners: @mrm9084
73-
# ServiceLabel: %App Configuration Provider
74-
75-
# PRLabel: %App Configuration Provider
76-
/sdk/appconfiguration/azure-appconfiguration-provider/ @albertofori @avanigupta @mrm9084 @rossgrambo
70+
/sdk/appconfiguration/ @albertofori @avanigupta @mrm9084 @rossgrambo @xiangyan99
7771

7872
# ServiceLabel: %Attestation
7973
# PRLabel: %Attestation

.vscode/cspell.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1873,6 +1873,14 @@
18731873
"Onvif"
18741874
]
18751875
},
1876+
{
1877+
"filename": "sdk/confidentialledger/azure-confidentialledger/**",
1878+
"words": [
1879+
"mrenclave",
1880+
"MRENCLAVE",
1881+
"oeverify"
1882+
]
1883+
},
18761884
{
18771885
"filename": "sdk/confidentialledger/azure-confidentialledger/azure/confidentialledger/_operations/*.py",
18781886
"words": [

doc/dev/how_to_request_a_feature_in_sdk.md

Lines changed: 0 additions & 26 deletions
This file was deleted.

eng/common/pipelines/templates/steps/set-vcpkg-cache-vars.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
2+
parameters:
3+
- name: TokenTimeoutInHours
4+
type: number
5+
default: 1
6+
17
steps:
28
- pwsh: |
39
Write-Host "Setting vcpkg cache variables for read only access to vcpkg binary and asset caches"
@@ -12,6 +18,7 @@ steps:
1218
azureSubscription: 'Azure SDK Artifacts'
1319
ScriptType: FilePath
1420
ScriptPath: eng/common/scripts/Set-VcpkgWriteModeCache.ps1
21+
ScriptArguments: -TokenTimeoutInHours ${{ parameters.TokenTimeoutInHours }}
1522
azurePowerShellVersion: LatestVersion
1623
pwsh: true
1724
# This step is idempotent and can be run multiple times in cases of

eng/common/pipelines/templates/steps/validate-all-packages.yml

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,23 @@
11
parameters:
2-
ArtifactPath: $(Build.ArtifactStagingDirectory)
3-
Artifacts: []
4-
ConfigFileDir: $(Build.ArtifactStagingDirectory)/PackageInfo
2+
- name: ArtifactPath
3+
type: string
4+
default: $(Build.ArtifactStagingDirectory)
5+
- name: Artifacts
6+
type: object
7+
default: []
8+
- name: ConfigFileDir
9+
type: string
10+
default: $(Build.ArtifactStagingDirectory)/PackageInfo
11+
- name: PackageInfoFiles
12+
type: object
13+
default: []
514

615
steps:
716
- ${{ if and(ne(variables['Skip.PackageValidation'], 'true'), and(ne(variables['Build.Reason'], 'PullRequest'), eq(variables['System.TeamProject'], 'internal'))) }}:
817
- pwsh: |
918
echo "##vso[task.setvariable variable=SetAsReleaseBuild]false"
1019
displayName: "Set as release build"
11-
condition: and(succeeded(), eq(variables['SetAsReleaseBuild'], ''))
20+
condition: and(succeededOrFailed(), eq(variables['SetAsReleaseBuild'], ''))
1221
1322
- task: AzureCLI@2
1423
inputs:
@@ -24,7 +33,8 @@ steps:
2433
-ConfigFileDir '${{ parameters.ConfigFileDir }}' `
2534
-BuildDefinition $(System.CollectionUri)$(System.TeamProject)/_build?definitionId=$(System.DefinitionId) `
2635
-PipelineUrl $(System.CollectionUri)$(System.TeamProject)/_build/results?buildId=$(Build.BuildId) `
27-
-IsReleaseBuild $$(SetAsReleaseBuild)
36+
-IsReleaseBuild $$(SetAsReleaseBuild) `
37+
-PackageInfoFiles ('${{ convertToJson(parameters.PackageInfoFiles) }}' | ConvertFrom-Json -NoEnumerate)
2838
workingDirectory: $(Pipeline.Workspace)
2939
displayName: Validate packages and update work items
3040
continueOnError: true

eng/common/scripts/Create-APIReview.ps1

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ Set-StrictMode -Version 3
2424
. (Join-Path $PSScriptRoot Helpers ApiView-Helpers.ps1)
2525

2626
# Submit API review request and return status whether current revision is approved or pending or failed to create review
27-
function Upload-SourceArtifact($filePath, $apiLabel, $releaseStatus, $packageVersion)
27+
function Upload-SourceArtifact($filePath, $apiLabel, $releaseStatus, $packageVersion, $packageType)
2828
{
2929
Write-Host "File path: $filePath"
3030
$fileName = Split-Path -Leaf $filePath
@@ -61,6 +61,13 @@ function Upload-SourceArtifact($filePath, $apiLabel, $releaseStatus, $packageVer
6161
$multipartContent.Add($releaseTagParamContent)
6262
Write-Host "Request param, setReleaseTag: $MarkPackageAsShipped"
6363

64+
$packageTypeParam = [System.Net.Http.Headers.ContentDispositionHeaderValue]::new("form-data")
65+
$packageTypeParam.Name = "packageType"
66+
$packageTypeParamContent = [System.Net.Http.StringContent]::new($packageType)
67+
$packageTypeParamContent.Headers.ContentDisposition = $packageTypeParam
68+
$multipartContent.Add($packageTypeParamContent)
69+
Write-Host "Request param, packageType: $packageType"
70+
6471
if ($releaseStatus -and ($releaseStatus -ne "Unreleased"))
6572
{
6673
$compareAllParam = [System.Net.Http.Headers.ContentDispositionHeaderValue]::new("form-data")
@@ -92,14 +99,14 @@ function Upload-SourceArtifact($filePath, $apiLabel, $releaseStatus, $packageVer
9299
return $StatusCode
93100
}
94101

95-
function Upload-ReviewTokenFile($packageName, $apiLabel, $releaseStatus, $reviewFileName, $packageVersion, $filePath)
102+
function Upload-ReviewTokenFile($packageName, $apiLabel, $releaseStatus, $reviewFileName, $packageVersion, $filePath, $packageType)
96103
{
97104
Write-Host "Original File path: $filePath"
98105
$fileName = Split-Path -Leaf $filePath
99106
Write-Host "OriginalFile name: $fileName"
100107

101108
$params = "buildId=${BuildId}&artifactName=${ArtifactName}&originalFilePath=${fileName}&reviewFilePath=${reviewFileName}"
102-
$params += "&label=${apiLabel}&repoName=${RepoName}&packageName=${packageName}&project=internal&packageVersion=${packageVersion}"
109+
$params +="&label=${apiLabel}&repoName=${RepoName}&packageName=${packageName}&project=internal&packageVersion=${packageVersion}&packageType=${packageType}"
103110
if($MarkPackageAsShipped) {
104111
$params += "&setReleaseTag=true"
105112
}
@@ -146,17 +153,18 @@ function Get-APITokenFileName($packageName)
146153
function Submit-APIReview($packageInfo, $packagePath)
147154
{
148155
$apiLabel = "Source Branch:${SourceBranch}"
156+
$packageType = $packageInfo.SdkType
149157

150158
# Get generated review token file if present
151159
# APIView processes request using different API if token file is already generated
152160
$reviewTokenFileName = Get-APITokenFileName $packageInfo.ArtifactName
153161
if ($reviewTokenFileName) {
154162
Write-Host "Uploading review token file $reviewTokenFileName to APIView."
155-
return Upload-ReviewTokenFile $packageInfo.ArtifactName $apiLabel $packageInfo.ReleaseStatus $reviewTokenFileName $packageInfo.Version $packagePath
163+
return Upload-ReviewTokenFile $packageInfo.ArtifactName $apiLabel $packageInfo.ReleaseStatus $reviewTokenFileName $packageInfo.Version $packagePath $packageType
156164
}
157165
else {
158166
Write-Host "Uploading $packagePath to APIView."
159-
return Upload-SourceArtifact $packagePath $apiLabel $packageInfo.ReleaseStatus $packageInfo.Version
167+
return Upload-SourceArtifact $packagePath $apiLabel $packageInfo.ReleaseStatus $packageInfo.Version $packageType
160168
}
161169
}
162170

eng/common/scripts/Detect-Api-Changes.ps1

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Param (
2222
$configFileDir = Join-Path -Path $ArtifactPath "PackageInfo"
2323

2424
# Submit API review request and return status whether current revision is approved or pending or failed to create review
25-
function Submit-Request($filePath, $packageName)
25+
function Submit-Request($filePath, $packageName, $packageType)
2626
{
2727
$repoName = $RepoFullName
2828
if (!$repoName) {
@@ -39,6 +39,7 @@ function Submit-Request($filePath, $packageName)
3939
$query.Add('packageName', $packageName)
4040
$query.Add('language', $LanguageShort)
4141
$query.Add('project', $DevopsProject)
42+
$query.Add('packageType', $packageType)
4243
$reviewFileFullName = Join-Path -Path $ArtifactPath $packageName $reviewFileName
4344
# If CI generates token file then it passes both token file name and original file (filePath) to APIView
4445
# If both files are passed then APIView downloads the parent directory as a zip
@@ -126,6 +127,7 @@ foreach ($packageInfoFile in $packageInfoFiles)
126127
{
127128
$packageInfo = Get-Content $packageInfoFile | ConvertFrom-Json
128129
$pkgArtifactName = $packageInfo.ArtifactName ?? $packageInfo.Name
130+
$packageType = $packageInfo.SdkType
129131

130132
LogInfo "Processing $($pkgArtifactName)"
131133

@@ -157,7 +159,7 @@ foreach ($packageInfoFile in $packageInfoFiles)
157159
if ($isRequired -eq $True)
158160
{
159161
$filePath = $pkgPath.Replace($ArtifactPath , "").Replace("\", "/")
160-
$respCode = Submit-Request -filePath $filePath -packageName $pkgArtifactName
162+
$respCode = Submit-Request -filePath $filePath -packageName $pkgArtifactName -packageType $packageType
161163
if ($respCode -ne '200')
162164
{
163165
$responses[$pkgArtifactName] = $respCode

0 commit comments

Comments
 (0)