-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
aws/provider: Fixes issue for aws_lb_ssl_negotiation_policy of already deleted ELB #12360
aws/provider: Fixes issue for aws_lb_ssl_negotiation_policy of already deleted ELB #12360
Conversation
Hi @bclodius Thanks for the work on this - please can you add an acceptance test to show that this works as expected? Please feel free to shout if you need some guidance in this area Thanks Paul |
@stack72 certainly. I will take a stab at this and let you know if I need some help. |
nice work @bclodius
LGTM! |
Thanks @stack72!!! |
…y deleted ELB (hashicorp#12360) * Ensures elb exists before negotiation policy check; Fixes hashicorp#11260 * Adds acceptance test case for missing elb * Adds back https properties for test elb
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. |
Fixes #11260