From 248c2adb2e00fb72514ab55d235a55eaf0ac97e0 Mon Sep 17 00:00:00 2001
From: Anmol Sethi <hi@nhooyr.io>
Date: Fri, 13 Nov 2020 17:08:05 -0500
Subject: [PATCH] helm: Fix README examples

Not sure where --name came from? Maybe an older version of helm.

Ah, it's from v2.16.7
---
 ci/helm-chart/README.md   | 23 +++++++++++++----------
 ci/helm-chart/values.yaml |  2 +-
 2 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/ci/helm-chart/README.md b/ci/helm-chart/README.md
index cb34d274..601596c1 100644
--- a/ci/helm-chart/README.md
+++ b/ci/helm-chart/README.md
@@ -10,8 +10,9 @@ This chart is community maintained by [@Matthew-Beckett](https://github.com/Matt
 ## TL;DR;
 
 ```console
-$ git clone https://github.com/cdr/code-server.git
-$ helm install code-server charts/code-server
+$ git clone https://github.com/cdr/code-server
+$ cd code-server
+$ helm upgrade --install code-server ci/helm-chart
 ```
 
 ## Introduction
@@ -26,10 +27,12 @@ package manager.
 
 ## Installing the Chart
 
-To install the chart with the release name `my-release`:
+To install the chart with the release name `code-server`:
 
 ```console
-$ helm install --name my-release charts/code-server
+$ git clone https://github.com/cdr/code-server
+$ cd code-server
+$ helm upgrade --install code-server ci/helm-chart
 ```
 
 The command deploys code-server on the Kubernetes cluster in the default
@@ -40,10 +43,10 @@ that can be configured during installation.
 
 ## Uninstalling the Chart
 
-To uninstall/delete the `my-release` deployment:
+To uninstall/delete the `code-server` deployment:
 
 ```console
-$ helm delete my-release
+$ helm delete code-server
 ```
 
 The command removes all the Kubernetes components associated with the chart and
@@ -97,9 +100,9 @@ Specify each parameter using the `--set key=value[,key=value]` argument to `helm
 install`. For example,
 
 ```console
-$ helm install --name my-release \
-  --set persistence.enabled=false \
-    deployment/chart
+$ helm install code-server \
+    ci/helm-chart \
+    --set persistence.enabled=false
 ```
 
 The above command sets the the persistence storage to false.
@@ -108,7 +111,7 @@ Alternatively, a YAML file that specifies the values for the above parameters
 can be provided while installing the chart. For example,
 
 ```console
-$ helm install --name my-release -f values.yaml deployment/chart
+$ helm install code-server ci/helm-chart -f values.yaml
 ```
 
 > **Tip**: You can use the default [values.yaml](values.yaml)
diff --git a/ci/helm-chart/values.yaml b/ci/helm-chart/values.yaml
index 5cc74185..e71499bf 100644
--- a/ci/helm-chart/values.yaml
+++ b/ci/helm-chart/values.yaml
@@ -112,7 +112,7 @@ persistence:
   ##
   # storageClass: "-"
   accessMode: ReadWriteOnce
-  size: 1Gi
+  size: 10Gi
   annotations: {}
   # existingClaim: ""
   # hostPath: /data