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
Fix default target group count #108
Fix default target group count #108
Conversation
see the original issue #94 if we want to change this back then at least lets prevent it from being created when both are set to fixed response |
Or maybe add a variable |
Thank you, have updated |
…kikuchi/terraform-aws-alb into linh-fix-default-target-group-count
/test all |
* Fix default target group count * Add variable default_target_group_enabled * Auto Format * Update variables to enable default target group * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
what
listener_https_fixed_response
but still need the default target group created in this module. It used to work before this PR (Addmodule.this.enabled
check to listener'scount
#97). Do not see why it should not get created when we havelistener_https_fixed_response