Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: allow for linux/amd64 profiles on docker builds #412

Merged

Conversation

maschad
Copy link
Member

@maschad maschad commented Feb 14, 2025

A user can now run

	export PLATFORM=linux/amd64

Then

COMPOSE_PROFILES=mistralrs-cpu,non-confidential docker compose up -d

This builds off the fixes in #409

@maschad maschad self-assigned this Feb 14, 2025
@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.34%. Comparing base (b7e8c14) to head (55081e6).
Report is 45 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   66.63%   68.34%   +1.71%     
==========================================
  Files          43       46       +3     
  Lines        7646     8931    +1285     
==========================================
+ Hits         5095     6104    +1009     
- Misses       2551     2827     +276     

see 4 files with indirect coverage changes

Copy link
Contributor

@jorgeantonio21 jorgeantonio21 left a comment

Choose a reason for hiding this comment

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

LGTM

context: https://github.com/EricLBuehler/mistral.rs.git
dockerfile: Dockerfile
image: ghcr.io/ericlbuehler/mistral.rs:cpu-0.4
platform: linux/amd64
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe we want to have support for arm64 in here as well (at least, building from source)

@jorgeantonio21 jorgeantonio21 merged commit 853b865 into atoma-network:main Feb 15, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants