Skip to content

Conversation

@alxtkr77
Copy link
Member

@alxtkr77 alxtkr77 commented Jan 6, 2026

Replace DatastoreProfileTDEngine with DatastoreProfilePostgreSQL for Community Edition deployments. TimescaleDB uses PostgreSQL protocol.

Related PR

This PR is coupled with mlrun/ce#234 and they should be merged together. It assumes that in 1.11, TimescaleDB replaces TDengine, so if merged before mlrun/ce#234, it will fail to find TimescaleDB. I verified that the TimescaleDB credentials in this PR correspond to the CE TimescaleDB installation in mlrun/ce#234.

@alxtkr77 alxtkr77 changed the title [Agent Deployer] Replace TDEngine with TimescaleDB for CE model monit… [Agent Deployer] Replace TDEngine with TimescaleDB for CE model monitoring Jan 6, 2026
…oring

Replace DatastoreProfileTDEngine with DatastoreProfilePostgreSQL for
Community Edition deployments. TimescaleDB uses PostgreSQL protocol.

- Add conditional import: try DatastoreProfilePostgreSQL first, fallback
  to DatastoreProfileTDEngine for older mlrun versions
- Configure TimescaleDB connection for CE mode (when available):
  - Host: timescaledb.{namespace}.svc.cluster.local
  - Port: 5432
  - Credentials: postgres/postgres
  - Database: postgres
- Maintain backwards compatibility with older mlrun versions using TDEngine

Reference: ML-11628
Copy link
Contributor

@royischoss royischoss left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@assaf758 assaf758 merged commit 0a4b5c8 into mlrun:development Jan 6, 2026
3 checks passed
Eyal-Danieli pushed a commit to Eyal-Danieli/functions that referenced this pull request Jan 11, 2026
…oring (mlrun#961)

Replace DatastoreProfileTDEngine with DatastoreProfilePostgreSQL for
Community Edition deployments. TimescaleDB uses PostgreSQL protocol.

- Add conditional import: try DatastoreProfilePostgreSQL first, fallback
  to DatastoreProfileTDEngine for older mlrun versions
- Configure TimescaleDB connection for CE mode (when available):
  - Host: timescaledb.{namespace}.svc.cluster.local
  - Port: 5432
  - Credentials: postgres/postgres
  - Database: postgres
- Maintain backwards compatibility with older mlrun versions using TDEngine

Reference: ML-11628

Co-authored-by: Alex Toker <alext@mckinsey.com>
Eyal-Danieli added a commit that referenced this pull request Jan 11, 2026
* update requirements.txt with stable mlrun version (#960)

* [Agent Deployer] Replace TDEngine with TimescaleDB for CE model monitoring (#961)

Replace DatastoreProfileTDEngine with DatastoreProfilePostgreSQL for
Community Edition deployments. TimescaleDB uses PostgreSQL protocol.

- Add conditional import: try DatastoreProfilePostgreSQL first, fallback
  to DatastoreProfileTDEngine for older mlrun versions
- Configure TimescaleDB connection for CE mode (when available):
  - Host: timescaledb.{namespace}.svc.cluster.local
  - Port: 5432
  - Credentials: postgres/postgres
  - Database: postgres
- Maintain backwards compatibility with older mlrun versions using TDEngine

Reference: ML-11628

Co-authored-by: Alex Toker <alext@mckinsey.com>

* Add kind to step and rename the `utils` category to `utilities` (#962)

* replace utils with utilities

* add kind to verify_schema

* add kind to the step template

---------

Co-authored-by: Roy Schossberger <85231212+royischoss@users.noreply.github.com>
Co-authored-by: alxtkr77 <3098237+alxtkr77@users.noreply.github.com>
Co-authored-by: Alex Toker <alext@mckinsey.com>
Co-authored-by: Daniel Perez <100069700+danielperezz@users.noreply.github.com>
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