Skip to content

Commit 2f6a08d

Browse files
Promote InstanceFlexibilityPolicy to GA (#12244)
1 parent cab7352 commit 2f6a08d

File tree

3 files changed

+3
-18
lines changed

3 files changed

+3
-18
lines changed

mmv1/third_party/terraform/services/compute/resource_compute_region_instance_group_manager.go.tmpl

+1-13
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,6 @@ func ResourceComputeRegionInstanceGroupManager() *schema.Resource {
9494
},
9595
},
9696

97-
{{- if ne $.TargetVersionName "ga" }}
9897
"instance_flexibility_policy": {
9998
Type: schema.TypeList,
10099
Optional: true,
@@ -134,7 +133,6 @@ func ResourceComputeRegionInstanceGroupManager() *schema.Resource {
134133
},
135134
},
136135
},
137-
{{- end }}
138136

139137
"name": {
140138
Type: schema.TypeString,
@@ -672,9 +670,7 @@ func resourceComputeRegionInstanceGroupManagerCreate(d *schema.ResourceData, met
672670
TargetSuspendedSize: int64(d.Get("target_suspended_size").(int)),
673671
TargetStoppedSize: int64(d.Get("target_stopped_size").(int)),
674672
{{- end }}
675-
{{- if ne $.TargetVersionName "ga" }}
676673
InstanceFlexibilityPolicy: expandInstanceFlexibilityPolicy(d),
677-
{{- end }}
678674
UpdatePolicy: expandRegionUpdatePolicy(d.Get("update_policy").([]interface{})),
679675
InstanceLifecyclePolicy: expandInstanceLifecyclePolicy(d.Get("instance_lifecycle_policy").([]interface{})),
680676
AllInstancesConfig: expandAllInstancesConfig(nil, d.Get("all_instances_config").([]interface{})),
@@ -878,11 +874,9 @@ func resourceComputeRegionInstanceGroupManagerRead(d *schema.ResourceData, meta
878874
return fmt.Errorf("Error setting target_stopped_size: %s", err)
879875
}
880876
{{- end }}
881-
{{- if ne $.TargetVersionName "ga" }}
882877
if err := d.Set("instance_flexibility_policy", flattenInstanceFlexibilityPolicy(manager.InstanceFlexibilityPolicy)); err != nil {
883878
return err
884879
}
885-
{{- end }}
886880
if err := d.Set("update_policy", flattenRegionUpdatePolicy(manager.UpdatePolicy)); err != nil {
887881
return fmt.Errorf("Error setting update_policy in state: %s", err.Error())
888882
}
@@ -956,7 +950,6 @@ func resourceComputeRegionInstanceGroupManagerUpdate(d *schema.ResourceData, met
956950
change = true
957951
}
958952

959-
{{- if ne $.TargetVersionName "ga" }}
960953
var targetSizePatchUpdate bool
961954
if d.HasChange("instance_flexibility_policy") {
962955
updatedManager.InstanceFlexibilityPolicy = expandInstanceFlexibilityPolicy(d)
@@ -969,7 +962,6 @@ func resourceComputeRegionInstanceGroupManagerUpdate(d *schema.ResourceData, met
969962
targetSizePatchUpdate = true
970963
}
971964
}
972-
{{- end }}
973965

974966
if d.HasChange("distribution_policy_target_shape") {
975967
updatedManager.DistributionPolicy = expandDistributionPolicyForUpdate(d)
@@ -1060,7 +1052,7 @@ func resourceComputeRegionInstanceGroupManagerUpdate(d *schema.ResourceData, met
10601052
}
10611053

10621054
// target size should use resize
1063-
if d.HasChange("target_size") {{- if ne $.TargetVersionName "ga" }} && !targetSizePatchUpdate {{- end}} {
1055+
if d.HasChange("target_size") && !targetSizePatchUpdate {
10641056
d.Partial(true)
10651057
targetSize := int64(d.Get("target_size").(int))
10661058
op, err := config.NewComputeClient(userAgent).RegionInstanceGroupManagers.Resize(
@@ -1210,7 +1202,6 @@ func flattenRegionUpdatePolicy(updatePolicy *compute.InstanceGroupManagerUpdateP
12101202
return results
12111203
}
12121204

1213-
{{- if ne $.TargetVersionName "ga" }}
12141205
func expandInstanceFlexibilityPolicy(d *schema.ResourceData) *compute.InstanceGroupManagerInstanceFlexibilityPolicy {
12151206
instanceFlexibilityPolicy := &compute.InstanceGroupManagerInstanceFlexibilityPolicy{}
12161207
oldFlexibilityPolicy, newFlexibilityPolicy := d.GetChange("instance_flexibility_policy")
@@ -1244,7 +1235,6 @@ func expandInstanceSelections(instanceSelections []any) map[string]compute.Insta
12441235
}
12451236
return instanceSelectionsMap
12461237
}
1247-
{{- end }}
12481238

12491239
func expandDistributionPolicyForUpdate(d *schema.ResourceData) *compute.DistributionPolicy {
12501240
dpts := d.Get("distribution_policy_target_shape").(string)
@@ -1281,7 +1271,6 @@ func expandDistributionPolicyForCreate(d *schema.ResourceData) *compute.Distribu
12811271
return distributionPolicy
12821272
}
12831273

1284-
{{- if ne $.TargetVersionName "ga" }}
12851274
func flattenInstanceFlexibilityPolicy(instanceFlexibilityPolicy *compute.InstanceGroupManagerInstanceFlexibilityPolicy) []map[string]any {
12861275
flattenedInstanceFlexibilityPolicy := []map[string]any{}
12871276
if instanceFlexibilityPolicy != nil {
@@ -1303,7 +1292,6 @@ func flattenInstanceSelections(instanceSelections map[string]compute.InstanceGro
13031292
}
13041293
return instanceSelectionsMap
13051294
}
1306-
{{- end }}
13071295

13081296
func flattenDistributionPolicy(distributionPolicy *compute.DistributionPolicy) []string {
13091297
zones := make([]string, 0)

mmv1/third_party/terraform/services/compute/resource_compute_region_instance_group_manager_test.go.tmpl

+1-4
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,6 @@ func TestAccRegionInstanceGroupManager_stoppedSuspendedTargetSize(t *testing.T)
431431
}
432432
{{- end }}
433433

434-
{{- if ne $.TargetVersionName "ga" }}
435434
func TestAccRegionInstanceGroupManager_instanceFlexibilityPolicy(t *testing.T) {
436435
t.Parallel()
437436

@@ -474,7 +473,7 @@ func TestAccRegionInstanceGroupManager_instanceFlexibilityPolicy(t *testing.T) {
474473
},
475474
})
476475
}
477-
{{- end }}
476+
478477
func TestAccRegionInstanceGroupManager_APISideListRecordering(t *testing.T) {
479478
t.Parallel()
480479

@@ -1969,7 +1968,6 @@ resource "google_compute_region_instance_group_manager" "sr-igm" {
19691968
}
19701969
{{- end }}
19711970

1972-
{{- if ne $.TargetVersionName "ga" }}
19731971
func testAccRegionInstanceGroupManager_instanceFlexibilityPolicy(network, template, igm string) string {
19741972
return fmt.Sprintf(`
19751973
data "google_compute_image" "my_image" {
@@ -2138,7 +2136,6 @@ resource "google_compute_region_instance_group_manager" "igm-basic" {
21382136
}
21392137
`, network, template, igm)
21402138
}
2141-
{{- end }}
21422139

21432140
{{- if ne $.TargetVersionName "ga" }}
21442141
func testAccRegionInstanceGroupManager_resourceManagerTags(template_name, tag_name, igm_name, project_id string) string {

mmv1/third_party/terraform/website/docs/r/compute_region_instance_group_manager.html.markdown

+1-1
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ group. You can specify one or more values. For more information, see the [offici
205205

206206
* `params` - (Optional [Beta](https://terraform.io/docs/providers/google/guides/provider_versions.html)) Input only additional params for instance group manager creation. Structure is [documented below](#nested_params). For more information, see [API](https://cloud.google.com/compute/docs/reference/rest/beta/instanceGroupManagers/insert).
207207

208-
* `instance_flexibility_policy` - (Optional [Beta](https://terraform.io/docs/providers/google/guides/provider_versions.html)) The flexibility policy for managed instance group. Instance flexibility allows managed instance group to create VMs from multiple types of machines. Instance flexibility configuration on managed instance group overrides instance template configuration. Structure is [documented below](#nested_instance_flexibility_policy).
208+
* `instance_flexibility_policy` - (Optional) The flexibility policy for managed instance group. Instance flexibility allows managed instance group to create VMs from multiple types of machines. Instance flexibility configuration on managed instance group overrides instance template configuration. Structure is [documented below](#nested_instance_flexibility_policy).
209209
- - -
210210

211211
The `standby_policy` block supports:

0 commit comments

Comments
 (0)