Skip to content

Conversation

stefwalter
Copy link
Contributor

This was failing during the build process:

Cloning ssh://****@github.com/instructlab/instructlab.git (to revision stable) to /tmp/pip-req-build-en5sp88r
Running command git clone --filter=blob:none --quiet 'ssh://****@github.com/instructlab/instructlab.git' /tmp/pip-req-build-en5sp88r
git@github.com: Permission denied (publickey).

Switch to an HTTP URL instead fixes the problem.

Changes

Which issue is resolved by this Pull Request:
Resolves #

Description of your changes:

This was failing during the build process:

    Cloning ssh://****@github.com/instructlab/instructlab.git (to revision stable) to /tmp/pip-req-build-en5sp88r
    Running command git clone --filter=blob:none --quiet 'ssh://****@github.com/instructlab/instructlab.git' /tmp/pip-req-build-en5sp88r
    git@github.com: Permission denied (publickey).

Switch to an HTTP URL instead fixes the problem.

Signed-off-by: Stef Walter <stefw@redhat.com>
@rhatdan
Copy link
Contributor

rhatdan commented Apr 25, 2024

I just hit this same issue.

LGTM

@@ -15,6 +15,6 @@ RUN export LD_LIBRARY_PATH="$LD_LIBRARY_PATH:/usr/local/cuda/lib64:/usr/local/cu
&& export XLA_TARGET=cuda120 \
&& export XLA_FLAGS=--xla_gpu_cuda_data_dir=/usr/local/cuda
RUN --mount=type=ssh,id=default CMAKE_ARGS="-DLLAMA_CUBLAS=on" python3.11 -m pip install --force-reinstall --no-cache-dir llama-cpp-python
RUN --mount=type=ssh,id=default python3.11 -m pip install git+ssh://git@github.com/instructlab/instructlab.git@stable
RUN --mount=type=ssh,id=default python3.11 -m pip install git+https://github.com/instructlab/instructlab@stable
Copy link
Contributor

Choose a reason for hiding this comment

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

is the --mount=type=..... still required ?

@stefwalter
Copy link
Contributor Author

This is now fixed bi #988 which was merged. Closing.

@stefwalter stefwalter closed this Apr 26, 2024
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.

3 participants