Migrating from /api to /ajax-api endpoint

JuliaAI / MLFlowClient.jl

Julia client for MLFlow.

https://juliaai.github.io/MLFlowClient.jl/

MIT License

46 stars 9 forks source link

Migrating from /api to /ajax-api endpoint #36

Closed pebeto closed 8 months ago

pebeto commented 8 months ago

The /ajax-api prefix is used by Dagshub to connect to the REST API. This is giving exactly the same responses as the previous /api prefix. This solves https://github.com/JuliaAI/MLFlowClient.jl/issues/35#issuecomment-1880158336.

I don't know the existance of both, but we must go through the one that is compatible with both solutions. In the future, if there's a case that is breaking this, we should think in a way to separate each different platform.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5a4237f) 77.64% compared to head (ac7c806) 77.71%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #36 +/- ## ========================================== + Coverage 77.64% 77.71% +0.06% ========================================== Files 9 9 Lines 331 332 +1 ========================================== + Hits 257 258 +1 Misses 74 74 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

deyandyankov commented 8 months ago

Thanks! Merged and triggered a new release: https://github.com/JuliaRegistries/General/pull/98410

freemin7 commented 6 months ago

This makes implementing GitLab support a lot harder as Gitlab is unaware of the undocumented (a search on the mlflow page found no mentions of this endpoint) ajax-api end point and uses the api endpoint.