-
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
feat: DBTP-913 - Run terraform plan in environment pipelines #110
Merged
Merged
Changes from all commits
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
04f785a
Initial role created with tests
antroy-madetech c0cb167
Added tags to role
antroy-madetech 3a76238
Added s3 bucket for artifact store
antroy-madetech 7b28fd3
Output test for s3 module
antroy-madetech a04a2d4
Working (dummy) codepipeline can now be created
antroy-madetech a5764a1
Added requires KMS perms
antroy-madetech 349e9df
Codebuild failing but producing logs
antroy-madetech 89f485e
Added S3 bucket perms to the codedeploy job
antroy-madetech 8c27004
Restricted role permissions for logs to our log group
antroy-madetech 9ebb2e4
Renamed roles.tf -> iam.tf
antroy-madetech 9d59f17
Codebuild 'install' job installs terraform, copilot, platform-helper
antroy-madetech 9a9af0a
Clean up
antroy-madetech be422d9
Remove Copilot string
WillGibson e803ee0
Add repository variable and mock AWS provider
WillGibson 61aea48
Start poking at the tests, some progress, still failing
WillGibson 5bdacce
Overriding data block with minimal JSON
antroy-madetech 01c828e
Refactored tag tests
antroy-madetech 8c76626
Merge branch 'main' into DBTP-911-Barebones-Pipeline
WillGibson 9d001eb
terraform fmt -recursive
WillGibson 342eb00
Add check on pipeline name and reuse the tags check
WillGibson af74114
Add commented out annoying asertion
WillGibson 31a36da
Start adding assertions for aws_codepipeline.codepipeline.stage.action
WillGibson 2b1cbcd
Strip it down to should be
WillGibson f155607
Add assertions for artifact_store
WillGibson 4f09e60
Rename stages etc.
WillGibson 51a3b5b
Test the pipeline tags
WillGibson 9d7a146
Rename stage back to build
WillGibson fe34a3b
Remove test_create_pipeline_with_different_application
WillGibson 2913249
Little reshuffle
WillGibson 972599d
Refactoring of names to bring in line with best practice guidelines
antroy-madetech c1827f5
Remaining iam tests
antroy-madetech e836abd
Merge branch 'main' into DBTP-911-Barebones-Pipeline
antroy-madetech c59dc0e
Fix Opensearch tests
antroy-madetech b0c5054
First codebuild test
antroy-madetech a8f92a6
Codebuild tests finished
antroy-madetech 2ad249f
terraform fmt
antroy-madetech b009673
PR comments addressed
antroy-madetech 77803d8
terraform fmt
antroy-madetech 1b1ecc3
Fixed tests
antroy-madetech 0ce3ac2
WIP
antroy-madetech e8eb219
Update opensearch/main.tf
antroy-madetech 3ef5295
Test changes for rename
antroy-madetech bddfda5
Reverted branch to main
antroy-madetech e9d8cab
Removed unnecessary comment
antroy-madetech 5d8a8cc
fmt
antroy-madetech 6099853
WIP
antroy-madetech e0a5596
Allow changing of branch. Use codestar to get modules repo
antroy-madetech 3fec9c6
Debugging
antroy-madetech a48ed75
Merge branch 'main' into DBTP-913-Pipleline-tf-plan
antroy-madetech 0946b0e
Merge error fixed
antroy-madetech d8e51fe
WIP
antroy-madetech 3450a30
Update CodeBuild image
JohnStainsby 46f0d6b
Merge branch 'DBTP-913-Pipleline-tf-plan' of github.com:uktrade/terra…
JohnStainsby 010ca2f
WIP
antroy-madetech ebde085
Add S3, KMS Key, and DynamoDB IAM policies
JohnStainsby c70aa6d
Bit of renaming
antroy-madetech ce0adc6
Added dependency for codepipeline on the codestar role
antroy-madetech 788b4a1
Added perms to read vpcs and subnets
antroy-madetech 7f6e2a8
Final few perms to get redis working
antroy-madetech 5dadd5b
SSM and further EC2 perms
antroy-madetech 64b5bb4
Add DNS account assume role policy
JohnStainsby e32d0c4
Add domain alias to providers
JohnStainsby cbf35ee
Add env to hosted zone name
JohnStainsby 1d33730
Revert route53 name change
JohnStainsby 45c16d4
Remove env name
JohnStainsby cee1ca4
Remove module source; Add variable for dns account it
JohnStainsby b4b9e06
Read properties from pipelines.yml variable
JohnStainsby a7f2963
Remove dns variable; Fix iam issue
JohnStainsby 4ef60b5
Fix existing tests
JohnStainsby 61ad11f
Add codepipeline tests
JohnStainsby e7f9846
Fix IAM role names for consistency; Add additional IAM tests
JohnStainsby 71e5dac
Fix formatting
JohnStainsby fbef95e
Merge branch 'main' into DBTP-913-Pipleline-tf-plan
JohnStainsby 215cfcd
Fix terraform fmt issues
JohnStainsby 39b9029
Update example for CodeBuild tests
JohnStainsby 0722cfe
Split policy statements for ec2 read access
JohnStainsby e5da8d6
Output terraform plan file
JohnStainsby File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
--- | ||
plan: | ||
name: "Plan" | ||
category: "Build" | ||
owner: "AWS" | ||
provider: "CodeBuild" | ||
input_artifacts: ["source_output"] | ||
output_artifacts: ["build_output"] | ||
version: "1" | ||
approve: | ||
run_order: 1 | ||
name: "AWS-Admin-Approval" | ||
category: "Approval" | ||
owner: "AWS" | ||
provider: "Manual" | ||
version: "1" | ||
input_artifacts: [] | ||
output_artifacts: [] | ||
|
||
apply: |
Oops, something went wrong.
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.
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.
Should these commands fall under the
pre_build
phase?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.
Yes probably once the rest of the copilot and platform-helper commands are added. This stuff is most likely going to be moved to a separate buildspec file anyway to work with a plan -> manual approve -> apply process