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

Memory requests/limits units convertion #728

Open
2 tasks done
chaos-mind opened this issue Sep 27, 2024 · 3 comments
Open
2 tasks done

Memory requests/limits units convertion #728

chaos-mind opened this issue Sep 27, 2024 · 3 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed pinned Prevents stalebot from removing

Comments

@chaos-mind
Copy link

chaos-mind commented Sep 27, 2024

What happened?

When memory resources set in Mi for an application, Goldilocks incorrectly displays them in Details.
For example, when I set 3125Mi, Goldilocks shows 3906250Ki. But when I set 3200M, Goldilocks shows 3200M.
image
image

What did you expect to happen?

Goldilocks shows correct values in M if resources set in Mi.
For example, if I set 3125Mi, Goldilocks shows 3200M.

How can we reproduce this?

Set memory requests/limits in Mi for your application and check if Goldilocks shows resources correctly in M.

Version

Application version v4.13.0, chart version 9.0.0

Search

  • I did search for other open and closed issues before opening this.

Code of Conduct

  • I agree to follow this project's Code of Conduct

Additional context

No response

@chaos-mind chaos-mind added bug Something isn't working triage This bug needs triage labels Sep 27, 2024
@sudermanjr
Copy link
Member

This has been an issue for a while. Would love to see someone propose a solution. Units conversion in K8s is a rather frustrating problem to work with.

@sudermanjr sudermanjr added help wanted Extra attention is needed good first issue Good for newcomers pinned Prevents stalebot from removing and removed triage This bug needs triage labels Sep 27, 2024
@sudermanjr
Copy link
Member

Previous report for more details #78

@sudermanjr
Copy link
Member

And here #395

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 help wanted Extra attention is needed pinned Prevents stalebot from removing
Projects
None yet
Development

No branches or pull requests

2 participants