Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
f9ab260
AMB-783 Created pull request
matthewaclark21 Jul 20, 2021
cbdda9b
AMB-783 Created pull request
matthewaclark21 Jul 20, 2021
362ee3e
AMB-783 Created separate _ping and _status endpoints logic
matthewaclark21 Jul 20, 2021
7e44d7d
AMB-783 re-arranged logic for status monitoring to make 503 implemnta…
matthewaclark21 Jul 20, 2021
92e90ba
AMB-783 added message for 503 status
matthewaclark21 Jul 20, 2021
f5f3cdf
AMB-783 fixed wrong syntax used (else)
matthewaclark21 Jul 20, 2021
142044c
AMB-783 Refactor set status step
Valentinolucas Jul 21, 2021
c632b5b
AMB-783 refactor monitoring steps
Valentinolucas Jul 21, 2021
b361929
AMB-783 fix if statement end of file error
Valentinolucas Jul 21, 2021
c077603
AMB-783 Removed commented out tests ready for merge
matthewaclark21 Jul 22, 2021
00d40dd
AMB-783 corrected grammar in wording
matthewaclark21 Jul 22, 2021
4e67ccb
Merge pull request #337 from NHSDigital/AMB-783-fix-monitoring-by-def…
matthewaclark21 Jul 22, 2021
28dfb5c
APM-2409 Add manual_approval_env overwrite
acarriedev Jul 23, 2021
9332bbc
APM-2409 Allow manual approval on prs
acarriedev Jul 23, 2021
003a96f
APM-2409 Add manual_approval_env to per env scoping
acarriedev Jul 23, 2021
e5d7a26
APM-2409 Remove manual_approval_env from per env scoping
acarriedev Jul 23, 2021
f167b0e
APM-2409 remove manual_approvaal_env from params
acarriedev Jul 23, 2021
fb858c9
APM-2409 Add back in
acarriedev Jul 23, 2021
6268238
APM-2409 Add default manual approval to params
acarriedev Jul 23, 2021
bcd2159
Merge pull request #339 from NHSDigital/apm-2409-add-manual-approval-…
acarriedev Jul 23, 2021
31ba25f
APM-2409 Accept changes
acarriedev Jul 23, 2021
1146c0a
APM-2409 Add manual_approval_env to not in list
acarriedev Jul 23, 2021
02f623c
APM-2409 Use flag for prod manual approval
acarriedev Jul 23, 2021
510ac18
APM-2409 Add prod approval to not in list
acarriedev Jul 23, 2021
f01c37c
APM-2409 Add time out to manual approval
acarriedev Jul 26, 2021
b329196
APM-2409 use manual-approval-prod env for prod
acarriedev Jul 26, 2021
fbddb6b
Merge pull request #338 from NHSDigital/apm-2409-allow-custom-manual-…
acarriedev Jul 26, 2021
b540d5a
Revert "AMB-783 Refactor monitoring by default feature"
Valentinolucas Jul 26, 2021
eb3be50
Merge pull request #342 from NHSDigital/revert-337-AMB-783-fix-monito…
acarriedev Jul 26, 2021
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
5 changes: 5 additions & 0 deletions azure/common/apigee-deployment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,10 @@ parameters:
displayName: Requires manual approval
type: boolean
default: true
- name: manual_approval_env
displayName: Custom manual approval env overwrite
type: string
default: 'manual-approval'
- name: _scoped_pipeline_vars
type: object
displayName: All the pipeline parameters which support per-environment scoping.
Expand Down Expand Up @@ -166,6 +170,7 @@ extends:
- environment: manual-approval
stage_name: manual_approval
depends_on: []
manual_approval_prod: true
${{ each var in parameters._scoped_pipeline_vars }}:
${{ var }}: ${{ parameters[var] }}
- ${{ each apigee_deployment in parameters.apigee_deployments }}:
Expand Down
5 changes: 4 additions & 1 deletion azure/common/deploy-manual-approval.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,19 @@ parameters:
- name: stage_name
- name: depends_on
type: object
- name: manual_approval_env
type: string

stages:
- stage: ${{ parameters.stage_name }}
dependsOn: ${{ parameters.depends_on }}
jobs:
- deployment: deploy
displayName: Manual Approval Required
timeoutInMinutes: 60
pool:
vmImage: "Ubuntu-16.04"
environment: "manual-approval"
environment: ${{ parameters.manual_approval_env }}
strategy:
runOnce:
deploy:
Expand Down
2 changes: 2 additions & 0 deletions azure/common/deploy-stages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ parameters:
type: boolean
- name: deploy_template
- name: prod_requires_approval
- name: manual_approval_env
type: string


stages:
Expand Down
33 changes: 21 additions & 12 deletions azure/common/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ parameters:
values:
- internal-dev
- internal-dev-sandbox
- manual-approval
- name: apigee_organization
- name: aws_account
- name: stage_name
Expand Down Expand Up @@ -54,17 +55,25 @@ parameters:
- name: hosted_target_healthcheck_suffix
type: string
default: ''

- name: manual_approval_env
type: string

stages:
- template: ./deploy-stage.yml
parameters:
${{ if eq(parameters.proxy_path, 'live')}}:
fully_qualified_service_name: ${{ parameters.service_name }}-${{ parameters.pr_label }}
${{ if ne(parameters.proxy_path, 'live')}}:
fully_qualified_service_name: ${{ parameters.service_name }}-${{ parameters.pr_label }}-${{ parameters.proxy_path }}
service_base_path: ${{ parameters.service_base_path }}-${{ parameters.pr_label }}
notify: true
${{ each param in parameters }}:
${{ if notIn(param.key, 'deploy_review_sandbox', 'fully_qualified_service_name', 'service_base_path') }}:
${{ param.key }}: ${{ param.value }}
- ${{ if eq(parameters.environment, 'manual-approval') }}:
- template: ./deploy-manual-approval.yml
parameters:
stage_name: ${{ parameters.stage_name }}
depends_on: ${{ parameters.depends_on }}
manual_approval_env: ${{ parameters.manual_approval_env }}
- ${{ if not(eq(parameters.environment, 'manual-approval')) }}:
- template: ./deploy-stage.yml
parameters:
${{ if eq(parameters.proxy_path, 'live')}}:
fully_qualified_service_name: ${{ parameters.service_name }}-${{ parameters.pr_label }}
${{ if ne(parameters.proxy_path, 'live')}}:
fully_qualified_service_name: ${{ parameters.service_name }}-${{ parameters.pr_label }}-${{ parameters.proxy_path }}
service_base_path: ${{ parameters.service_base_path }}-${{ parameters.pr_label }}
notify: true
${{ each param in parameters }}:
${{ if notIn(param.key, 'deploy_review_sandbox', 'fully_qualified_service_name', 'service_base_path', 'manual_approval_env') }}:
${{ param.key }}: ${{ param.value }}
11 changes: 10 additions & 1 deletion azure/common/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,22 @@ parameters:
type: boolean
- name: enable_status_monitoring
type: boolean
- name: manual_approval_env
type: string
- name: manual_approval_prod
type: boolean
default: false

stages:
- ${{ if eq(parameters.environment, 'manual-approval') }}:
- template: ./deploy-manual-approval.yml
parameters:
stage_name: ${{ parameters.stage_name }}
depends_on: ${{ parameters.depends_on }}
${{ if eq(true, parameters.manual_approval_prod) }}:
manual_approval_env: manual-approval-prod
${{ if eq(false, parameters.manual_approval_prod) }}:
manual_approval_env: ${{ parameters.manual_approval_env }}
- ${{ if not(eq(parameters.environment, 'manual-approval')) }}:
- template: ./deploy-stage.yml
parameters:
Expand All @@ -69,5 +78,5 @@ stages:
${{ if ne('', parameters.fully_qualified_service_name) }}:
fully_qualified_service_name: ${{ parameters.fully_qualified_service_name }}
${{ each param in parameters }}:
${{ if notIn(param.key, 'fully_qualified_service_name') }}:
${{ if notIn(param.key, 'fully_qualified_service_name', 'manual_approval_env', 'manual_approval_prod') }}:
${{ param.key }}: ${{ param.value }}