From 8c00c05d0c81d9a816610aa001321140d018ab5e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E2=80=9CNaeel=E2=80=9D?= Date: Fri, 14 Aug 2026 18:52:48 +0400 Subject: [PATCH] =?UTF-8?q?HISTORY:=20=D1=84=D0=B8=D0=BA=D1=81=D1=8B=20?= =?UTF-8?q?=D0=B3=D0=BE=D0=BD=D0=BE=D0=BA=20map,=20=D0=B8=D0=BD=D1=86?= =?UTF-8?q?=D0=B8=D0=B4=D0=B5=D0=BD=D1=82=20=D1=81=20=D1=80=D0=B5=D1=81?= =?UTF-8?q?=D1=82=D0=B0=D1=80=D1=82=D0=B0=D0=BC=D0=B8=20=D0=BF=D0=BE=D0=B4?= =?UTF-8?q?=D0=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- HISTORY/2026-08-14-session-log.md | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/HISTORY/2026-08-14-session-log.md b/HISTORY/2026-08-14-session-log.md index 8fe76ae..9a9ab89 100644 --- a/HISTORY/2026-08-14-session-log.md +++ b/HISTORY/2026-08-14-session-log.md @@ -568,3 +568,28 @@ DeleteQueue всегда успешен; NextSequenceNumber пересоздаё **Осталось**: логин в консоль для применения лимита 50 к существующему тенанту, затем повторный 30-мин нагрузочный тест. +## Фиксы гонок map — под падал под нагрузкой (14.08.2026) + +**Инцидент**: 30-мин прогон показал 4 сбоя /health; под перезапускался 2 раза +(Exit Code 2). Логи упавшего контейнера: `fatal error: concurrent map read and map write`, +горутина `ChangeMessageVisibilityV1`, `change_message_visibility.go:50` — +чтение `SyncQueues.Queues[key]` БЕЗ блокировки; писатель — DeleteQueueV1 (delete под Lock), +массово вызываемый тестом. + +**Анализ Соннета** (подтверждён): 3 файла читают map до Lock; смежный риск nil-deref +(очередь удаляется между проверкой и Lock); receive_message.go имел два RLock с окном +nil-deref; рекомендован перенос проверки внутрь Lock + defer Unlock. + +**Исправлено (v0.1.35, digest `sha256:7436efd8…`)**: +- `change_message_visibility.go`: проверка существования внутри Lock, defer Unlock. +- `change_message_visibility_batch.go`: то же. +- `delete_message_batch.go`: то же. +- `send_message.go`: nil-guard под Lock + повторная проверка OOM-лимита под Lock. +- `send_message_batch.go`: nil-guard под Lock. +- `receive_message.go`: один RLock вместо двух (nil-deref окно закрыто). +- `errors.go`: добавлен код ошибки `OverLimit` (раньше отсутствовал — zero SqsErrorType). + +**Проверки**: go build/vet OK, models-тесты OK; короткий прогон 120с: неожиданных 2 +(оба сетевые ReadTimeout), сервисных 0, health_fail 0; под restarts 0. +30-мин прогон запущен 18:52:18 +04 — в процессе. +