Uh oh!
There was an error while loading. Please reload this page.
[ENH] Make TEST_SERVER_URL configurable via environment variable - #1663
[ENH] Make TEST_SERVER_URL configurable via environment variable#1663Sandipmandal25 wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR makes the test server URL configurable via the OPENML_TEST_SERVER_URL environment variable, enabling developers to run tests against local or custom test servers without modifying source code. This addresses issue #1657 about failures when running tests against locally replicated test servers and supports the dockerized CI testing workflow from PR #1629.
Changes:
- Adds
OPENML_TEST_SERVER_URL_ENV_VARconstant following existing naming conventions - Makes
TEST_SERVER_URLread from environment variable with fallback to defaulthttps://test.openml.org
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
geetu040
left a comment
There was a problem hiding this comment.
I am not sure if we need this in the long-term since we expect multiple API versions in future. But let's keep this PR open until we reach a conclusion.
Metadata
TEST_SERVER_URLconfigurable viaOPENML_TEST_SERVER_URLenvironment variable"Details
Running tests against a local server currently requires editing
openml/config.pydirectly — fragile and CI-unfriendly. This PR introducesOPENML_TEST_SERVER_URLso developers can point tests elsewhere without touching source code:Follows the existing pattern of
OPENML_CACHE_DIRandOPENML_TEST_SERVER_ADMIN_KEY. No behavior change when unset — defaults tohttps://test.openml.orgas before.This is also the approach requested in #1629 (review by @geetu040 ) to avoid modifying source code for CI-based local server testing.