Skip to content

Commit 2618bd8

Browse files
simbasimba
authored andcommitted
fix: restore Package.resolved and add CI retry for HF downloads
- Package.resolved was accidentally deleted when switching to local path for fork development. Regenerated with resolved fork commit. - e2e-test.yml: added 3-attempt retry loop for transient HuggingFace API failures (HTTP 500). Set HF_HUB_DOWNLOAD_TIMEOUT=120. - build.yml: added feature/* branches to trigger CI builds.
1 parent 29e1d64 commit 2618bd8

3 files changed

Lines changed: 25 additions & 2 deletions

File tree

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: Build
22

33
on:
44
push:
5-
branches: [main, develop]
5+
branches: [main, develop, feature/*]
66
pull_request:
77
branches: [main]
88

.github/workflows/e2e-test.yml

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,23 @@ jobs:
4242
key: mlx-model-qwen2.5-0.5b-4bit
4343

4444
- name: Run E2E tests
45+
env:
46+
HF_HUB_DOWNLOAD_TIMEOUT: "120"
4547
run: |
4648
chmod +x tests/test-server.sh
47-
tests/test-server.sh .build/release/mlx-server 15413
49+
# Retry up to 2 times for transient HuggingFace download failures
50+
for attempt in 1 2 3; do
51+
echo "Attempt $attempt of 3..."
52+
if tests/test-server.sh .build/release/mlx-server 15413; then
53+
exit 0
54+
fi
55+
if [ "$attempt" -lt 3 ]; then
56+
echo "Test failed, retrying in 10s..."
57+
sleep 10
58+
fi
59+
done
60+
echo "All attempts failed"
61+
exit 1
4862
4963
- name: Upload test logs on failure
5064
if: failure()

Package.resolved

Lines changed: 9 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)