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
Add secondary sources #73
Conversation
Add secondary sources
@@ -280,6 +280,23 @@ resource "aws_codebuild_project" "default" { | |||
} | |||
} | |||
|
|||
dynamic "secondary_sources"{ | |||
iterator = secondary_source |
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.
remove pls iterator
for consistency with other modules
@@ -280,6 +280,23 @@ resource "aws_codebuild_project" "default" { | |||
} | |||
} | |||
|
|||
dynamic "secondary_sources"{ | |||
iterator = secondary_source | |||
for_each = length(var.secondary_sources) > 0 ? var.secondary_sources : [] |
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 condition isn't required
iterator = secondary_source | ||
for_each = length(var.secondary_sources) > 0 ? var.secondary_sources : [] | ||
content { | ||
git_clone_depth = secondary_source.value.git_clone_depth |
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.
need to add fallback to null
for non-required vars (see
terraform-aws-codebuild/main.tf
Line 324 in 974c9d5
status = lookup(s3_logs.value, "status", null) |
This pull request is now in conflict. Could you fix it @cfir? 🙏 |
This Pull Request has been updated, so we're dismissing all reviews.
This reverts commit 55d0d2b.
Hey all, what needs to be done to get this integraded ? We need the same feature so if I can help ... |
This feature has been added to the module in #87 |
what
I am adding dynamic object to support secindary sources
why
You are not supporting it right now and we need it.