Update from master TF repo #1

…aad_check` properties (#4168) * Expose Principal Type attribute * Update azurerm/resource_arm_role_assignment_test.go Co-Authored-By: Tom Harvey <[email protected]> * Update azurerm/resource_arm_role_assignment_test.go Co-Authored-By: kt <[email protected]> * Updates per PR comments * Missed a comma * Revert name change * Update per PR review * Change of direction * Removing unused resource * added set to read CRUD * Update read * gofmt * Changes per PR comments * Added principal_type as a computed attribute

add `pricipal_type` and `skip_servicepricipal_aad_check` properties

* WIP: Initial checkin * Updates to Frontdoor * PIP * PIP * Finalized Schema * WIP Current state * WIP adding validation * WIP Finished validation code * WIP Expand RoutingRules working * WIP All expand func implemented * WIP: Functional CreateUpdate done * Cast values and start on flatten functions * WIP: Last fix routing rules * WIP: Fully working Frontdoor * WIP: Wasnt exactly done... fixed destroy * gofmt code * WIP: Added first test * Fixed casing issue when interacting with portal * Fix bug in fronend endpoint enum * Bug fix for Frontend Endpoint flatten * Add data source * Slight update to documentation * Added Frontend client code * Extending frontend endpoints schema * schema and exposed enable and disable for frontend * Enabled custom HTTPS domain setting * [WIP] Last code changes * Tweeking defaults and adding validation rules * [WIP] Update test case * Updated documentation and exposed cname * fixed flatten for Cert Source AzureKeyVault * Update to latest * Few changes per PR review * Fixed lint errors * Missed one client declaration * Fix test client * Removed data source fix lint errs * update CheckDestroy func * Update website/docs/r/front_door.html.markdown Co-Authored-By: kt <[email protected]> * Update azurerm/resource_arm_front_door.go Co-Authored-By: kt <[email protected]> * Update azurerm/resource_arm_front_door.go Co-Authored-By: kt <[email protected]> * Update test and docs * Added test case * Update test per PR * Address PR issues * Update website/docs/r/front_door.html.markdown Co-Authored-By: kt <[email protected]> * Added link to internal bug tracking casing issue * gofmt * Fixes per PR comments * Moving package(s) and refactor * gofmt

changing the `ip_configuration` property to no longer force new resource

On https://www.terraform.io/docs/providers/azurerm/r/api_management_backend.html the `validate_certificate_name` section was rendered as-is. The only difference I could find is the item list item not having an explicit enter beneath is in the source. Hopefully this fixes the problem.

In the file Terraform Hackathon\Examples\sql-azure\database\main.tf, there was a small reference mistake on line 19.

Fixes: #4278

Reverting fix #3747 for SDK bug # 5157

Fixes #4118

While trying to import a resource from Azure, I met a weird error message ```Error: unknown resource type: azurerm_databrick_workspace```, it took me sometimes to figure out the issue was just a simple typo.

Fixed a typo in the import sample

refactor: adding a Base Client

Mariadb storage autogrow

Mysql storage autogrow

r/private_dns_zone: polling on the future instead of sleeping

fixes #3115

…ual_network_link

… it's gone ``` $ acctests azurerm TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_ === RUN TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_basic === PAUSE TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_basic === RUN TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_requiresImport --- SKIP: TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_requiresImport (0.00s) resource_arm_private_dns_zone_virtual_network_link_test.go:42: Skipping since resources aren't required to be imported === RUN TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_withTags === PAUSE TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_withTags === CONT TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_basic --- PASS: TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_basic (226.26s) === CONT TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_withTags --- PASS: TestAccAzureRMPrivateDnsZoneVirtualNetworkLink_withTags (372.72s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 599.035s ```

New resource "azurerm_private_dns_zone_virtual_network_link"

…mapped fieldg

* Increase the timeout of upserting a CosmosDB account * Increase the timeout of deleting a CosmosDB account

Dependencies: updated `compute` API version to 2018-10-01

This avoids a circular reference where the resources in the Compute package required access to the common client, which required the client in the Compute package

``` $ ARM_PROVIDER_STRICT=true acctests azurerm TestAccAzureRMMarketplaceAgreement 2019/09/12 12:27:16 [DEBUG] Registering Data Sources for "Compute".. 2019/09/12 12:27:16 [DEBUG] Registering Resources for "Compute".. === RUN TestAccAzureRMMarketplaceAgreement === RUN TestAccAzureRMMarketplaceAgreement/basic === RUN TestAccAzureRMMarketplaceAgreement/basic/basic === RUN TestAccAzureRMMarketplaceAgreement/basic/requiresImport --- PASS: TestAccAzureRMMarketplaceAgreement (145.03s) --- PASS: TestAccAzureRMMarketplaceAgreement/basic (145.03s) --- PASS: TestAccAzureRMMarketplaceAgreement/basic/basic (67.86s) --- PASS: TestAccAzureRMMarketplaceAgreement/basic/requiresImport (77.17s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 145.092s ```

…et_id

…ment New Resource: `azurerm_marketplace_agreement`

r/container_registry - support for configuring Virtual Network Rules to Subnets

…ties `azurerm_stream_analytics_job` Removing requirement of optional properties

Update fork

Patch: `azurerm_bot_connection`

…-3673 network: reverting the locking changes from #3673

This commit changes the `certificate_data` field of an `azurerm_keyvault_certificate` to a hexadecimal representation, in common with the `thumbprint`. This is a better approach than calling `string(x)` directly on a byte array, since certificate data usually contains invalid UTF-8, and therefore cannot be processed by a wide array of tools downstream. Documentation is updated to reflect the new format, but since this is a computed value, no state migration function is supplied.

vendor: Bump terraform to 0.12.8

For the `storage_data_disk` block, there may be some confusion as to how to properly use the `create_option` parameter. In order to successfully deploy a data disk while using a template that does not initially write data to a data disk, the `Empty` parameter value must be specified. This was not clear during my initial attempts to deploy. Thanks.

[fixes: #3186 ]

compute: upgrading the API Version to 2019-07-01

r/public_ip: reading the `public_ip_prefix_id` field

keyvault_certificate: Store raw data as hex

VMSS and Standard Load Balancers are now GA, so removing the note of them as preview features. https://github.com/Azure/AKS/releases/tag/2019-09-09

Remove VMSS and standard LB as preview features

Bump Azure SDK to v33.2.0

…tead of hard coded ones

…alues Use SDK values for kusto_cluster sku names

…iner

The validation allows `basic` to be used as the SKU. This updates the documentation to go with that

docs: Update search_service SKUs

Adding data source 'azurerm_public_ip_prefix'

The docs did not include any information about this block. https://github.com/terraform-providers/terraform-provider-azurerm/blob/master/azurerm/resource_arm_front_door.go#L159 This should serve at least as a rough draft to make the docs more complete for this resource.

Shared Dashboard Resource

…me` property (#3154) * Flatten arm_api_management sku * Updates for backwards compatibility * Updated test cases * Addressing some PR comments * Fix for test pass * Rmoved comments and fixed docs * Fixed ImportStateVerify * Added SKU back into datasource * Updated code per PR comments * Removed newline * Remove unused function

…irtualMachineScaleSets.

azurerm_stream_analytics_stream_input_eventhub mentioned instead of azurerm_stream_analytics_stream_input_blob

…ty (#4409)

corrected the resource defined in key_url

…an_id

Co-Authored-By: Tom Harvey <[email protected]>

Add zone redundancy for service bus namespace

…y-check azurerm_container_registry - Checks if name is available on `create`

…blob-container Analysis services server server_full_name & backup_blob_container_uri

The string "Key Vault" was just twice in this sentence

Remove duplicated string

The resource name is `azurerm_frontdoor`

…fter create/update to update all the computed properties The previous behavior of the create was to read the resource like almost all others to make sure the computed properties like `url` are set. but with the change [1], it was changed to update to make sure create was done correctly, but update doesn't have a read call making the create failing to set the `url` property [1]: cab451f#diff-8e833d66ec7540aaf7ec57fd375ba62aR177-R178

For most resources, the resource_group_name attribute is used to specify the group into which the new resource will be added. In the case of DNS records, the resource_group_name specifies the group in which the parent can be found.

Clarify DNS record resource_group_name usage in documentation.

azurerm/resource_arm_storage_blob.go: make sure storage blob is read after create/update to update all the computed properties

… a new resource to be created

Removed ForceNew for resource_arm_function_app for the app_service_plan_id

Move the note on Linus app service plan to be below the kind field.

The check for `partner_id` happens within the setUserAgent function so we can instead delegate to that and ensure this is always set

Move linux note

clients: ensuring the user agent is set even if there's no partner id

provider: Use real Terraform version

- Support sending the slot diagnostic logs to blob storage - Support sending the slot HTTP logs to blob storage and the filesystem

…NetworkingKubenet

…Network

…ificate New data source: app_service_certificate

Example update: AKS advanced networking with multiple agent pool on VMSS

…9/datalakestore/

…ipal r/kubernetes_cluster: allowing the Service Principal to be updated

This is a minor change, just updating the URLs in the test application to HTTPS. The plan phase fails if the URLs are in HTTP rather HTTPS.

This fixes #4373 by adding the default_site_hostname attribute to the list of attributes.

Adding default_site_hostname attribute

This ticket fixes #4343 by adding the note about needing to delete the disks.

…ogging Support log settings for app service slot

…ather than the name of the storage account & fixing PR comments

[#3319] datalakestore gen2 filesystem resource

Fix docs of api_management_product_group

`azurerm_app_service_custom_hostname_binding`: support for the `ssl_state` and `thumbprint` properties

fixes #3234

Update redis_cache.html.markdown

…-password Bug Fix: `azurerm_hdinight_...` diff suppress passwords that are `*****` from the api

Correct Page heading. Resource name (azurerm_dns_a_record) was incorrect.

Fix operation-timeouts link

Update page heading to reflect correct Resource

Commits on Sep 28, 2019

Update CHANGELOG.md

katbyte authored Sep 28, 2019

Configuration menu

View commit details

Copy full SHA for bc14d7c

Browse repository at this point

Copy the full SHA

bc14d7c View commit details

Browse the repository at this point in the history

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update from master TF repo #1

Update from master TF repo #1

Commits on Sep 7, 2019

Commits on Sep 9, 2019

Commits on Sep 10, 2019

Commits on Sep 11, 2019

Commits on Sep 12, 2019

Commits on Sep 13, 2019

Commits on Sep 14, 2019

Commits on Sep 16, 2019

Commits on Sep 17, 2019

Commits on Sep 18, 2019

Commits on Sep 19, 2019

Commits on Sep 20, 2019

Commits on Sep 21, 2019

Commits on Sep 22, 2019

Commits on Sep 23, 2019

Commits on Sep 24, 2019

Commits on Sep 25, 2019

Commits on Sep 26, 2019

Commits on Sep 27, 2019

Commits on Sep 28, 2019

Commits on Sep 30, 2019

Commits on Oct 1, 2019

Commits on Oct 2, 2019

Commits on Oct 3, 2019