docs(tradein/domclick): два комментария описывали пул, которого нет с миграции 253 #3240

Merged

lekss361 merged 1 commit from docs/3189-stale-pool-comments into main

2026-08-29 16:13:54 +00:00

lekss361 commented

2026-08-29 16:02:44 +00:00

Owner

Summary

providers/domclick/serp.py:332 — докстринг утверждал «в пуле 3 включённых узла, 2 забанены Домкликом». В пуле четыре узла, у всех provider_affinity='any', выделенного домклик-прокси нет с миграции 253 (#2800).
Тот же докстринг называл не тот ограничитель: свип кладёт не счётчик провалов lease, а break по первому DomClickBlockedError (#2854) — до ротации дело не доходит ни при каком счётчике, отсюда buckets_completed=0 во всех прогонах.
data/sql/175_..._domclick_detail_backfill.sql:41-44 — комментарий к max_consecutive_blocks обосновывал значение тем, что «DomClick has no IP-rotation recovery on block (one dedicated residential proxy, not a pool)». Само значение 3 остаётся, обоснование помечено как устаревшее.

Только комментарии, поведение не меняется.

Зачем

Именно на эти два места опирался тикет #3189, заведённый под «калибровку» ограничения, которого не существует уже несколько недель. Тикет закрыт как построенный на опровергнутой посылке — но текст, породивший посылку, остался бы в репо и подставил бы следующего.

Test plan

ruff check на serp.py — passed
Миграция 175 уже применена на проде; _schema_migrations трекает по имени файла (PRIMARY KEY filename, без checksum — .forgejo/workflows/deploy-tradein.yml:971), поэтому правка текста её не перезапустит
CI зелёный

## Summary - `providers/domclick/serp.py:332` — докстринг утверждал «в пуле 3 включённых узла, 2 забанены Домкликом». В пуле четыре узла, у всех `provider_affinity='any'`, выделенного домклик-прокси нет с миграции 253 (#2800). - Тот же докстринг называл не тот ограничитель: свип кладёт не счётчик провалов lease, а `break` по первому `DomClickBlockedError` (#2854) — до ротации дело не доходит ни при каком счётчике, отсюда `buckets_completed=0` во всех прогонах. - `data/sql/175_..._domclick_detail_backfill.sql:41-44` — комментарий к `max_consecutive_blocks` обосновывал значение тем, что «DomClick has no IP-rotation recovery on block (one dedicated residential proxy, not a pool)». Само значение 3 остаётся, обоснование помечено как устаревшее. Только комментарии, поведение не меняется. ## Зачем Именно на эти два места опирался тикет #3189, заведённый под «калибровку» ограничения, которого не существует уже несколько недель. Тикет закрыт как построенный на опровергнутой посылке — но текст, породивший посылку, остался бы в репо и подставил бы следующего. ## Test plan - [x] `ruff check` на `serp.py` — passed - [x] Миграция 175 уже применена на проде; `_schema_migrations` трекает по имени файла (PRIMARY KEY `filename`, без checksum — `.forgejo/workflows/deploy-tradein.yml:971`), поэтому правка текста её не перезапустит - [ ] CI зелёный

lekss361 added 1 commit 2026-08-29 16:02:44 +00:00

docs(tradein/domclick): два комментария описывали пул, которого нет с миграции 253

CI Trade-In / changes (pull_request) Successful in 9s

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI / changes (pull_request) Successful in 10s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 4m48s

Details

3059f09e53

Оба места утверждали, что у Домклика один выделенный резидентный прокси и
пула нет. Это перестало быть правдой ещё в #2800 (миграция 253 сняла
резервацию узла), но текст остался — и именно на него опирался тикет #3189,
поставленный под «калибровку» ограничения, которого не существует.
Свип к тому же ходит через пул давно (serp.py:359), а бэкфилл подключён к
нему в PR #3222.

Заодно докстринг называл не тот ограничитель: свип кладёт не счётчик
провалов lease, а break по первому DomClickBlockedError (#2854) — до
ротации дело не доходит ни при каком счётчике, отсюда buckets_completed=0.

Только комментарии, поведение не меняется. Миграция 175 уже применена, а
_schema_migrations трекает по имени файла без checksum — правка текста
её не перезапустит.