Skip to content

Conversation

@saliceti
Copy link
Contributor

@saliceti saliceti commented Aug 15, 2025

Description

Allow deploy a review app for each pull request. It's an opt-in feature, triggered when the deploy label is added to the PR.
It triggers github actions and azure devops to create the review app.
The URL is displayed as a PR comment.
When the PR is merged or closed, the review app is deleted.

There are now detailed infrastructure docs in docs/infrastructure.

Jira link

https://nhsd-jira.digital.nhs.uk/browse/DTOSS-10199

Review notes

Try closing and opening the PR to create/delete the app
Check pipelines in Github actions and Azure devops
Validate the new documentation

After merge

Check the dev environment

@github-actions
Copy link

github-actions bot commented Aug 15, 2025

The review app is available at this URL:
https://pr-294.manage-breast-screening.non-live.screening.nhs.uk
You must authenticate with Entra ID

Also ask for confirmation to execute the first bicep
The GHtoADO managed identity is used by Github actions to trigger a
pipeline in Azure devops
Add the RBAC role assignment for the infra keyvault
Rename kvSecretUser to kvSecretsUser to avoid confusion and reflect the
role name accurately
Update gitleaksignore
Github action workflows and Azure devops pipelines
The review app builds when the PR is labelled with "deploy"
The workflows now use a Github environment for each separate environment
configuration, i.e. 'review' and 'dev' as opposed to 'azure'. Each one
contains the secrets required for each ADO pipeline.
The ADO pipeline now runs for only one environment as opposed to a list
of environments. And it accepts a pull request number for review apps.
Github action workflow and Azure devops pipeline
Triggered when the pull request is closed or merged. Tries to run
terraform destroy if the PR has a "deploy" label.
Ensure:
- 2 deployments don't run at the same time on the same environment
- deploy and destroy a review app don't run at the same time
Fails the Github action workflow explicitly if Azure devops pipeline fails
@josielsouzanordcloud josielsouzanordcloud merged commit bb77d49 into main Aug 19, 2025
34 of 37 checks passed
@josielsouzanordcloud josielsouzanordcloud deleted the DTOS-10199-review-apps-pipeline branch August 19, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants