Skip to content

Commit

Permalink
Remove update_strategy from rigm.
Browse files Browse the repository at this point in the history
  • Loading branch information
rileykarson committed Dec 6, 2018
1 parent 68c1019 commit 5ac2f13
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -172,10 +172,9 @@ func resourceComputeRegionInstanceGroupManager() *schema.Resource {
<% if version.nil? || version == 'ga' -%>
"update_strategy": &schema.Schema{
Type: schema.TypeString,
Deprecated: "This field is deprecated as it has no functionality anymore. It will be removed in 3.0.0.",
Deprecated: "This field is removed.",
Optional: true,
Computed: true,
ValidateFunc: validation.StringInSlice([]string{"NONE", "ROLLING_UPDATE"}, false),
},
<% end -%>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,6 @@ func TestAccRegionInstanceGroupManager_updateStrategy(t *testing.T) {
ResourceName: "google_compute_region_instance_group_manager.igm-update-strategy",
ImportState: true,
ImportStateVerify: true,
ImportStateVerifyIgnore: []string{"update_strategy"},
},
},
})
Expand Down Expand Up @@ -998,7 +997,6 @@ resource "google_compute_region_instance_group_manager" "igm-update-strategy" {
base_instance_name = "rigm-update-strategy"
region = "us-central1"
target_size = 2
update_strategy = "NONE"
named_port {
name = "customhttp"
port = 8080
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,6 @@ The following arguments are supported:
* `project` - (Optional) The ID of the project in which the resource belongs. If it
is not provided, the provider project is used.

* `update_strategy` - (Optional, Default `"NONE"`) This field is deprecated as it has no functionality anymore.
It previously turned update behaviour on and off. This field is only present in the `google` provider.

* `target_size` - (Optional) The target number of running instances for this managed
instance group. This value should always be explicitly set unless this resource is attached to
an autoscaler, in which case it should never be set. Defaults to `0`.
Expand Down

0 comments on commit 5ac2f13

Please sign in to comment.