TF updates - SFR3 and contribution guide (#2134)

Author: matebarabas

docs/content/contributing/terraform/composition.md

@@ -159,16 +159,11 @@ Please refer to the [Terraform Interfaces]({{% siteparam base %}}/specs/tf/inter
 
 ## Telemetry
 
-To meet [SFR3]({{% siteparam base %}}/spec/SFR3) & [SFR4]({{% siteparam base %}}/spec/SFR4).
-We use a telemetry provider [modtm](https://registry.terraform.io/providers/Azure/modtm/latest).
-This is a lightweight telemetry provider that sends telemetry data to Azure Application Insights via a HTTP POST front end service.
+To meet the requirements of [SFR3]({{% siteparam base %}}/spec/SFR3) & [SFR4]({{% siteparam base %}}/spec/SFR4), we use the [modtm](https://registry.terraform.io/providers/Azure/modtm/latest) telemetry provider. This lightweight telemetry provider sends telemetry data to Azure Application Insights via a HTTP POST front end service.
 
-The telemetry provider is included in the module by default and is enabled by default.
-You do not need to change the configuration included in the [template repo](https://github.com/Azure/terraform-azurerm-avm-template/blob/main/main.telemetry.tf).
+The `modtm` telemetry provider is included in all Terraform modules and is enabled by default through the [main.telemetry.tf](https://github.com/Azure/terraform-azurerm-avm-template/blob/main/main.telemetry.tf) file being automatically distributed from the template repo. You do not need to change this configuration.
 
-You must make sure to have the `modtm` provider in your `required_providers`.
-The linter will check this for you.
-However, inside your `terraform.tf` file please make sure you have this entry:
+Make sure that the `modtm` provider is listed under the `required_providers` section in the module's `terraform.tf` file using the following entry. This is also validated by the linter.
 
 ```terraform
 terraform {
@@ -182,8 +177,6 @@ terraform {
 }
 ```
 
-{{% include file="/static/includes/sample.telem.tf" language="terraform" options="linenos=false" %}}
-
 ## Eventual Consistency
 
 When creating modules, it is important to understand that the Azure Resource Manager (ARM) API is sometimes eventually consistent.

docs/content/specs-defs/includes/shared/shared/functional/SFR3.md

@@ -89,3 +89,23 @@ An example deployment name for a shortened module name would be: `46d3xbcp.res.d
 - [Terraform]({{% siteparam base %}}/contributing/terraform/)
 
 {{% /notice %}}
+
+### Terraform
+
+To enable telemetry data collection for Terraform modules, the [modtm](https://registry.terraform.io/providers/Azure/modtm/latest) telemetry provider **MUST** be used. This lightweight telemetry provider sends telemetry data to Azure Application Insights via a HTTP POST front end service.
+
+The `modtm` telemetry provider is included in all Terraform modules and is enabled by default through the [main.telemetry.tf](https://github.com/Azure/terraform-azurerm-avm-template/blob/main/main.telemetry.tf) file being automatically distributed from the template repo.
+
+The `modtm` provider **MUST** be listed under the `required_providers` section in the module's `terraform.tf` file using the following entry. This is also validated by the linter.
+
+```terraform
+terraform {
+  required_providers {
+    # .. other required providers as needed
+    modtm = {
+      source = "Azure/modtm"
+      version = "~> 0.3"
+    }
+  }
+}
+```