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

Replicas + resources.total.limits #721

Closed
devoxel opened this issue Dec 21, 2023 · 2 comments · Fixed by #725
Closed

Replicas + resources.total.limits #721

devoxel opened this issue Dec 21, 2023 · 2 comments · Fixed by #725
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@devoxel
Copy link

devoxel commented Dec 21, 2023

The resource sizing logic does not take into account the number of replicas assigned.

To demonstrate, here is the following TempoStack definition

apiVersion: tempo.grafana.com/v1alpha1
kind: TempoStack
metadata:
  name: simplest
spec:
  images:
    tempo: docker.io/grafana/tempo:x.y.z
    tempoQuery: docker.io/grafana/tempo-query:x.y.z
    tempoGateway: quay.io/observatorium/api
    tempoGatewayOPA: quay.io/observatorium/opa-openshift
  storage:
    secret:
      name: minio-test
      type: s3
  resources:
    total:
      limits:
        memory: 50Gi
        cpu: 10000m
# uncomment and test the difference in output
# template:
#   compactor:
#     replicas: 5
  storageSize: 1Gi

The diff between the version with a comment and without is:

<   replicas: 5
---
>   replicas: 1

I think this behaviour is not intuitive, and might lead to people over-provisioning their cluster by accident.

@andreasgerstmayr andreasgerstmayr added bug Something isn't working good first issue Good for newcomers labels Dec 21, 2023
@andreasgerstmayr
Copy link
Collaborator

Thanks for the report! Yep, the assigned limits per component should be divided by the number of replicas for this component.

@rafiramadhana
Copy link
Contributor

can I help with this? thx

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

Successfully merging a pull request may close this issue.

3 participants