Limita conexoes Redis idle

[rondinelisaad]

O que esse PR faz?

Configura django-redis para usar BlockingConnectionPool com limite de conexoes, timeout de pool, socket timeouts e health check em local e producao.

Adiciona limites para pools Redis usados pelo Celery broker/backend para reduzir acumulo de sockets idle por processo.

Preserva CELERY_BROKER_URL quando definido no entrypoint de producao, permitindo separar broker/cache sem sobrescrever com REDIS_URL.

Onde a revisão poderia começar?

Apliquei uma correção em:
config/settings/production.py (line 27): django-redis agora usa BlockingConnectionPool, com REDIS_CACHE_MAX_CONNECTIONS, timeout de pool, socket timeout e health check.

config/settings/local.py (line 16): mesma configuração para local.

config/settings/base.py (line 490): limites para pools Redis do Celery: CELERY_BROKER_POOL_LIMIT=5 e CELERY_REDIS_MAX_CONNECTIONS=20 por padrão.

compose/production/django/entrypoint (line 10): não força mais CELERY_BROKER_URL=REDIS_URL se você definir CELERY_BROKER_URL explicitamente.

Como este poderia ser testado manualmente?

Validei com Docker, sim.

O que passou:

docker compose -f local.yml config: Compose parseou ok. Só apareceu o aviso já esperado de version obsoleto.
Build da imagem local scielo_core_local_django: passou.
Redis local subiu e respondeu.
config.settings.local: cache.set/get funcionou contra Redis.
config.settings.production: validei a parte Redis/Celery sem django.setup(), porque a imagem local não tem anymail, dependência de produção.
O pool aplicado ficou correto:
pool_class=BlockingConnectionPool
max_connections=20
pool_timeout=2
health_check_interval=30
celery_broker_pool_limit=5
celery_redis_max_connections=20
Sintaxe do /entrypoint: bash -n /entrypoint passou.
redis-cli INFO clients após os testes mostrou connected_clients:1, sem conexões presas pelos containers one-shot.
Parei o Redis local que eu tinha subido para a validação.
Resumo: as mudanças estão funcionais no Docker e os limites de pool estão efetivamente sendo aplicados.

Referências

Correção elabora pelo codex gpt

[Copilot]

Pull request overview

Este PR ajusta a configuração de conexões Redis para evitar acúmulo de sockets idle (especialmente em cenários com múltiplos processos/replicas), limitando pools no cache (django-redis) e no Celery, e tornando o entrypoint mais flexível ao não sobrescrever um CELERY_BROKER_URL já definido.

Changes:

• Configura o cache Redis (local e produção) para usar redis.BlockingConnectionPool com limites e timeouts via variáveis de ambiente.
• Adiciona limites padrão para pools Redis do Celery (CELERY_BROKER_POOL_LIMIT e CELERY_REDIS_MAX_CONNECTIONS).
• Ajusta o entrypoint de produção para preservar CELERY_BROKER_URL quando já estiver definido.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
config/settings/production.py	Aplica BlockingConnectionPool, limites e socket timeouts no cache Redis em produção.
config/settings/local.py	Espelha a configuração do pool/timeouts do cache Redis para ambiente local.
config/settings/base.py	Define limites padrão de pool para conexões Redis usadas pelo Celery.
compose/production/django/entrypoint	Não sobrescreve CELERY_BROKER_URL se já estiver definido, caindo para REDIS_URL caso contrário.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.