resource/servicebus_subscription: Allow the setting of default_message_ttl

[stack72]

Fixes: #1566

This wasn't being passed to the Create func even though the user could
define it

▶ acctests azurerm TestAccAzureRMServiceBusSubscription_
=== RUN   TestAccAzureRMServiceBusSubscription_importBasic
--- PASS: TestAccAzureRMServiceBusSubscription_importBasic (243.63s)
=== RUN   TestAccAzureRMServiceBusSubscription_basic
--- PASS: TestAccAzureRMServiceBusSubscription_basic (223.77s)
=== RUN   TestAccAzureRMServiceBusSubscription_defaultTtl
--- PASS: TestAccAzureRMServiceBusSubscription_defaultTtl (217.08s)
=== RUN   TestAccAzureRMServiceBusSubscription_updateEnableBatched
--- PASS: TestAccAzureRMServiceBusSubscription_updateEnableBatched (281.08s)
=== RUN   TestAccAzureRMServiceBusSubscription_updateRequiresSession
--- PASS: TestAccAzureRMServiceBusSubscription_updateRequiresSession (397.46s)
=== RUN   TestAccAzureRMServiceBusSubscription_updateForwardTo
--- PASS: TestAccAzureRMServiceBusSubscription_updateForwardTo (208.94s)
PASS
ok	github.com/terraform-providers/terraform-provider-azurerm/azurerm	1354.925s

[tombuildsstuff]

LGTM - thanks for this 👍

[tombuildsstuff]

hi @stack72

Just to let you know that is now available in [v1.10 of the AzureRM Provider(https://github.com/terraform-providers/terraform-provider-azurerm/blob/v1.10.0/CHANGELOG.md#1100-july-21-2018) - which you can update to this version by specifying it in the Provider block, like so:

provider "azurerm" {
  version = "=1.10.0"
}

Thanks!

[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!