MagicStack / MagicStack/asyncpg
Pool connect ignores DNS ttl
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 8.1k
- Forks
- 468
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
- asyncpg version: 0.14
- PostgreSQL version: 9.5/9.6
- Do you use a PostgreSQL SaaS? If so, which? Can you reproduce
the issue with a local PostgreSQL install?: Using AWS RDS - Python version: 3.6
- Platform: Linux in docker
- Do you use pgbouncer?: Not in this instances
- Did you install asyncpg with pip?: yes
- If you built asyncpg locally, which version of Cython did you use?:
- Can the issue be reproduced under both asyncio and
uvloop?: We only use asyncio
After performing a failover/reboot of an AWS RDS instance asyncpg does not appear to reconnect gracefully - it looks like the code in Pool caches the IP of the PG instance after dns resolution and does not respect whatever TTL we might have set on the cname that points at our master: https://github.com/MagicStack/asyncpg/blob/master/asyncpg/pool.py#L133
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza en asyncpg/pool.py alrededor de la línea 133 y sigue cómo Pool resuelve y reutiliza el host de PostgreSQL. Comprueba el escenario de failover de AWS RDS reportado y cómo se gestionan los cambios de DNS en asyncio. Se considera terminado cuando el pool vuelve a conectarse a la dirección actualizada después del failover en lugar de conservar la IP resuelta antigua, con cobertura de regresión para este comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- postgresql, python
- Área
- backend, databases
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100