GabrielBrascher commented on pull request #4575:
URL: https://github.com/apache/cloudstack/pull/4575#issuecomment-782089537
Applied changes in order to use the `toString()` method from HostVO. These
changes consider the proposal from PR #4706 where `toString()` would present a
"_JSON like"_ String.
- How it is today: `Host[-123-Routing]` (yep ... nowadays logs that use the
toString have `-` before the ID)
- Proposal from #4706: `Host [{id: "123", name: "hostName", uuid:
"1e374e50-g231-4345-d6h3-123abcdrfas143", type="Routing"}]`
The idea is that it would be good for:
1. Debug via DB filtering with the host ID
2. Run commands on API/cloudmonkey by copying the uuid presented at the log
3. Identify the host quickly as name makes logs human-friendly
4. Name also tends to be resolved to the host address; therefore, admins can
jump into the host (SSH) by copying the name from the log.
Ping for discussion/review: @rafaelweingartner @sureshanaparti @shwstppr
@RodrigoDLopez @DaanHoogland @rhtyd @wido @ustcweizhou @svenvogel
@andrijapanicsb and others ...
----------------------------------------------------------------
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]
