ocket8888 commented on a change in pull request #4374: Create API v2 tests
URL: https://github.com/apache/trafficcontrol/pull/4374#discussion_r382845286
 
 

 ##########
 File path: traffic_ops/client/deliveryservice.go
 ##########
 @@ -394,6 +394,16 @@ func (to *Session) GetDeliveryServiceCapacity(id string) 
(*tc.DeliveryServiceCap
 
 // GetDeliveryServiceServer returns associations between Delivery Services and 
servers using the
 // provided pagination controls.
+func (to *Session) GetDeliveryServiceRouting(id string) 
(*tc.DeliveryServiceRouting, ReqInf, error) {
 
 Review comment:
   So this route does exist in TO API v2.0, but the client method doesn't exist 
in method, and as written it won't work. There's nothing wrong with leaving it 
in - but it needs to be fixed. Basically just add an API endpoint above and use 
it here instead of `deliveryServiceRoutingEp(id)` - as that function no longer 
exists.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to