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

[demo] -Set flagd memory limit to 50Mi #1161

Merged
merged 4 commits into from
May 1, 2024

Conversation

puckpuck
Copy link
Contributor

@puckpuck puckpuck commented Apr 30, 2024

This replaces #1131 and fixes issue 1536 in the demo repo.

Increases the memory limit for FlagD to be 50Mi instead of 20Mi

This also bumps the Flagd version the latest release (0.10.1)

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
…lagd-memory-limit

# Conflicts:
#	charts/opentelemetry-demo/Chart.yaml
#	charts/opentelemetry-demo/examples/bring-your-own-observability/rendered/component.yaml
#	charts/opentelemetry-demo/examples/bring-your-own-observability/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/bring-your-own-observability/rendered/serviceaccount.yaml
#	charts/opentelemetry-demo/examples/collector-as-daemonset/rendered/component.yaml
#	charts/opentelemetry-demo/examples/collector-as-daemonset/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/collector-as-daemonset/rendered/grafana-dashboards.yaml
#	charts/opentelemetry-demo/examples/collector-as-daemonset/rendered/serviceaccount.yaml
#	charts/opentelemetry-demo/examples/custom-environment-variables/rendered/component.yaml
#	charts/opentelemetry-demo/examples/custom-environment-variables/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/custom-environment-variables/rendered/grafana-dashboards.yaml
#	charts/opentelemetry-demo/examples/custom-environment-variables/rendered/serviceaccount.yaml
#	charts/opentelemetry-demo/examples/default/rendered/component.yaml
#	charts/opentelemetry-demo/examples/default/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/default/rendered/grafana-dashboards.yaml
#	charts/opentelemetry-demo/examples/default/rendered/serviceaccount.yaml
#	charts/opentelemetry-demo/examples/kubernetes-infra-monitoring/rendered/component.yaml
#	charts/opentelemetry-demo/examples/kubernetes-infra-monitoring/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/kubernetes-infra-monitoring/rendered/grafana-dashboards.yaml
#	charts/opentelemetry-demo/examples/kubernetes-infra-monitoring/rendered/serviceaccount.yaml
#	charts/opentelemetry-demo/examples/public-hosted-ingress/rendered/component.yaml
#	charts/opentelemetry-demo/examples/public-hosted-ingress/rendered/flagd-config.yaml
#	charts/opentelemetry-demo/examples/public-hosted-ingress/rendered/grafana-dashboards.yaml
#	charts/opentelemetry-demo/examples/public-hosted-ingress/rendered/serviceaccount.yaml
Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
@TylerHelmuth TylerHelmuth merged commit 7b8b56e into open-telemetry:main May 1, 2024
3 checks passed
@puckpuck puckpuck deleted the demo.flagd-memory-limit branch May 1, 2024 02:36
12ushan pushed a commit to giffgaff/opentelemetry-helm-charts that referenced this pull request Jul 22, 2024
* Set flagd memory limit to 50Mi

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

* Set flagd memory limit to 50Mi

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

* generate examples after merge

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

---------

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flagd CrashLoopBackOff
4 participants