Skip to content

APPT-1652 Point in time restore pipeline#1636

Draft
Dave-O-NHS wants to merge 91 commits intomainfrom
appt-1652/point-in-time-restore-pipeline
Draft

APPT-1652 Point in time restore pipeline#1636
Dave-O-NHS wants to merge 91 commits intomainfrom
appt-1652/point-in-time-restore-pipeline

Conversation

@Dave-O-NHS
Copy link
Copy Markdown
Contributor

@Dave-O-NHS Dave-O-NHS commented Apr 23, 2026 •
edited
Loading

Description

BST
image
GMT
image

Checklist:

  • My work is behind a feature toggle (if appropriate)
  • If my work is behind a feature toggle, I've added a full suite of tests for both the ON and OFF state
  • The ticket number is in the Pull Request title, with format "APPT-XXX: My Title Here"
  • I have ran npm tsc / lint (in the future these will be ran automatically)
  • My code generates no new .NET warnings (in the future these will be treated as errors)
  • If I've added a new Function, it is disabled in all but one of the terraform groups (e.g. http_functions)
  • If I've added a new Function, it has both unit and integration tests. Any request body validators have unit tests also
  • If I've made UI changes, I've added appropriate Playwright and Jest tests
  • If I've added/updated an end-point, I've added the appropriate annotations and tested the Swagger documentation reflects the change

ste-banks and others added 30 commits April 22, 2026 13:04
@ste-banks
Initial check in for PITR yml file
e50bb73
@ste-banks
Moving location of file
3faddea
@ste-banks
Stop app services for dev only - WIP
0bbe4af
@Dave-O-NHS
adding validation and PITR stages to new PITR pipeline
b513f64
@Dave-O-NHS
udating resourceGroup and accountName
b32c66f
@Dave-O-NHS
updating azure subscription
4010c50
@Dave-O-NHS
fixing pipeline triggering on every push
c7a6970
@Dave-O-NHS
changing subscription
07db01c
@Dave-O-NHS
fixing scriptType
50a8772
@Dave-O-NHS
fixing parameter name
e2a97ec
@ste-banks
Updating script to stop aggregator and audit container apps as well
7c2b360
@Dave-O-NHS
converting UK local time to UTC
2bfb5f9
@ste-banks
Adding subscriptionId
fa1b80c
@ste-banks
Fix typo
ef5b696
@ste-banks
Update to script to use az CLI
c58bf91
@ste-banks
Rename display name
75b49a3
@Dave-O-NHS
creating keyvault and allowing PITR to set active db name
d6bdf38
@ste-banks
Update to stopping container apps in script
6a7a341
@ste-banks
Update to set max-replicas to 1 - effectively stopping the container …
3eeb5f3 
…app - WIP
@pchoi-al
Merge David O changes and resolve conflict with APPT-2315 script
0d5b67c
@pchoi-al
Merge branch 'appt-1652/point-in-time-restore-pipeline' of https://gi…
4c6217a 
…thub.com/NHSDigital/nbs-appointments-management-service into appt-1652/point-in-time-restore-pipeline
@ste-banks
Deactivating revisions within the container app instead
9bca116
@Dave-O-NHS
fixing date issue
50f8b9a
@ste-banks
Query update
3e66ba5
@Dave-O-NHS
updating active db in key vault
2103a8e
@ste-banks
Stopping the booking and capacity extract jobs
197f184
@pchoi-al
Integrate RestartServices stage with Environment approval and start c…
508fbe2 
…ommands
@Dave-O-NHS
updating active db config file
1ade9d2
@ste-banks
Change to delete the container app jobs instead of stopping
2e2cdf8
@Dave-O-NHS
changing parameter name for enabling rbac on key vault
ea26559
@ste-banks ste-banks force-pushed the appt-1652/point-in-time-restore-pipeline branch from 66b937d to 294fb69 Compare April 29, 2026 15:09
pchoi-al and others added 16 commits April 29, 2026 20:53
@pchoi-al
Merge branch 'appt-1652/point-in-time-restore-pipeline' of https://gi…
d04440e 
…thub.com/NHSDigital/nbs-appointments-management-service into appt-1652/point-in-time-restore-pipeline
@Dave-O-NHS
loading endpoint and token from key vault
50b18dc
@Dave-O-NHS
container apps and data extracts load key vault
310a2cb
@ste-banks
Creating a new audit blob storage account and updating the container …
f328edb 
…app to point to it
@Dave-O-NHS
fixing configuration issues
5aac57e
@Dave-O-NHS
fix
360bf81
@pchoi-al
Added a stage to take active DB offline
9abe0cb
@ste-banks
Update restart step's depend on
97d2e2c
@ste-banks
Fix
7c159d3
@Dave-O-NHS
adding role assignment for new key vault
aeee702
@ste-banks
Formatting change - also adding error
5b1cf01
@ste-banks
Changing to single line command as pipeline is giving syntax errors a…
1326856 
…nd fixing other issues
@ste-banks
Update depends on
a65652f
@ste-banks
Removing spaces
87e4dc8
@Dave-O-NHS
terraform issues
251124d
@Dave-O-NHS
terraform issue
3d13444
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 1, 2026

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
58.2% Coverage on New Code
1.1% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Dave-O-NHS @ste-banks @pchoi-al