29 lines
3.4 KiB
Markdown
29 lines
3.4 KiB
Markdown
 
|
# Исправление: удаление организации возвращает 409 (изменение 007)
|
|||
|
|
|
|||
|
|
## Context
|
|||
|
|
`DELETE /api/v1/organizations/5` отвечает `409 «Запись с такими значениями уже существует»`, хотя у организации нет префиксов, устройств и операторов
|
|||
|
|
(проверено по БД: у org 5 только служебный VRF `default`). Так же падает удаление организации 4.
|
|||
|
|
|
|||
|
|
## Причина (подтверждена логом PostgreSQL)
|
|||
|
|
`ERROR: update or delete on table "organizations" violates foreign key constraint "vrfs_organization_id_fkey" ... Key (id)=(5) is still referenced from table "vrfs"`.
|
|||
|
|
В `delete_org` (`app/api/v1/refs.py:67-79`) VRF удаляются через `db.delete(v)`, затем `db.delete(o)`, всё одним `commit`. Между `Vrf` и `Organization` нет ORM-`relationship()`,
|
|||
|
|
поэтому unit of work не гарантирует порядок DELETE, и `DELETE FROM organizations` уходит раньше `DELETE FROM vrfs`. `IntegrityError` в `commit()` (`app/services.py:52`)
|
|||
|
|
превращается в общий 409 с вводящим в заблуждение текстом.
|
|||
|
|
|
|||
|
|
## Исправление
|
|||
|
|
1. `app/api/v1/refs.py`, `delete_org`: заменить цикл `for v in ...: db.delete(v)` на `db.execute(delete(Vrf).where(Vrf.organization_id == id))`
|
|||
|
|
(добавить `delete` в импорт `sqlalchemy`) — выполняется немедленно, до `db.delete(o)`. Аудит и проверка «занято» без изменений.
|
|||
|
|
2. Схема БД, миграции и UI не меняются.
|
|||
|
|
3. Остальные `db.delete` в проекте (`vrfs`, `device-types`, `devices`, `isps`, `prefixes`, `addresses`, `users`) удаляют одиночную запись без зависимых строк, вне области правки.
|
|||
|
|
|
|||
|
|
## Артефакты (правила проекта)
|
|||
|
|
- `docs/changes/007-org-delete-fix/PLAN.md` (этот план) и `SUMMARY.md` — причина, правка, проверка.
|
|||
|
|
- `README.md`: короткая пометка в разделе об удалении организаций/поведении API (если такой раздел есть), иначе строка в списке изменений.
|
|||
|
|
|
|||
|
|
## Тест (минимум)
|
|||
|
|
Один тест в `tests/test_api.py`: создать организацию → `DELETE` → 204 → `GET` → 404; организация с префиксом → `DELETE` → 409 с текстом «Нельзя удалить…».
|
|||
|
|
|
|||
|
|
## Проверка
|
|||
|
|
1. `docker compose -p ipam_control_006 up -d --build app` (пересборка только приложения; БД и данные сохраняются, прежняя поставка `ipam_control-*` не затрагивается).
|
|||
|
|
2. `venv/bin/python -m pytest -q tests/test_api.py` — против запущенного стенда (BASE из `tests/conftest.py`).
|
|||
|
|
3. Вручную в UI (http://192.168.5.9:8088): удалить организации «ООО «Дата-Центр»» (id 5) и «ООО «СтройМонтаж»» (id 4) — 204; попытка удалить «ООО «Технологии связи»» (есть префиксы) — 409 «Нельзя удалить…»; записи `organization.deleted` появились в журнале.
|