[hlf-couchdb] Update Owners + bug fixes and best-practices (#7246)

* Add OWNERS file

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

* Add Hyperledger Fabric logo

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

* Typo fix

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

* Update persistence according to Helm best practices

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

* Bump version up

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

* Fix typo

Signed-off-by: Alejandro Vicente Grabovetsky <sasha@aid.technology>

stable/hlf-couchdb/.helmignore

@@ -19,3 +19,6 @@
 .project
 .idea/
 *.tmproj
+
+# OWNERS file for Helm repository
+OWNERS

stable/hlf-couchdb/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 description: CouchDB instance for Hyperledger Fabric (these charts are created by AID:Tech and are currently not directly associated with the Hyperledger project)
 name: hlf-couchdb
-version: 1.0.4
+version: 1.0.5
 appVersion: 0.4.9
 keywords:
   - blockchain
@@ -16,5 +16,4 @@ maintainers:
     email: sasha@aid.technology
   - name: nicolapaoli
     email: nicola@aid.technology
-## Icon not included while sorting out trademark question with Hyperledger project
-# icon: https://www.hyperledger.org/wp-content/uploads/2018/04/fabric-logo.png
+icon: https://www.hyperledger.org/wp-content/uploads/2018/04/fabric-logo.png

stable/hlf-couchdb/OWNERS

@@ -0,0 +1,6 @@
+approvers:
+- alexvicegrab
+- nicolapaoli
+reviewers:
+- alexvicegrab
+- nicolapaoli

stable/hlf-couchdb/README.md

@@ -86,7 +86,7 @@ The following table lists the configurable parameters of the Hyperledger Fabric
 
 ## Persistence
 
-The volume stores the Fabric CouchdDB data and configurations at the `/opt/couchdb/data` path of the container.
+The volume stores the Fabric CouchDB data and configurations at the `/opt/couchdb/data` path of the container.
 
 The chart mounts a [Persistent Volume](http://kubernetes.io/docs/user-guide/persistent-volumes/) at this location. The volume is created using dynamic volume provisioning through a PersistentVolumeClaim managed by the chart.
 

stable/hlf-couchdb/templates/pvc.yaml

@@ -16,6 +16,10 @@ spec:
     requests:
       storage: {{ .Values.persistence.size | quote }}
   {{- if .Values.persistence.storageClass }}
-  storageClassName: {{ .Values.persistence.storageClass | quote }}
+  {{- if (eq "-" .Values.persistence.storageClass) }}
+  storageClassName: ""
+  {{- else }}
+  storageClassName: "{{ .Values.persistence.storageClass }}"
+  {{- end }}
   {{- end }}
 {{- end }}

stable/hlf-couchdb/values.yaml

@@ -29,7 +29,12 @@ ingress:
 persistence:
   enabled: true
   annotations: {}
-  ## If unset or "", use "default" storage class.
+  ## If defined, storageClassName: <storageClass>
+  ## If set to "-", storageClassName: "", which disables dynamic provisioning
+  ## If undefined (the default) or set to null, no storageClassName spec is
+  ##   set, choosing the default provisioner.  (gp2 on AWS, standard on
+  ##   GKE, AWS & OpenStack)
+  ##
   storageClass: ""
   accessMode: ReadWriteOnce
   size: 1Gi