-
Notifications
You must be signed in to change notification settings - Fork 60
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
Prometheus PVC name too long #563
Comments
@furynick so after some testing I would suggest that if you're using openebs to add the following to the configuration to make this work.
The main issue is with After talking about this issue with the team I think the best path forward is to make the configuration of the volume claim optional in tobs and document how to make it work depending on the option you are wanting to set. |
Sorry for late answer but it doesn't seem to make any difference unless I made a mistake in syntax or misunderstood your comment.
|
I fixed command line : kube-prometheus-stack.prometheus.prometheusSpec.storageSpec.volumeClaimTemplate.metadata.name instead of kube-prometheus-stack.storageSpec.volumeClaimTemplate.metadata.name Any other idea to workaround or any clue of a mistake on my command line ? |
The PR I created (prometheus-community/helm-charts#2415) to allow you to set the PVC name through kube-prometheus-stack was just merged. So once tobs is updated to use that version you should be able to template the PVC names of the volumes that are created with kube-prometheus-stack. |
Thanks, I'll keep an eye on new tobs releases and give it a try. |
Fixed ! PVC names are slightly shorter with following syntax and OpenEBS volumes are successfully provisionned. helm install \
-n monitoring \
--create-namespace \
--set timescaledbExternal.enabled=true \
--set opentelemetry-operator.enabled=false \
--set kube-prometheus-stack.prometheus.prometheusSpec.storageSpec.volumeClaimTemplate.metadata.name=data \
--set promscale.connection.uri="postgres://$DB_USER:$DB_PASS@$DB_HOST:$DB_PORT/$DB_NAME?sslmode=require" \
--set timescaledb-single.enabled=false \
tobs timescale/tobs |
What happened?
Installing with tobs helm chart, Prometheus DB volume PVC Name is more than 63 characters which is not supported by CSI.
Did you expect to see something different?
PVC Name should be less than 63 characters
How to reproduce it (as minimally and precisely as possible):
helm install \ -n monitoring \ --create-namespace \ --set opentelemetry-operator.enabled=false \ --set promscale.connection.uri=postgres://prometheus:xxxxxx@externaldatabase:5432/prometheus?sslmode=require \ --set timescaledb-single.enabled=false \ tobs timescale/tobs
Environment
Kubernetes version information:
kubectl version
v1.24.3
Kubernetes cluster kind:
On Premise kubeadm
tobs Logs:
The text was updated successfully, but these errors were encountered: