Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update protobuf to include control plane #616

Merged
merged 7 commits into from
Jun 6, 2022

Conversation

adriangonz
Copy link
Contributor

Fixes #600

Changelog

  • Merge model repository endpoints with data plane's service
  • Keep previous model repository API but flag as deprecated (will be removed in MLServer 1.2.0)

@adriangonz adriangonz force-pushed the update-data-plane branch from 2c4dd25 to a5e8add Compare June 6, 2022 11:14
@adriangonz adriangonz merged commit ec8780b into SeldonIO:master Jun 6, 2022
@adriangonz adriangonz deleted the update-data-plane branch June 6, 2022 13:21
kserve-oss-bot pushed a commit to kserve/modelmesh-runtime-adapter that referenced this pull request May 26, 2023
#### Motivation

Related to [updating the MLServer runtime image](kserve/modelmesh-serving#355), the `ModelRepository` endpoint was deprecated. 

References:
kserve/modelmesh-serving#159
SeldonIO/MLServer#616

#### Modifications
- Updated protobuf 
- Updated mock server testing
- Updated runtime-adapter code to call new endpoint

#### Result
- MLServer runtime adapter no longer uses the deprecated model repository API

Signed-off-by: Rafael Vasquez <[email protected]>
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.

Merge v2 control and data separate protos in one common place
1 participant