This is an automated email from the ASF dual-hosted git repository.
raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 515f724690 GH-47692: [CI][Python] Do not fallback to return 404 if
wheel is found on emscripten jobs (#49007)
515f724690 is described below
commit 515f7246901f6ad6b630d92f5136959c3251b694
Author: Raúl Cumplido <[email protected]>
AuthorDate: Tue Jan 27 13:36:24 2026 +0100
GH-47692: [CI][Python] Do not fallback to return 404 if wheel is found on
emscripten jobs (#49007)
### Rationale for this change
When looking for the wheel the script was falling back to returning a 404
even when the wheel was found:
```
+ python scripts/run_emscripten_tests.py
dist/pyarrow-24.0.0.dev31-cp312-cp312-pyodide_2024_0_wasm32.whl
--dist-dir=/pyodide --runtime=chrome
127.0.0.1 - - [27/Jan/2026 01:14:50] code 404, message File not found
```
Timing out the job and failing.
### What changes are included in this PR?
Correct logic and only return 404 if the file requested wasn't found.
### Are these changes tested?
Yes via archery
### Are there any user-facing changes?
No
* GitHub Issue: #47692
Authored-by: Raúl Cumplido <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
---
python/scripts/run_emscripten_tests.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/python/scripts/run_emscripten_tests.py
b/python/scripts/run_emscripten_tests.py
index 53d3dd52bd..406dfc54e4 100644
--- a/python/scripts/run_emscripten_tests.py
+++ b/python/scripts/run_emscripten_tests.py
@@ -45,7 +45,7 @@ class TemplateOverrider(http.server.SimpleHTTPRequestHandler):
self.end_headers()
with PYARROW_WHEEL_PATH.open(mode="rb") as wheel:
self.copyfile(wheel, self.wfile)
- if self.path.endswith("/test.html"):
+ elif self.path.endswith("/test.html"):
body = b"""
<!doctype html>
<html>
