-
Notifications
You must be signed in to change notification settings - Fork 148
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: create Cloudbuild Source submodule #167
feat: create Cloudbuild Source submodule #167
Conversation
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.
Thanks @daniel-cit some minor comments
modules/tf_cloudbuild_core/main.tf
Outdated
source = "terraform-google-modules/cloud-storage/google//modules/simple_bucket" | ||
version = "~> 3.2" | ||
|
||
name = "${module.cloudbuild_project.project_id}_cloudbuild" |
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.
What do we use this bucket for?
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.
this is the default cloud build bucket, the one used for source staging.
if not created here it will be created in the fist time a build is run and it will be always created in the "US" location, independent of where the bootstrap infrastructure is been created.
same as terraform-google-modules/terraform-example-foundation#666
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.
Got it, lets add a comment here with any upstream docs if any.
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.
added an explanation
@bharathkkb could you PTAL?
@daniel-cit |
Submodule that creates: