RodrigoDLopez commented on pull request #4575: URL: https://github.com/apache/cloudstack/pull/4575#issuecomment-762318256
Hey @shwstppr what do you suggest? I totally agree with @GabrielBrascher, normally when we are talking about a host we use their names to refer to it, I never met someone who uses UUID for this purpose. Maybe, in some cases, the UUID turn easy to find these hosts on the database, but with a simple change on the query, we have the same result. And for me, something like > Operation Failed vm's original host: node23.clusterA (ID: 98c37930-599f-11eb-ae93-0242ac130002) new host: node09.clusterA (ID: bdb22ea8-599f-11eb-ae93-0242ac130002) host before state transition: node01.clusterA (ID: bdb22598-599f-11eb-ae93-0242ac130002) it is too verbose and does not add anything to the debug factor. Presenting only the hostname in the log makes it easy to identify, as it is the way we use it to reference it daily. And with the name, we can find the UUID if necessary. Don't you agree? ---------------------------------------------------------------- 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: [email protected]
