Skip to content

Commit 45e87fb

Browse files
AlancereAlancere
committed
Merge branch 'main' of https://github.com/Azure/azure-sdk-for-go into generator_typespec_tool_0614
2 parents 520b212 + 4e0b68a commit 45e87fb

File tree

252 files changed

+471
-321
lines changed

Some content is hidden

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

252 files changed

+471
-321
lines changed

‎.github/CODEOWNERS

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,10 @@
7979
/sdk/resourcemanager/ @ArcturusZhang @lirenhe @tadelesh @raych1
8080

8181
# AzureSDKOwners: @jhendrixMSFT
82-
# ServiceOwner: @siminsavani-msft
82+
# ServiceOwner: @tanyasethi-msft
8383
# ServiceLabel: %Azure.Core
8484
# PRLabel: %Storage
85-
/sdk/storage/ @siminsavani-msft @souravgupta-msft @jhendrixMSFT @gapra-msft @vibhansa-msft @tanyasethi-msft
85+
/sdk/storage/ @souravgupta-msft @jhendrixMSFT @gapra-msft @vibhansa-msft @tanyasethi-msft
8686

8787
# AzureSDKOwners: @jhendrixMSFT
8888
# ServiceLabel: %OpenTelemetry

‎eng/common/pipelines/templates/steps/set-default-branch.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ parameters:
22
WorkingDirectory: '$(System.DefaultWorkingDirectory)'
33
RemoteRepo: 'origin'
44
DefaultBranchVariableName: DefaultBranch
5+
Condition: 'succeeded()'
56
steps:
67
- pwsh: |
78
$setDefaultBranch = (git remote show ${{ parameters.RemoteRepo }} | Out-String) -replace "(?ms).*HEAD branch: (\w+).*", '$1'
@@ -13,4 +14,5 @@ steps:
1314
Write-Host "##vso[task.setvariable variable=${{ parameters.DefaultBranchVariableName }}]$setDefaultBranch"
1415
displayName: "Setup Default Branch"
1516
workingDirectory: ${{ parameters.workingDirectory }}
17+
condition: ${{ parameters.Condition }}
1618
ignoreLASTEXITCODE: true

‎eng/common/pipelines/templates/steps/verify-links.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ parameters:
1212

1313
steps:
1414
- template: /eng/common/pipelines/templates/steps/set-default-branch.yml
15+
parameters:
16+
Condition: ${{ parameters.Condition }}
1517
- task: PowerShell@2
1618
displayName: Link verification check
1719
condition: ${{ parameters.Condition }}

‎eng/common/scripts/Generate-PR-Diff.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ $changedServices = Get-ChangedServices -ChangedFiles $changedFiles
4545
$result = [PSCustomObject]@{
4646
"ChangedFiles" = $changedFiles
4747
"ChangedServices" = $changedServices
48-
"PRNumber" = $env:SYSTEM_PULLREQUEST_PULLREQUESTNUMBER
48+
"PRNumber" = if ($env:SYSTEM_PULLREQUEST_PULLREQUESTNUMBER) { $env:SYSTEM_PULLREQUEST_PULLREQUESTNUMBER } else { "-1" }
4949
}
5050

5151
$result | ConvertTo-Json | Out-File $ArtifactName

‎eng/common/scripts/Package-Properties.ps1

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,30 @@ function Get-PkgProperties
105105
return $null
106106
}
107107

108+
function Get-PrPkgProperties([string]$InputDiffJson) {
109+
$packagesWithChanges = @()
110+
111+
$allPackageProperties = Get-AllPkgProperties
112+
$diff = Get-Content $InputDiffJson | ConvertFrom-Json
113+
$targetedFiles = $diff.ChangedFiles
114+
115+
foreach($pkg in $allPackageProperties)
116+
{
117+
$pkgDirectory = Resolve-Path "$($pkg.DirectoryPath)"
118+
119+
foreach($file in $targetedFiles)
120+
{
121+
$filePath = Resolve-Path (Join-Path $RepoRoot $file)
122+
$shouldInclude = $filePath -like "$pkgDirectory*"
123+
if ($shouldInclude) {
124+
$packagesWithChanges += $pkg
125+
}
126+
}
127+
}
128+
129+
return $packagesWithChanges
130+
}
131+
108132
# Takes ServiceName and Repo Root Directory
109133
# Returns important properties for each package in the specified service, or entire repo if the serviceName is not specified
110134
# Returns a Table of service key to array values of PS Object with properties @ { pkgName, pkgVersion, pkgDirectoryPath, pkgReadMePath, pkgChangeLogPath }

‎eng/common/scripts/Save-Package-Properties.ps1

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,10 @@ filename as track 1 packages (e.g. same artifact name or package name), the
1515
track 2 package properties will be written.
1616
1717
.PARAMETER serviceDirectory
18-
Service directory in which to search for packages
18+
Service directory in which to search for packages.
19+
20+
.PARAMETER prDiff
21+
A file path leading to a file generated from Generate-PR-Diff.json. This parameter takes precedence over serviceDirectory, do not provide both.
1922
2023
.PARAMETER outDirectory
2124
Output location (generally a package artifact directory in DevOps) for JSON
@@ -32,10 +35,10 @@ Verison property in that file.
3235

3336
[CmdletBinding()]
3437
Param (
35-
[Parameter(Mandatory=$True)]
3638
[string] $serviceDirectory,
3739
[Parameter(Mandatory=$True)]
3840
[string] $outDirectory,
41+
[string] $prDiff,
3942
[switch] $addDevVersion
4043
)
4144

@@ -92,7 +95,16 @@ function GetRelativePath($path) {
9295
}
9396

9497
$exportedPaths = @{}
95-
$allPackageProperties = Get-AllPkgProperties $serviceDirectory
98+
99+
$allPackageProperties = @()
100+
101+
if ($prDiff) {
102+
$allPackageProperties = Get-PrPkgProperties $prDiff
103+
}
104+
else {
105+
$allPackageProperties = Get-AllPkgProperties $serviceDirectory
106+
}
107+
96108
if ($allPackageProperties)
97109
{
98110
if (-not (Test-Path -Path $outDirectory))
@@ -137,6 +149,6 @@ if ($allPackageProperties)
137149
}
138150
else
139151
{
140-
Write-Error "Package properties are not available for service directory $($serviceDirectory)"
152+
Write-Error "Package properties are not available for service directory $serviceDirectory or $prdiff"
141153
exit 1
142154
}

‎eng/common/scripts/Test-SampleMetadata.ps1

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,7 @@ begin {
203203
"azure-genomics",
204204
"azure-hdinsight",
205205
"azure-hdinsight-rserver",
206+
"azure-health-data-services",
206207
"azure-health-insights",
207208
"azure-hpc-cache",
208209
"azure-immersive-reader",

‎sdk/azidentity/CHANGELOG.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,15 @@
11
# Release History
22

3+
## 1.8.0-beta.2 (Unreleased)
4+
5+
### Features Added
6+
7+
### Breaking Changes
8+
9+
### Bugs Fixed
10+
11+
### Other Changes
12+
313
## 1.8.0-beta.1 (2024-07-17)
414

515
### Features Added

‎sdk/azidentity/cache/CHANGELOG.md

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,25 @@
11
# Release History
22

3-
## 0.3.0 (Unreleased)
3+
## 0.3.1 (Unreleased)
44

55
### Features Added
66

77
### Breaking Changes
8-
* Removed optional fallback to plaintext storage. `azidentity/cache` now
9-
always returns an error when it can't encrypt a persistent cache.
108

119
### Bugs Fixed
1210

1311
### Other Changes
1412

13+
## 0.3.0 (2024-07-18)
14+
15+
### Features Added
16+
* Added `New`, a constructor for persistent caches. See `azidentity` docs,
17+
in particular the `PersistentUserAuthentication` example, for usage details.
18+
19+
### Breaking Changes
20+
* Removed optional fallback to plaintext storage. `azidentity/cache` now
21+
always returns an error when it can't encrypt a persistent cache.
22+
1523
## 0.2.2 (2024-05-07)
1624

1725
### Bugs Fixed

‎sdk/azidentity/cache/go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ retract v0.1.0
66

77
require (
88
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0
9-
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.7.0
9+
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.8.0-beta.1
1010
github.com/AzureAD/microsoft-authentication-extensions-for-go/cache v0.1.1
1111
github.com/AzureAD/microsoft-authentication-library-for-go v1.2.2
1212
github.com/google/uuid v1.6.0

‎sdk/azidentity/cache/go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0 h1:GJHeeA2N7xrG3q30L2UXDyuWRzDM900/65j70wcM4Ww=
22
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0/go.mod h1:l38EPgmsp71HHLq9j7De57JcKOWPyhrsW1Awm1JS6K0=
3-
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.7.0 h1:tfLQ34V6F7tVSwoTf/4lH5sE0o6eCJuNDTmH09nDpbc=
4-
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.7.0/go.mod h1:9kIvujWAA58nmPmWB1m23fyWic1kYZMxD9CxaWn4Qpg=
3+
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.8.0-beta.1 h1:iw4+KCeCoieuKodp1d5YhAa1TU/GgogCbw8RbGvsfLA=
4+
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.8.0-beta.1/go.mod h1:AP8cDnDTGIVvayqKAhwzpcAyTJosXpvLYNmVFJb98x8=
55
github.com/Azure/azure-sdk-for-go/sdk/internal v1.10.0 h1:ywEEhmNahHBihViHepv3xPBn1663uRv2t2q/ESv9seY=
66
github.com/Azure/azure-sdk-for-go/sdk/internal v1.10.0/go.mod h1:iZDifYGJTIgIIkYRNWPENUnqx6bJ2xnSDFI2tjwZNuY=
77
github.com/AzureAD/microsoft-authentication-extensions-for-go/cache v0.1.1 h1:WJTmL004Abzc5wDB5VtZG2PJk5ndYDgVacGqfirKxjM=

‎sdk/azidentity/cache/version.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@
77
package cache
88

99
//lint:ignore U1000 used by automation
10-
const version = "v0.3.0"
10+
const version = "v0.3.1"

‎sdk/azidentity/device_code_credential.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ type DeviceCodeCredentialOptions struct {
3737
// ClientID is the ID of the application to which users will authenticate. When not set, users
3838
// will authenticate to an Azure development application, which isn't recommended for production
3939
// scenarios. In production, developers should instead register their applications and assign
40-
// appropriate roles. See https://aka.ms/identity/AppRegistrationAndRoleAssignment for more
40+
// appropriate roles. See https://aka.ms/azsdk/identity/AppRegistrationAndRoleAssignment for more
4141
// information.
4242
ClientID string
4343

‎sdk/azidentity/go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ go 1.18
44

55
require (
66
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0
7-
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.2.3
7+
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.3.0
88
github.com/Azure/azure-sdk-for-go/sdk/internal v1.10.0
99
github.com/AzureAD/microsoft-authentication-library-for-go v1.2.2
1010
github.com/golang-jwt/jwt/v5 v5.2.1

‎sdk/azidentity/go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0 h1:GJHeeA2N7xrG3q30L2UXDyuWRzDM900/65j70wcM4Ww=
22
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.13.0/go.mod h1:l38EPgmsp71HHLq9j7De57JcKOWPyhrsW1Awm1JS6K0=
3-
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.2.3 h1:BAUsn6/icUFtvUalVwCO0+hSF7qgU9DwwcEfCvtILtw=
4-
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.2.3/go.mod h1:QlAsNp4gk9zLD2wiZIvIuv699ynpZ2Tq2ZBp+6MrSEw=
3+
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.3.0 h1:+m0M/LFxN43KvULkDNfdXOgrjtg6UYJPFBJyuEcRCAw=
4+
github.com/Azure/azure-sdk-for-go/sdk/azidentity/cache v0.3.0/go.mod h1:PwOyop78lveYMRs6oCxjiVyBdyCgIYH6XHIVZO9/SFQ=
55
github.com/Azure/azure-sdk-for-go/sdk/internal v1.10.0 h1:ywEEhmNahHBihViHepv3xPBn1663uRv2t2q/ESv9seY=
66
github.com/Azure/azure-sdk-for-go/sdk/internal v1.10.0/go.mod h1:iZDifYGJTIgIIkYRNWPENUnqx6bJ2xnSDFI2tjwZNuY=
77
github.com/AzureAD/microsoft-authentication-extensions-for-go/cache v0.1.1 h1:WJTmL004Abzc5wDB5VtZG2PJk5ndYDgVacGqfirKxjM=

‎sdk/azidentity/img/mermaidjs/DefaultAzureCredentialAuthFlow.md

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,29 @@
11
```mermaid
22
%% STEPS TO GENERATE IMAGE
33
%% =======================
4-
%% 1. Install mermaid CLI (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md)
4+
%% 1. Install mermaid CLI v10.9.1 (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md):
5+
%% npm i -g @mermaid-js/[email protected]
56
%% 2. Run command: mmdc -i DefaultAzureCredentialAuthFlow.md -o DefaultAzureCredentialAuthFlow.svg
67
7-
flowchart LR;
8-
A(Environment):::deployed ==> B(Workload Identity):::deployed ==> C(Managed Identity):::deployed ==> D(Azure CLI):::developer ==> E(Azure Developer CLI):::developer;
8+
%%{
9+
init: {
10+
'theme': 'base',
11+
'themeVariables': {
12+
'tertiaryBorderColor': '#fff',
13+
'tertiaryColor': '#fff'
14+
}
15+
}
16+
}%%
917
18+
flowchart LR;
1019
subgraph CREDENTIAL TYPES;
1120
direction LR;
12-
Deployed(Deployed service):::deployed --- Developer(Developer):::developer;
21+
Deployed(Deployed service):::deployed ~~~ Developer(Developer):::developer;
22+
end;
1323
14-
%% Hide links between boxes in the legend by setting width to 0. The integers after "linkStyle" represent link indices.
15-
linkStyle 4 stroke-width:0px;
24+
subgraph CREDENTIALS;
25+
direction LR;
26+
A(Environment):::deployed --> B(Workload Identity):::deployed --> C(Managed Identity):::deployed --> D(Azure CLI):::developer --> E(Azure Developer CLI):::developer;
1627
end;
1728
1829
%% Define styles for credential type boxes

0 commit comments

Comments
 (0)