From 773aadbc34b71edfe81124bc4cf3f196f6c08e68 Mon Sep 17 00:00:00 2001
From: xyctruth <398041993@qq.com>
Date: Fri, 18 Feb 2022 15:00:20 +0800
Subject: [PATCH] update charts parameters and docs

---
 charts/README.md   | 56 ++++++++++++++++++++++++++++++----------------
 charts/values.yaml | 12 +++++-----
 2 files changed, 42 insertions(+), 26 deletions(-)

diff --git a/charts/README.md b/charts/README.md
index e0833d6..2af1863 100644
--- a/charts/README.md
+++ b/charts/README.md
@@ -1,35 +1,53 @@
-# DTM
+# DTM charts
 
 ## Usage
 
-To install the dtm chart:
+Install the dtm chart:
 
-    helm install --create-namespace -n dtm-system dtm .
+```bash
+helm install --create-namespace -n dtm-system dtm ./charts
+```
 
-To install the dtm chart:
+Upgrade the dtm chart:
 
-    helm upgrade -n dtm-system dtm .
+```bash
+helm upgrade -n dtm-system dtm ./charts
+```
 
-To uninstall the chart:
+Uninstall the chart:
 
-    helm delete -n dtm-system dtm
+```bash
+helm delete -n dtm-system dtm
+```
 
 ## Parameters
 
 ### Configuration parameters
 
-| Key           | Description                                        | Value |
-| ------------- | -------------------------------------------------- | ----- |
-| configuration | DTM configuration. Specify content for config.yaml | ""    |
+| Key             | Description                                                                                                                           | Value |
+|-----------------|---------------------------------------------------------------------------------------------------------------------------------------|-------|
+| `configuration` | DTM configuration. Specify content for `config.yaml`, ref: [sample config](https://github.com/dtm-labs/dtm/blob/main/conf.sample.yml) | `""`  |
+
+
+
+### Autoscaling Parameters
+
+| Name                                            | Description                               | Value   |
+|-------------------------------------------------|-------------------------------------------|---------|
+| `autoscaling.enabled`                           | Enable Horizontal POD autoscaling for DTM | `false` |
+| `autoscaling.minReplicas`                       | Minimum number of DTM replicas            | `1`     |
+| `autoscaling.maxReplicas`                       | Maximum number of DTM replicas            | `10`    |
+| `autoscaling.targetCPUUtilizationPercentage`    | Target CPU utilization percentage         | `80`    |
+| `autoscaling.targetMemoryUtilizationPercentage` | Target Memory utilization percentage      | `80`    |
 
 ### Ingress parameters
 
-| Key                          | Description                                                                     | Value             |
-| ---------------------------- | ------------------------------------------------------------------------------- | ----------------- |
-| ingress.enabled              | Enable ingress record generation for DTM                                        | false             |
-| ingress.className            | IngressClass that will be be used to implement the Ingress (Kubernetes 1.18+)   | "nginx"           |
-| ingress.annotations          | To enable certificate autogeneration, place here your cert-manager annotations. | {}                |
-| ingress.hosts.host           | Default host for the ingress record.                                            | "your-domain.com" |
-| ingress.hosts.paths.path     | Default path for the ingress record                                             | "/"               |
-| ingress.hosts.paths.pathType | Ingress path type                                                               | "Prefix"          |
-| ingress.tls                  | Enable TLS configuration for the host defined at ingress.hostname parameter     | []                |
+| Key                            | Description                                                                      | Value               |
+|--------------------------------|----------------------------------------------------------------------------------|---------------------|
+| `ingress.enabled`              | Enable ingress record generation for DTM                                         | `false`             |
+| `ingress.className`            | IngressClass that will be be used to implement the Ingress (Kubernetes 1.18+)    | `"nginx"`           |
+| `ingress.annotations`          | To enable certificate auto generation, place here your cert-manager annotations. | `{}`                |
+| `ingress.hosts.host`           | Default host for the ingress record.                                             | `"your-domain.com"` |
+| `ingress.hosts.paths.path`     | Default path for the ingress record                                              | `"/"`               |
+| `ingress.hosts.paths.pathType` | Ingress path type                                                                | `"Prefix"`          |
+| `ingress.tls`                  | Enable TLS configuration for the host defined at ingress.hostname parameter      | `[]`                |
diff --git a/charts/values.yaml b/charts/values.yaml
index cd5827e..ef1717f 100644
--- a/charts/values.yaml
+++ b/charts/values.yaml
@@ -26,12 +26,10 @@ imagePullSecrets: []
 nameOverride: ""
 fullnameOverride: ""
 
-podSecurityContext:
-  {}
+podSecurityContext: {}
   # fsGroup: 2000
 
-securityContext:
-  {}
+securityContext: {}
   # capabilities:
   #   drop:
   #   - ALL
@@ -47,9 +45,9 @@ resources:
 autoscaling:
   enabled: false
   minReplicas: 1
-  maxReplicas: 100
+  maxReplicas: 10
   targetCPUUtilizationPercentage: 80
-  # targetMemoryUtilizationPercentage: 80
+  targetMemoryUtilizationPercentage: 80
 
 nodeSelector: {}
 
@@ -64,7 +62,7 @@ service:
     grpc: 36790
 
 ingress:
-  enabled: false
+  enabled: true
   className: "nginx"
   annotations:
     {}