azurerm_function_app - support for daily_memory_time_quota

[aqche]

Fixes: #6076

Adds the daily_memory_time_quota option to the azurerm_function_app resource.

=== RUN   TestAccAzureRMFunctionApp_dailyMemoryTimeQuota
=== PAUSE TestAccAzureRMFunctionApp_dailyMemoryTimeQuota
=== CONT  TestAccAzureRMFunctionApp_dailyMemoryTimeQuota
--- PASS: TestAccAzureRMFunctionApp_dailyMemoryTimeQuota (200.69s)
PASS
ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/web/tests   200.725s

[aqche]

looks like if the value of DailyMemoryTimeQuota is set to 0 or a negative value, then the quota will be turned off. I know schema.TypeInt will default to a value of 0 if not provided. Do we want to explicitly set a default value here?

[tombuildsstuff]

I think that should be fine - since the quota itself only affects functions inside a consumption plan?

[tombuildsstuff]

LGTM 👍

[tombuildsstuff]

I think that should be fine - since the quota itself only affects functions inside a consumption plan?

[tombuildsstuff]

Ignoring a couple of known test errors (and the linting errors fixed in #6118) this otherwise LGTM:

[ghost]

This has been released in version 2.2.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.2.0"
}
# ... other configuration ...

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!