[Feature/Documentation Request] - Add GCE support to regionInstanceGroupManagers
- Dominant language
- Python
- Stars
- 7
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
### Feature Request
Currently, only zonal MIGs are supported. I'm requesting a upgrade to the architecture in order to support regional instance group managers for future-proofing and also for:
- Increased quota
- Better handle zonal-failure
### Documentation Request
Meanwhile, please add to the documentation notes on how the plugin supports **only** zonal MIGs to avoid confusion. The following is an example suggestion:
>- Template configuration
> - (...)
> - `gcp_zone` : Zone of the existing zonal MIG.
> - `gcp_instance_group`: Name of the existing zonal MIG.
### Reference
- [regionInstanceGroupManagers](https://cloud.google.com/compute/docs/reference/rest/v1/regionInstanceGroupManagers)
- https://github.com/google/symphony-gcp/blob/6e12b54ea2deb255e5a6b0483473e90adb41693b/hf-provider/src/gce_provider/commands/request_machines.py#L56-L64
Contributor guide
Research direction
Start with hf-provider/src/gce_provider/commands/request_machines.py around lines 56-64 and review the regionInstanceGroupManagers reference. Define how regional instance group managers should be supported alongside the current zonal behavior, then update the documentation to state that only zonal MIGs are currently supported. Done means the regional behavior is implemented and the documentation no longer leaves the supported scope ambiguous.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- gcp, python
- Domain
- cloud, infrastructure
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100