> On Nov. 12, 2014, 4:02 a.m., Koushik Das wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 1348 > > <https://reviews.apache.org/r/27868/diff/1/?file=757725#file757725line1348> > > > > What operation, which hypervisor and VM? Also the info log is not > > required as the exception message will be present in logs.
I can see the exception messages at debug log level, Think it will be good if we say something at the info level as well. updated the log with relavent info. - bharat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27868/#review60943 ----------------------------------------------------------- On Nov. 11, 2014, 2:19 p.m., bharat kumar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27868/ > ----------------------------------------------------------- > > (Updated Nov. 11, 2014, 2:19 p.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Repository: cloudstack-git > > > Description > ------- > > InvalidParameter Exception with stacktrace in MS log > https://issues.apache.org/jira/browse/CLOUDSTACK-7348 > > we do not throw the exceptions now. Added some info in the log, to say what > happend to the command. > > > Diffs > ----- > > server/src/com/cloud/vm/UserVmManagerImpl.java > 77ace7a5f6c7f6e423a7e97bbf4c4bb5fdcb730a > > Diff: https://reviews.apache.org/r/27868/diff/ > > > Testing > ------- > > > Thanks, > > bharat kumar > >
