test

Author: satvshr

.github/workflows/test.yml

@@ -101,29 +101,32 @@ jobs:
         echo "BEFORE=$git_status" >> $GITHUB_ENV
         echo "Repository status before tests: $git_status"
 
-    - name: Clone Services
+    - name: Configure Host Network (The "Magic" Step)
+      run: |
+        # Map 'nginx' to localhost so the Host machine can resolve the URLs in the database
+        echo "127.0.0.1 nginx" | sudo tee -a /etc/hosts
+        ping -c 1 nginx
+
+    - name: Clone Services & Apply Universal Patch
       if: matrix.os == 'ubuntu-latest'
       run: |
         git clone --depth 1 https://github.com/openml/services.git
         cd services
-        
         git config user.email "ci@openml.org"
         git config user.name "CI"
-
-        git fetch origin pull/13/head:pr-13
-        git merge pr-13 --no-edit
-
-        git fetch origin pull/15/head:pr-15
-        git merge pr-15 --no-edit
-        sed -i 's/localhost:8000/172.28.0.2:8000/g' config/database/update.sh
+        git fetch origin pull/13/head:pr-13 && git merge pr-13 --no-edit
+        git fetch origin pull/15/head:pr-15 && git merge pr-15 --no-edit
+        
+        # === PATCH 1: Use 'nginx' hostname ===
+        # This works inside Docker (DNS) and on Host (via /etc/hosts hack above)
+        sed -i 's/localhost:8000/nginx:8000/g' config/database/update.sh
         
-        # === PATCH 2: Fix MinIO Path Mismatch ===
-        # The PR uses '/minio/' but Nginx usually expects '/data/' for MinIO
-        # We replace '/minio/' with '/data/' in the URL rewrite script
+        # === PATCH 2: Fix Path Mismatch ===
+        # Ensure we use /data/ which Nginx recognizes
         sed -i 's|/minio/|/data/|g' config/database/update.sh
         
         echo "=== Patched Update Script ==="
-        cat config/database/update.sh | grep "172.28.0.2"
+        cat config/database/update.sh | grep "nginx"
 
     - name: Start Docker Services
       if: matrix.os == 'ubuntu-latest'