Skip to content

Comments

fix: resolve Docker build compilation errors#77

Merged
santi1234567 merged 1 commit intomigalabs:masterfrom
barnabasbusa:fix/docker-build-compilation-errors
Aug 28, 2025
Merged

fix: resolve Docker build compilation errors#77
santi1234567 merged 1 commit intomigalabs:masterfrom
barnabasbusa:fix/docker-build-compilation-errors

Conversation

@barnabasbusa
Copy link
Contributor

This commit addresses several compilation errors that prevented successful Docker builds:

  1. Fix unused import: Remove unused "encoding/binary" import from pkg/networks/ethereum/enr.go

  2. Update dependencies: Upgrade Go modules to resolve prometheus/common retracted version warning:

    • prometheus/common: v0.50.0 → v0.65.0 (resolves retracted version issue)
    • prometheus/client_golang: v1.19.0 → v1.20.4
    • Go toolchain: 1.21 → 1.23.0
    • Other dependency updates for compatibility
  3. Fix missing TopicsPerPeer method: Replace non-existent gs.PubsubService.TopicsPerPeer() with compatible implementation using gs.PubsubService.ListPeers() in pkg/gossipsub/metrics.go

  4. Fix missing ArrivalTime field: Replace msg.ArrivalTime (non-existent field) with time.Now() in message handlers at pkg/networks/ethereum/gossip_handlers.go

This commit addresses several compilation errors that prevented successful Docker builds:

1. **Fix unused import**: Remove unused "encoding/binary" import from pkg/networks/ethereum/enr.go

2. **Update dependencies**: Upgrade Go modules to resolve prometheus/common retracted version warning:
   - prometheus/common: v0.50.0 → v0.65.0 (resolves retracted version issue)
   - prometheus/client_golang: v1.19.0 → v1.20.4
   - Go toolchain: 1.21 → 1.23.0
   - Other dependency updates for compatibility

3. **Fix missing TopicsPerPeer method**: Replace non-existent gs.PubsubService.TopicsPerPeer()
   with compatible implementation using gs.PubsubService.ListPeers() in pkg/gossipsub/metrics.go

4. **Fix missing ArrivalTime field**: Replace msg.ArrivalTime (non-existent field) with time.Now()
   in message handlers at pkg/networks/ethereum/gossip_handlers.go

These changes maintain the original functionality while ensuring compatibility with standard
go-libp2p-pubsub library APIs, allowing successful Docker builds.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@santi1234567 santi1234567 self-requested a review August 28, 2025 13:47
Copy link
Collaborator

@santi1234567 santi1234567 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks!

@santi1234567 santi1234567 merged commit 64c4fd0 into migalabs:master Aug 28, 2025
1 check passed
@barnabasbusa barnabasbusa deleted the fix/docker-build-compilation-errors branch August 28, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants