Merge pull request #5 from puckpuck/puckpuck-patch-1

[seashell] - clarify default for rbac.readAll

Author: puckpuck

charts/seashell/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: seashell
 description: A tool that provides a shell to debug Kubernetes workloads
 type: application
-version: 0.3.1
+version: 0.3.2
 appVersion: 0.1
 keywords:
   - debug

charts/seashell/README.md

@@ -3,18 +3,6 @@
 Installs and configures [seashell](https://github.com/puckpuck/seashell) to
 provide tools for debugging Kubernetes workloads.
 
-## Warning
-
-The ClusterRole used to set up this tool provides read access to **ALL**
-resources
-in your Kubernetes cluster. This isn't recommended for anything that even comes
-close to a production environment. Please know what the fuck you are doing
-before
-you deploy this thing blindly to your Kubernetes cluster.
-
-You can modify this by setting `rbac.create` to `false` and providing a
-ClusterRole and ClusterRoleBinding for the ServiceAccount used with this tool.
-
 ## Installing the chart
 
 This chart should run without the need to specify any custom chart values.
@@ -26,6 +14,11 @@ helm install seashell puckpuck/seashell
 
 ## Parameters
 
+> **Warning**
+> Setting `rbac.readAll` to true will grant read access to ALL resources in your Kubernetes cluster. 
+> This isn't recommended for anything that even comes close to a production environment. 
+> Please know what the fuck you are doing before you deploy this thing blindly to your Kubernetes cluster.
+
 | Parameter                    | Description                                                            | Default                                          |
 |------------------------------|------------------------------------------------------------------------|--------------------------------------------------|
 | `image.repository`           | seashell Image repository                                              | `puckpuck/seashell`                              |