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

[redisstorageextension] allow redis key prefix to be specified in config #37677

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HubertFraczek
Copy link

Description

The purpose of this PR is to allow the user to specify the redis key prefix. The lack of this feature has become problematic when using multiple replicas in a kubernetes cluster - until now, the key prefix has been based on the storage extension and receiver names, which are the same for each replica because they share the same configuration and therefore overwrite the same key in redis. With the changes in this PR, the user can specify the prefix as an environmental variable (which can have different value for each replica), for example.

Testing

opentelemetry-collector-contrib (0.118.0) was built with this change and tested on a kubernetes cluster

Config used for testing:

redis_storage:
  endpoint: redis.monitoring.svc.cluster.local:6379
  prefix: ${K8S_NODE_NAME}

Documentation

Updated README.md to add new config parameter.

@HubertFraczek HubertFraczek requested review from atoulme and a team as code owners February 4, 2025 14:55
Copy link

linux-foundation-easycla bot commented Feb 4, 2025

CLA Signed

  • ✅login: HubertFraczek / (a270e3f)

The committers listed above are authorized under a signed CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants