generated from ministryofjustice/template-repository
-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
LGA-3086: Setup CI/CD #3
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2f7dba7
to
f7c3582
Compare
9b8e722
to
ee285f3
Compare
ee285f3
to
ec55a4f
Compare
c23bc5e
to
ddfd7e4
Compare
ddfd7e4
to
6c265b2
Compare
helm_deploy/laa-access-civil-legal-aid/templates/deployment.yaml
Outdated
Show resolved
Hide resolved
a8b4f64
to
ff53d4b
Compare
ff53d4b
to
9024ed5
Compare
179aa37
to
daebd24
Compare
Pheonnexx
approved these changes
May 22, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM good catch by Jav on somethings!
digitalali-moj
approved these changes
May 22, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Legend.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this pull request do?
Adds a CI/CD Pipeline using GitHub Actions' workflows
Important
All ingress hosts are only accessible whilst using the Global Protect VPN.
Feature Branch Pipeline
Static Analysis:
Testing:
Build:
docker build
creating an image with a tag relating to the latest commit sha of the branch.docker push
pushing the image to the AWS Elastic Container Registry in thelaa-access-civil-legal-aid-production
namespace.Deploy Dev:
[branch name]-access-cla.cloud-platform.service.gov.uk
Deploy UAT:
laa-get-access
Release to staging & production
Static Analysis
Testing
Build
Deploy Staging:
laa-get-access
Deploy Production:
laa-get-access
Clean up dev release
Once a Pull Request has been closed, or merged in main, the 'Clean up' workflow will run. This uninstalls the feature's Helm chart from the dev namespace, removing all deployments, ingresses and pods.
Any other changes that would benefit highlighting?
Adds a Flask app which returns the text "Hello, World!" on the "/" route to test the deployment and ingress works as expected.
Checklist