Skip to content
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

Feature gcp region autoscaler #12928

Closed
wants to merge 13 commits into from
Closed

Feature gcp region autoscaler #12928

wants to merge 13 commits into from

Conversation

corianderpulm
Copy link

Ready for review! Previously it had the WIP tag but it was finished, only requiring a quick review.

Note: The new resource 'google_compute_region_autoscaler' was created to work with the 'google_compute_region_instance_group_manager' resource introduced in PR #10907.

I noticed the normal Google Cloud autoscaler requires a 'zone' variable. For region level autoscalers this should not be required as it crosses zones. Trying to create an autoscaler set in one zone fails as the logic inside the plugin itself is looking the autoscaler up by zone and name instead of region and name. While this is useful for single-zone autoscalers, it conflicts with how they work on a cross-zone basis. This PR is a lightly modified 'compute_autoscaler' resource that has explicit logic for region instance group managers.

@corianderpulm
Copy link
Author

corianderpulm commented Mar 21, 2017

Be advised, this is kind of a weird PR as it requires #10907 to be also merged in so CI tests might fail.

@Pryz
Copy link
Contributor

Pryz commented Oct 24, 2017

Any update on this ?

@apparentlymart
Copy link
Contributor

Hi @CoryPulm!

Sorry this wasn't reviewed before. At the time you submitted this we were starting the process of splitting the provider code out into separate repositories, so the Google Cloud provider now has its own repository and its own team of maintainers.

This change unfortunately isn't mergable as-is because the rest of the provider code isn't here anymore. With that said, it looks like there is a google_compute_region_autoscaler resource that was added in hashicorp/terraform-provider-google#544 which I assume exposes the same underlying APIs as what was added here, so I'm going to close this.

Thanks for working on this @CoryPulm, and sorry it got caught up in the confusion while the providers were being migrated.

@ghost
Copy link

ghost commented Apr 6, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants