Skip to content

Commit

Permalink
'none' as additional option for deployment_type in CI pipelines (micr…
Browse files Browse the repository at this point in the history
…osoft#190)

Added `"none"` option for `deployment_type` in platform CI pipeline,
including the three versions: ADO (no change needed), Github and
Jenkins. If option is set to "none", then deployment is skipped.

Closes microsoft#189
  • Loading branch information
mariamedp authored Jul 25, 2024
1 parent bf3d980 commit 9dcdfe4
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .azure-pipelines/platform_ci_dev_pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ parameters:
- name: use_case_base_path
displayName: "type of model to execute"
- name: deployment_type
displayName: "determine type of deployment - aks, aml, docker, webapp"
displayName: "determine type of deployment - aks, aml, docker, webapp, none"
- name: registry_details
displayName: "determine the variable containing acr details"
- name: RESOURCE_GROUP_NAME
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/platform_ci_dev_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ on:
default: "web_classification"
deployment_type:
type: string
description: "Determine type of deployment - aml, aks, docker, webapp"
description: "Determine type of deployment - aml, aks, docker, webapp, none"
required: true
secrets:
azure_credentials:
Expand Down Expand Up @@ -165,6 +165,7 @@ jobs:
# Execute platform_cd_dev_workflow flow for deployment of flows
#=====================================
deploy-flow:
if: ${{ inputs.deployment_type != 'none' }}
uses: ./.github/workflows/platform_cd_dev_workflow.yml
needs: flow-experiment-and_evaluation
with:
Expand Down
5 changes: 4 additions & 1 deletion .jenkins/pipelines/platform_ci_dev
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ pipeline {
parameters {
string(name: 'use_case_base_path', defaultValue: 'named_entity_recognition', description: 'The flow use-case to execute')
string(name: 'env_name', defaultValue: 'dev', description: 'Execution Environment')
string(name: 'deployment_type', defaultValue: 'webapp', description: 'Determine type of deployment - aml, aks, webapp')
string(name: 'deployment_type', defaultValue: 'webapp', description: 'Determine type of deployment - aml, aks, webapp, none')
string(name: 'rg_name', description: 'Azure Resource Group Name')
string(name: 'ws_name', description: 'AzureML Workspace Name')
string(name: 'kv_name', description: 'Azure Key Vault Name')
Expand Down Expand Up @@ -173,6 +173,9 @@ load_dotenv('.env')

// Execute Platform CD Pipeline to deploy flow to specified deployment type
stage('Execute Platform CD Pipeline to deploy flow') {
when {
expression { params.deployment_type != 'none' }
}
steps {
script {
build job: 'platform_cd_dev', parameters: [
Expand Down

0 comments on commit 9dcdfe4

Please sign in to comment.