Redis statestore field redisPassword incorrectly marked as required in docs
jake-engelberg opened this issue · comments
EDIT (contrib maintainer): The contrib repo metadata was correct. The docs are wrong. The docs instead need updating.
Expected Behavior
redisPassword
should be a required field in the statestore .yaml
as per the docs:
https://docs.dapr.io/reference/components-reference/supported-pubsub/setup-redis-pubsub/
Actual Behavior
redisPassword
is incorrectly set as required: false
.
https://github.com/dapr/components-contrib/blob/787b23d5d0a3486fc14255f26b36cf365c9bd2bf/state/redis/metadata.yaml#L24-L29
Steps to Reproduce the Problem
N/A
Release Note
RELEASE NOTE: FIX Redis statestore field redisPassword
now correctly marked as required.
The docs are incorrect and the metadata file is actually correct. I'll transfer this issue to the docs repo and you can then update the docs there instead.
Thanks!
@jake-engelberg @berndverst it looks like the docs are already updated to show redisPassword
as a required field in all Redis component specs.
@jake-engelberg @berndverst it looks like the docs are already updated to show
redisPassword
as a required field in all Redis component specs.
ope, just realized what we're asking for. I'll update the docs to show as not required.