fix(tradein/scraper-kit): проброс config= в pipeline.py::fetch_newbuilding call
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 8s
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 1m37s
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 8s
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 1m37s
Найдено при code-review этого же PR (F1, #2349) и независимо подтверждено F3-аудитом (#2351): run_cian_city_sweep's houses-фаза звала fetch_newbuilding(zhk_url) без config=, хотя config — mandatory параметр enclosing-функции, тривиально в scope. Сейчас dormant (USE_KIT_SCHEDULER=False + cian_city_sweep.enabled=false в SQL seed), но при включении флага/расписания каждый house молча падал бы AssertionError (перехватывается except → houses_failed, sweep не крашится целиком, но newbuilding-enrichment был бы 100% failing silently).
This commit is contained in:
parent
b89e0c3daf
commit
ccf716119c
1 changed files with 1 additions and 1 deletions
|
|
@ -2456,7 +2456,7 @@ async def run_cian_city_sweep(
|
|||
zhk_url: str = hrow["cian_zhk_url"]
|
||||
counters.houses_attempted += 1
|
||||
try:
|
||||
nb_enrichment = await fetch_newbuilding(zhk_url)
|
||||
nb_enrichment = await fetch_newbuilding(zhk_url, config=config)
|
||||
if nb_enrichment is not None:
|
||||
save_newbuilding_enrichment(db, house_id_val, nb_enrichment)
|
||||
counters.houses_enriched += 1
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue