Skip to content

Commit

Permalink
chore(influxdb3-clustered): update clustered helm chart (#701)
Browse files Browse the repository at this point in the history
* fix(influxdb3-clustered): rename "hostingEnvironment.eks" to "hostingEnvironment.aws"

* chore(influxdb3-clustered): add "seviceMonitor" configuration

* fix: specify "target-branch" for ct

* chore(influxdb3-clustered): increment chart version

* chore(influxdb3-clustered): add "resources.catalog" key
  • Loading branch information
bnpfeife authored Jan 22, 2025
1 parent bea415a commit ec62b46
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/helm-charts-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,13 +35,13 @@ jobs:
- name: Run chart-testing (list-changed)
id: list-changed
run: |
changed=$(ct list-changed)
changed=$(ct list-changed --target-branch=master)
if [[ -n "$changed" ]]; then
echo "changed=true" >> "$GITHUB_OUTPUT"
fi
- name: Run chart-testing (lint)
run: ct lint
run: ct lint --target-branch=master

- name: Create kind cluster
uses: helm/kind-action@v1
Expand Down
2 changes: 1 addition & 1 deletion charts/influxdb3-clustered/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v2
type: application

version: 0.1.9
version: 0.1.10
appVersion: "20241217-1494922"
name: influxdb3-clustered
description: InfluxDB 3.0 Clustered
Expand Down
10 changes: 8 additions & 2 deletions charts/influxdb3-clustered/templates/app-instance.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ spec:
spec:
{{- if .Values.hostingEnvironment}}
hostingEnvironment:
{{- if hasKey .Values.hostingEnvironment "eks"}}
eks: {{.Values.hostingEnvironment.eks | include "mapTrim" | trim | nindent 10}}
{{- if hasKey .Values.hostingEnvironment "aws"}}
aws: {{.Values.hostingEnvironment.aws | include "mapTrim" | trim | nindent 10}}
{{- else if hasKey .Values.hostingEnvironment "openshift"}}
openshift: {{.Values.hostingEnvironment.openshift | include "mapTrim" | trim | nindent 10}}
{{- else if hasKey .Values.hostingEnvironment "gke"}}
Expand Down Expand Up @@ -67,6 +67,12 @@ spec:
{{- if .Values.featureFlags}}
featureFlags: {{.Values.featureFlags | toYaml | nindent 6}}
{{- end}}
{{- if .Values.observability.serviceMonitor}}
observability:
serviceMonitor:
interval: {{required "missing observability.serviceMonitor.interval" .Values.observability.serviceMonitor.interval}}
scrapeTimeout: {{required "missing observability.serviceMonitor.scrapeTimeout" .Values.observability.serviceMonitor.scrapeTimeout}}
{{- end}}
---
{{- end }}

Expand Down
12 changes: 11 additions & 1 deletion charts/influxdb3-clustered/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ operatorImageOverride: ""
skipOperator: false

hostingEnvironment: {}
# eks:
# aws:
# eksRoleArn: "" # "arn:aws:iam::111111111111:role/your-influxdb-clustered-role"
# # Note: there are currently no OpenShift-specific parameters. The empty object is still required for proper function
# openshift: {}
Expand Down Expand Up @@ -184,6 +184,10 @@ resources:
router:
requests:
limits:
# The Catalog stores metadata related to schema information and physical locations of partitions.
catalog:
requests:
limits:

admin:
# The list of users to grant access to Clustered via influxctl
Expand Down Expand Up @@ -221,3 +225,9 @@ egress:

# This can be used to enable certain features that are still in testing
featureFlags: []

# Parameters to tune observability configuration, such as Prometheus ServiceMonitor's.
observability: {}
# serviceMonitor:
# interval: 10s
# scrapeTimeout: 30s

0 comments on commit ec62b46

Please sign in to comment.