redis receiver should allow to use a username in addition to password

[nsteinmetz]

Component(s)

receiver/redis

What happened?

Description

With Redis 7, a new ACL mechanism was deployed and there is no longer a single user (ie requirepass property).

Thus, it would be nice to pass a username and a password to redis when you have a dedicated user for monitoring instead of default one.

Steps to Reproduce

Set ACL in redis.conf

user default on > some_password ~* +@all
user monitoring_user on > monitoring_password ~* +@admin +@dangerous

Expected Result

receivers:
    redis:
      endpoint: localhost:6379
      collection_interval: 10s
      username: monitoring_user
      password: monitoring_password

Actual Result

Only default password works to authenticate to redis

receivers:
    redis:
      endpoint: localhost:6379
      collection_interval: 10s
      password: some_password

Collector version

0.103.0

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

[github-actions]

Pinging code owners:

• receiver/redis: @dmitryax @hughesjj

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[rogercoll]

The current redis receiver has a username configuration option: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/redisreceiver/config.go#L29

Is this what you were referring to? Or an issue with the actual username configuration?

In the OpenTelemetry demo we are using the default ACL for Valkey which only requires a username: https://github.com/open-telemetry/opentelemetry-demo/blob/main/src/otelcollector/otelcol-config.yml#L18

[crobert-1]

Adding the documentation label as the config option should be in the README

[nsteinmetz]

@rogercoll my bad, I only refered to the documentation, which does not mention it

I can confirm it works once username is added.

So only a documentation issue.