Skip to content

feat: add approval_required config to Cloud Build triggers#101

Merged
harrison-carter merged 2 commits into
mainfrom
feat/add-approval-config-to-cloudbuild-triggers
Mar 19, 2026
Merged

feat: add approval_required config to Cloud Build triggers#101
harrison-carter merged 2 commits into
mainfrom
feat/add-approval-config-to-cloudbuild-triggers

Conversation

@harrison-carter

@harrison-carter harrison-carter commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

https://nandosuk.atlassian.net/browse/SRMD-6983

Summary

  • Add an approval_required variable to the cloud-cloudbuild-trigger module, which uses a dynamic approval_config block to optionally require manual approval before builds execute
  • Pass the approval_required variable through the cloud-run-v2 module so consumers can enable approval gating without reaching into the trigger submodule directly
  • Defaults to false so existing usage is unaffected

Test plan

  • Verify terraform plan with approval_required = false (default) produces no diff on existing triggers
  • Verify terraform plan with approval_required = true adds the approval_config block as expected
  • Confirm the variable is correctly forwarded from cloud-run-v2 to cloud-cloudbuild-trigger

- Add `approval_required` boolean variable (defaults to `false`)
- Use a dynamic `approval_config` block so it's only included when approval is required
@harrison-carter harrison-carter force-pushed the feat/add-approval-config-to-cloudbuild-triggers branch from 076cd61 to 31547c4 Compare March 18, 2026 16:32
Comment thread gcp/cloud-cloudbuild-trigger/main.tf Outdated
- Add `approval_required` variable to the cloud-run-v2 module and forward it to the trigger provision submodule
@harrison-carter harrison-carter force-pushed the feat/add-approval-config-to-cloudbuild-triggers branch from 31547c4 to 235e367 Compare March 19, 2026 11:19
@harrison-carter harrison-carter merged commit dd6e31a into main Mar 19, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.71.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants