Alternate configuration for Redis

[Oumph]

Actuellement redis est dans un conteneur différent de la partie RoR (sur le même hôte). Il y a donc besoin de surcharger la configuration pour ce redis qui n'est pas sur 127.0.0.1 / localhost. Par contre on ne fait rien niveau authentification ou support TLS, donc à réserver à un réseau local...

Les REDIS_HOST/PORT/URL semblent être des variables standards. Le REDIS_CACHE_URL est spécifique à la configuration de notre CMS.

[echarp]

Normalement, tu peux aussi utiliser la syntaxe: ENV.fetch("REDIS_CACHE_URL", "redis://localhost:6379/1/cache"), ça permet de gérer la valeur par défaut.

[Trim]

Pour information, j‘avais eu justement un besoin similaire pour configurer redis pour l‘environnement de développement pour les containers: https://github.com/linuxfrorg/linuxfr.org/blob/0f970f466c01307e145a9c5d362b50539851cfa4/deployment/default.env#L5-L9 ```env # Redis service is used as a caching service and a publish/subscribe service # to allow the different LinuxFr tools work together REDIS_HOST=redis REDIS_PORT=6379 REDIS_URL=redis://redis:6379/0 ``` C‘est ensuite utilisé dans la ligne de commande pour démarrer le service `img-Linuxfr.org`: https://github.com/linuxfrorg/linuxfr.org/blob/0f970f466c01307e145a9c5d362b50539851cfa4/deployment/linuxfr-img/Dockerfile#L51 ```dockerfile CMD ["sh", "-c", "exec ./img-LinuxFr.org -a 0.0.0.0:8000 -r \"${REDIS_URL##redis://}\""] ``` C‘est aussi utilisé pour `board`, il faut lui partager les 2 variables d'environnement (`REDIS_HOST` et `REDIS_PORT`): https://github.com/linuxfrorg/board-sse-linuxfr.org/blob/43959c1712c87baa8baba34a48466099ac6ca13b/lib/board-sse-linuxfr.org/redis_plugin.rb#L15 ```ruby @redis = Redis.new(host: ENV['REDIS_HOST'] || "localhost", port: ENV['REDIS_PORT'] || 6379) ``` Dans cette PR, on pourrait peut être ajouter une note sur les service `img-LinuxFr.org` et `board` qui doivent être connectés au même service Redis pour qu'ils interagissent correctement avec LinuxFr. Enfin, dans la branche pour Rails 8, nous avons le fichier `config/cable.yml` qui utilise aussi cette variable, mais je ne sais pas à quoi ça correspond: https://github.com/linuxfrorg/linuxfr.org/blob/07d30220aa598ac2bd2f242d816bf1b7b1d0c32e/config/cable.yml#L9 ```yml url: <%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" } %> ```

[Oumph]

Ce qui semble utilisé comme convention par d'autres :

https://www.rubydoc.info/gems/redis/5.4.1
exemple avec Redis.new(url: ENV["REDIS_URL"]) et le code lib/redis/client.rb avec url: -> { ENV["REDIS_URL"] }

ou bien

./share-LinuxFr.org/tests/mastodon/mastodon/spec/lib/mastodon/redis_configuration_spec.rb:        ClimateControl.modify REDIS_PASSWORD: 'testpass', REDIS_HOST: 'redis.example.com', REDIS_PORT: '3333', REDIS_DB: '3' do

Dans notre cas on a besoin de:

• REDIS_HOST (localhost par défaut)
• REDIS_PORT (6379 par défaut)
• REDIS_URL (redis://[[username:]password@]host[:port][/database] tant qu'on n'a pas mis de TLS) cf https://github.com/redis/lettuce/wiki/Redis-URI-and-connection-details
• REDIS_DB (0 pour les données et 1 pour le cache)

par contre le redis_cache_store dit prendre une REDIS_URL (cf https://guides.rubyonrails.org/caching_with_rails.html#activesupport-cache-rediscachestore ) et du coup je ne comprends pas la config actuelle avec le /cache dans "redis://localhost:6379/1/cache" (dedans les clés sont en /views/... pas en /cache par exemple)

[Oumph]

irb(main):001:0> require "redis"
=> true
irb(main):005:0> redis = Redis.new(url: "redis://x.x.x.x:6379/1/sdflshgdjkghuieuihjkghjkhgdjkhgd")
=> #<Redis client v4.0.2 for redis://x.x.x.x:6379/1>
irb(main):006:0> redis.get("mykey")
=> nil
irb(main):007:0> redis.set("mykey", "toto")
=> "OK"
irb(main):008:0> redis.get("mykey")
=> "toto"

a priori le /cache est juste là pour faire joli

[Oumph]

Cf 649f5a0 qui précise que c'est le 'namespace' https://github.com/nono/redis-store

-  config.cache_store = :redis_store, "redis://linuxfr.org:6379/0"
+  config.cache_store = :redis_store, "redis://linuxfr.org:6379/0/cache"

https://github.com/nono/redis-store/blob/1e611c18bb1292042b2ef9ae6916164ce49fbfe4/lib/redis/store.rb#L25

Et j'avais raté ça dans la doc https://api.rubyonrails.org/v8.1.2/classes/ActiveSupport/Cache/RedisCacheStore.html

No namespace is set by default. Provide one if the Redis cache server is shared with other apps: namespace: 'myapp-cache'.

[Oumph]

(conseil de lecture: mettre la musique de Retour vers le futur)

nono-redis-store avait une notion de namespace.

52c7ae3

-  gem "nono-redis-store",      "~>1.0"
+  gem "redis-store",           "~>1.0"

redis-store 1.0 avait une notion de namespace cf redis-store/redis-store@7b24cb1

b955509

-  gem "redis-store",           "~>1.0"
+  gem "redis-activesupport",   "~>3.2"

puis f602114 "Use the new redis cache store from Rails 5.2"

ce qui nous ramène à

No namespace is set by default. Provide one if the Redis cache server is shared with other apps: namespace: 'myapp-cache'.

bref c'est un namespace pour ActiveSupport https://api.rubyonrails.org/v8.1.2/classes/ActiveSupport/Cache/RedisCacheStore.html , pas pour Redis lui-même. Et a priori faut le garder.