weizhouapache commented on a change in pull request #4601:
URL: https://github.com/apache/cloudstack/pull/4601#discussion_r563773324
##########
File path: server/src/main/java/com/cloud/server/StatsCollector.java
##########
@@ -684,18 +684,17 @@ protected void runInContext() {
return;
}
// collect the vm disk statistics(total) from hypervisor. added by
weizhou, 2013.03.
- s_logger.trace("Running VM disk stats ...");
- try {
- Transaction.execute(new TransactionCallbackNoReturn() {
- @Override
- public void doInTransactionWithoutResult(TransactionStatus
status) {
- s_logger.debug("VmDiskStatsTask is running...");
+ s_logger.debug("VmDiskStatsTask is running...");
Review comment:
@DaanHoogland the message and level are not changed. moved out from try
block.
Removed the line which is not needed as we have debug message.
`s_logger.trace("Running VM disk stats ...");`
----------------------------------------------------------------
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]
