Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/main' into release-app-Microsoft…
Browse files Browse the repository at this point in the history
….App-2024-10-02-preview
  • Loading branch information
jijohn14 committed Oct 22, 2024
2 parents 37712e1 + de278c0 commit 3801d72
Show file tree
Hide file tree
Showing 20,020 changed files with 2,827,459 additions and 82,176 deletions.
The diff you're trying to view is too large. We only load the first 3000 changed files.
1 change: 1 addition & 0 deletions .azure-pipelines/azure-pipelines-data-container.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ schedules:
always: true

pool:
name: azsdk-pool-mms-ubuntu-2204-general
vmImage: 'Ubuntu-22.04'

variables:
Expand Down
38 changes: 21 additions & 17 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
/specification/applicationinsights/data-plane/Monitor.Exporters/ @ramthi @trask @hectorhdzg @lzchen @Azure/api-stewardship-board

# PRLabel: %Container Apps
/specification/app/ @jijohn14 @Juliehzl
/specification/app/ @Juliehzl

# PRLabel: %Code Signing
/specification/codesigning/data-plane @Azure/api-stewardship-board
Expand All @@ -62,7 +62,7 @@
/specification/billing/ @wilcobmsft @asarkar84

# PRLabel: %Network - CDN
/specification/cdn/ @jorinmejia @yunhemsft @jessicl-ms @rrahulms @t-bzhan @ChenglongLiu
/specification/cdn/ @jorinmejia @yunhemsft @jessicl-ms @rrahulms @ChenglongLiu @Ptnan7

# PRLabel: %Cognitive Services
/specification/cognitiveservices/ @felixwa @yangyuan
Expand Down Expand Up @@ -111,7 +111,7 @@
/specification/deviceregistry/ @marcodalessandro @rohankhandelwal @riteshrao

# PRLabel: %Device Update
/specification/deviceupdate/data-plane/ @mikekistler @Azure/api-stewardship-board
/specification/deviceupdate/data-plane/ @Azure/api-stewardship-board

/specification/documentdb/ @dmakwana

Expand Down Expand Up @@ -142,7 +142,7 @@
/specification/keyvault/ @heaths @randallilama @jlichwa

# PRLabel: %Load Test Service
/specification/loadtestservice/data-plane/ @mikekistler @Azure/api-stewardship-board
/specification/loadtestservice/data-plane/ @Azure/api-stewardship-board

# PRLabel: %Logic App
/specification/logic/ @pankajsn @tonytang-microsoft-com
Expand All @@ -162,7 +162,7 @@
/specification/monitor/ @gucalder

# PRLabel: %Network
/specification/network/ @GuptaVertika
/specification/network/ @kamboj-prjwl

# PRLabel: %Notification Hub
/specification/notificationhubs/ @amolr @smithab
Expand All @@ -182,7 +182,7 @@
/specification/powerbidedicated/ @tarostok

# PRLabel: %Purview
/specification/purview/data-plane @mikekistler @Azure/api-stewardship-board
/specification/purview/data-plane @Azure/api-stewardship-board

# PRLabel: %PostgreSQL
/specification/postgresql/** @Azure/azure-sdk-write-postgresql
Expand Down Expand Up @@ -230,7 +230,7 @@
/specification/sql/ @jamestao @ericshape @jeremyfrosti

# PRLabel: %Storage
/specification/storage/ @jasonyang-msft
/specification/storage/resource-manager/ @blueww @yifanz7
/specification/storage/data-plane/ @seanmcc-msft @Azure/api-stewardship-board

# PRLabel: %Import Export
Expand Down Expand Up @@ -260,6 +260,9 @@
# PRLabel: %Web Apps
/specification/web/ @naveedaz @Azure/azure-app-service-control-plane

# PRLabel: %Video Indexer
/specification/vi/ @dazilb @tshoham @omerhaimov @bermanraz @Einav94

# PRLabel: %AzureML - Compute Instance
/specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/**/machineLearningServices.json @Azure/aml-compute-instance

Expand All @@ -275,13 +278,14 @@
###########
# Eng Sys
###########
/.gitattributes @weshaggard @mikeharder @konrad-jamrozik
/.gitignore @weshaggard @mikeharder @konrad-jamrozik
/.prettierrc.json @weshaggard @mikeharder @konrad-jamrozik
/package-lock.json @weshaggard @mikeharder @konrad-jamrozik
/package.json @weshaggard @mikeharder @konrad-jamrozik
/tsconfig.json @weshaggard @mikeharder @konrad-jamrozik
/.azure-pipelines/ @weshaggard @mikeharder @konrad-jamrozik @benbp
/.github/ @weshaggard @mikeharder @konrad-jamrozik @benbp
/eng/ @weshaggard @mikeharder @konrad-jamrozik @benbp
/scripts/ @weshaggard @mikeharder @konrad-jamrozik
/.gitattributes @weshaggard @mikeharder
/.gitignore @weshaggard @mikeharder
/.prettierrc.json @weshaggard @mikeharder
/package-lock.json @weshaggard @mikeharder
/package.json @weshaggard @mikeharder
/tsconfig.json @weshaggard @mikeharder
/.azure-pipelines/ @weshaggard @mikeharder @benbp
/.github/ @weshaggard @mikeharder @benbp
/eng/ @weshaggard @mikeharder @benbp
/scripts/ @weshaggard @mikeharder
/.github/CODEOWNERS @Azure/azure-sdk-eng
3 changes: 2 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE/control_plane_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ What's the purpose of this PR? Check the specific option that applies. This is *
- [ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to [OpenAPI specs PR creation guidance](https://aka.ms/azsdkdocs/createopenapispec)).
- [ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
- [ ] Update existing version to fix OpenAPI spec quality issues in S360.
- [ ] Convert existing [OpenAPI spec to TypeSpec spec](https://aka.ms/typespec/conversion) (do not combine this with implementing changes for a new API version).
- [ ] Other, please clarify:
- _edit this with your clarification_

Expand Down Expand Up @@ -51,7 +52,7 @@ If one or multiple validation error/warning suppression(s) is detected in your P
## Getting help

- First, please carefully read through this PR description, from top to bottom. Please fill out the `Purpose of this PR` and `Due diligence checklist`.
- If you don't have permissions to remove or add labels to the PR, request `write access` per [aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories](https://aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories")
- If you don't have permissions to remove or add labels to the PR, request `write access` per [aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories](https://aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories)
- To understand what you must do next to merge this PR, see the `Next Steps to Merge` comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
- For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
Expand Down
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE/data_plane_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ The [Azure API Review Board](https://aka.ms/azsdk/onboarding/restapischedule) is
## Getting help

- First, please carefully read through this PR description, from top to bottom.
- If you don't have permissions to remove or add labels to the PR, request `write access` per [aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories](https://aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories")
- If you don't have permissions to remove or add labels to the PR, request `write access` per [aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories](https://aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories)
- To understand what you must do next to merge this PR, see the `Next Steps to Merge` comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
- For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
Expand Down
10 changes: 10 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
version: 2
updates:
- package-ecosystem: "npm"
directory: "/"
schedule:
interval: "daily"
groups:
typespec:
patterns:
- "*typespec*"
12 changes: 0 additions & 12 deletions .github/policies/resourceManagement.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2530,18 +2530,6 @@ configuration:
- removeLabel:
label: no-recent-activity
description:
- if:
- payloadType: Issue_Comment
- and:
- isPullRequest
then:
- if:
- commentContains:
pattern: "^/pr RequestMerge$"
isRegex: true
then:
- addLabel:
label: MergeRequested
- if:
- payloadType: Pull_Request
then:
Expand Down
29 changes: 28 additions & 1 deletion .github/workflows/_reusable-eng-tools-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,17 @@ on:
description: Name of package under eng/tools
required: true
type: string
sparse-checkout-paths:
description: Paths for sparse checkout
type: string
lint:
description: Run 'npm run lint' if true
required: false
type: boolean
prettier:
description: Run 'npm run prettier' if true
required: false
type: boolean

jobs:
test:
Expand All @@ -24,9 +35,15 @@ jobs:
runs-on: ${{ matrix.os }}

steps:
- if: runner.os == 'Windows'
run: git config --global core.longpaths true
shell: pwsh

- uses: actions/checkout@v4
with:
sparse-checkout: eng
sparse-checkout: |
eng
${{ inputs.sparse-checkout-paths }}
- name: Use Node ${{ matrix.node-version }}.x
uses: actions/setup-node@v4
Expand All @@ -44,6 +61,16 @@ jobs:
shell: pwsh
working-directory: ./eng/tools/${{ inputs.package }}

- run: npm run lint
if: inputs.lint == true
shell: pwsh
working-directory: ./eng/tools/${{ inputs.package }}

- run: npm run prettier
if: inputs.prettier == true
shell: pwsh
working-directory: ./eng/tools/${{ inputs.package }}

- run: npm run test:ci
shell: pwsh
working-directory: ./eng/tools/${{ inputs.package }}
Expand Down
45 changes: 45 additions & 0 deletions .github/workflows/protected-files.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
name: Protected Files

on: pull_request

env:
# Users allowed to edit protected files without failing check
user-allowed: ${{ github.event.pull_request.user.login == 'azure-sdk' || github.event.pull_request.user.login == 'dependabot[bot]' }}

jobs:
protected-files:
name: Protected Files

runs-on: ubuntu-latest

steps:
# Since check is required, the job must pass instead of being skipped
- name: User allowed
if: ${{ env.user-allowed == 'true' }}
run: echo "Account '${{ github.event.pull_request.user.login }}' is allowed to update protected files"

- uses: actions/checkout@v4
if: ${{ env.user-allowed != 'true' }}
with:
# Required since "HEAD^" is passed to Get-ChangedFiles
fetch-depth: 2

- name: Detect changes to protected files
if: ${{ env.user-allowed != 'true' }}
run: |
. eng/scripts/ChangedFiles-Functions.ps1
$protectedFiles = @("package.json", "package-lock.json")
$changedFiles = @(Get-ChangedFiles -baseCommitish HEAD^ -targetCommitish HEAD -diffFilter "")
$matchedFiles = @($protectedFiles | Where-Object { $changedFiles -contains $_})
if ($matchedFiles.Count -gt 0) {
foreach ($file in $matchedFiles) {
Write-Output "::error file=$file::File '$file' should only be updated by the Azure SDK team. If intentional, the PR may be merged by the Azure SDK team via bypassing the branch protections."
}
exit 1
}
else {
Write-Output "No changes to protected files: [$($protectedFiles -join ', ')]"
}
shell: pwsh
27 changes: 27 additions & 0 deletions .github/workflows/specs-model-test.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
name: Specs Model - Test

on:
push:
branches:
- main
- typespec-next
pull_request:
paths:
- package-lock.json
- package.json
- tsconfig.json
- .github/workflows/_reusable-eng-tools-test.yaml
- .github/workflows/specs-model-test.yaml
- eng/tools/package.json
- eng/tools/tsconfig.json
- eng/tools/specs-model/**
workflow_dispatch:

jobs:
specsModel:
name: Specs Model
uses: ./.github/workflows/_reusable-eng-tools-test.yaml
with:
package: specs-model
lint: true
prettier: true
31 changes: 31 additions & 0 deletions .github/workflows/tsp-client-test.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
name: tsp-client - Test

on:
push:
branches:
- main
pull_request:
branches:
- main
paths:
- package-lock.json
- package.json
- tsconfig.json
- .github/workflows/_reusable-eng-tools-test.yaml
- .github/workflows/tsp-client-test.yaml
- eng/tools/package.json
- eng/tools/tsconfig.json
- eng/tools/tsp-client-tests/**
- specification/common-types
- specification/keyvault
- specification/sphere

jobs:
tsp-client:
uses: ./.github/workflows/_reusable-eng-tools-test.yaml
with:
package: tsp-client-tests
sparse-checkout-paths: |
specification/common-types
specification/keyvault
specification/sphere
23 changes: 23 additions & 0 deletions .github/workflows/typespec-requirement-test.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: TypeSpec Requirement - Test

on:
push:
branches:
- main
pull_request:
paths:
- package-lock.json
- package.json
- tsconfig.json
- .github/workflows/_reusable-eng-tools-test.yaml
- .github/workflows/typespec-requirement-test.yaml
- eng/scripts/**
- eng/tools/package.json
- eng/tools/tsconfig.json
- eng/tools/typespec-requirement/**

jobs:
typespec-requirement:
uses: ./.github/workflows/_reusable-eng-tools-test.yaml
with:
package: typespec-requirement
36 changes: 36 additions & 0 deletions .github/workflows/typespec-requirement.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
name: TypeSpec Requirement

on: pull_request

jobs:
TypeSpec-Requirement:
name: TypeSpec Requirement
strategy:
fail-fast: false
matrix:
spec-type: [data-plane, resource-manager]

runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4
with:
# Required since "HEAD^" is passed to TypeSpec-Requirement.ps1
fetch-depth: 2

- name: Use Node 20.0
uses: actions/setup-node@v4
with:
node-version: 20.0

- run: npm ci

- run: npm ls -a
continue-on-error: true

- run: |
eng/scripts/TypeSpec-Requirement.ps1 `
-BaseCommitish HEAD^ `
-TargetCommitish HEAD `
-SpecType ${{ matrix.spec-type }}
shell: pwsh
Loading

0 comments on commit 3801d72

Please sign in to comment.