Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

modules/aws: configure ELB idle timeout #725

Merged
merged 1 commit into from
May 17, 2017

Conversation

s-urbaniak
Copy link
Contributor

Fixes #390

@s-urbaniak
Copy link
Contributor Author

ok to test

Copy link
Contributor

@alexsomesan alexsomesan left a comment

Choose a reason for hiding this comment

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

LGTM.
We've discussed this offline and decided hardcoding the values. There seem to be very few situations where they would need tweaking in the context of Tectonic.

@alexsomesan alexsomesan merged commit 0a96415 into coreos:master May 17, 2017
aknuds1 pushed a commit to aknuds1/tectonic-installer that referenced this pull request May 19, 2017
alekssaul pushed a commit to alekssaul/tectonic-installer that referenced this pull request May 30, 2017
wking added a commit to wking/openshift-installer that referenced this pull request Aug 28, 2019
It is not valid for network load balancers [1].  We've been setting
this in our Terraform config since 0a96415 (modules/aws: configure
ELB idle timeout, coreos/tectonic-installer#725) when we were using
classic load balancers and should have dropped it in 16dfbb3
(data/aws: use nlbs instead of elbs, 2018-11-01, openshift#594).

[1]: hashicorp/terraform-provider-aws@d25a227#diff-f4b0dbdc7e3eede6ba70cd286c834f37R78
wking added a commit to wking/openshift-installer that referenced this pull request Aug 28, 2019
It is not valid for network load balancers [1].  We've been setting
this in our Terraform config since 0a96415 (modules/aws: configure
ELB idle timeout, coreos/tectonic-installer#725) when we were using
classic load balancers and should have dropped it in 16dfbb3
(data/aws: use nlbs instead of elbs, 2018-11-01, openshift#594).

[1]: hashicorp/terraform-provider-aws@d25a227#diff-f4b0dbdc7e3eede6ba70cd286c834f37R78
alaypatel07 pushed a commit to alaypatel07/installer that referenced this pull request Nov 13, 2019
It is not valid for network load balancers [1].  We've been setting
this in our Terraform config since 0a96415 (modules/aws: configure
ELB idle timeout, coreos/tectonic-installer#725) when we were using
classic load balancers and should have dropped it in 16dfbb3
(data/aws: use nlbs instead of elbs, 2018-11-01, openshift#594).

[1]: hashicorp/terraform-provider-aws@d25a227#diff-f4b0dbdc7e3eede6ba70cd286c834f37R78
jhixson74 pushed a commit to jhixson74/installer that referenced this pull request Dec 6, 2019
It is not valid for network load balancers [1].  We've been setting
this in our Terraform config since 0a96415 (modules/aws: configure
ELB idle timeout, coreos/tectonic-installer#725) when we were using
classic load balancers and should have dropped it in 16dfbb3
(data/aws: use nlbs instead of elbs, 2018-11-01, openshift#594).

[1]: hashicorp/terraform-provider-aws@d25a227#diff-f4b0dbdc7e3eede6ba70cd286c834f37R78
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.

2 participants