Adds an endpoint to get a registered model by id

[alexcreasy]

Description

Adds an endpoint to fetch registered models by id.

How Has This Been Tested?

Test rig setup:

1. Deploy MR backend (you can follow the guide using kind in the docs)
2. Deploy BFF with e.g. make run PORT=4000 MOCK_K8S_CLIENT=true
3. Add a model to one of the registries:

curl -i -X POST "http://localhost:4000/api/v1/model-registry/model-registry/registered_models" \
     -H "Content-Type: application/json" \
     -d '{
  "customProperties": {
    "my-label9": {
      "metadataType": "MetadataStringValue",
      "string_value": "val"
    }
  },
  "description": "bella description",
  "externalId": "9927",
  "name": "bella",
  "owner": "eder",
  "state": "LIVE"
}'

Test routine:

1. Verify the endpoint returns a 200 and the model from the endpoint: curl -i localhost:4000/api/v1/model-registry/model-registry/registered_models/1
2. Try with a different non existent id and verify that a 404 is returned: curl -i localhost:4000/api/v1/model-registry/model-registry/registered_models/fluglebinder

Merge criteria:

• All the commits have been signed-off (To pass the DCO check)

• The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
• Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
• The developer has manually tested the changes and verified that the changes work.
• Code changes follow the kubeflow contribution guidelines.

If you have UI changes

• The developer has added tests or explained why testing cannot be added.
• Included any necessary screenshots or gifs if it was a UI change.
• Verify that UI/UX changes conform the UX guidelines for Kubeflow.

[Griffin-Sullivan]

FYI there's a typo in the commit message. Adds and endpoint to get individual registered models

[Griffin-Sullivan]

Tested the changes. Everything is working for me.

[lucferbux]

Changes seem to work fine, great work here, I subscribe to @Griffin-Sullivan questions, once that's sorted out we can lgtm it!

[Griffin-Sullivan]

Changes are good to me

[alexcreasy]

/approve

[alexcreasy]

/lgtm

[google-oss-prow]

@alexcreasy: you cannot LGTM your own PR.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[alexcreasy]

I mean, that's fair that I can't LGTM it myself technically, but Griffen just did :P

[ederign]

@alexcreasy just some small changes related to our API design. Another option is to use https://pkg.go.dev/github.com/julienschmidt/httprouter#Router.RedirectTrailingSlash , but we would need to investigate better if there is any side effect of using it.

Also, could you please add the example from the PR in our README?

[alexcreasy]

@ederign think we should be gtg now

[ederign]

/lgtm

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexcreasy, ederign, Griffin-Sullivan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• clients/ui/OWNERS [alexcreasy,ederign]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment