Skip to content

Commit 8f7175f

Browse files
committed
fix: correct Java version matrix and API endpoint compatibility
- Fix compatibility report to check Java 8 and 17 (not 11 and 17) - Change /api/v2 to /api/v1 for better compatibility with ChromaDB 1.x - Remove obsolete V2 documentation files - .DS_Store already in .gitignore
1 parent cf8b845 commit 8f7175f

File tree

5 files changed

+12
-1023
lines changed

5 files changed

+12
-1023
lines changed

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

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ jobs:
104104
run: |
105105
echo "Waiting for ChromaDB to be ready..."
106106
for i in {1..30}; do
107-
if curl -f http://localhost:8000/api/v2 > /dev/null 2>&1; then
107+
if curl -f http://localhost:8000/api/v1 > /dev/null 2>&1; then
108108
echo "ChromaDB is ready!"
109109
break
110110
fi
@@ -113,12 +113,12 @@ jobs:
113113
done
114114
115115
# Verify ChromaDB is responding
116-
curl -v http://localhost:8000/api/v2 || true
116+
curl -v http://localhost:8000/api/v1 || true
117117
118118
- name: Get ChromaDB Version Info
119119
run: |
120120
echo "Testing against ChromaDB version: ${{ matrix.chroma-version }}"
121-
curl -s http://localhost:8000/api/v2/version || echo "Version endpoint not available"
121+
curl -s http://localhost:8000/api/v1/version || echo "Version endpoint not available"
122122
123123
- name: Compile project
124124
run: mvn clean compile -DskipTests
@@ -205,20 +205,20 @@ jobs:
205205
run: |
206206
echo "# V2 API Compatibility Matrix" >> $GITHUB_STEP_SUMMARY
207207
echo "" >> $GITHUB_STEP_SUMMARY
208-
echo "| ChromaDB Version | Java 11 | Java 17 |" >> $GITHUB_STEP_SUMMARY
209-
echo "|------------------|---------|---------|" >> $GITHUB_STEP_SUMMARY
208+
echo "| ChromaDB Version | Java 8 | Java 17 |" >> $GITHUB_STEP_SUMMARY
209+
echo "|------------------|--------|---------|" >> $GITHUB_STEP_SUMMARY
210210
211211
# Check test results for each combination
212212
for chroma_version in "1.0.0" "1.1.0" "latest"; do
213-
java11_status="❓"
213+
java8_status="❓"
214214
java17_status="❓"
215215
216-
if [ -d "test-artifacts/test-results-v2-chroma-${chroma_version}-java-11" ]; then
217-
if grep -q 'failures="0"' test-artifacts/test-results-v2-chroma-${chroma_version}-java-11/TEST-*.xml 2>/dev/null && \
218-
grep -q 'errors="0"' test-artifacts/test-results-v2-chroma-${chroma_version}-java-11/TEST-*.xml 2>/dev/null; then
219-
java11_status="✅"
216+
if [ -d "test-artifacts/test-results-v2-chroma-${chroma_version}-java-8" ]; then
217+
if grep -q 'failures="0"' test-artifacts/test-results-v2-chroma-${chroma_version}-java-8/TEST-*.xml 2>/dev/null && \
218+
grep -q 'errors="0"' test-artifacts/test-results-v2-chroma-${chroma_version}-java-8/TEST-*.xml 2>/dev/null; then
219+
java8_status="✅"
220220
else
221-
java11_status="❌"
221+
java8_status="❌"
222222
fi
223223
fi
224224
@@ -231,7 +231,7 @@ jobs:
231231
fi
232232
fi
233233
234-
echo "| ${chroma_version} | ${java11_status} | ${java17_status} |" >> $GITHUB_STEP_SUMMARY
234+
echo "| ${chroma_version} | ${java8_status} | ${java17_status} |" >> $GITHUB_STEP_SUMMARY
235235
done
236236
237237
echo "" >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)