azurerm_mssql_database - license_type is now also Computed
#18230
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
mbfrahry
changed the title
azurerm_mssql_database - license_type is now also Computed
mbfrahry
added a commit
that referenced
this pull request
Sep 2, 2022
|
This functionality has been released in v3.22.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Some tests of resource
azurerm_mssql_databasefailed with "After applying this test step, the plan was not empty."The tests fail because in some cases the property
LicenseTypeis set to default valueLicenseIncluded. Checking the current API behavior as follows:When the value of createMode is set to
Defaultand the value of LicenseType is not set, the API returns the value ofLicenseTypeasLicenseIncluded.When the value of
createModeis set toCopy, the value ofLicenseTypeis not set and theLicenseTypeof the source database is set toBasePrice, the API returns the value ofLicenseTypeasBasePrice.When the value of
createModeis set toCopy, the value ofLicenseTypeis set toLicenseIncludedand theLicenseTypeof the source database is set toBasePrice, the API returns the value ofLicenseTypeasLicenseIncluded.Per the above API behavior, I submitted this PR to add
Computed = truefor propertylicense_typeto fix the tests failure.