Add DataPlane API Support #936
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #936 +/- ##
========================================
+ Coverage 7.04% 7.09% +0.04%
========================================
Files 282 284 +2
Lines 65329 65542 +213
========================================
+ Hits 4602 4648 +46
- Misses 60418 60584 +166
- Partials 309 310 +1 ☔ View full report in Codecov by Sentry. |
| {{.Comment " // " 80}} | ||
| {{.Name}} {{.Package.Name}}.{{.Name}}Interface | ||
| {{end}}{{end}} | ||
| } | ||
|
|
||
| // Returns a new OAuth scoped to the authorization details provided. | ||
| // It will return an error if the CredentialStrategy does not support OAuth tokens. | ||
| // |
There was a problem hiding this comment.
We don't need this on the WorkspaceClient, but the Client itself.
There was a problem hiding this comment.
@hectorcast-db, I left a few Go-style suggestions — please have a look. Happy to discuss these offline.
| return nil, err | ||
| } | ||
| if response.{{(index .DataPlaneInfoFields 0).PascalName}} == nil { | ||
| return nil, errors.New("resource does not support direct Data Plane access") |
There was a problem hiding this comment.
When would this case happen? Is this a bug in the API definition? Or is it possible that some model serving endpoints support direct-to-dataplane access and others don't?
There was a problem hiding this comment.
DataPlane access needs to be enabled on a resource level. If you call this method for a Model which is not "optimized" (their flag for DataPlane), this field won't be set (the endpointURL does not exists)
Changes
Add DataPlane API Support
Tests
Manual test:
make testpassingmake fmtapplied