From 0eeb7fdb4f1f42bb82a41d64b075ce8cbfc40605 Mon Sep 17 00:00:00 2001 From: ReSearchITEng Date: Thu, 8 Jun 2017 18:45:26 +0300 Subject: [PATCH] fine tuning Great addition! Few suggestions: i. namespace used till now for operator was: rook (not rook-operator). ii. `helm delete --purge` (note the --purge), so one can retry again later with the same name. With regards to charts.rook.io -> why not in the https://github.com/kubernetes/charts/tree/master/incubator , so you won't need to deploy a helmet yourself? --- demo/helm/rook-operator/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/demo/helm/rook-operator/README.md b/demo/helm/rook-operator/README.md index c4e6dc4fe..841d0da19 100644 --- a/demo/helm/rook-operator/README.md +++ b/demo/helm/rook-operator/README.md @@ -15,7 +15,7 @@ Alternatively, to deploy from a local checkout of the rook codebase (until the r ```console $ cd demo/helm/rook-operator -$ helm install --name rook-operator --namespace rook-operator . +$ helm install --name rook-operator --namespace rook . ``` Introduction @@ -66,7 +66,7 @@ Uninstalling the Chart To uninstall/delete the `rook-operator` deployment: ```console -$ helm delete rook-operator +$ helm delete --purge rook-operator ``` The command removes all the Kubernetes components associated with the chart and deletes the release.