-
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
provider/google: Add support for session_affinity to google_compute_region_backend_service #11228
Conversation
cc: @evandbrown @danawillow FYI :) |
Config: testAccComputeRegionBackendService_withSessionAffinity(serviceName, checkName), | ||
Check: resource.ComposeTestCheckFunc( | ||
testAccCheckComputeRegionBackendServiceExists( | ||
"google_compute_region_backend_service.foobar", &svc), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also check here that the new region backend service not just exists but has the correct session affinity?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
Result of modified acceptance test:
|
Is it possible to have an ETA when this will be merged ? And is it possible to know when next release will be ? Just to know if I need to build terraform locally to have this feature :) |
@danawillow Ok, thx for merging, I'll build terraofrm using master for the moment and will be waiting for next release :) I must say things are moving quickly here. That's always nice to have :) |
Thanks for the review and merge, @danawillow! Appreciated :) @zopanix -- Minor revisions of Terraform are cut on average every two weeks. Last release was cut a week ago, so maybe another week or so? There are never guarantees, but it won't be long that's for sure. |
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 #11199.