Skip to content

[Model Monitoring] Remove v3io from mlrun config, remove deprecated MM ENDPOINT_STORE_CONNECTION#263

Merged
GiladShapira94 merged 4 commits intomlrun:developmentfrom
royischoss:ceml-229
Feb 26, 2026
Merged

[Model Monitoring] Remove v3io from mlrun config, remove deprecated MM ENDPOINT_STORE_CONNECTION#263
GiladShapira94 merged 4 commits intomlrun:developmentfrom
royischoss:ceml-229

Conversation

@royischoss
Copy link
Contributor

@royischoss royischoss commented Feb 24, 2026

This change is part of https://iguazio.atlassian.net/browse/CEML-358 , https://iguazio.atlassian.net/browse/CEML-229
Includes:

  1. Adding MLRUN_MODEL_ENDPOINT_MONITORING__STORE_PREFIXES__USER_SPACE, MLRUN_MODEL_ENDPOINT_MONITORING__STORE_PREFIXES__MONITORING_APPLICATION to override mlrun defualt config to remove v3io definitions.
  2. Remove MLRUN_MODEL_ENDPOINT_MONITORING__ENDPOINT_STORE_CONNECTION that was deprecated.

No breaking changes, Model monitoring works with/without this change

…R_SPACE and MLRUN_MODEL_ENDPOINT_MONITORING__STORE_PREFIXES__MONITORING_APPLICATION

plus removes MLRUN_MODEL_ENDPOINT_MONITORING__ENDPOINT_STORE_CONNECTION
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes deprecated v3io-related configuration from MLRun model monitoring and introduces new S3-based store prefix configurations. The changes align with the broader initiative (CEML-358, CEML-229) to migrate away from v3io storage and establish explicit S3 paths for model monitoring data.

Changes:

  • Added two new model monitoring store prefix configurations: USER_SPACE and MONITORING_APPLICATION pointing to S3 paths
  • Removed deprecated ENDPOINT_STORE_CONNECTION configuration that referenced a DSN template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mlrun mlrun deleted a comment from Copilot AI Feb 24, 2026
@GiladShapira94 GiladShapira94 merged commit ba417c1 into mlrun:development Feb 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants