MagicStack / MagicStack/asyncpg
Pool connect ignores DNS ttl
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 8.1k
- Forks
- 468
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
- 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
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in asyncpg/pool.py ungefähr bei Zeile 133 und verfolge, wie Pool den PostgreSQL-Host auflöst und wiederverwendet. Prüfe das gemeldete AWS-RDS-Failover-Szenario und wie DNS-Änderungen unter asyncio behandelt werden. Die Aufgabe ist abgeschlossen, wenn der Pool sich nach dem Failover wieder mit der aktualisierten Adresse verbindet, statt die alte aufgelöste IP beizubehalten, und eine Regressionstestabdeckung für dieses Verhalten vorhanden ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- postgresql, python
- Bereich
- backend, databases
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100