fix: use the external names in the error from GET model_version

[pboyd]

Description

GET /api/model_registry/v1alpha3/model_version returned errors using its internal variable names which were returned to the user. Renaming the variables so that the internal and external names are the same.

Fixes #2009

How Has This Been Tested?

On a local dev environment.

Before:

$ curl http://localhost:8080/api/model_registry/v1alpha3/model_version
{"code":"Bad Request","message":"invalid parameters call, supply either (versionName and registeredModelId), or externalId: bad request"}

After:

$ curl http://localhost:8080/api/model_registry/v1alpha3/model_version
{"code":"Bad Request","message":"invalid parameters call, supply either (name and parentResourceId), or externalId: bad request"}

Merge criteria:

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

• The commits have meaningful messages
• Automated tests are provided as part of the PR for major new functionalities; 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.

[pboyd]

/retest

[jonburdo]

/lgtm

[Al-Pragliola]

• fix: use the external names in the error from GET model_version #2012 (review)

/lgtm

/approve

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Al-Pragliola

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [Al-Pragliola]

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