cetic / helm-fadi

Helm Chart for FADI
https://github.com/cetic/fadi
Apache License 2.0
13 stars 4 forks source link

Adding InfluxDB Helm chart #44

Closed zakaria2905 closed 2 years ago

zakaria2905 commented 2 years ago

Adding InfluxDB in Chart.yaml file Adding enable configuration in values.yaml file Adding doc link in README.md file

We chose the influxdb chart of bitnami because it release activity is higher: https://artifacthub.io/packages/search?ts_query_web=influxdb&sort=relevance&page=1

banzo commented 2 years ago

Seems there is a linting issue:

>> Building chart...
>>> helm lint /root/project/fadi
coalesce.go:196: warning: cannot overwrite table with non table for service (map[annotations:map[] clusterIP: loadBalancerIP: loadBalancerSourceRanges:[] nodePorts:map[http: rpc:] port:8086 rpcPort:8088 sessionAffinity: sessionAffinityConfig:map[] type:ClusterIP])
coalesce.go:196: warning: cannot overwrite table with non table for service (map[annotations:map[] clusterIP: loadBalancerIP: loadBalancerSourceRanges:[] nodePorts:map[http: rpc:] port:8086 rpcPort:8088 sessionAffinity: sessionAffinityConfig:map[] type:ClusterIP])
==> Linting /root/project/fadi
[ERROR] templates/: template: fadi/charts/influxdb/templates/influxdb/service.yaml:11:19: executing "fadi/charts/influxdb/templates/influxdb/service.yaml" at <.Values.influxdb.service.annotations>: can't evaluate field annotations in type interface {}

Error: 1 chart(s) linted, 1 chart(s) failed
banzo commented 2 years ago

Some preliminary review comments: