Backport of docs: update 1.4.0 Helm docs per Docs team feedback into …

…release/1.4.x (#3715)

backport of commit 9b309da

Co-authored-by: Michael Zalimeni <[email protected]>

charts/consul/values.yaml

@@ -623,7 +623,7 @@ global:
     # @type: boolean
     disableAgentHostName: false
 
-    # Configures consul agent underlying host metrics. Only applicable if
+    # Configures consul agent underlying host metrics. Default is false.
     # Only applicable if `global.metrics.enabled` and `global.metrics.enableAgentMetrics` is true.
     # @type: boolean
     enableHostMetrics: false
@@ -646,14 +646,12 @@ global:
     # @type: boolean
     enableTelemetryCollector: false
 
-    # This configures the list of filter rules to apply for allowing/blocking
+    # Configures the list of filter rules to apply for allowing or blocking
     # metrics by prefix in the following format:
     #
     # A leading "+" will enable any metrics with the given prefix, and a leading "-" will block them.
     # If there is overlap between two rules, the more specific rule will take precedence.
     # Blocking will take priority if the same prefix is listed multiple times.
-    #
-    # - allowList:
     prefixFilter:
       # @type: array<string>
       allowList: []
@@ -693,8 +691,7 @@ global:
 
       # Configures Kubernetes Prometheus/OpenMetrics auto-discovery annotations for use with Datadog.
       # This configuration is less common and more for advanced usage with custom metrics monitoring
-      # configurations. See https://docs.datadoghq.com/containers/kubernetes/prometheus/?tab=kubernetesadv2 for more details
-      # surround further configuration.
+      # configurations. Refer to the [Datadog documentation](https://docs.datadoghq.com/containers/kubernetes/prometheus/?tab=kubernetesadv2) for more details.
       openMetricsPrometheus:
         # @default: false
         # @type: boolean
@@ -905,14 +902,14 @@ global:
 
   # Consul feature flags that will be enabled across components.
   # Supported feature flags:
-  # * `resource-apis`:
-  #   _**Danger**_! This feature is under active development. It is not
+  # - `resource-apis`:
+  #   _**Warning**_! This feature is under active development. It is not
   #   recommended for production use. Setting this flag during an
   #   upgrade could risk breaking your Consul cluster.
   #   If this flag is set, Consul components will use the
   #   V2 resources APIs for all operations.
-  # * `v2tenancy`:
-  #   _**Danger**_! This feature is under active development. It is not
+  # - `v2tenancy`:
+  #   _**Warning**_! This feature is under active development. It is not
   #   recommended for production use. Setting this flag during an
   #   upgrade could risk breaking your Consul cluster.
   #   If this flag is set, Consul V2 resources (catalog, mesh, auth, etc)
@@ -1164,7 +1161,7 @@ server:
     enabled: true
 
     # The maximum number of unavailable pods. In most cases you should not change this as it is automatically set to
-    # the correct number when left as null. This setting has been kept to not break backwards compatibility.
+    # the correct number when left as null. This setting has been kept to preserve backwards compatibility.
     #
     # By default, this is set to 1 internally in the chart. When server pods are stopped gracefully, they leave the Raft
     # consensus pool. When running an odd number of servers, one server leaving the pool does not change the quorum