Skip to content

Conversation

@jwhitcraft
Copy link
Contributor

PR o'clock

Description

This adds support for slow_start to the target_groups which fixes #86

Checklist

  • terraform fmt and terraform validate both work from the root and examples/alb_test_fixture directories (look in CI for an example)
  • Tests for the changes have been added and passing (for bug fixes/features)
  • Test results are pasted in this PR (in lieu of CI)
  • Docs have been added/updated (for bug fixes/features)
  • Any breaking changes are noted in the description above

colindean added a commit to arcadia/terraform-aws-alb that referenced this pull request Oct 26, 2018
Add slow_start option to target_group defs
@jwhitcraft
Copy link
Contributor Author

ping @brandoconnor

@brandonjbjelland
Copy link
Contributor

brandonjbjelland commented Oct 30, 2018 via email

@jwhitcraft
Copy link
Contributor Author

@brandoconnor I just updated this with the latest master? Any chance of a merge?

Copy link
Contributor

@brandonjbjelland brandonjbjelland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM merging.

@brandonjbjelland brandonjbjelland merged commit bc0c5f1 into terraform-aws-modules:master Dec 3, 2018
@github-actions
Copy link

I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add option for slow_start aws_lb_target_group

2 participants