search

OT-CONTAINER-KIT / helm-charts

A repository which that will contain helm charts with best and security practices.
https://ot-container-kit.github.io/helm-charts
49 stars 84 forks source link

Helm Chart CRDs for Redis Operator are not up to date #197

Open alita1991 opened 3 months ago

alita1991 commented 3 months ago

Does this issue reproduce with the latest release? Yes

What operating system and processor architecture are you using (kubectl version)?

kubectl version Output
$ kubectl version
Client Version: v1.29.3

What did you do? Installed the latest redis-operator helm chart (tested with the helm chart from helm-charts repo and also with the one from redis-operator repo)

What did you expect to see? redisReplicationPassword field should exist in RedisSentinel CRD

What did you see instead? redisReplicationPassword field is not found in RedisSentinel CRD

Note The CRD update is important to allow Redis Sentinel to connect to Redis Replication, when auth is activated

drivebyer commented 3 months ago

The Redis Operator Helm chart is maintained at https://github.com/OT-CONTAINER-KIT/redis-operator/tree/master/charts/redis-operator. If you have any issues, feel free to submit them to the Redis Operator repository. Thank you!

alita1991 commented 3 months ago

Hi @drivebyer, I even looked into the helm chart from the Redis Operator repository, we have the same issue there, the CRDs were not updated.

drivebyer commented 3 months ago

Absolutely, the CRD hasn't been updated yet. We'll make sure to update it following the next image release. @alita1991

pasztorl commented 2 months ago

@drivebyer I see that redis-operator version now 0.17.0. Please also push an image to https://quay.io/repository/opstree/redis-operator because the latest version is 0.15.1 Thanks!

drivebyer commented 2 months ago

@pasztorl, actually I do not have permission for quay.io.