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

[kube-prometheus-stack] Shards PVC has too long name so that it cannot be provisioned #4523

Closed
kworkbee opened this issue May 8, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@kworkbee
Copy link

kworkbee commented May 8, 2024

Describe the bug a clear and concise description of what the bug is.

I'm deploying Prometheus via kube-prometheus-stack helm chart on AWS EKS environment, but operator is giving too long PVC name even though fullnameOverride value is short. Therefore, there's an issue that PVC cannot be provisioned.

image

What's your helm version?

3.14.3

What's your kubectl version?

1.28.2

Which chart?

kube-prometheus-stack

What's the chart version?

55.8.1

What happened?

No response

What you expected to happen?

No response

How to reproduce it?

No response

Enter the changed values of values.yaml?

No response

Enter the command that you execute and failing/misfunctioning.

Sync through ArgoCD

Anything else we need to know?

No response

@kworkbee kworkbee added the bug Something isn't working label May 8, 2024
@kworkbee kworkbee changed the title PVC has too long name so that it cannot be provisioned [kube-prometheus-stack] Shards PVC has too long name so that it cannot be provisioned May 8, 2024
@zeritti
Copy link
Member

zeritti commented May 8, 2024

Setting volumeClaimTemplate.metadata.name may help: #3392 (comment)

@kworkbee
Copy link
Author

kworkbee commented May 9, 2024

@zeritti It works! Thanks for your help 😄

@kworkbee kworkbee closed this as completed May 9, 2024
@kworkbee kworkbee reopened this May 9, 2024
@kworkbee
Copy link
Author

kworkbee commented May 9, 2024

Some of them still have long names...

@kworkbee
Copy link
Author

kworkbee commented May 9, 2024

AWS EFS access points have a limit of 100 characters for the full path, as described here
https://docs.aws.amazon.com/efs/latest/ug/efs-access-points.html#enforce-root-directory-access-point

But already unique id occupies 36 characters, so we have to track this issue

@kworkbee
Copy link
Author

kworkbee commented May 9, 2024

Aside from such issues, it certainly seems problematic that Prometheus is repeated in the currently generated PVC naming.

@jkroepke
Copy link
Member

jkroepke commented May 14, 2024

This to set the value

cleanPrometheusOperatorObjectNames: false

to true.

Other note: Please mention that it expected that the TSDB (prometheus database) may get corrupt on NFS based volumes, since it not an supported file storage.

@kworkbee
Copy link
Author

@jkroepke It solved clearly. Thank you for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants