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

Fix: docker build #624

Merged
merged 1 commit into from
Jan 23, 2025
Merged

Fix: docker build #624

merged 1 commit into from
Jan 23, 2025

Conversation

soichisumi
Copy link
Contributor

@soichisumi soichisumi commented Jan 22, 2025

What

To fix docker build failure, add rust toolchain

Why

When we build container, it'll fail due to a lack of rust toolchain.

[+] Building 21.0s (10/17)                                                                                docker:desktop-linux
 => [internal] load build definition from Dockerfile                                                                      0.0s
 => => transferring dockerfile: 1.07kB                                                                                    0.0s
 => WARN: FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 2)                                            0.0s
 => [internal] load metadata for docker.io/library/python:3.11-slim                                                       0.8s
 => [internal] load .dockerignore                                                                                         0.0s
 => => transferring context: 2B                                                                                           0.0s
 => [internal] load build context                                                                                         0.0s
 => => transferring context: 4.09kB                                                                                       0.0s
 => [builder 1/7] FROM docker.io/library/python:3.11-slim@sha256:6ed5bff4d7d377e2a27d9285553b8c21cfccc4f00881de1b24c9bc8  1.1s
 => => resolve docker.io/library/python:3.11-slim@sha256:6ed5bff4d7d377e2a27d9285553b8c21cfccc4f00881de1b24c9bc8d90016e8  0.0s
 => => sha256:8c2bfb64ec8eed5b02227a699c0eefcaf56f703acb33deecd0f389278c356b42 16.13MB / 16.13MB                          0.9s
 => => sha256:1c510bbba8451812391d414bbe2506eeb4b2c5bed5906900f20acc9b034c9df0 249B / 249B                                0.2s
 => => extracting sha256:8c2bfb64ec8eed5b02227a699c0eefcaf56f703acb33deecd0f389278c356b42                                 0.2s
 => => extracting sha256:1c510bbba8451812391d414bbe2506eeb4b2c5bed5906900f20acc9b034c9df0                                 0.0s
 => [builder 2/7] WORKDIR /app                                                                                            0.0s
 => [builder 3/7] RUN apt-get update && apt-get install -y --no-install-recommends     build-essential     && rm -rf /va  8.9s
 => [builder 4/7] COPY requirements.txt .                                                                                 0.0s
 => [builder 5/7] COPY lightrag/api/requirements.txt ./lightrag/api/                                                      0.0s
 => ERROR [builder 6/7] RUN pip install --user --no-cache-dir -r requirements.txt                                        10.2s
------
 > [builder 6/7] RUN pip install --user --no-cache-dir -r requirements.txt:
....
....
9.521   × Preparing metadata (pyproject.toml) did not run successfully.
9.521   │ exit code: 1
9.521   ╰─> [6 lines of output]
9.521
9.521       Cargo, the Rust package manager, is not installed or is not on PATH.
9.521       This package requires Rust and Cargo to compile extensions. Install it through
9.521       the system's package manager or via https://rustup.rs/
9.521
9.521       Checking for Rust toolchain....
9.521       [end of output]

@soichisumi
Copy link
Contributor Author

soichisumi commented Jan 22, 2025

Can anyone check plz?
cc @LarFii

@LarFii
Copy link
Collaborator

LarFii commented Jan 23, 2025

Sorry for the late reply. Thanks for your contribution!

@LarFii LarFii merged commit e485d7f into HKUDS:main Jan 23, 2025
1 check passed
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