Skip to content

onboard Github Pages PEP503 compatible simple index for llm-d-fs-offloading connector#442

Open
Gregory-Pereira wants to merge 1 commit intollm-d:mainfrom
Gregory-Pereira:onboard-gh-pages-wheel-index
Open

onboard Github Pages PEP503 compatible simple index for llm-d-fs-offloading connector#442
Gregory-Pereira wants to merge 1 commit intollm-d:mainfrom
Gregory-Pereira:onboard-gh-pages-wheel-index

Conversation

@Gregory-Pereira
Copy link
Copy Markdown
Member

…oading connector

Signed-off-by: greg pereira <grpereir@redhat.com>
@Gregory-Pereira
Copy link
Copy Markdown
Member Author

Works:

.github/scripts/generate-pip-index.sh kv_connectors/llmd_fs_backend/wheels /tmp/pip-index-test

cd /tmp/pip-index-test && python3 -m http.server 8123 &>/dev/null & SERVER_PID=$!
trap 'kill $SERVER_PID 2>/dev/null; rm -rf /tmp/pip-index-test' EXIT

pip install --dry-run --no-deps \
      --index-url http://localhost:8123/simple/ \
      --trusted-host localhost \
      --platform linux_x86_64 \
      --python-version 3.12 \
      --implementation cp \
      --abi cp312 \
      --only-binary :all: \
      llmd-fs-connector
Generated PEP 503 index at /tmp/pip-index-test/simple/
Packages: llmd-fs-connector
[1] 82375
Looking in indexes: http://localhost:8123/simple/
Collecting llmd-fs-connector
  Downloading http://localhost:8123/simple/llmd-fs-connector/llmd_fs_connector-0.17.1-cp312-cp312-linux_x86_64.whl (10.9 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 10.9/10.9 MB 576.6 MB/s  0:00:00
Would install llmd_fs_connector-0.17.1
☁  pip-index-test

The only issue here is that we may not want to keep the wheel in git

@kfirtoledo
Copy link
Copy Markdown
Collaborator

@Gregory-Pereira can it be a separate Git? Also, it is worth updating the fs-connector Readme

@Gregory-Pereira
Copy link
Copy Markdown
Member Author

This does mean we can remove the wheels eventually from main. It will instead be tracked on the gh-pages branch

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