Address review comments

Signed-off-by: Aylei <rayingecho@gmail.com>
pingcap · Apr 16, 2019 · f6bebaf · f6bebaf
1 parent 2b43106
commit f6bebaf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/charts/tidb-cluster/templates/tidb-service.yaml b/charts/tidb-cluster/templates/tidb-service.yaml
@@ -18,7 +18,7 @@ spec:
   externalTrafficPolicy: {{ .Values.tidb.service.externalTrafficPolicy }}
   {{- end }}
   {{- if .Values.tidb.service.loadBalancerIP }}
-  loadBalancerIP: {{ .Values.tidb.service.loadBalancerIP }}
+  loadBalancerIP: {{ .Values.tidb.service.loadBalancerIP }
   {{- end }}
   ports:
   - name: mysql-client

diff --git a/docs/setup.md b/docs/setup.md
@@ -125,6 +125,8 @@ $ rm -rf /mnt/disks/local-pv01
 $ rm -rf /data/local-pv01
 ```
 
+You should also delete the related entries in `/etc/fstab` at the same time, otherwise the automount may cause problems when the machine restarts.
+
 > Note: The local-volume plugin expects paths to be stable, if you remove a previous mount-point in the discovery directory (default to `/mnt/disks/`), you should remove the PV manually to keep consistency.
 
 ## Install TiDB Operator