Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
95 commits
Select commit Hold shift + click to select a range
83e1531
Use the correct path to the cache directory for the task
PGijsbers Jan 20, 2026
f90036d
Push configuration of test server URL exclusively to config.py
PGijsbers Jan 21, 2026
3a257ab
Update the test to use a dataset which does not have a parquet file
PGijsbers Jan 28, 2026
3b79017
Replace hard-coded cache directory by configured one
PGijsbers Jan 28, 2026
f524d75
Update test to use dataset file that is already in cache
PGijsbers Jan 28, 2026
7ef12c2
Windows test
satvshr Jan 29, 2026
a5601e3
relax assumptions on local file structure
PGijsbers Jan 29, 2026
d862be2
Do not use static cache directory
PGijsbers Jan 29, 2026
16699e6
Update expected number to match initial server state
PGijsbers Jan 29, 2026
7c14c68
bug fixing
satvshr Jan 29, 2026
78b2038
merge main
satvshr Jan 29, 2026
16ceeaa
remove db refresh every test
satvshr Jan 29, 2026
015acf4
bug fixing
satvshr Jan 29, 2026
937fc77
bug fixing
satvshr Jan 29, 2026
30972f8
bug fixing
satvshr Jan 29, 2026
775dcf7
Add symlink to regular test cache directory
PGijsbers Jan 30, 2026
319cb35
Skip test for 1.8 since expected results differ too much
PGijsbers Jan 30, 2026
a680ebe
Simplify path to static cache directory
PGijsbers Jan 30, 2026
b161b3b
Update symbolic link to be relative
PGijsbers Jan 30, 2026
0b989d1
Fix typo
PGijsbers Jan 30, 2026
892ea6c
trying ot fix multiple threads issue
satvshr Jan 31, 2026
ae3befb
removed test file
satvshr Jan 31, 2026
5f396a0
removed unnecessary code (?)
satvshr Jan 31, 2026
8a319cd
Trigger Build
satvshr Jan 31, 2026
4ba4239
Clean up code
satvshr Feb 1, 2026
0292404
comment fixing
satvshr Feb 1, 2026
a7b5d76
attempted bug fixing
satvshr Feb 1, 2026
9b0f3d7
attempted bug fixing
satvshr Feb 1, 2026
630f240
attempted bug fixing
satvshr Feb 1, 2026
c61d410
attempted bug fixing reverts
satvshr Feb 1, 2026
1ab42b7
disabling parallel runs
satvshr Feb 1, 2026
06405c8
disabling parallel runs
satvshr Feb 2, 2026
e22b7ca
disabling windows CI
satvshr Feb 2, 2026
1b00a7f
removed docker from pytest default
satvshr Feb 6, 2026
cc6e673
change mysql port
satvshr Feb 6, 2026
c1bf558
Change order of ci flow
satvshr Feb 6, 2026
1a794fe
CI testing
satvshr Feb 11, 2026
dbe7782
CI testing
satvshr Feb 11, 2026
d8be5f1
CI testing
satvshr Feb 11, 2026
b204845
CI testing
satvshr Feb 11, 2026
54725fa
Windows CI bugfixing
satvshr Feb 11, 2026
abc44a5
merging 2 branches
satvshr Feb 11, 2026
b034687
merging 2 branches
satvshr Feb 11, 2026
b8826f5
merging 2 branches
satvshr Feb 11, 2026
445cbe8
merging 2 branches
satvshr Feb 11, 2026
295ef93
curl to verify server is running
satvshr Feb 11, 2026
488f409
path fix
satvshr Feb 11, 2026
93d7409
Merge branch 'update-tests-for-local' into i1614
satvshr Feb 11, 2026
45e7257
run all test server tests
satvshr Feb 11, 2026
7fcf039
fix 'Cleanup Docker setup'
satvshr Feb 11, 2026
37cfb2e
skipping windows given docker binaries do not match
satvshr Feb 11, 2026
9290010
testing out locally
satvshr Feb 12, 2026
bbfa193
replacing with 8080
satvshr Feb 12, 2026
4531cbc
test
satvshr Feb 12, 2026
d90615a
test
satvshr Feb 12, 2026
9b12d6f
test
satvshr Feb 12, 2026
45d3423
test
satvshr Feb 12, 2026
16f22b1
test
satvshr Feb 12, 2026
dd2ce68
test
satvshr Feb 12, 2026
ebeccea
test
satvshr Feb 12, 2026
a0ac6b9
test
satvshr Feb 12, 2026
439e683
test
satvshr Feb 12, 2026
f87051b
test
satvshr Feb 12, 2026
4077a56
test
satvshr Feb 12, 2026
fad1ee7
test
satvshr Feb 12, 2026
4086730
test
satvshr Feb 12, 2026
fecebbc
windows fix?
satvshr Feb 12, 2026
4845a1e
windows fix?
satvshr Feb 12, 2026
a247050
windows fix?
satvshr Feb 12, 2026
e4a6807
Do not include ports in cache path. ':' not supported by windows
PGijsbers Feb 13, 2026
03bf396
Merge branch 'main' into update-tests-for-local
PGijsbers Feb 13, 2026
4c6bd2f
merge 1630
satvshr Feb 13, 2026
ba0e480
revert test.yml changes
satvshr Feb 13, 2026
519d5cb
Update openml/config.py
PGijsbers Feb 16, 2026
30fd44d
Merge branch 'main' into update-tests-for-local
PGijsbers Feb 16, 2026
c908993
Merge branch 'main' into update-tests-for-local
PGijsbers Feb 17, 2026
06b9741
Keep port as part of cache directory path
PGijsbers Feb 17, 2026
015cca3
merge update-tests-for-local
satvshr Feb 18, 2026
8de78af
merge main
satvshr Feb 22, 2026
4482a2c
req changes
satvshr Feb 22, 2026
278b546
changing retries
satvshr Feb 22, 2026
cf30367
fixes
satvshr Feb 23, 2026
b41a9b2
testing replacement
satvshr Feb 23, 2026
f737cb1
bug fix
satvshr Feb 23, 2026
b40d702
added skip tests
satvshr Feb 23, 2026
5f079ba
final touches
satvshr Feb 23, 2026
f75b2de
Added skip tests
satvshr Feb 24, 2026
e59a990
revery ruff
satvshr Feb 26, 2026
7423b88
revert ruff
satvshr Feb 26, 2026
cf7d0a1
revert ruff
satvshr Feb 26, 2026
d8bb385
skip tests
satvshr Feb 26, 2026
6f2a47e
skip tests
satvshr Feb 26, 2026
bf2a73e
skip test
satvshr Feb 26, 2026
27d3aba
Merge branch 'main' into i1614
PGijsbers Mar 4, 2026
e31df09
Merge branch 'main' into i1614
PGijsbers Mar 9, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
trying ot fix multiple threads issue
  • Loading branch information
satvshr committed Jan 31, 2026
commit 892ea6c85ce7eecd5ae0541ad46b2a0f459786b5
45 changes: 31 additions & 14 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@
import multiprocessing
import sys

import fasteners

multiprocessing.set_start_method("spawn", force=True)

from collections.abc import Iterator
Expand Down Expand Up @@ -300,23 +302,38 @@ def with_test_cache(test_files_directory, request):
if tmp_cache.exists():
shutil.rmtree(tmp_cache)

@pytest.fixture(scope="session", autouse=True)
def openml_docker_stack():
subprocess.run(["docker", "compose", "up", "-d"], check=True)
def _is_server_responding():
"""Check if the Docker API is already listening."""
try:
requests.get("http://localhost:9001/api/v2/", timeout=1)
return True
except (requests.exceptions.ConnectionError, requests.exceptions.Timeout):
return False

def _start_docker():
"""Logic to spin up the containers and wait for initialization."""
subprocess.run(["docker", "compose", "up", "-d"], check=True, capture_output=True, text=True)
subprocess.run(["docker", "wait", "openml-test-setup-ci"], check=True)

@pytest.fixture(scope="session", autouse=True)
def openml_docker_stack(tmp_path_factory, worker_id):
Comment thread
satvshr marked this conversation as resolved.
Outdated
# For local development, single-process runs
if worker_id == "master":
_start_docker()
yield
subprocess.run(["docker", "compose", "down", "-v"], check=True)
return

# Case 2: Running in CI with multiple workers (xdist)
root_tmp_dir = tmp_path_factory.getbasetemp().parent
lock_file = root_tmp_dir / "docker_setup.lock"

timeout = 10
start = time.time()
while time.time() - start < timeout:
try:
requests.get("http://localhost:9001/api/v2/")
break
except requests.exceptions.ConnectionError:
time.sleep(1)

lock = fasteners.InterProcessLock(str(lock_file))
with lock:
if not _is_server_responding():
_start_docker()

yield

subprocess.run(["docker", "compose", "down", "-v"], check=True)

@pytest.fixture
def static_cache_dir():
Expand Down
Loading