Update from master TF repo #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…aad_check` properties (#4168) * Expose Principal Type attribute * Update azurerm/resource_arm_role_assignment_test.go Co-Authored-By: Tom Harvey <tombuildsstuff@users.noreply.github.com> * Update azurerm/resource_arm_role_assignment_test.go Co-Authored-By: kt <kt@katbyte.me> * 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 <kt@katbyte.me> * Update azurerm/resource_arm_front_door.go Co-Authored-By: kt <kt@katbyte.me> * Update azurerm/resource_arm_front_door.go Co-Authored-By: kt <kt@katbyte.me> * 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 <kt@katbyte.me> * 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.
Reverting fix #3747 for SDK bug # 5157
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
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
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
nitzanm
pushed a commit
that referenced
this pull request
Oct 29, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.