Open fmichaelobrien opened 1 month ago
Thanks for raising this issue. This is identified on our internal roadmap as work to address in the next major round of updates for v5 (sometime in H2 this year). The answer may or may not be SSM, due to the limitation that SSM is currently an invitation only service.
Closing this issue, and as eeaton noted, we will address this as part of our H2 roadmap.
Reopening, as I didn't notice it was marked as backlog.
Sounds good. I am currently working a PR patch in our fork for later submission.
One additional aspect to address when we work on this:
It was identified in #1273 that running the docker tests documented in CONTRIBUTING.md also have a dependency on Cloud Build & CSR, so this will fail in any new organizations that haven't previously used CSR. It's not yet clear to me whether we can unpick this locally, or it's an upstream issue with the CFT test framework.
Update: comment from apeabody suggests that this might be the API enablement in the bootstrap project, not necessarily the framework:
Hi @eeaton - I suspect (without seeing diagnostic output) that the make docker_test_prepare dependency on Cloud Source Repositories is due to the sourcerepo API activation in this repo's test/setup: https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/test/setup/main.tf#L65C6-L65C16 That line could likely be commented out to verify or if not using Cloud Source Repositories.
TL;DR
CSR is undergoing deprecation in favour of SSM
Impact to CICD
Terraform Resources
Detailed design
Additional information
fmichaelobrien will look into the SSM addition unless this work is already assigned in the roadmap