lucas-a-martins commented on PR #8603:
URL: https://github.com/apache/cloudstack/pull/8603#issuecomment-2101020576

   > @lucas-a-martins I understand some error messages are unnecessary for 
normal users, not only this one, but also most other exceptions in vm the 
deployment, migration, etc. I agree we should display different error messages 
to normal users.
   > 
   > However I hope, as a root admin, we can get more information from error 
message on the UI and in the API response. Please bear in mind , not all root 
admins have access to the management server log and have the ability to find 
the root cause from massvie logs.
   > 
   > you can explore the options, for example in the API response or UI, 
display "Unable to complete this operation. Contact your cloud admin." 
(configurable) to normal users, and display the detailed error message to root 
admins. It could apply on ALL exceptions.
   
   Hey, @weizhouapache 
   
   Considering these scenarios, I agree that having an exception to Root Admins 
makes sense and could be applied to all exceptions; however, this goes out of 
the scope of this PR. We have a discussion about patterns about logging (#8746) 
that could address these cases as well; we can discuss that there.


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to