sureshanaparti commented on pull request #5424:
URL: https://github.com/apache/cloudstack/pull/5424#issuecomment-915904754


   > I'm not sure if this helps to address the original issue. Moreover, this 
further complicates it by again changing the listPods API response.
   > In my opinion, we can just update the description for the `startip` and 
`endip` params of the response, stating that they will return lists (and not a 
single IP)
   
   The original issue doesn't make sense as the Pod can hold multiple ip 
ranges, and I'm (-1) with changing the description. The response then doesn't 
maintain proper relation between the range parameters, instead these have 
changed to List from String (I feel, that is not the correct way).


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to