`service_logs` API exposes unimplemented POST method for `/api/service_logs/v1/cluster_logs/clusters/{uuid}/cluster_logs`
Closed this issue · 4 comments
The service_logs
API as currently generated makes the Add
method available by way of the ClusterLogsClient
for the path /api/service_logs/v1/cluster_logs/clusters/{uuid}/cluster_logs
. The backend however only implements List
for this path making the sdk
somewhat confusing.
ocm-sdk-go/servicelogs/v1/cluster_client.go
Lines 45 to 53 in 1c9fb31
@gshilin-sdb @Lir10 I think that @Ajpantuso is right. I believe this happens because in the model we are using the same ClusterLogs resource for both .../v1/cluster_logs
and .../v1/clusters/{uuid}/cluster_logs
. That resource definition contains the List
and Add
methods, but for .../v1/clusters/{uuid}/cluster_logs
we want List
only. To fix that we will have to create a new resource similar to the existing ClusterLogs
but with the List
method only. @gshilin-sdb @Lir10 can you take care of that?
sure I'll work on that shortly
This is being resolved by @Lir10 in openshift-online/ocm-api-model#501. Once that is merged we will re-generate the code of the SDK.