-
-
Notifications
You must be signed in to change notification settings - Fork 675
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
8.3.0 alb target group stickiness not working. #275
Comments
Its hard to say from the info you have provided, you don't provide a full reproduction or even the values for stickiness used, but the changing the |
+1 the same error |
Today, I have the same problem.
With module in version 8.3.0, I have the error... I don't understand why it doesn't pass the |
@jcollin-zmg thanks for the reproduction, I'll look into this now |
This issue has been resolved in version 8.3.1 🎉 |
Only to clarify, this wasn't solved in version 8.3.1. The issue could have been solved for NLB but ALB has the same problem. This is adding and attaching another security group, which, shouldn't be created. |
That is what was intended so I believe it is working as intended |
Not yet. This happened to me today, it created extra security without declared inbound rules and it attached to ALB existing, which, already had a security group created and attached. |
@rubenesda This is what I explained in the MR that added this feature. #273 (comment) #273 (comment) |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Please provide a clear and concise description of the issue you are encountering, and a reproduction of your configuration (see the
examples/*
directory for references that you can copy+paste and tailor to match your configs if you are unable to copy your exact configuration). The reproduction MUST be executable by runningterraform init && terraform apply
without any further changes.If your request is for a new feature, please use the
Feature request
template.Before you submit an issue, please perform the following first:
.terraform
directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!):rm -rf .terraform/
terraform init
Versions
Module version [Required]: 8.3.0
Terraform version: 1.3.6
Reproduction Code [Required]
target_groups spec
[{
backend_port = "${v.backend_port}"
backend_protocol = "${v.backend_protocol}"
deregistration_delay = 60
protocol_version = "HTTP1"
target_type = "instance"
targets = { for target_id in (var.ec2 > 0 ? module.ec2.*.id : v.target_ids) :
target_id => {
"target_id" = "${target_id}"
"port" = "${v.backend_port}"
}
}
health_check = {
enabled = true
interval = 30
path = "${v.backend_health_check_path}"
port = "traffic-port"
healthy_threshold = 2
unhealthy_threshold = 3
timeout = 10
protocol = "HTTP"
matcher = "200-399"
}
load_balancing_algorithm_type = v.lb_algorithm_type
stickiness = v.stickiness
}]
Steps to reproduce the behavior:
use 8.3.0 version
source = "terraform-aws-modules/alb/aws"
version = "8.3.0"
Expected behavior
alb create success
Actual behavior
Terminal Output Screenshot(s)
guess problem
part of main.tf(8.3.0)
dynamic "stickiness" {
for_each = try([var.target_groups[count.index].stickiness], [])
}
I think you should replace lookup with try.
The text was updated successfully, but these errors were encountered: