[PGijsbers]

We've made some changes to services lately which allows us to run fewer services, so this should decrease startup time.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.91%. Comparing base (d35b799) to head (fb90c71).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1782   +/-   ##
=======================================
  Coverage   80.91%   80.91%           
=======================================
  Files          63       63           
  Lines        5230     5230           
=======================================
  Hits         4232     4232           
  Misses        998      998           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[PGijsbers]

Will update PR once openml/services#34 is merged.

[PGijsbers]

Current issues:

• tests still use url uploading, will be disbaled in [pre-commit.ci] pre-commit autoupdate #1783.
• Windows tests use actual test server instead of services because the scripts setup steps don't work on windows. There seems to be a problematic tagging response from the test.openml.org elasticsearch instance, which seems out of scope for a fix in openml-python.

[PGijsbers]

Docker compose would take a long time to start with the old setup since elastic search indexing needed to happen. I think this required a workaround to properly wait for services to be up. Either that or at some point a container which is supposed to exit gracefully during startup (e.g., database-setup) was included and this conflicts with --wait. Either way, with the new profile definition we don't have this issue and --wait works as expected.