Closed alismx closed 2 months ago
Reviewed the files and the terraform plan outputs. The only warning of interest was in the Prod output there are a few
Warning: Value for undeclared variable
warnings. Are these intended to be future variables?I don't see this as a must to change I was just interested in the intent of the variables. LGTM
I appreciate the call-out, with all the flux in this project, I think they're safe to ignore for now. I'm going to go ahead and merge this.
Related Issue
Changes Proposed
Additional Information
10933
TargetWorkerPool
to a value that doesn't exist will create a pool or if it needs to be set to a value predefined in Octopus.Testing
Checklist for Primary Reviewer
Infrastructure
terraform-plan
job inside the "Terraform Checks" workflow run for this PR. Confirm that there are no unexpected changes!Security
Cloud
dev
environment for verification.