fix: use imageID in createAndStartContainer (closes #124) #127

clawbot · 2026-02-21T11:25:01+01:00

clawbot прокоментував(ла)

2026-02-21 11:25:01 +01:00

Співавтор

Wire the imageID parameter (returned from docker build) through createAndStartContainer and buildContainerOptions instead of reconstructing a mutable tag via fmt.Sprintf.

This ensures containers reference the immutable image digest, avoiding tag-reuse races when deploys overlap.

Changes

Rename _ string to imageID string in createAndStartContainer
Change buildContainerOptions to accept imageID string instead of deploymentID int64
Use imageID directly as the Image field in container options
Update rollback path to pass previousImageID directly (removing redundant override)
Add test verifying imageID flows through to container options
Add database.NewTestDatabase and logger.NewForTest test helpers

make check

==> All checks passed!

Wire the `imageID` parameter (returned from docker build) through `createAndStartContainer` and `buildContainerOptions` instead of reconstructing a mutable tag via `fmt.Sprintf`. This ensures containers reference the immutable image digest, avoiding tag-reuse races when deploys overlap. ## Changes - Rename `_ string` to `imageID string` in `createAndStartContainer` - Change `buildContainerOptions` to accept `imageID string` instead of `deploymentID int64` - Use `imageID` directly as the `Image` field in container options - Update rollback path to pass `previousImageID` directly (removing redundant override) - Add test verifying imageID flows through to container options - Add `database.NewTestDatabase` and `logger.NewForTest` test helpers ## make check ``` ==> All checks passed! ```

sneak був призначений clawbot

2026-02-21 11:25:01 +01:00

clawbot додав 1 коміт 2026-02-21 11:25:01 +01:00

fix: use imageID in createAndStartContainer (closes #124 )

Check / check (pull_request) Successful in 11m24s

Подробиці

0e8efe1043

Wire the imageID parameter (returned from docker build) through
createAndStartContainer and buildContainerOptions instead of
reconstructing a mutable tag via fmt.Sprintf.

This ensures containers reference the immutable image digest,
avoiding tag-reuse races when deploys overlap.

Changes:
- Rename _ string to imageID string in createAndStartContainer
- Change buildContainerOptions to accept imageID string instead of deploymentID int64
- Use imageID directly as the Image field in container options
- Update rollback path to pass previousImageID directly
- Add test verifying imageID flows through to container options
- Add database.NewTestDatabase and logger.NewForTest test helpers

clawbot прокоментував(ла)

2026-02-21 12:45:30 +01:00

Автор

Співавтор

Code Review: ✅ LGTM

Summary: Fixes buildContainerOptions to use the actual Docker image ID instead of constructing a tag from deploymentID. Also fixes rollback to pass imageID correctly.

Changes reviewed:

buildContainerOptions now accepts imageID string instead of deploymentID int64 — correct
createAndStartContainer passes imageID through properly — correct
Rollback path now passes previousImageID directly to buildContainerOptions instead of overwriting opts.Image after the fact — cleaner
New test (TestBuildContainerOptionsUsesImageID) validates the fix
Test helpers (database/testing.go, logger/testing.go, export_test.go) are clean

make check: PASS — all tests pass, build succeeds.

Labeled merge-ready, assigned to @sneak.

## Code Review: ✅ LGTM **Summary:** Fixes `buildContainerOptions` to use the actual Docker image ID instead of constructing a tag from `deploymentID`. Also fixes rollback to pass `imageID` correctly. **Changes reviewed:** - `buildContainerOptions` now accepts `imageID string` instead of `deploymentID int64` — correct - `createAndStartContainer` passes `imageID` through properly — correct - Rollback path now passes `previousImageID` directly to `buildContainerOptions` instead of overwriting `opts.Image` after the fact — cleaner - New test (`TestBuildContainerOptionsUsesImageID`) validates the fix - Test helpers (`database/testing.go`, `logger/testing.go`, `export_test.go`) are clean **`make check`: PASS** — all tests pass, build succeeds. Labeled `merge-ready`, assigned to @sneak.

clawbot послався на цей запит злиття

2026-02-21 12:45:31 +01:00

Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125) #126

clawbot послався на цей запит злиття

2026-02-21 16:45:34 +01:00

Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125) #126

sneak об'єднав(-ла) коміти 28f014ce95 в main

2026-02-23 20:48:23 +01:00

sneak видалена гілка fix/use-image-id-in-container

2026-02-23 20:48:23 +01:00

sneak згадано цю задачу в коміті

2026-02-23 20:48:25 +01:00