WHOOSH-REQ-001: Fix Docker Client API Compilation Error in swarm_manager.go #2

New Issue

tony · 2025-09-21T06:18:50Z

tony commented

2025-09-21 06:18:50 +00:00

Problem

Critical compilation error blocking WHOOSH build due to outdated Docker client API usage.

Location

File: internal/orchestrator/swarm_manager.go
Line: 459

Error Details

# github.com/chorus-services/whoosh/internal/orchestrator
internal/orchestrator/swarm_manager.go:459:19: undefined: types.ContainerLogsOptions

Root Cause

Using deprecated Docker client type. The types.ContainerLogsOptions has been moved or renamed in newer Docker client versions.

Required Fix

Update import to use current Docker client API
Replace types.ContainerLogsOptions with container.LogsOptions
Verify other Docker client types are up-to-date
Test compilation after fix

Technical Details

Current (broken):

// Line 459
opts := types.ContainerLogsOptions{
    // ... options
}

Required fix:

// Import: github.com/docker/docker/api/types/container
opts := container.LogsOptions{
    // ... options  
}

Priority

Critical - Blocks all WHOOSH development until resolved

Requirement Traceability

@goal: WHOOSH-REQ-001 - Basic compilation and build functionality

Acceptance Criteria

WHOOSH project compiles without errors
Docker client API usage updated to current version
All container management operations work correctly
No deprecated API warnings in build output

## Problem Critical compilation error blocking WHOOSH build due to outdated Docker client API usage. ## Location **File:** `internal/orchestrator/swarm_manager.go` **Line:** 459 ## Error Details ``` # github.com/chorus-services/whoosh/internal/orchestrator internal/orchestrator/swarm_manager.go:459:19: undefined: types.ContainerLogsOptions ``` ## Root Cause Using deprecated Docker client type. The `types.ContainerLogsOptions` has been moved or renamed in newer Docker client versions. ## Required Fix 1. Update import to use current Docker client API 2. Replace `types.ContainerLogsOptions` with `container.LogsOptions` 3. Verify other Docker client types are up-to-date 4. Test compilation after fix ## Technical Details **Current (broken):** ```go // Line 459 opts := types.ContainerLogsOptions{ // ... options } ``` **Required fix:** ```go // Import: github.com/docker/docker/api/types/container opts := container.LogsOptions{ // ... options } ``` ## Priority **Critical** - Blocks all WHOOSH development until resolved ## Requirement Traceability @goal: WHOOSH-REQ-001 - Basic compilation and build functionality ## Acceptance Criteria - [ ] WHOOSH project compiles without errors - [ ] Docker client API usage updated to current version - [ ] All container management operations work correctly - [ ] No deprecated API warnings in build output

tony added the

bzzz-task

label 2025-09-21 06:30:09 +00:00

tony commented

2025-09-21 07:26:07 +00:00

Assigned to: Claude AI Assistant

Status: Starting implementation

Plan:

Update Docker client API import in internal/orchestrator/swarm_manager.go:459
Replace types.ContainerLogsOptions with container.LogsOptions
Verify other Docker client types are current
Test compilation
Create pull request

Root Cause: Using deprecated Docker client type that has been moved in newer versions

ETA: Immediate implementation

**Assigned to**: Claude AI Assistant **Status**: Starting implementation **Plan**: 1. Update Docker client API import in `internal/orchestrator/swarm_manager.go:459` 2. Replace `types.ContainerLogsOptions` with `container.LogsOptions` 3. Verify other Docker client types are current 4. Test compilation 5. Create pull request **Root Cause**: Using deprecated Docker client type that has been moved in newer versions **ETA**: Immediate implementation

tony referenced this issue from a commit

2025-09-21 07:27:40 +00:00

fix(orchestrator): resolve Docker client API compilation error in swarm_manager.go

tony referenced a pull request that will close this issue

2025-09-21 07:28:34 +00:00

WHOOSH-REQ-001: Fix Docker Client API Compilation Error in swarm_manager.go #5

tony commented

2025-09-21 07:28:50 +00:00

✅ Implementation Complete

Status: Fixed and ready for review

Changes Made:

Updated types.ContainerLogsOptions to container.LogsOptions
Fixed Docker client API compilation error
Added @goal: WHOOSH-REQ-001 requirement traceability
Preserved all existing functionality

Testing:
✅ Orchestrator package compiles without errors
✅ Full WHOOSH project compiles successfully
✅ No breaking changes to Docker operations

Pull Request: #5 - Ready for review and merge
URL: #5

Impact: WHOOSH development is no longer blocked - full project compilation works

Next Steps: Pull request review and merge to close this issue

✅ **Implementation Complete** **Status**: Fixed and ready for review **Changes Made**: - Updated `types.ContainerLogsOptions` to `container.LogsOptions` - Fixed Docker client API compilation error - Added `@goal: WHOOSH-REQ-001` requirement traceability - Preserved all existing functionality **Testing**: ✅ Orchestrator package compiles without errors ✅ Full WHOOSH project compiles successfully ✅ No breaking changes to Docker operations **Pull Request**: #5 - Ready for review and merge **URL**: https://gitea.chorus.services/tony/WHOOSH/pulls/5 **Impact**: WHOOSH development is no longer blocked - full project compilation works **Next Steps**: Pull request review and merge to close this issue

tony closed this issue

2025-10-12 21:15:24 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tony/WHOOSH#2