Use memcached modern recommended options instead (#1221)

* Use memcached modern recommended options instead

* Allow setting custom Memcached options with a sane default

* Bump major version since this might break backwards compatibilty by changing defaults

* don't use '-o' if extendedOptions is empty

* Update Chart.yaml

* We set default for memcached verbosity to 'v' in values.yml

* fix options

stable/memcached/Chart.yaml

@@ -1,5 +1,5 @@
 name: memcached
-version: 1.0.1
+version: 1.1.0
 description: Free & open source, high-performance, distributed memory object caching system.
 keywords:
 - memcached

stable/memcached/templates/statefulset.yaml

@@ -37,7 +37,11 @@ spec:
         command:
         - memcached
         - -m {{ .Values.memcached.maxItemMemory  }}
-        {{- if default "" .Values.memcached.verbosity }}
+        {{- if .Values.memcached.extendedOptions }}
+        - -o
+        - {{ .Values.memcached.extendedOptions }}
+        {{- end }}
+        {{- if .Values.memcached.verbosity }}
         - -{{ .Values.memcached.verbosity }}
         {{- end }}
         ports:
@@ -54,4 +58,4 @@ spec:
           initialDelaySeconds: 5
           timeoutSeconds: 1
         resources:
-{{ toYaml .Values.resources | indent 10 }}
+{{ toYaml .Values.resources | indent 10 }}

stable/memcached/values.yaml

@@ -19,6 +19,7 @@ memcached:
   ##
   maxItemMemory: 64
   verbosity: v
+  extendedOptions: modern
 
 ## Configure resource requests and limits
 ## ref: http://kubernetes.io/docs/user-guide/compute-resources/