----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7153/#review11681 -----------------------------------------------------------
Ship it! Ship It! - edison su On Sept. 19, 2012, 12:56 a.m., Hiroaki Kawai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7153/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2012, 12:56 a.m.) > > > Review request for cloudstack. > > > Description > ------- > > add logging for debugging. > > The default value for local.storage.path does not exists by > default in CentOS 6. By default, this results in NullPointerException > silently. Without this log message, administrator can't figure out > the reason at all. > > > Diffs > ----- > > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java > d6236a0d6038bbf251450a424ea0089c01739e00 > > Diff: https://reviews.apache.org/r/7153/diff/ > > > Testing > ------- > > > Thanks, > > Hiroaki Kawai > >
