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

[flagd-ui] increase memory to 100M #2120

Merged
merged 2 commits into from
Mar 14, 2025

Conversation

puckpuck
Copy link
Contributor

@puckpuck puckpuck commented Mar 14, 2025

Changes

Fixes #1825

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.

@puckpuck puckpuck requested a review from a team as a code owner March 14, 2025 02:01
@github-actions github-actions bot added the helm-update-required Requires an update to the Helm chart when released label Mar 14, 2025
@julianocosta89
Copy link
Member

@puckpuck, reviewing this PR I've noticed that we have flagd on the minimal version of the demo, but we do not have flagd-ui.

Should we add it?

Also, it would be great to know if anyone is actually using the minimal.
The Demo is way too big now, I'm not sure if there are folks using the minimal version at all.

@julianocosta89 julianocosta89 merged commit c8cee24 into open-telemetry:main Mar 14, 2025
31 checks passed
@puckpuck puckpuck deleted the flagd-ui.increase-memory branch March 15, 2025 01:11
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.

flagdui container crashing due to OOMKilled
2 participants