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

Update memory limits #1931

Merged
merged 3 commits into from
Jan 22, 2025
Merged

Conversation

mviitane
Copy link
Member

@mviitane mviitane commented Jan 21, 2025

Changes

Increase memory limits for

  • frontend-proxy
  • kafka
  • grafana
  • opensearch

Fix for #1831

Measurement

After running the app for around 5 hours on Docker / Ubuntu / x86_64, I see the following memory usage:

$ docker stats --no-stream --format "table {{.Name}}\t{{.MemPerc}}\t{{.MemUsage}}" | sort -k 2 -h
NAME              MEM %     MEM USAGE / LIMIT
image-provider    4.34%     5.211MiB / 120MiB
recommendation    8.95%     44.77MiB / 500MiB
valkey-cart       17.68%    3.535MiB / 20MiB
shipping          18.91%    3.781MiB / 20MiB
currency          19.86%    3.973MiB / 20MiB
prometheus        30.72%    92.16MiB / 300MiB
frontend          39.31%    98.29MiB / 250MiB
quote             40.11%    16.04MiB / 40MiB
cart              41.37%    66.19MiB / 160MiB
email             47.32%    47.32MiB / 100MiB
flagd-ui          49.86%    37.4MiB / 75MiB
payment           59.42%    71.3MiB / 120MiB
checkout          62.25%    12.45MiB / 20MiB
product-catalog   68.85%    13.77MiB / 20MiB
load-generator    70.95%    1.039GiB / 1.465GiB
accounting        72.90%    87.48MiB / 120MiB
fraud-detection   73.02%    219.1MiB / 300MiB
jaeger            73.66%    883.9MiB / 1.172GiB
ad                75.09%    225.3MiB / 300MiB
otel-collector    78.05%    156.1MiB / 200MiB
flagd             83.37%    62.53MiB / 75MiB
frontend-proxy    88.43%    57.48MiB / 65MiB
kafka             88.50%    548.7MiB / 620MiB
opensearch        94.22%    1.018GiB / 1.08GiB
grafana           96.48%    110MiB / 114MiB

Merge Requirements

For new features contributions, please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@mviitane mviitane requested a review from a team as a code owner January 21, 2025 12:09
@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Jan 21, 2025
Copy link
Contributor

@rogercoll rogercoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mviitane mviitane merged commit 513586b into open-telemetry:main Jan 22, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm-update-required Requires an update to the Helm chart when released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants