-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add CI Terraform Plan for EKS #383
Conversation
669941d
to
e5f368c
Compare
This work has a dependency on trello card iam-eks-configure-fulladmin-and-readonly-access-levels so we will wait for next sprint. |
0727a83
to
2919d94
Compare
2919d94
to
35f7d3a
Compare
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.
If you rebase, there's a refactored version of the Terraform script that you can use.
I was using that but forgot to remove the old script, will do now. |
35f7d3a
to
c1d636b
Compare
Since we switch to EKS from ECS, this PR adds the terraform plans for the EKS modules and removes the ECS ones. **Testing** Tested by setting the CI pipeline to the one defined here and checking that the 2 terraform plans are done and reported to GitHub **Notes** We use plan with `-lock=false` and `-refresh=false` as done previously in ECS with justifications in #297 and #306 respectively. Ref: 1. [EKS cluster](#372) 2. [EKS cluster addons](#377) 3. [Concourse EKS pipeline](#378) 4. [add CI user to k8s](#391)
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.
Cool. Looks good, cheers for getting this running again.
Since we switch to EKS from ECS, this PR adds the terraform plans for
the EKS modules and removes the ECS ones.
Testing
Tested by setting the CI pipeline to the one defined here and
checking that the 2 terraform plans are done and reported to
GitHub
Notes
We use plan with
-lock=false
and-refresh=false
as donepreviously in ECS with justifications in #297 and #306 respectively.
Ref: