Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(influxdb3-clustered): update clustered helm chart #701

Merged
merged 5 commits into from
Jan 22, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
Loading