Skip to content

Commit

Permalink
Adding in a reference for the older "App" App Service Plans (#1156)
Browse files Browse the repository at this point in the history
  • Loading branch information
tombuildsstuff authored Apr 24, 2018
1 parent 85ca751 commit 37ad400
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions azurerm/resource_arm_app_service_plan.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ func resourceArmAppServicePlan() *schema.Resource {
Default: "Windows",
ForceNew: true,
ValidateFunc: validation.StringInSlice([]string{
// @tombuildsstuff: I believe `app` is the older representation of `Windows`
// thus we need to support it to be able to import resources without recreating them.
"App",
"FunctionApp",
"Linux",
"Windows",
Expand Down
2 changes: 1 addition & 1 deletion website/docs/r/app_service_plan.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ The following arguments are supported:

* `location` - (Required) Specifies the supported Azure location where the resource exists. Changing this forces a new resource to be created.

* `kind` - (Optional) The kind of the App Service Plan to create. Possible values are `Windows`, `Linux` and `FunctionApp` (for a Consumption Plan). Defaults to `Windows`. Changing this forces a new resource to be created.
* `kind` - (Optional) The kind of the App Service Plan to create. Possible values are `Windows` (also available as `App`), `Linux` and `FunctionApp` (for a Consumption Plan). Defaults to `Windows`. Changing this forces a new resource to be created.

~> **NOTE:** When creating a `Linux` App Service Plan, the `reserved` field must be set to `true`.

Expand Down

0 comments on commit 37ad400

Please sign in to comment.