Incorrect documentation for lb_target_group

[jleclanche]

1.3.0 changelog claims aws_lb_target_group no longer has a default on matcher, but "200" is still documented:
https://www.terraform.io/docs/providers/aws/r/lb_target_group.html

Additionally, timeout default seems to be 10 now, not 5.

[wjglerum]

Like I mentioned here: #2251 path is also not default anymore for a health_check block.

[jurajseffer]

I've also just noticed this change. The defaults have changed as follows:

health_check.0.healthy_threshold:   "5" => "3"
health_check.0.matcher:             "200" => ""
health_check.0.timeout:             "5" => "10"
health_check.0.unhealthy_threshold: "2" => "3"

[catsby]

Hey @jurajseffer could you possibly provide an example configuration that shows that diff, minus any secret info? I'd like to use it for a regression test to make sure our fix handles that. Thanks!

[catsby]

We're also going to track this in #2327 just so I can consolidate issues

[jurajseffer]

@catsby

resource "aws_alb_target_group" "target" {
...
    health_check {
        path = "${var.health_check_path}"
    }
...

The above config had no changes planned under aws provider <1.3. After upgrade the same config showed changes I posted earlier

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!