DaanHoogland commented on pull request #5782:
URL: https://github.com/apache/cloudstack/pull/5782#issuecomment-996616882


   > This is the same for the `listHosts`. Do we need to fix other searches as 
well, such as for hosts?
   > 
   > Additionally, this will change the behavior of the UI searches. I 
particularly have been using a lot the list filtering to give hosts/clusters 
with name matching with the provided string. I am not sure what would be the 
best option here, but particularly I like having the possibility to filter 
hosts by name. If we want a specific host we can already filter by uuid.
   
   I don't think it does @GabrielBrascher, the UI uses the keyword parameter 
and not the name (unless there's a bug there as well.
   
   > 
   > However, if the expected behavior of the API is to list by matching 
exactly with the name, then this PR is OK. But we might need to refactor the UI.
   
   The problem is mainly custom scripts I think, but we could label it a bug 
that has been featured. I that case we would need to deprecate the existing 
bahaviour and implement a new API with correct behaviour (to be very formally 
correct)


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