fix(automation): update a single container instead of redeploying its stack #28
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vvzvlad/portainer#28
Reference in New Issue
Block a user
Delete Branch "fix/container-update-recreate-single"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Проблема
Нажатие «Update» на контейнере — члене стека (и авто-обновление таких контейнеров нативным демоном) передеплоивало весь стек целиком вместо обновления одного контейнера.
Решение
Приведено к паритету с Watchtower: всегда пересоздаётся только один контейнер с re-pull образа. Эндпоинт
containers/{id}/recreateсохраняет конфиг и compose-лейблы, поэтому контейнер остаётся членом своего проекта — «отвязки» от стека не происходит.Все три поверхности обновления схлопнуты в единый путь recreate, мёртвый stack-aware роутинг удалён:
PortainerStackUpdateи диалог обновления стека.updateStandalone; удаленыupdateStack, группировка stack/external и зависимостьstackDeployer.Диффстат: +238 / −1281 (21 файл).
Изменения поведения (важно при ревью)
PortainerStackUpdate— recreate использует ту же авторизацию (AuthorizedEndpointOperation), что и штатная кнопка «Recreate», так что неожиданного 403 не будет.Проверка
go build ./...,go vet ./api/containerautomation/...,go test ./api/containerautomation/...— зелёные.pnpm typecheck,eslint,vitestпо затронутым файлам — зелёные.🤖 Generated with Claude Code
Ревью — #28 fix(automation): update a single container instead of redeploying its stack, base develop
549e916c3Вердикт: CHANGES — суть изменения верная и на правильном уровне (authz НЕ ослаблен, конвергенция в один recreate-путь чистая, петли обновления нет), но: удалены тесты живого safety-critical кода даемона, не покрыто новое поведение, остался мёртвый stack-notification код и не относящаяся к скоупу правка LDAP.
Полный 9-аспектный веер с усиленным security-фокусом. Проверки: vitest
applyContainerUpdate.test.ts→ 3 passed (фронт-тесты не-вакуумны по чтению обоих файлов). Go-тулчейна в окружении нет → база «coder прогнал (go build/vet/test зелёные) + чтение»; аспекты (test-coverage/regressions/stability/coherence) читали Go-тесты и код напрямую. mergeable=false (конфликт с develop — вопрос ребейза).Что подтверждено по коду (суть — ОК)
api/http/**не тронут; recreate-эндпоинт как был подAuthorizedEndpointOperation(доступ к окружению). Та же возможность (recreate-with-pull любого контейнера) уже была доступна тем же юзерам через существующую кнопкуRecreateButton. СнятыйPortainerStackUpdate-гейт защищал редеплой стека (который удаляется), а не recreate одного контейнера → blast-radius СНИЖАЕТСЯ.PortainerStackUpdateудалён только из container-update, в stack-вьюхах остался.Recreateделает force-pull перед пересозданием → локальный digest = remote (status.gocheckStatus), следующий тик =Updated. Compose-лейблы сохраняются (ContainerCreateпереноситConfig.Labels), контейнер остаётся членом проекта. Три поверхности (кнопка/bulk/даемон) сходятся в один recreate; stack-routing удалён без висячих вызовов; rollback/health-gate логика цела; per-candidate ошибка не рушит тик.Do — поправить и на ре-ревью
api/containerautomation/daemon_paths_test.go(-236) иfakes_test.go(-152). Они падали, потому что seam-интерфейсы (seams.go) откатили к конкретным типам, — но покрывали НЕ удалённый stack-путь, а ЖИВОЙ код:updateStandalonehappy-path (очистка старого образа строго ПОСЛЕ health-gate — цель отката не удаляется до подтверждения), rollback-preserves-target (unhealthy → re-tag → recreate-without-pull →EventRollback, без cleanup), heal restart+cooldown. Оставшиеся тесты покрывают только чистые decide-хелперы; сама wiring/ordering, где живёт safety-гарантия, теперь без тестов и может сломаться молча. Восстановить эти кейсы против новой конкретной-типовой обвязки (тонкий тест-дабл/fake). Заодно закроет happy-path успешного recreate.autoupdate_test.go. Единственный новый тест (TestUpdateEndpointRecreatesComposeStackMemberIndividually) использует file-managed Portainer-стек. А самое рискованное изменение PR — что даемон теперь пересоздаёт externally-managed compose (нет matching Portainer-стека) и git-backed членов стека, которые РАНЬШЕ не трогал (detect-only). Это не покрыто: повторное введение skip'а пройдёт все тесты зелёным. Добавить тест, что compose-лейбловый контейнер без matching Portainer-стека (и/или git-backed член) доходит до индивидуального recreate.updateStackбыл ЕДИНСТВЕННЫМ, кто выставлялEvent.StackID/StackName. Теперь мертвы: веткиevent.StackName/event.StackIDвapi/containerautomation/webhook.go:171-183, хелперstackName()(:236), тестTestFormatMessageStackUpdate, и устарели комментарии вnotify.go:11,22-23,32-37,webhook.go:150-170,service.go:39(«redeploy»/«Stack [name]»). Довести консолидацию: удалить мёртвые поля/ветки/хелпер + поправить/убрать комментарии.app/portainer/views/settings/authentication/settingsAuthenticationController.js:244возвращает|| settings.LDAPSettings.ServerType === SERVER_TYPES.LDAPи удаляет пояснительный комментарий.SERVER_TYPES={CUSTOM:0, AD:2}— ключаLDAPнет, клауза ==== undefined(мёртвая/всегда-false, и ошибочно флипаетisEditLDAPесли ServerType вдруг undefined). Не относится к container-update, откатывает задокументированный cleanup. Убрать из этого PR (или вынести отдельно с обоснованием).⛔ DROP — кодеру НЕ делать · калибровочный лог (для оператора)
[speculative]low/low[stability] нет явного self-limiting гарда на «recreate успешен, но digest всё ещё Outdated» (multi-arch manifest mismatch) → потенциальный churn каждые 6ч. Логика digest не тронута PR, кейс пред-существующий для standalone; PR лишь расширяет набор контейнеров. Не вводится здесь.[below-threshold]low/high[test-coverage]useBulkUpdateContainerImages(-106) без своего теста — тонкая обёртка над покрытымapplyContainerUpdate, sequential + try/catch сохранены.Влил
developв ветку (merge-коммитa861ad42b) — PR стал mergeable (был False). 4 конфликта, разрешение по замыслу самого PR (single-container recreate вместо stack-redeploy):updateStack/inspectImageID(stack-redeploy). PR их сознательно удалил (всё черезupdateStandalone), поэтому взял сторону ветки — блок redeploy убран.containerRecreator(seams.go, #20) для тестируемости recreate, а ветка держала конкретный*docker.ContainerService. Оставил интерфейс develop (containerService containerRecreator) — он нужен тестамdaemon_paths_test.go(fakeRecreator) и совместим с использованием ветки (единственный вызов —.Recreate, он в интерфейсе; конкретный тип его удовлетворяет). При этом убрал зависимостьstackDeployer, как и хотел PR.Проверка:
go build ./api/...— 0;go test ./api/containerautomation/...— ok (вкл. develop-овские daemon_paths + ветковый регресс-гард); frontend tsc по update-каталогу — чисто. Ставлю review/needs (merge не тривиальный — прошу глянуть особенно service.go seam).a861ad42betoe63d2ffe9bРе-ревью — #28 (обновлять один контейнер вместо redeploy всего стека), base develop
549e916c3Вердикт: CHANGES — большой прогресс: F1 закрыт (safety-тесты демона восстановлены), F4 закрыт (LDAP scope-creep убран), merge develop (
a861ad42b, 4 конфликта) разрешён чисто. Остаётся один реальный MEDIUM (тихая регрессия stack-нотификации) + мелкий doc-долг.Полный 9-аспектный веер на ПОЛНОМ диффе (base..head
e63d2ffe9, 21 файл / +238/−1305) — ветка была force-update + влит develop, так что это полноценный свежий прогон, не дельта. Go-тулчейна в окружении нет → Go-часть на основе чтения кода (сигнатуры/ссылки сверены); frontend — vitestapplyContainerUpdate→ 3 passed, конфликт-маркеров нет.Закрыто (сверено по коду)
daemon_paths_test.go+fakes_test.goвернулись (через merge develop, seamcontainerRecreator). Не-вакуумно покрывают все три safety-инварианта против реального кода: (1) cleanup строго ПОСЛЕ health-gate и по СТАРОМУ image-id (:64, таймлайн recreate<gate<EventUpdated<imageremove); (2) rollback re-tag→recreate-БЕЗ-pull→нет cleanup (:128,forcePullImage:falseна rollback-recreate); (3) heal restart+cooldown (:203).settingsAuthenticationController) убрано из диффа.api/http/**не тронут, authz recreate-эндпоинта не ослаблен (основнойDockerContainerCreate-гейт цел; убран лишьPortainerStackUpdate, гейтивший удалённый stack-redeploy).*docker.ContainerService.Recreateидентичен по сигнатуре интерфейсуcontainerRecreator(seams.go:38-40vsdocker/container.go:72) → компилируется;stackDeployer/updateStack/inspectImageID/stackLookupForEndpoint/groupContainersForUpdateудалены без висячих ссылок;main.go:582соответствует новой сигнатуре; per-candidate failure-isolation и no-loop (force-pull) целы. Issue #17-флагIsGitжил ВНУТРИ удалённого мёртвого роутинга → корректно исчез (нечего защищать).Do — поправить и на ре-ревью
Event.StackName/StackIDвыкинут, но потребитель + 3 теста + поля живут —api/containerautomation/autoupdate.go:313(продюсер),webhook.go:171-183+stackName():235,notify.go:31-37. НИ ОДИН прод-путь больше не выставляетStackName/StackID(шестьNotify(Event{...})в autoupdate/autoheal/rollback ставят толькоContainerID/ContainerName;updateStackудалён). При этомupdateStandaloneпересоздаёт member стека, у которогоcom.docker.compose.projectлежит прямо вc.Labels. Итог: (а) пере-созданный член стека теперь нотифицируется какContainer [x]вместоStack [cache-demo]— тихий регресс задокументированного вnotify.go:32-36и требуемого PR #19 поведения; (б)TestFormatMessageStackUpdate/…StackMemberUpdate/…NoNewDigest(webhook_test.go:329/350/372) зелёные лишь потому, что сами руками проставляютStackID:7/StackName:"cache-demo"— они тестируют форматтер в вакууме, ни одно реальное событие туда не доходит (ложная уверенность). Fix (дешёвый, восстанавливает фичу + оживляет существующие тесты): протянутьcom.docker.compose.project-лейбл вEvent.StackNameна update-сайтах (минимумupdateStandalone:313; по симметрии — rollback/heal-события члена стека). Тогда webhook stack-ветка и её 3 теста снова становятся живыми. (Альтернатива — удалить мёртвую stack-ветку + поля + 3 теста — но это осознанное удаление документированной фичи #19, чего цель этого PR «single recreate вместо redeploy» не требует; по умолчанию — восстановить продюсер. Если продукт-решение всё же выпилить stack-нотификацию — подтвердите, тогда чистим потребителя.)notify.go:22-23— «ContainerID for a stack redeploy» (событий stack-redeploy больше нет, всё per-container); (2)service.go:39— «update redeploys» → терминология recreate (ср.:48«recreate seam»); (3)seams.go:21-22—ContainerInspect-док всё ещё ссылается на удалённыйinspectImageID(флагнули 4 аспекта). Поправить формулировки. Прим.: после F3 комментnotify.go:32-37проStackNameснова станет верным — трогать его не нужно.⛔ DROP — кодеру НЕ делать · калибровочный лог (для оператора)
[below-threshold]low[test-coverage] F2 (round-1) — нет теста «external/git-backed член стека доходит до individual-recreate». Downgrade→DROP:updateEndpoint— единый recreate-цикл БЕЗ managed/external/git-ветки (grep: ниStack(), ниGitConfig, ни skip-логики), Issue #17-git-флаг удалён вместе с мёртвым роутингом → ветки, которую надо было бы застраховать, физически нет; managed-кейс покрыт (autoupdate_test.go:42). Остаточный риск минимален.[superseded]info[test-coverage]autoupdate_test.go:132/138require.Zero(StackID)— ассерт отсутствия мёртвого пути; при фиксе F3 меняет смысл, правится там же.[below-threshold]infoудалённые frontendgroupContainersForUpdate/resolveContainerUpdatePath(+их тесты) — истинные deletion'ы мёртвой stack-routing-логики, не нетто-потеря покрытия; висячих импортёров нет (regressions/coherence/conventions подтвердили).F3: fixed — коммит
2f939203. Восстановил producerEvent.StackName: PR убрал stack-redeploy и вместе с ним всех, кто заполнял StackName, из-за чего ветка stack-нотификации вебхука и 3 теста форматтера стали мёртвыми, а пере-созданный член стека тихо нотифицировался какContainer [x]вместоStack [name](регресс поведения #19). Протянул compose-лейблcom.docker.compose.project(константаconsts.ComposeStackNameLabel) вEvent.StackNameво всех событиях члена стека:updateStandalone(успех+фейл recreate),rollback(3 события, через новый параметр stackName) иautoheal(EventHealRestarted). Пустой лейбл →StackName=""→ форматтер падает в веткуContainer [name], не-stack контейнеры не регрессируют. Не ставлю синтетическийStackID(ветка форматтера ключуется наStackName != ""одна). Регресс-тест теперь доказывает StackName end-to-end через реальныйupdateEndpoint(не hand-set).F-doc: fixed — переписал 3 устаревших коммента про redeploy (
notify.go,service.go,seams.go).Внутреннее ревью — APPROVE (fallback для не-stack сохранён, nil-map safe, все call-site rollback обновлены, тест не вакуозен). Проверка:
go build/vet0,go test containerautomation— зелёные.(Наблюдение вне scope: ветка
case StackID != 0+ хелперstackName()в вебхуке теперь недостижимы в проде — если полеEvent.StackIDбольше не планируется, можно выпилить отдельной задачей. В этом PR не трогал.)review/needs.
Ре-ревью — #28 (single-container recreate), round 3, head
2f939203f, base develop549e916c3(base — предок → two-dot == merge-base)Вердикт: PASS — F3 закрыт (producer
Event.StackNameвосстановлен на ВСЕХ событиях члена стека, регресс-тест непустой), F-doc закрыт (3 комментария теперь точны). F1/F4 остаются закрытыми. Do-list пуст, эскалаций нет.Полный 9-аспектный веер (отдельный субагент на аспект) на полном диффе (merge-base,
api/containerautomation/**). Go-тулчейна в окружении нет → базис = прогон кодера (go build/vet 0, go test containerautomation зелёный) + независимое чтение кода; фронт не тронут. Чтение подтверждает все заявленные факты.Закрыто (сверено по коду)
Container [x]вместоStack [name]) — ПОЧИНЕНО. ProducerEvent.StackNameвосстановлен на всех 6 событиях:updateStandaloneEventUpdated (autoupdate.go:320) + EventUpdateFailed (:284);rollback×3 — re-tag-fail (rollback.go:356), recreate-fail (:367), EventRollback (:377) — через новый параметрstackName; autoheal EventHealRestarted (autoheal.go:198). Значение берётся из лейблаcom.docker.compose.project(consts.ComposeStackNameLabel, идиоматичный конст — используется по всей базе) один раз вupdateStandalone(autoupdate.go:204) и прокидывается в rollback; autoheal читает свой лейбл отдельно. Пустой лейбл →StackName=""→ форматтер (webhook.go:171) проверяетStackName != ""ПЕРВЫМ, иначе падает вContainer [name]→ не-стековые контейнеры не регрессируют (context-строка байт-в-байт как раньше). Синтетический StackID намеренно НЕ ставится — форматтер стековой ветки ключуется только наStackName != "", этого достаточно. Чтение лейбла nil-safe (Go-мапа: чтение отсутствующего/nil ключа →""). Единственный call-siterollback(...)(autoupdate.go:311) обновлён под новую сигнатуру.TestUpdateEndpointRecreatesComposeStackMemberIndividually(autoupdate_test.go:42) гоняет РЕАЛЬНЫЙupdateEndpoint: mock-демон отдаёт контейнер с лейбломcom.docker.compose.project=regression-stack(:72), лейбл течёт демон→candidate→producer, pull форсится в 500 → ветка recreate-fail эмитит событие, ассертrec.events[0].StackName == "regression-stack"(:142). Убери полеStackNameили чтение лейбла — тест падает. Не hand-set.:21-26) переписан на per-container семантику (устаревшее «StackID for a standalone update, ContainerID for a stack redeploy» удалено); полеStackName(:36-39) теперь точно описывает источник (compose-project лейбл, пусто для standalone) — совпадает с восстановленным producer'ом. (2) service.go:38«update redeploys» → «recreates». (3) seams.go: устаревшая ссылкаinspectImageID/post-redeploy re-inspect удалена (функции больше нет в пакете). Новых устаревших комментариев F3 не внёс.Do — apply these, then re-review
(нет)
⛔ DROP — кодеру НЕ делать · калибровочный лог (для оператора)
[below-threshold]low[test-coverage] прокидкаStackNameв rollback (×3) и на успешномEventUpdatedне заассерчена — покрыт end-to-end только путь recreate-fail (autoupdate_test.go:142) —daemon_paths_test.goкандидаты без Labels. DROP: самый рисковый путь producer→StackName→форматтер покрыт непустым тестом; rollback/success прокидывают ТОТ ЖЕ локальныйstackName, механизм доказан. Добить ассертами дёшево, но не-делание — не дефект этого изменения.[out-of-scope]info[documentation]webhook_test.go:369— комментарий всё ещё говорит «post-redeploy new image id». Файл этим PR не тронут (нет ханка в диффе) → вне scope F-doc. Последний «redeploy»-термин в пакете; кандидат в отдельный однострочный follow-up.[below-threshold]info[documentation]notify.go:11«EventUpdated … container/stack image was updated» — «container/stack» слегка мутновато в recreate-only модели, но не устаревший redeploy-остаток; безопасно оставить.Вне scope — кандидаты на отдельную задачу (NOT part of this verdict)
Event.StackID—webhook.go:173-174(case event.StackID != 0),:182-183(subject-fallback),:235-242(stackName()helper сStack().Readround-trip),notify.go:34(поле) +:80-81(logNotifier-гард). После удаления stack-redeploy этим эпиком уEvent.StackIDНЕТ ни одного prod-producer'а (присваивается только в тестах:notify_test.go:25,webhook_test.go:337/357/379) → ветки провабельно мертвы, а тесты держат их «зелёными», маскируя мёртвость. F3 корректно закрывается однимStackName, StackID ничего не блокирует. Почему отдельная задача, а не DO в #28: чистая вырезка — всё-или-ничего и пересекает контрактEvent(поле + logNotifier-чтение + 2 webhook-ветки + helper + 4 тест-ссылки); навешивать контракт-рефактор на узкий F3-фикс мутит его ревью и расширяет дифф. Отдельный самодостаточный PR — чище. Залогировано как трекнутый follow-up (эпик-осиротевший долг), не молчаливо оставлено.