Skip to content

Commit

Permalink
ci.yml compatibility updates, ensure dev_requirements is present and …
Browse files Browse the repository at this point in the history
…updated, disable a few checks that won't work with legacy code
  • Loading branch information
scbedd committed Apr 12, 2021
1 parent df73fa8 commit 1051a43
Show file tree
Hide file tree
Showing 281 changed files with 937 additions and 538 deletions.
9 changes: 0 additions & 9 deletions eng/pipelines/templates/jobs/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,15 +74,6 @@ jobs:
vmImage: MMSUbuntu18.04

steps:
- template: /eng/common/pipelines/templates/steps/verify-links.yml
parameters:
${{ if eq(variables['Build.Reason'], 'PullRequest') }}:
Directory: ''
Urls: (eng/common/scripts/get-markdown-files-from-changed-files.ps1)
${{ if ne(variables['Build.Reason'], 'PullRequest') }}:
Directory: sdk/${{ parameters.ServiceDirectory }}
CheckLinkGuidance: $true

- template: ../steps/analyze.yml
parameters:
ServiceDirectory: ${{ parameters.ServiceDirectory }}
Expand Down
6 changes: 0 additions & 6 deletions eng/pipelines/templates/stages/archetype-python-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,6 @@ stages:
ServiceDirectory: "template"
TestPipeline: ${{ parameters.TestPipeline }}

- ${{if ne(artifact.skipVerifyChangeLog, 'true')}}:
- template: /eng/common/pipelines/templates/steps/verify-changelog.yml
parameters:
PackageName: ${{artifact.name}}
ServiceName: ${{parameters.ServiceDirectory}}
ForRelease: true
- pwsh: |
Get-ChildItem -Recurse $(Pipeline.Workspace)/${{parameters.ArtifactName}}/${{artifact.name}}
workingDirectory: $(Pipeline.Workspace)
Expand Down
8 changes: 0 additions & 8 deletions eng/pipelines/templates/steps/analyze.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,6 @@ steps:
ServiceDirectory: "template"
TestPipeline: ${{ parameters.TestPipeline }}

- ${{ each artifact in parameters.Artifacts }}:
- ${{if ne(artifact.skipVerifyChangeLog, 'true')}}:
- template: /eng/common/pipelines/templates/steps/verify-changelog.yml
parameters:
PackageName: ${{artifact.name}}
ServiceName: ${{parameters.ServiceDirectory}}
ForRelease: false

# Using --always-succeed so as not to block the build. Once package
# target is based on data available per-package the --always-succeed should
# be removed so this script can help enforce correct practices
Expand Down
4 changes: 0 additions & 4 deletions eng/pipelines/templates/steps/build-artifacts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,3 @@ steps:
ArtifactPath: '$(Build.SourcesDirectory)/_docs'
CustomCondition: ${{ parameters.BuildDocs }}
ArtifactName: 'documentation'

- template: /eng/common/pipelines/templates/steps/create-apireview.yml
parameters:
Artifacts: ${{ parameters.Artifacts }}
1 change: 1 addition & 0 deletions sdk/advisor/azure-mgmt-advisor/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/advisor/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,5 @@ extends:
parameters:
ServiceDirectory: advisor
Artifacts:
- name: azure_mgmt_advisor
safeName: azuremgmtadvisor
- name: azure-mgmt-advisor
safeName: azuremgmtadvisor
2 changes: 2 additions & 0 deletions sdk/aks/azure-mgmt-devspaces/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/aks/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,5 @@ extends:
parameters:
ServiceDirectory: aks
Artifacts:
- name: azure_mgmt_devspaces
safeName: azuremgmtdevspaces
- name: azure-mgmt-devspaces
safeName: azuremgmtdevspaces
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/alertsmanagement/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,5 @@ extends:
parameters:
ServiceDirectory: alertsmanagement
Artifacts:
- name: azure_mgmt_alertsmanagement
safeName: azuremgmtalertsmanagement
- name: azure-mgmt-alertsmanagement
safeName: azuremgmtalertsmanagement
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/apimanagement/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,5 @@ extends:
parameters:
ServiceDirectory: apimanagement
Artifacts:
- name: azure_mgmt_apimanagement
safeName: azuremgmtapimanagement
- name: azure-mgmt-apimanagement
safeName: azuremgmtapimanagement
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
../../core/azure-core
-e ../../../tools/azure-devtools
-e ../../../tools/azure-sdk-tools
../../core/azure-core
-e ../../identity/azure-identity
aiohttp>=3.0; python_version >= '3.5'
aiodns>=2.0; python_version >= '3.5'
msrest>=0.6.10
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@

aiohttp>=3.0; python_version >= '3.5'
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
11 changes: 6 additions & 5 deletions sdk/appconfiguration/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -19,6 +19,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -35,7 +36,7 @@ extends:
parameters:
ServiceDirectory: appconfiguration
Artifacts:
- name: azure_appconfiguration
- name: azure-appconfiguration
safeName: azureappconfiguration
- name: azure_mgmt_appconfiguration
safeName: azuremgmtappconfiguration
- name: azure-mgmt-appconfiguration
safeName: azuremgmtappconfiguration
Original file line number Diff line number Diff line change
@@ -1 +1,4 @@
-e ../../../tools/azure-sdk-tools
msrestazure
../../core/azure-core
-e ../../../tools/azure-devtools
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/applicationinsights/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,7 +30,7 @@ extends:
parameters:
ServiceDirectory: applicationinsights
Artifacts:
- name: azure_mgmt_applicationinsights
- name: azure-mgmt-applicationinsights
safeName: azuremgmtapplicationinsights
- name: azure_applicationinsights
- name: azure-applicationinsights
safeName: azureapplicationinsights
2 changes: 2 additions & 0 deletions sdk/appplatform/azure-mgmt-appplatform/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
4 changes: 3 additions & 1 deletion sdk/appplatform/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +17,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +31,5 @@ extends:
parameters:
ServiceDirectory: appplatform
Artifacts:
- name: azure_mgmt_appplatform
- name: azure-mgmt-appplatform
safeName: azuremgmtappplatform
1 change: 1 addition & 0 deletions sdk/appservice/azure-mgmt-web/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
9 changes: 5 additions & 4 deletions sdk/appservice/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,5 @@ extends:
parameters:
ServiceDirectory: appservice
Artifacts:
- name: azure_mgmt_web
safeName: azuremgmtweb
- name: azure-mgmt-web
safeName: azuremgmtweb
2 changes: 2 additions & 0 deletions sdk/attestation/azure-mgmt-attestation/dev_requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-e ../../../tools/azure-sdk-tools
-e ../../../tools/azure-devtools
11 changes: 7 additions & 4 deletions sdk/attestation/ci.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# DO NOT EDIT THIS FILE
# This file is generated automatically and any changes will be lost.
# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
branches:
include:
- master
- main
- hotfix/*
- release/*
- restapi*
Expand All @@ -16,6 +16,7 @@ pr:
branches:
include:
- master
- main
- feature/*
- hotfix/*
- release/*
Expand All @@ -29,5 +30,7 @@ extends:
parameters:
ServiceDirectory: attestation
Artifacts:
- name: azure_mgmt_attestation
safeName: azuremgmtattestation
- name: azure-mgmt-attestation
safeName: azuremgmtattestation
- name: azure-security-attestation
safeName: azuresecurityattestation
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
-e ../../../tools/azure-devtools
-e ../../../tools/azure-sdk-tools
azure-identity
Loading

0 comments on commit 1051a43

Please sign in to comment.