[stable/influxdb] Move bind_address under rpc header and update image…

… to 1.7 (helm#9384) * Update image to 1.7 Signed-off-by: Naseem Ullah <naseemkullah@gmail.com> * move bind_address under rpc heading Signed-off-by: Naseem Ullah <naseemkullah@gmail.com> * Version stable, adding OWNERS, adding Upgrading in README Signed-off-by: Naseem Ullah <naseemkullah@gmail.com>
FairwindsOps · Dec 31, 2018 · b63f665 · b63f665
1 parent 2622e6a
commit b63f665
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 10 deletions.
diff --git a/stable/influxdb/Chart.yaml b/stable/influxdb/Chart.yaml
@@ -1,6 +1,6 @@
 name: influxdb
-version: 0.12.1
-appVersion: 1.6
+version: 1.0.0
+appVersion: 1.7
 description: Scalable datastore for metrics, events, and real-time analytics.
 keywords:
 - influxdb

diff --git a/stable/influxdb/OWNERS b/stable/influxdb/OWNERS
@@ -0,0 +1,4 @@
+approvers:
+- jackzampolin
+reviewers:
+- jackzampolin
diff --git a/stable/influxdb/README.md b/stable/influxdb/README.md
@@ -80,3 +80,9 @@ Influxdb requires also a user to be set in order for authentication to be enforc
 To handle this setup on startup, a job can be enabled in `values.yaml` by setting `.Values.setDefaultUser.enabled` to `true`.
 
 Make sure to uncomment or configure the job settings after enabling it. If a password is not set, a random password will be generated.
+
+## Upgrading
+
+### From < 1.0.0 To >= 1.0.0
+
+Values `.Values.config.bind_address` and `.Values.exposeRpc` no longer exist. They have been replaced with `.Values.config.rpc.bind_address` and `.Values.config.rpc.enabled` respectively. Please adjust your values file accordingly.
diff --git a/stable/influxdb/templates/config.yaml b/stable/influxdb/templates/config.yaml
@@ -11,7 +11,7 @@ metadata:
 data:
   influxdb.conf: |+
     reporting-disabled = {{ .Values.config.reporting_disabled | default false }}
-    bind-address = ":{{ .Values.config.bind_address }}"
+    bind-address = ":{{ .Values.config.rpc.bind_address }}"
 
     [meta]
       dir = "{{ .Values.config.storage_directory }}/meta"

diff --git a/stable/influxdb/templates/service.yaml b/stable/influxdb/templates/service.yaml
@@ -58,10 +58,10 @@ spec:
     port: {{ .Values.config.opentsdb.bind_address }}
     targetPort: {{ .Values.config.opentsdb.bind_address }}
   {{- end }}
-  {{- if .Values.exposeRpcService }}
+  {{- if .Values.config.rpc.enabled }}
   - name: rpc
-    port: {{ .Values.config.bind_address }}
-    targetPort: {{ .Values.config.bind_address }}
+    port: {{ .Values.config.rpc.bind_address }}
+    targetPort: {{ .Values.config.rpc.bind_address }}
   {{- end }}
   selector:
     app: {{ template "influxdb.fullname" . }}
diff --git a/stable/influxdb/values.yaml b/stable/influxdb/values.yaml
@@ -2,7 +2,7 @@
 ## ref: https://hub.docker.com/r/library/influxdb/tags/
 image:
   repo: "influxdb"
-  tag: "1.6-alpine"
+  tag: "1.7-alpine"
   pullPolicy: IfNotPresent
 
 ## Specify a service type
@@ -117,8 +117,10 @@ env:
 ## ref: https://docs.influxdata.com/influxdb/v1.1/administration/config/
 config:
   reporting_disabled: false
-  bind_address: 8088
   storage_directory: /var/lib/influxdb
+  rpc:
+    enabled: true
+    bind_address: 8088
   meta:
     retention_autocreate: true
     logging_enabled: true
@@ -231,5 +233,3 @@ config:
     log_enabled: true
     enabled: true
     run_interval: 1s
-
-exposeRpcService: true