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

EVEREST-1516 | Use VictoriaMetrics Helm chart to install Everest monitoring #416

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft
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
7 changes: 5 additions & 2 deletions charts/everest/Chart.lock
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,8 @@ dependencies:
- name: everest-db-namespace
repository: file://charts/everest-db-namespace
version: 0.0.0
digest: sha256:de45f56b3032db8bc6e21fc0858bc56d1b8701e7c1edfcc18c61285945235826
generated: "2024-11-05T15:38:39.638426+05:30"
- name: victoria-metrics-operator
repository: https://victoriametrics.github.io/helm-charts
version: 0.37.0
digest: sha256:5a31bb7a9b06f09f6ede94ac79bf15182d312ecfea36881be64e2981f1b5ae1a
generated: "2024-11-15T23:27:17.821571+05:30"
5 changes: 5 additions & 0 deletions charts/everest/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,8 @@ dependencies:
version: 0.0.0
repository: "file://charts/everest-db-namespace"
condition: "everest-db-namespace.enabled"
- name: victoria-metrics-operator
alias: monitoring
version: 0.37.0
repository: "https://victoriametrics.github.io/helm-charts"
condition: "monitoring.enabled"
8 changes: 7 additions & 1 deletion charts/everest/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,13 @@ The following table shows the configurable parameters of the Percona Everest cha
|-----|------|---------|-------------|
| compatibility.openshift | bool | `false` | Enable OpenShift compatibility. If set, ignores olm.install and olm.namespace settings. |
| everest-db-namespace.enabled | bool | `false` | Do not enable. |
| monitoring.namespace | string | `"everest-monitoring"` | Namespace where monitoring is installed. Do no change unless you know what you are doing. |
| monitoring.admissionWebhooks.enabled | bool | `false` | |
| monitoring.certManager.enabled | bool | `false` | |
| monitoring.crds.enabled | bool | `false` | |
| monitoring.crds.plain | bool | `true` | |
| monitoring.enabled | bool | `true` | If set, installs VictoriaMetrics operator. |
| monitoring.nameOverride | string | `"vm-operator"` | |
| monitoring.namespaceOverride | string | `"everest-monitoring"` | |
| namespaceOverride | string | `""` | Namespace override. Defaults to the value of .Release.Namespace. |
| olm.catalogSourceImage | string | `"perconalab/everest-catalog"` | Image to use for Everest CatalogSource. |
| olm.image | string | `"quay.io/operator-framework/olm@sha256:1b6002156f568d722c29138575733591037c24b4bfabc67946f268ce4752c3e6"` | Image to use for the OLM components. |
Expand Down
5 changes: 0 additions & 5 deletions charts/everest/templates/everest-monitoring/hooks.yaml

This file was deleted.

4 changes: 0 additions & 4 deletions charts/everest/templates/everest-monitoring/namespace.yaml

This file was deleted.

This file was deleted.

12 changes: 0 additions & 12 deletions charts/everest/templates/everest-monitoring/subscription.yaml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ spec:
{{- if .Values.operator.enableLeaderElection }}
- --leader-elect
{{- end }}
- --monitoring-namespace={{ .Values.monitoring.namespace }}
- --monitoring-namespace={{ .Values.monitoring.namespaceOverride }}
- --system-namespace={{ include "everest.namespace" . }}
command:
- /manager
Expand Down
6 changes: 6 additions & 0 deletions charts/everest/templates/monitoring-namespace.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{{- if .Values.monitoring.enabled }}
apiVersion: v1
kind: Namespace
metadata:
name: {{ .Values.monitoring.namespaceOverride }}
{{- end }}
16 changes: 13 additions & 3 deletions charts/everest/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,19 @@ olm:
tlsCert: ""
# -- Client key for the PackageServer APIService. Overrides the tls.type setting.
tlsKey: ""
monitoring:
# -- Namespace where monitoring is installed. Do no change unless you know what you are doing.
namespace: everest-monitoring
everest-db-namespace:
# -- Do not enable.
enabled: false

monitoring:
# -- If set, installs VictoriaMetrics operator.
enabled: true
namespaceOverride: everest-monitoring
nameOverride: vm-operator
crds:
enabled: false
plain: true
admissionWebhooks:
enabled: false
certManager:
enabled: false
Loading