Skip to content

fix: DTOSS-10762 CD Pipeline conditions improvement #1587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
patrickmoore-nc merged 4 commits into from
Sep 3, 2025
Merged

fix: DTOSS-10762 CD Pipeline conditions improvement #1587

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
910ffcf
fix: CD Pipeline conditions improvement
patrickmoore-nc Sep 3, 2025
4a5206c
Update cd-infrastructure-core-common.yaml
patrickmoore-nc Sep 3, 2025
6337f2c
Update cd-infrastructure-core-common.yaml
patrickmoore-nc Sep 3, 2025
d6a7785
Update cd-infrastructure-core-common.yaml
patrickmoore-nc Sep 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 13 additions & 25 deletions .azuredevops/templates/cd-infrastructure-core-common.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,7 @@ stages:
# Only required until commit hashes are passed in by calling pipeline, except for the
# PreProd pipeline which needs to retag images to move them from the Dev ACR to the Prod ACR
displayName: ACR re-tag
condition: |
and(
eq(variables['Build.Reason'], 'Manual'),
in('${{ parameters.retagImages }}', 'true')
)
condition: in('${{ parameters.retagImages }}', 'true')
pool:
name: ${{ parameters.hostPoolName }}
jobs:
Expand All @@ -48,12 +44,11 @@ stages:
serviceConnection: ${{ parameters.serviceConnection }}

- stage: terraform_deploy
displayName: Terraform Deploy
condition: |
and(
eq(variables['Build.Reason'], 'Manual'),
in(dependencies.re_tag_stage.result, 'Succeeded', 'Skipped')
)
${{ if eq(parameters.terraformActions, 'Apply') }}:
displayName: Terraform Deploy
${{ else }}:
displayName: Terraform Plan
condition: in(dependencies.re_tag_stage.result, 'Succeeded', 'Skipped')
variables:
tfVarsFile: ${{ parameters.tfVarsFile }}
pool:
Expand All @@ -75,7 +70,7 @@ stages:
condition: |
and(
eq(dependencies.init_and_plan.outputs['TerraformPlan.changesPresent'], 'true'),
in('${{ parameters.terraformActions }}', 'Apply')
eq('${{ parameters.terraformActions }}', 'Apply')
)
strategy:
runOnce:
Expand All @@ -88,12 +83,9 @@ stages:
- stage: db_changes_stage
displayName: Database changes
condition: |
and(
eq(variables['Build.Reason'], 'Manual'),
or(
in(dependencies.re_tag_stage.result, 'Succeeded', 'Skipped'),
eq(dependencies.terraform_deploy.result, 'Succeeded')
)
or(
eq(dependencies.re_tag_stage.result, 'Succeeded'),
eq(dependencies.terraform_deploy.jobs.terraform_apply.result, 'Succeeded')
)
pool:
name: ${{ parameters.hostPoolName }}
Expand All @@ -113,13 +105,9 @@ stages:
- stage: restart_functions_stage
displayName: Restart Function Apps
condition: |
and(
eq(variables['Build.Reason'], 'Manual'),
or(
in(dependencies.re_tag_stage.result, 'Succeeded', 'Skipped'),
eq(dependencies.terraform_deploy.result, 'Succeeded'),
in(dependencies.db_changes_stage.result, 'Succeeded', 'Skipped')
)
or(
eq(dependencies.re_tag_stage.result, 'Succeeded'),
eq(dependencies.terraform_deploy.jobs.terraform_apply.result, 'Succeeded')
)
pool:
name: ${{ parameters.hostPoolName }}
Expand Down
Loading