App Service Re-allignment #19685

jackofallops · 2022-12-14T15:31:15Z

This PR collects together a number of fixes and breaking changes to address the drift of the App and slot resources in App Service. Due to this drift, several behaviours and values have been changed in the service resulting in the resources not working as originally intended, not being correctly configurable, or not be accurately reflected in the Portal.

This PR supersedes: #19532 #19343 #18948 #18227 #18948 #18177 #18155 #18076 #17656

There will be a follow up PR to collect and address the Docker related updates for Web and Function app resources.

Due to the imminent holiday period and a number of breaking changes within, this PR will not be merged until after the holidays.

TODO:

Update Docs for affected resources

katbyte

LGTM 🧮

xiaxyi · 2023-01-04T08:40:49Z

Thanks @jackofallops for the pr, I may have some suggestions towards the dotnet application stack.

I see v3.0 is a valid value for dotnet runtime in both windows function/ web app, I'm afraid 3.0 is not a valid value for dotnet app stack since the underlying iis server doesn't have version 3.0. if user have the setting like:

 site_config {
    application_stack {
      dotnet_version = "v3.0"
    }
  }

The app service won't be started (shows 503 service unavailable) as the .netFrameworkVersion is sets to 3.0 which is not supported.

You can check my pr #18177 for more information. In short words, we need to ignore the user's input if the value is 3.0, then system will assign a default value to .netFrameworkVersion property.

jackofallops · 2023-01-04T08:48:22Z

Thanks @jackofallops for the pr, I may have some suggestions towards the dotnet application stack.

I see v3.0 is a valid value for dotnet runtime in both windows function/ web app, I'm afraid 3.0 is not a valid value for dotnet app stack since the underlying iis server doesn't have version 3.0. if user have the setting like:
 site_config {
    application_stack {
      dotnet_version = "v3.0"
    }
  }
The app service won't be started (shows 503 service unavailable) as the .netFrameworkVersion is sets to 3.0 which is not supported.

You can check my pr #18177 for more information. In short words, we need to ignore the user's input if the value is 3.0, then system will assign a default value to .netFrameworkVersion property.

Thanks

Thanks @jackofallops for the pr, I may have some suggestions towards the dotnet application stack.

I see v3.0 is a valid value for dotnet runtime in both windows function/ web app, I'm afraid 3.0 is not a valid value for dotnet app stack since the underlying iis server doesn't have version 3.0. if user have the setting like:
 site_config {
    application_stack {
      dotnet_version = "v3.0"
    }
  }
The app service won't be started (shows 503 service unavailable) as the .netFrameworkVersion is sets to 3.0 which is not supported.

You can check my pr #18177 for more information. In short words, we need to ignore the user's input if the value is 3.0, then system will assign a default value to .netFrameworkVersion property.

Hi @xiaxyi - I appreciate the feedback, however, this only applies to newly created resources. If a customer has an application deployed from before the v3.0 -> v4.0 change, it will still be a valid value and removing it here would potentially break their configuration, so we'll need to leave it here for now and look to remove in a future release.

mbfrahry

Hey @jackofallops, this is a doozy of a PR. I think the bulk of it looks good but I've called out some areas that need investigation and some logic issues that might not be a problem but are worth looking at to confirm.

mbfrahry · 2023-01-04T23:16:59Z

internal/services/appservice/helpers/app_stack.go

+					Type:       pluginsdk.TypeString,
+					Optional:   true,
+					Computed:   true,
+					Deprecated: "This property is deprecated. Values set are not used by the service.",


Do we want to add a todo to deprecate this in 4.0?

Yes, I believe so. However, I think that should be a follow up PR as the schema here will end up very difficult to read and this change is already complex /awkward to review.

mbfrahry · 2023-01-04T23:17:37Z

internal/services/appservice/helpers/app_stack.go

+				"java_container": {
+					Type:       pluginsdk.TypeString,
+					Optional:   true,
+					Deprecated: "this property has been deprecated in favour of `tomcat_version` and `java_embedded_server_enabled`",


Remove for 4.0?

internal/services/appservice/helpers/common_web_app_schema.go

mbfrahry · 2023-01-04T23:43:12Z

internal/services/appservice/helpers/function_app_schema.go

 		MaxItems: 1,
 		Elem: &pluginsdk.Resource{
 			Schema: map[string]*pluginsdk.Schema{
 				"dotnet_version": {
 					Type:     pluginsdk.TypeString,
 					Optional: true,
+					Default:  "v4.0",


Potential breaking change?

internal/services/appservice/helpers/function_app_schema.go

internal/services/appservice/helpers/function_app_slot_schema.go

internal/services/appservice/helpers/web_app_slot_schema.go

internal/services/appservice/helpers/windows_web_app_schema.go

tombuildsstuff

Left some comments inline but otherwise 👍

website/docs/r/windows_web_app.html.markdown

internal/services/appservice/windows_web_app_slot_resource.go

internal/services/appservice/windows_web_app_resource_test.go

tombuildsstuff · 2023-01-05T10:18:18Z

internal/services/appservice/windows_function_app_slot_resource.go

@@ -888,7 +884,12 @@ func (m *WindowsFunctionAppSlotModel) unpackWindowsFunctionAppSettings(input web
 		case "FUNCTIONS_EXTENSION_VERSION":


(line above this) we need to strings to upper this to handle casing?

~~"System" app_settings are always in upper, but I don't believe there's any reason not to do this to ensure typos are caught.~~
Scratch that, some settings are in mixed case and afaik they need to be explicitly cased, e.g. AzureWebJobsStorage__accountName

tombuildsstuff · 2023-01-05T10:18:35Z

internal/services/appservice/windows_function_app_slot_resource.go

+			switch *v {
+			case "dotnet-isolated":
+				m.SiteConfig[0].ApplicationStack[0].DotNetIsolated = true
+			case "custom":
+				m.SiteConfig[0].ApplicationStack[0].CustomHandler = true


we should be lower-casing v here prior to doing this switch?

To the best of my knowledge these are case sensitive values so I think it's safe to leave this as is?

internal/services/appservice/helpers/common_web_app_schema.go

tombuildsstuff · 2023-01-05T15:27:32Z

internal/services/appservice/helpers/common_web_app_schema.go

+	// Remove the settings the service adds for legacy reasons.
+	for _, v := range unmanagedSettings { //nolint:typecheck
+		delete(appSettings, v)
+	}


do we want to case-insensitively check the key casings here?

tombuildsstuff · 2023-01-05T15:27:48Z

internal/services/appservice/helpers/common_web_app_schema.go

+	if app.VirtualPath == nil || app.PhysicalPath == nil {
+		return false
+	}
+	if *app.VirtualPath == "/" && *app.PhysicalPath == "site\\wwwroot" && *app.PreloadEnabled && app.VirtualDirectories == nil {


we should check these case-insensitively

tombuildsstuff · 2023-01-05T15:28:09Z

internal/services/appservice/helpers/common_web_app_schema.go

+		if inputSKU == sku {
+			result = true
+		}
+	}
+	for _, sku := range sharedSkus {
+		if inputSKU == sku {


do we need to case-insensitively check these?

internal/services/appservice/helpers/web_app_slot_schema.go

xiaxyi · 2023-01-06T08:24:58Z

Thanks @jackofallops for the pr, I may have some suggestions towards the dotnet application stack.
I see v3.0 is a valid value for dotnet runtime in both windows function/ web app, I'm afraid 3.0 is not a valid value for dotnet app stack since the underlying iis server doesn't have version 3.0. if user have the setting like:
 site_config {
    application_stack {
      dotnet_version = "v3.0"
    }
  }
The app service won't be started (shows 503 service unavailable) as the .netFrameworkVersion is sets to 3.0 which is not supported.
You can check my pr #18177 for more information. In short words, we need to ignore the user's input if the value is 3.0, then system will assign a default value to .netFrameworkVersion property.
Thanks
Thanks @jackofallops for the pr, I may have some suggestions towards the dotnet application stack.
I see v3.0 is a valid value for dotnet runtime in both windows function/ web app, I'm afraid 3.0 is not a valid value for dotnet app stack since the underlying iis server doesn't have version 3.0. if user have the setting like:
 site_config {
    application_stack {
      dotnet_version = "v3.0"
    }
  }
The app service won't be started (shows 503 service unavailable) as the .netFrameworkVersion is sets to 3.0 which is not supported.
You can check my pr #18177 for more information. In short words, we need to ignore the user's input if the value is 3.0, then system will assign a default value to .netFrameworkVersion property.
Hi @xiaxyi - I appreciate the feedback, however, this only applies to newly created resources. If a customer has an application deployed from before the v3.0 -> v4.0 change, it will still be a valid value and removing it here would potentially break their configuration, so we'll need to leave it here for now and look to remove in a future release.

Thanks @jackofallops , but if the user creates the app service with DOTNET version sets to 4.0, the app won't be started. Do you think we should only allow it in 3.0 provider and remove the value in 4.0?

* CHANGELOG hashicorp#19722 * CHANGELOG hashicorp#19669 * Docs: Fix possible values mismatch between document and schema validator (hashicorp#19690) * New Resource : `azurerm_cognitive_deployment` (hashicorp#19526) * CHANGELOG hashicorp#19526 * New Resource: `azurerm_resource_deployment_script` (hashicorp#19436) Fix hashicorp#13581 * CHANGELOG hashicorp#19436 * Docs: more documents fix for syntax, optional/required and possible values (hashicorp#19733) * `azurerm_stream_analytics_output_blob` - fix type conversion for `batch_min_rows` (hashicorp#19732) fixes hashicorp#19725 * `azurerm_monitor_diagnostic_setting` - fix tests caused by service API behavior change (hashicorp#19762) * Docs: Fix missed/odd default values in document (hashicorp#19760) * Docs: fix outdated timeout values, or add missed timeouts lines of all resources (hashicorp#19756) * `azurerm_postgresql_flexible_server` - add nil check for `authentication` (hashicorp#19755) fixes hashicorp#19752 * `azurerm_key_vault` - fix mismatch document of `access_policy` (hashicorp#19753) * New resource `azurerm_key_vault_certificate_contacts` (hashicorp#19743) Close hashicorp#10256 Close hashicorp#16642 * CHANGELOG hashicorp#19743 * new resource: `azurerm_spring_cloud_customized_accelerator` (hashicorp#19736) * CHANGELOG hashicorp#19736 * new resource: `azurerm_billing_account_cost_management_export` (hashicorp#19723) Fixes hashicorp#14726 * CHANGELOG hashicorp#19723 * v3.37.0 * `azurerm_synapse_linked_service` - update document example (hashicorp#19768) * `azurerm_web_pubsub` - moving docs sub-category to messaging (hashicorp#19775) * Update settings.kt * Batch Fix LRO call bugs * update code * update code * update code * update code * update code * update code * `azurerm_postgresql_flexible_server` - Fix test case failure with "HA is disabled for region westus2" issue (hashicorp#19791) * Docs : Fix wrong description of `data.azurerm_key_vault_secret` in tf doc (hashicorp#19787) Fix hashicorp#19785 * Replace "primary_key" with "primary_connection_string" (hashicorp#19786) * update code * Updating based on c2554b6e * Fix missing/redundent props in doc (hashicorp#19821) * `azurerm_resource_deployment_script` - fix document error of `identity` (hashicorp#19820) * `azurerm_hdinsight_spark_cluster`, `azurerm_hdinsight_kafka_cluster`, `azurerm_hdinsight_interactive_query_cluster`, `azurerm_hdinsight_hbase_cluster`, `azurerm_hdinsight_hadoop_cluster`: Update documentations to match variable names in resources (hashicorp#19811) * `azurerm_app_service_connection`, `azurerm_spring_cloud_connection`, `azurerm_dev_test_global_vm_shutdown_schedule`, `azurerm_dev_test_policy`, `azurerm_dev_test_schedule`, `azurerm_stream_analytics_job`: Update documentations (hashicorp#19812) * `azurerm_site_recovery_replicated_vm`, `azurerm_sentinel_alert_rule_scheduled` and `azurerm_data_share_dataset_kusto_cluster` - fix document (hashicorp#19804) * `azurerm_kubernetes_cluster` - Fix incorrect allowed values and default value in document. (hashicorp#19773) Co-authored-by: zjhe <[email protected]> * `azurerm_datadog_monitor_sso_configuration` - fix default value for `name` to be a literal string (hashicorp#19771) * updating to include hashicorp#19792 * fixing inconsistency in document and code (hashicorp#19824) * Fixing doc of iothub (hashicorp#19829) * hardcode go version for link-milestone * fix (hashicorp#19838) * updating to include hashicorp#19698 * dependencies: updating to version `v0.20230103.1090844` of `github.com/hashicorp/go-azure-sdk` * doc fix: `azurerm_data_protection_backup_policy_disk` removes `resource_group_name` in doc (hashicorp#19839) * `document` remove unsetable `resource_group_name` in `azurerm_iot_time_series_insights_access_policy` (hashicorp#19835) * `document` - fix mismatch `zone` in `azurerm_dedicated_host_group` (hashicorp#19833) * `document` - Fix property name `protocol` in `azurerm_windows_virtual_machine` (hashicorp#19832) * Fix doc mismatch in `azurerm_orchestrated_virtual_machine_scale_set` (hashicorp#19831) * Update settings.kt * `document` - remove `tenant_id`in `azurerm_virutal_machine` which is not exported (hashicorp#19830) * `azurerm_monitor_activity_log_alert` - fix block conflicts (hashicorp#19827) resolves hashicorp#19797 * doc fix: add `azurerm_monitor_scheduled_query_rules_log` and `azurerm_monitor_scheduled_query_rules_alert` missed properties (hashicorp#19837) Co-authored-by: jackofallops <[email protected]> * Adding multiple data sources (hashicorp#19628) * CHANGELOG hashicorp#19628 * `azurerm_virtual_desktop_application_group` - Fix `azurerm_virtual_desktop_application_group` force new on `host_pool_id` change (hashicorp#19689) Fix hashicorp#19687 * CHANGELOG hashicorp#19689 * azurerm_linux/windows_web_app - add application_stack node 18-lts (hashicorp#19763) fixes hashicorp#19289 * `azurerm_route_server` - update resource creation future. (hashicorp#19772) * `azurerm_automation_software_update_configuration` - fix `expiry_time` is optional/computed (hashicorp#19774) Fixes hashicorp#19738 * CHANGELOG hashicorp#19774 * `azurerm_batch_pool` - Fix terraform fails to execute(re-run) if `azure_batch_pool` is deleted outside of terraform (hashicorp#19780) Fix hashicorp#19770 * CHANGELOG hashicorp#19780 * [WIP] `azurerm_mssql_database` : Fix test cases failing with "Provisioning is restricted in this region" issue (hashicorp#19794) * azurerm_storage_account_customer_managed_key - support ManagedHSM Key Vaults (hashicorp#19801) * fix documents (hashicorp#19826) Co-authored-by: kt <[email protected]> * `azurerm_postgresql_flexible_server` - fix acctest for `authentication` (hashicorp#19800) * CHANGELOG hashicorp#19800 * `azurerm_proximity_placement_group` - support `allowed_vm_sizes` and `zone` (hashicorp#19675) Close hashicorp#19379 * CHANGELOG hashicorp#19675 * `hdinsight_kafka_cluster_resource`: deprecate `kafka_management_node` in 4.0 (hashicorp#19423) * CHANGELOG hashicorp#19423 * `document` - fix vmss `ultra_ssd_disk_iops_read_write` and `ultra_ssd_disk_mbps_read_write` * `azurerm_kubernetes_cluster` - `scale_down_mode` can be updated (hashicorp#19823) * Update for hashicorp#19823 * updating to include hashicorp#19840 * updating to include hashicorp#19719 * New Resource: `azurerm_network_manager_network_group` (hashicorp#19593) * CHANGELOG hashicorp#19593 * `azurerm_stream_analytics_reference_input_blob`: Make `storage_account_key` optional when MSI auth is used (hashicorp#19676) Co-authored-by: Alexander Guth <[email protected]> * CHANGELOG hashicorp#19676 * `azurerm_eventhub` - fix capture feature not being set issue (hashicorp#19836) fix hashicorp#19782 * Fix several wrong logic with 404 response in read * dependencies: updating to `v0.20230105.1121404` of `github.com/hashicorp/go-azure-sdk` * Docs: Fix incorrect properties (hashicorp#19864) * updating to include hashicorp#19872 * updating to include hashicorp#19871 * `azurerm_synapse_linked_service` - report error during create/update (hashicorp#19849) * Update changelog for hashicorp#19849 * `azurerm_dedicated_host` - add support for`LSv3-Type1` type (hashicorp#19875) * Update changelog for hashicorp#19875 * Fix typo in README.md (hashicorp#19878) * `azurerm_storage_share` - Update document for property `access_tier` (hashicorp#19880) Fixes hashicorp#19819 * v3.38.0 * `azurerm_nginx_deployment` - add new GA sku for nginx deployment (hashicorp#19882) * add insensitive parser for APIM Api until migrated to go-azure-sdk * `azurerm_stream_analytics_stream_input_eventhub_v2` - Add an example for MSI auth (hashicorp#19679) * `document` add example for data source `azurerm_disk_encryption_set` (hashicorp#19907) * fix app conf feature no label update (hashicorp#19900) * data source `azurerm_shared_image` - support `purchase_plan` (hashicorp#19873) * Update for hashicorp#19873 and #199000 * `azurerm_logic_app_action_http` - `body` property support "@" symbol (hashicorp#19754) * Update for hashicorp#19754 * `document` - fix import command of `azurerm_api_management_identity_profider_aadb2c` (hashicorp#19910) * Update for hashicorp#19866 * support spark verison 3.3 (hashicorp#19866) * GHA - Add waiting-response label and comment for failures (hashicorp#19895) * `azurerm_databox_edge_device` - swap to typed sdk, add data source. (hashicorp#19914) * `azurerm_cost_anomaly_alert` - New resource (hashicorp#19899) Fixes hashicorp#18062 * CHANGELOG hashicorp#19899 * `azurerm_media_services_account` - support for `encryption`, `public_network_access_enabled` properties (hashicorp#19891) * CHANGELOG hashicorp#19891 * New Data Source: `azurerm_private_dns_resolver` (hashicorp#19885) * CHANGELOG hashicorp#19885 * `azurerm_eventhub_namespace` - adding wait before deletion (hashicorp#19165) * `azurerm_sentinel_automation_rule` - Support for `triggers_on`, `triggers_when` and `condition_json` (deprecates the `condition` property) (hashicorp#19309) Co-authored-by: kt <[email protected]> Fix hashicorp#18620 * CHANGELOG hashicorp#19309 * `Site Recovery` - swap sdk and upgrade to 2022-10-01, support `network_interface.is_primary` property (hashicorp#19571) Co-authored-by: kt <[email protected]> * CHANGELOG hashicorp#19571 * New Resource `azurerm_network_manager_subscription_connection` (hashicorp#19617) * CHANGELOG hashicorp#19617 * `azurerm_storage_account` - Add 403 (previously only 401) as a valid status code for lacking permissions to list keys (hashicorp#19645) Co-authored-by: kt <[email protected]> Fix hashicorp#19622 * CHANGELOG hashicorp#19645 * `azurerm_storage_account` - Add code check for `restore_policy` prerequisites (hashicorp#19822) Co-authored-by: kt <[email protected]> Fix hashicorp#19799 * New Resource: azurerm_lab_service_lab (hashicorp#19852) * CHANGELOG hashicorp#19852 * `azurerm_public_ip` - fix ddos diff from old version (hashicorp#19860) resolves hashicorp#19855 * `azurerm_maintenance_configuration` - support `in_guest_user_patch_mode` and `install_patches` properties (hashicorp#19865) fixes hashicorp#19790 * CHANGELOG hashicorp#19865 * `azurerm_web_pubsub_hub` - modify event handler type from typeSet to TypeList to respect the user's input order (hashicorp#19886) fix hashicorp#19876 * CHANGELOG hashicorp#19886 * CHANGELOG hashicorp#19825 * `azurerm_api_management_custom_domain` - Allow setting IdentityClientId for retrieving SSL cert from key vault (hashicorp#19881) * `azurerm_mysql_flexible_server` add suport for customer_managed_key (hashicorp#19905) fixes hashicorp#19842 * CHANGELOG hashicorp#19905 * `azurerm_storage_account` - Support `allowed_copy_scope` (hashicorp#19906) Fix hashicorp#19888 * CHANGELOG hashicorp#19906 * New Resource: `azurerm_private_endpoint_application_security_group_association` (hashicorp#19825) Co-authored-by: kt <[email protected]> Fixes hashicorp#17665 * Doc: `policy_set_definition`'s `policy_definition_id` should not use `policy_set_definition_id` (hashicorp#19929) Fixes hashicorp#19003 * Docs: fix missed properties in document of `automation`, `network`, `fluidrelay server`, etc. (hashicorp#19913) * Docs: fix tf doc issues of `azurerm_media_asset_filter`, `azurerm_media_live_event_output`, `azurerm_sql_database`, `azurerm_analysis_services_server` and `azurerm_mssql_managed_instance_vulnerability_assessment` (hashicorp#19937) * dox fix * update code * Initial check-in... (hashicorp#19927) * Updating based on 3fd84505 * `azurerm_kubernetes_cluster` - `api_server_vnet_integration_enabled` and `api_server_subnet_id` added (PREVIEW) (hashicorp#19438) * `azurerm_kubernetes_cluster` - `api_server_vnet_integration_enabled` and `api_server_subnet_id` added * AccTests * Docs * Make move from subnet to no subnet impossible * Rewrite of `api_server_*` properties to `api_server_access_profile` block * Add subnet id validation * Fix read * Enhance other tests and last doc thingiez * fmt * Fix tests with `private_cluster_on` * Implement comment * Update for hashicorp#19927 and hashicorp#19438 * `azurerm_log_analytics_data_export_rule` - fix `destination_resource_id` doesn't accept Event Hub Namespace (hashicorp#19868) * fix issue 19861 * update code * update code * Update for hashicorp#19868 * update api mangement diagnostic document (hashicorp#19960) Co-authored-by: xuzhang3 <Zhangxu894765> * update machine learning workspace doc (hashicorp#19959) Co-authored-by: xuzhang3 <Zhangxu894765> * `azurerm_vmware_netapp_volume_attachment` - update docs (hashicorp#19958) * fix delete logic for netapp volume * update netapp docs * update file extension * `azurerm_policy_set_definition` - fix empty group names on update (hashicorp#19890) * Add missing groupNames field to policy_set_definition update method (hashicorp#13791) * convert set to string[] * Add acceptance test for `policy_definition_reference.policy_group_names` * fix alignment of comment * remove unneeded comments * Update for hashicorp#19890 * Update GHA for PR Labelling to `CodelyTV/pr-size-labeler@v1` (hashicorp#19961) * resource `azurerm_storage_queue` - add `resource_manager_id` * `azurerm_storage_data_lake_gen2_path`/`azurerm_storage_data_lake_gen2_filesystem` - ACLs generated by default are no longer stored in state to prevent perpetual state diffs (hashicorp#18494) * Update changelog for hashicorp#18494 * Minor documentation change to fix issue 19766 (hashicorp#19974) * Correct changelog typo (hashicorp#19973) * minor frontdoor docs update (hashicorp#19975) * updating to include hashicorp#19969 * `azurerm_app_configuration_key` - fix no label regression issue (hashicorp#19979) * `azurerm_spring_cloud_api_portal` - fix failed testcases (hashicorp#19978) * `azurerm_monitor_diagnostic_setting` - deprecate `log` in favour of `enabled_log` (hashicorp#19504) * deprecate log in favour of enabked_log and disabled_log * deprecate log in favour of enabled_log * add check for empty string * add 4.0 beta flag and review comments * fix ids * use parser in update * fix tests * review comments * fix import id error * update changelog for hashicorp#19504 * `document` - remove redundant space in import commands (hashicorp#19986) * App Service Re-allignment (hashicorp#19685) * Updated for hashicorp#19685 * `document` - Fix wrong import commands (hashicorp#19984) * `azurerm_app_service_source_control_token` - fix import command (hashicorp#19987) * `external_id` needs to reference tenant id when adding AAD group (hashicorp#19981) * publish contributor docs with mkdocs (hashicorp#19883) * mkdocs update permissions (hashicorp#19990) * `azurerm_cosmosdb_account` - remove default value for `default_identity_type` per new API behavior (hashicorp#19956) * Update changelog for hashicorp#19956 * New Data Source: `azurerm_private_dns_resolver_dns_forwarding_ruleset` (hashicorp#19941) * Update changelog for hashicorp#19941 * New Data Source: `azurerm_private_dns_resolver_outbound_endpoint` (hashicorp#19950) * Update changelog for hashicorp#19950 * r/lab_service_lab: passwords are now sensitive / the flatten blocks now set an object (hashicorp#19988) * New Data Source: `azurerm_private_dns_resolver_inbound_endpoint` (hashicorp#19948) * Update changelog for hashicorp#19948 * New Resource `azurerm_media_services_account_filter` (hashicorp#19964) * CHANGELOG hashicorp#19964 * New Resource: `azurerm_lab_service_user` (hashicorp#19957) * CHANGELOG hashicorp#19957 * New Data Source: `azurerm_private_dns_resolver_virtual_network_link` (hashicorp#19951) * Update changelog for hashicorp#19951 * Update CHANGELOG.md (hashicorp#19993) * New resource - `azurerm_application_insights_standard_web_test` (hashicorp#19954) Fixes hashicorp#14184 * CHANGELOG hashicorp#19954 * New Data Source: `azurerm_private_dns_resolver_forwarding_rule` (hashicorp#19947) * Update changelog for hashicorp#19947 * `azurerm_spring_cloud_gateway` - support for the `application_performance_monitoring_types `, `environment_variables` `sensitive_environment_variables` properties (hashicorp#19884) * CHANGELOG hashicorp#19884 * New Resource `azurerm_network_manager_management_group_connection` (hashicorp#19621) * CHANGELOG hashicorp#19621 * New resource: `azurerm_storage_account_local_user` (hashicorp#19592) Co-authored-by: Tom Harvey <[email protected]> Co-authored-by: kt <[email protected]> Fix hashicorp#19536 * CHANGELOG hashicorp#19592 * New resource: `azurerm_sentinel_data_connector_threat_intelligence_taxii` (hashicorp#19209) * CHANGELOG hashicorp#19209 * v3.39.0 * `azurerm_cosmosdb_sql_container` - Fix property `included_path` can not be removed issue (hashicorp#19998) * fix issue 19901 * update code * update code * update code * `azurerm_log_analytics `- fixing crash during read (hashicorp#20011) * Update changelog for hashicorp#20011 * Update changelog for hashicorp#19998 * v3.39.1 * fugue PR 2: Remove calls to listKeys on storage * fugue PR 7: Make location optional for network interface * fugue PR 11: fixes for storage account * fugue PR 17: Toggle Printf logging with TF_LOG * fugue PR 18: remove validation func for snapshot name * fugue PR 20: Allow "all" access policy permissions for key vault * fugue PR 25: add configuration set to mysql and postgres * fugue PR 27: Remove storageAccessKey from sqldb extended_auditing_policy * fugue PR 28: Reader only access, adjust validations, dont read secrets * fugue PR 34: Removes admin_username and admin_password from the Container Registry schema * fugue PR 35: Reader permissions support for CosmosDB and Redis * fugue PR 36: Disable validation and remove password field in container_group * fugue PR 39: Added role_type to the Read function. * fugue PR 40: allow for missing OsProfile ComputerName and AdminUsername * fugue PR 42: Handle nil sku in vm scale sets * fugue PR 46: Profile strings can be nil * fugue PR 47: Azure scan failing for azurerm_virtual_machine_scale_set Looks like all the profile is nil * fugue PR 48: Impose 30-second timeout for azurerm_postgresql_configuration_set * fugue PR 51: Add rate limiting * chore: fixed the ID to fetch only the mysql/postgres server * chore: added debug info * chore: added update metodas so we can run it with terraform * feat: add provider module (CLOUD-1078) (#54) * fix permissions for azurerm_storage_account (#55) * comment out list keys Co-authored-by: kt <[email protected]> Co-authored-by: Xu Wu <[email protected]> Co-authored-by: Elena Xin <[email protected]> Co-authored-by: Zhenhua Hu <[email protected]> Co-authored-by: Neil Ye <[email protected]> Co-authored-by: Tao <[email protected]> Co-authored-by: Yichun Ma <[email protected]> Co-authored-by: Heng Lu <[email protected]> Co-authored-by: Xiaxin <[email protected]> Co-authored-by: keiranm <[email protected]> Co-authored-by: hc-github-team-tf-azure <> Co-authored-by: magodo <[email protected]> Co-authored-by: JT <[email protected]> Co-authored-by: ziyeqf <[email protected]> Co-authored-by: lonegunmanb <[email protected]> Co-authored-by: zjhe <[email protected]> Co-authored-by: Tom Harvey <[email protected]> Co-authored-by: Yun Liu <[email protected]> Co-authored-by: Steph <[email protected]> Co-authored-by: tombuildsstuff <[email protected]> Co-authored-by: jackofallops <[email protected]> Co-authored-by: steweg <[email protected]> Co-authored-by: Pierre-Rudolf Gerlach <[email protected]> Co-authored-by: Martijn Baay <[email protected]> Co-authored-by: Alexander Guth <[email protected]> Co-authored-by: Alexander Guth <[email protected]> Co-authored-by: Matthew Frahry <[email protected]> Co-authored-by: Assaf Giladi <[email protected]> Co-authored-by: Vaz <[email protected]> Co-authored-by: jackofallops <[email protected]> Co-authored-by: catriona-m <[email protected]> Co-authored-by: aristosvo <[email protected]> Co-authored-by: Paul Chvl <[email protected]> Co-authored-by: William <[email protected]> Co-authored-by: bwilczynski <[email protected]> Co-authored-by: Wodans Son <[email protected]> Co-authored-by: xuzhang3 <[email protected]> Co-authored-by: aurelschwitter <[email protected]> Co-authored-by: Stephen Jennings <[email protected]> Co-authored-by: dkuzmenok <[email protected]> Co-authored-by: Neil McG <[email protected]> Co-authored-by: Oscar <[email protected]> Co-authored-by: Robert Brandsø <[email protected]> Co-authored-by: ricardo-snyk <[email protected]> Co-authored-by: Evan Nemerson <[email protected]>

github-actions · 2023-01-13T19:16:26Z

This functionality has been released in v3.39.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!

github-actions · 2023-02-14T02:15:40Z

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.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

jackofallops requested a review from tombuildsstuff December 14, 2022 15:31

jackofallops self-assigned this Dec 14, 2022

github-actions bot added documentation service/app-service size/XL labels Dec 14, 2022

jackofallops mentioned this pull request Dec 14, 2022

azurerm_windows_function_app_slot - prevent removing application_stack related settings when stack block is not changed #18948

Closed

This was referenced Dec 22, 2022

Initial Creation of resource "azurerm_windows_web_app_slot" forces it to be created as 32 bit, regardless of setting "use_32_bit_worker" to false #19765

Closed

Support for Go 1.18 and 1.19 in windows/linux web app #19615

Closed

0gust1 mentioned this pull request Jan 3, 2023

azurerm_windows_web_app, azurerm_windows_web_app_slot - add node 18 support and fix node version not being set issue. #19343

Closed

katbyte approved these changes Jan 3, 2023

View reviewed changes

jackofallops force-pushed the ste/as-realign branch from 61b85a4 to bb9b485 Compare January 4, 2023 07:50

jackofallops mentioned this pull request Jan 4, 2023

azurerm_windows_web_app site config application stack is not in alignment with the supported versions #19850

Closed

1 task

mbfrahry requested changes Jan 5, 2023

View reviewed changes

github-actions bot added the waiting-response label Jan 5, 2023

tombuildsstuff suggested changes Jan 5, 2023

View reviewed changes

CalvinRossSmith reviewed Jan 5, 2023

View reviewed changes

internal/services/appservice/helpers/web_app_slot_schema.go Show resolved Hide resolved

github-actions bot removed the waiting-response label Jan 6, 2023

jackofallops mentioned this pull request Jan 6, 2023

APIM - add insensitive parser for APIM Api until migrated to go-azure-sdk #19887

Merged

jackofallops added 8 commits January 6, 2023 11:27

app-stacks pt1

26ad2fa

app stack changes for windows web app

8dd94d4

add node18 test

2bdd80c

java strings are fun

4a0c047

linuxFxVersion string processing

0a97d3d

add linuxFxString error capability

4d9c7b5

update decoding and tests for java on linux

9f6f11e

finish Go support on linux

5b5358b

jackofallops and others added 17 commits January 6, 2023 11:27

minor refactor for separation

1680563

windows function apps

507cece

windows function app slot

9900cf2

goimports

9ee52fc

update linux function app and slot and tests

4243501

fixup win func schema

8e328de

test fixes

2708ee3

node flatten fix

fe86fca

normalise cors

78ab076

more stack tweaks

e0f9ca0

linting fixes

5033a14

more linting fixes

bb205ec

unparam ignore on test config

94249db

update application_stack docks

4a08434

remove old node workaround

51689c0

review changes pt1

b382598

review changes pt2

78362e1

jackofallops force-pushed the ste/as-realign branch from bb9b485 to 78362e1 Compare January 9, 2023 10:35

jackofallops and others added 2 commits January 10, 2023 17:23

fix docker parts for windows slot

c5d59b9

remove completed TODOs

38ceaf4

jackofallops merged commit a5b0ec2 into main Jan 12, 2023

jackofallops deleted the ste/as-realign branch January 12, 2023 16:28

github-actions bot added this to the v3.39.0 milestone Jan 12, 2023

jackofallops added a commit that referenced this pull request Jan 12, 2023

Updated for #19685

3644c11

MattJeanes mentioned this pull request Feb 1, 2023

64 Bit platform and PowerShell Core Version not set for azurerm_windows_function_app #17029

Closed

1 task

Mralbert93 mentioned this pull request Feb 8, 2023

health_check_eviction_time_in_min is not applied to azurerm_linux_web_app in site_config block #16302

Closed

1 task

github-actions bot locked as resolved and limited conversation to collaborators Feb 14, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

App Service Re-allignment #19685

App Service Re-allignment #19685

jackofallops commented Dec 14, 2022

katbyte left a comment

xiaxyi commented Jan 4, 2023

jackofallops commented Jan 4, 2023

mbfrahry left a comment

mbfrahry Jan 4, 2023

jackofallops Jan 11, 2023

mbfrahry Jan 4, 2023

mbfrahry Jan 4, 2023

tombuildsstuff left a comment

tombuildsstuff Jan 5, 2023

jackofallops Jan 6, 2023 •

edited

Loading

tombuildsstuff Jan 5, 2023

jackofallops Jan 6, 2023

tombuildsstuff Jan 5, 2023

tombuildsstuff Jan 5, 2023

tombuildsstuff Jan 5, 2023

xiaxyi commented Jan 6, 2023

github-actions bot commented Jan 13, 2023

github-actions bot commented Feb 14, 2023

		@@ -888,7 +884,12 @@ func (m *WindowsFunctionAppSlotModel) unpackWindowsFunctionAppSettings(input web
		case "FUNCTIONS_EXTENSION_VERSION":

App Service Re-allignment #19685

App Service Re-allignment #19685

Conversation

jackofallops commented Dec 14, 2022

katbyte left a comment

Choose a reason for hiding this comment

xiaxyi commented Jan 4, 2023

jackofallops commented Jan 4, 2023

mbfrahry left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tombuildsstuff left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jackofallops Jan 6, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xiaxyi commented Jan 6, 2023

github-actions bot commented Jan 13, 2023

github-actions bot commented Feb 14, 2023

jackofallops Jan 6, 2023 •

edited

Loading