Skip to content

Commit ba0e480

Browse files
committed
revert test.yml changes
1 parent 4c6bd2f commit ba0e480

1 file changed

Lines changed: 13 additions & 68 deletions

File tree

.github/workflows/test.yml

Lines changed: 13 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -101,11 +101,11 @@ jobs:
101101
echo "BEFORE=$git_status" >> $GITHUB_ENV
102102
echo "Repository status before tests: $git_status"
103103
104-
- name: Configure Host Network
105-
if: matrix.os == 'ubuntu-latest'
106-
run: |
107-
# Map 'nginx' to localhost so the Host machine can resolve the URLs in the database
108-
echo "127.0.0.1 nginx" | sudo tee -a /etc/hosts
104+
# - name: Configure Host Network
105+
# if: matrix.os == 'ubuntu-latest'
106+
# run: |
107+
# # Map 'nginx' to localhost so the Host machine can resolve the URLs in the database
108+
# echo "127.0.0.1 nginx" | sudo tee -a /etc/hosts
109109

110110
- name: Clone Services & Apply Universal Patch
111111
if: matrix.os == 'ubuntu-latest'
@@ -117,71 +117,16 @@ jobs:
117117
git fetch origin pull/13/head:pr-13 && git merge pr-13 --no-edit
118118
git fetch origin pull/15/head:pr-15 && git merge pr-15 --no-edit
119119
120-
# === PATCH 1: Use 'nginx' hostname ===
121-
# This works inside Docker (DNS) and on Host (via /etc/hosts hack above)
122-
sed -i 's/localhost:8000/nginx:8000/g' config/database/update.sh
123-
124-
# === PATCH 2: Fix Path Mismatch ===
125-
# Ensure we use /data/ which Nginx recognizes
126-
sed -i 's|/minio/|/data/|g' config/database/update.sh
127-
128-
echo "=== Patched Update Script ==="
129-
cat config/database/update.sh | grep "nginx"
130-
131-
- name: Start Docker Services
132-
if: matrix.os == 'ubuntu-latest'
133-
working-directory: ./services
134-
run: |
135-
sudo systemctl stop mysql.service
136-
docker compose --profile rest-api --profile minio --profile evaluation-engine up -d --build
137-
138-
echo "1. Waiting for Database population..."
139-
docker wait openml-test-database-setup
140-
141-
echo "2. Waiting for Elasticsearch (this is the slow part)..."
142-
# Wait up to 5 minutes for ES to go green
143-
timeout 300s bash -c 'until [ "$(docker inspect -f {{.State.Health.Status}} openml-elasticsearch)" == "healthy" ]; do sleep 5; done'
144-
145-
echo "3. Waiting for PHP API..."
146-
# Wait up to 5 minutes for PHP to accept connections
147-
timeout 300s bash -c 'until [ "$(docker inspect -f {{.State.Health.Status}} openml-php-rest-api)" == "healthy" ]; do sleep 5; done'
148-
149-
- name: Finalize Setup & Verify Splits
150-
if: matrix.os == 'ubuntu-latest'
151-
run: |
152-
echo "1. Forcing Elasticsearch Indexing Sync (With Auth)..."
153-
# We append the default test API Key (AD0...0) to authorized the admin action
154-
curl -s "http://nginx:8000/api/v1/xml/admin/index/sync?api_key=AD000000000000000000000000000000" || echo "Sync request failed"
155-
156-
echo "2. Waiting for Evaluation Engine to process Task 119..."
157-
echo "Targeting Task 119 (The primary failure point)."
158-
159-
# We give it 5 minutes (300s) to handle the queue.
160-
count=0
161-
while [ $count -lt 30 ]; do
162-
code=$(curl -s -o /dev/null -w "%{http_code}" http://nginx:8000/api_splits/get/119/Task_119_splits.arff)
163-
if [ "$code" == "200" ]; then
164-
echo "✅ SUCCESS: Task 119 splits are ready!"
165-
exit 0
166-
fi
167-
echo " ... waiting for split generation (Current Status: $code)"
168-
sleep 10
169-
count=$((count+1))
170-
done
171-
172-
echo "❌ ERROR: Evaluation Engine timed out."
173-
174-
echo "=== DEBUG: LISTING LOG DIR ==="
175-
docker exec openml-evaluation-engine ls -R /logs/ || echo "Dir empty"
176-
177-
echo "=== DEBUG: DUMPING ALL LOGS ==="
178-
# Use wildcard to catch whatever the filename actually is
179-
docker exec openml-evaluation-engine sh -c "cat /logs/*.log" || echo "Could not read logs"
120+
# # === PATCH 1: Use 'nginx' hostname ===
121+
# # This works inside Docker (DNS) and on Host (via /etc/hosts hack above)
122+
# sed -i 's/localhost:8000/nginx:8000/g' config/database/update.sh
180123
181-
# Also check the cron log again, just in case
182-
docker exec openml-evaluation-engine cat /cron.log || echo "Cron log empty"
124+
# # === PATCH 2: Fix Path Mismatch ===
125+
# # Ensure we use /data/ which Nginx recognizes
126+
# sed -i 's|/minio/|/data/|g' config/database/update.sh
183127
184-
exit 1
128+
# echo "=== Patched Update Script ==="
129+
# cat config/database/update.sh | grep "nginx"
185130
186131
- name: Show installed dependencies
187132
run: python -m pip list

0 commit comments

Comments
 (0)