Modification in the Courtesy Push pipeline (#21147)

* Simplification of courtesy push pipeline

* Fixing order of parameters and removing redundant variables

Author: surajitshil-03

azure-pipelines.yml

@@ -17,14 +17,19 @@ resources:
     ref: refs/tags/release
 
 parameters:
+- name: task_deployment
+  displayName: Choose deployment options
+  type: string
+  default: 'Deploy task of Sprint'
+  values:
+  - 'Deploy all Tasks'
+  - 'Deploy task of Sprint'
+  - 'Deploy Hotfix'
 - name: task_name
-  displayName: Task Name (List of tasks to build (e.g. BashV3 [, AzureCLIV1]))
+  displayName: |
+    To 'Deploy Hotfix' provide task names (e.g. BashV3,AzureCLIV1,PowerShellV2) else leave to default
   type: string
-  default: TaskNameVN
-- name: deploy_all_tasks
-  displayName: Build all tasks for deployment
-  type: boolean
-  default: false
+  default: 'TaskNameVN'
 - name: enableCodeQL
   displayName: Enable CodeQL for run
   type: boolean
@@ -33,10 +38,6 @@ parameters:
   displayName: Flag to update LocalPackages buildconfig (for testing, this will be made  default later)
   type: boolean
   default: false # note: keep in sync with ci\ci-test-tasks\canary-tests-v2.yml
-- name: skip_tests
-  displayName: skipTests (Not for production use)
-  type: boolean
-  default: false
 - name: skip_publish
   displayName: skipPublish (Not for production use)
   type: boolean
@@ -52,7 +53,7 @@ variables:
 - name: taskName
   value: ${{ parameters.task_name }}
 - name: taskNameIsSet
-  value: ${{ not(eq(parameters.task_name, 'TaskNameVN')) }}
+  value: ${{ not(or(eq(parameters.task_deployment, 'Deploy all Tasks'), eq(parameters.task_deployment, 'Deploy task of Sprint'))) }}
 - name: runCodeQl
   value: ${{ eq(parameters.enableCodeQL, true) }}
 - name: system.debug
@@ -67,18 +68,13 @@ variables:
     value: '1'
   ${{ else }}:
     value: ''
-- name: tasksSkipTests
-  ${{ if eq(parameters.skip_tests, true) }}:
-    value: 'true'
-  ${{ else }}:
-    value: 'false'
 - name: tasksSkipPublish
   ${{ if eq(parameters.skip_publish, true) }}:
     value: 'true'
   ${{ else }}:
     value: 'false'
 - name: DEPLOY_ALL_TASKSVAR
-  ${{ if eq(parameters.deploy_all_tasks, true) }}:
+  ${{ if eq(parameters.task_deployment,'Deploy all Tasks') }}:
     value: 'true'
   ${{ else }}:
     value: 'false'
@@ -155,7 +151,7 @@ extends:
             ArtifactName: IndividualNuGetPackages
         steps:
         - template: /ci/publish-steps.yml@self
-
+        
       # Courtesy Push
       - job: courtesy_push
         displayName: Courtesy Push
@@ -220,7 +216,7 @@ extends:
         steps:
         - template: /ci/build-all-tasks.yml@self
           parameters:
-            deploy_all_tasks: ${{ parameters.deploy_all_tasks }}
+            deploy_all_tasks: ${{ eq(parameters.task_deployment, 'Deploy all Tasks') }}
 
       # All tasks on Linux
       - job: build_all_linux

ci/build-all-steps.yml

@@ -222,4 +222,4 @@ steps:
         eq(variables['Build.SourceBranch'], 'refs/heads/master')
       )
     continueOnError: true
-    displayName: Create Release
+    displayName: Create Release