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] release 1.11.0 #1232

Merged
merged 7 commits into from
Jun 28, 2024
Merged

Conversation

rogercoll
Copy link
Contributor

@rogercoll rogercoll commented Jun 25, 2024

  1. The OpenTelemetry demo replaced the Redis usage in favor of Valkey: [Valkey] Replace Redis with Valkey opentelemetry-demo#1619

The current helm chart needs to be updated before next OpenTelemetry demo release. If used demo's nightly image for the deployment, the following error arised for the cartservice pod:

Defaulted container "cartservice" out of: cartservice, wait-for-redis (init)
VALKEY_ADDR environment variable is required.
  1. Increase memory limit for account service: https://github.com/open-telemetry/opentelemetry-demo/pull/1538/files#diff-e45e45baeda1c1e73482975a664062aa56f20c03dd9d64a827aba57775bed0d3R32

output files of running make generate-examples
@rogercoll
Copy link
Contributor Author

Test are not passing because it's not using nightly demo images. Should we merge/release this once the new demo images are generated?

@julianocosta89
Copy link
Member

This change should take affect just after a new Demo version is released.

@rogercoll rogercoll changed the title [demo] Replace Redis with Valkey [demo] release 1.11.0 Jun 28, 2024
charts/opentelemetry-demo/Chart.yaml Outdated Show resolved Hide resolved
@TylerHelmuth TylerHelmuth merged commit 829ce87 into open-telemetry:main Jun 28, 2024
3 checks passed
@rogercoll rogercoll deleted the replace_redis_valkey branch June 29, 2024 12:27
12ushan pushed a commit to giffgaff/opentelemetry-helm-charts that referenced this pull request Jul 22, 2024
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.

4 participants