Skip to content

Commit 405f1d7

Browse files
committed
fix: replace curl with wget in all workflow healthchecks
- Replace curl with wget in healthcheck commands (curl not available in ChromaDB container) - Fix nightly workflow to use curl from runner instead of docker exec - Update all action versions from v5 to v4 (v5 doesn't exist) - Fixes container initialization failures across all workflows
1 parent 8f7175f commit 405f1d7

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed

.github/workflows/v2-api-nightly.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ jobs:
3636

3737
steps:
3838
- name: Checkout code
39-
uses: actions/checkout@v5
39+
uses: actions/checkout@v4
4040

4141
- name: Set up Java ${{ matrix.java-version }}
42-
uses: actions/setup-java@v5
42+
uses: actions/setup-java@v4
4343
with:
4444
java-version: ${{ matrix.java-version }}
4545
distribution: 'temurin'
@@ -57,7 +57,7 @@ jobs:
5757
# Wait for ChromaDB to be ready
5858
echo "Waiting for ChromaDB to start..."
5959
for i in {1..60}; do
60-
if docker exec chroma-${{ matrix.chroma-version }} curl -f http://localhost:8000/api/v1 > /dev/null 2>&1; then
60+
if curl -f http://localhost:8000/api/v1 > /dev/null 2>&1; then
6161
echo "ChromaDB is ready!"
6262
break
6363
fi
@@ -119,15 +119,15 @@ jobs:
119119
IS_PERSISTENT: 'TRUE'
120120
PERSIST_DIRECTORY: '/chroma/data'
121121
options: >-
122-
--health-cmd "curl -f http://localhost:8000/api/v1 || exit 1"
122+
--health-cmd "wget -q --spider http://localhost:8000/api/v1 || exit 1"
123123
--health-interval 10s
124124
--health-timeout 5s
125125
--health-retries 5
126126
--mount type=tmpfs,destination=/chroma/data
127127
128128
steps:
129129
- name: Checkout code
130-
uses: actions/checkout@v5
130+
uses: actions/checkout@v4
131131

132132
- name: Set up Java 17
133133
uses: actions/setup-java@v4

.github/workflows/v2-api-pr-validation.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,14 @@ jobs:
2121
ALLOW_RESET: 'TRUE'
2222
IS_PERSISTENT: 'FALSE'
2323
options: >-
24-
--health-cmd "curl -f http://localhost:8000/api/v1 || exit 1"
24+
--health-cmd "wget -q --spider http://localhost:8000/api/v1 || exit 1"
2525
--health-interval 10s
2626
--health-timeout 5s
2727
--health-retries 5
2828
2929
steps:
3030
- name: Checkout PR branch
31-
uses: actions/checkout@v5
31+
uses: actions/checkout@v4
3232
with:
3333
fetch-depth: 0
3434

@@ -214,7 +214,7 @@ jobs:
214214

215215
steps:
216216
- name: Checkout code
217-
uses: actions/checkout@v5
217+
uses: actions/checkout@v4
218218

219219
- name: Set up Java 11
220220
uses: actions/setup-java@v4

.github/workflows/v2-api-release.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ jobs:
2323

2424
steps:
2525
- name: Checkout code
26-
uses: actions/checkout@v5
26+
uses: actions/checkout@v4
2727
with:
2828
fetch-depth: 0
2929

3030
- name: Set up Java 8
31-
uses: actions/setup-java@v5
31+
uses: actions/setup-java@v4
3232
with:
3333
java-version: '8'
3434
distribution: 'adopt'
@@ -163,14 +163,14 @@ jobs:
163163
env:
164164
ALLOW_RESET: 'TRUE'
165165
options: >-
166-
--health-cmd "curl -f http://localhost:8000/api/v1 || exit 1"
166+
--health-cmd "wget -q --spider http://localhost:8000/api/v1 || exit 1"
167167
--health-interval 10s
168168
--health-timeout 5s
169169
--health-retries 5
170170
171171
steps:
172172
- name: Checkout code
173-
uses: actions/checkout@v5
173+
uses: actions/checkout@v4
174174

175175
- name: Set up Java ${{ matrix.java-version }}
176176
uses: actions/setup-java@v4
@@ -210,7 +210,7 @@ jobs:
210210

211211
steps:
212212
- name: Checkout code
213-
uses: actions/checkout@v5
213+
uses: actions/checkout@v4
214214

215215
- name: Set up Java 11
216216
uses: actions/setup-java@v4
@@ -224,7 +224,7 @@ jobs:
224224
gpg-passphrase: MAVEN_GPG_PASSPHRASE
225225

226226
- name: Download release artifacts
227-
uses: actions/download-artifact@v5
227+
uses: actions/download-artifact@v4
228228
with:
229229
name: release-artifacts-v2
230230
path: ./release

0 commit comments

Comments
 (0)