Skip to content
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

Merged
merged 18 commits into from
Jul 15, 2022

Conversation

daniel-cit
Copy link
Contributor

Submodule that creates:

  • Project for Cloud Build.
  • Default Cloud Build bucket.
  • Bucket for Cloud Build artifacts.
  • Set of Cloud Source Repos.

@daniel-cit daniel-cit marked this pull request as ready for review July 8, 2022 22:22
@daniel-cit daniel-cit requested a review from bharathkkb July 8, 2022 23:27
Copy link
Member

@bharathkkb bharathkkb left a 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

examples/tf_cloudbuild_core_simple/main.tf Outdated Show resolved Hide resolved
modules/tf_cloudbuild_core/main.tf Outdated Show resolved Hide resolved
source = "terraform-google-modules/cloud-storage/google//modules/simple_bucket"
version = "~> 3.2"

name = "${module.cloudbuild_project.project_id}_cloudbuild"
Copy link
Member

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?

Copy link
Contributor Author

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

Copy link
Member

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.

Copy link
Contributor Author

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?

modules/tf_cloudbuild_core/variables.tf Outdated Show resolved Hide resolved
modules/tf_cloudbuild_core/variables.tf Outdated Show resolved Hide resolved
modules/tf_cloudbuild_core/README.md Outdated Show resolved Hide resolved
modules/tf_cloudbuild_core/outputs.tf Outdated Show resolved Hide resolved
modules/tf_cloudbuild_core/variables.tf Outdated Show resolved Hide resolved
@daniel-cit daniel-cit changed the title feat: create Cloudbuild Core submodule feat: create Cloudbuild Source submodule Jul 13, 2022
@daniel-cit daniel-cit requested a review from bharathkkb July 13, 2022 20:40
modules/tf_cloudbuild_source/main.tf Outdated Show resolved Hide resolved
@comment-bot-dev
Copy link

@daniel-cit
Thanks for the PR! 🚀
✅ Lint checks have passed.

@bharathkkb bharathkkb merged commit 2dc083f into terraform-google-modules:master Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants