> On July 17, 2013, 12:24 a.m., Sheng Yang wrote: > > server/src/com/cloud/template/TemplateManagerImpl.java, line 999 > > <https://reviews.apache.org/r/12612/diff/1/?file=322288#file322288line999> > > > > How can this happen?? Add Min Chen in reviewer. > > Venkata Siva Vijayendra Bhamidipati wrote: > This is because vmware tools or xenserver tools ISOs aren't actual ISOs. > They're internally provisioned by vmware/xenserver, so the URL will be null. > Since it isn't really resident anywhere, the image store will also be absent > for it, i.e., null.
Vijay, I don't quite understand this. This check is to check if VM is at running state. Original logic is that if VM that we are going to attach ISO to is not at running, we just directly return true and update DB to put iso id for this VM. Why did you change to return false here? - Min ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12612/#review23217 ----------------------------------------------------------- On July 17, 2013, 12:39 a.m., Venkata Siva Vijayendra Bhamidipati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12612/ > ----------------------------------------------------------- > > (Updated July 17, 2013, 12:39 a.m.) > > > Review request for cloudstack, Chip Childers, edison su, Kelven Yang, Sateesh > Chodapuneedi, and Sheng Yang. > > > Bugs: CLOUDSTACK-3554 > > > Repository: cloudstack-git > > > Description > ------- > > Fixing NPE that occurs when attaching vmware-tools.iso to a guest VM on ESX. > > > Diffs > ----- > > > plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java > 4113803 > server/src/com/cloud/template/TemplateManagerImpl.java c7cc818 > > Diff: https://reviews.apache.org/r/12612/diff/ > > > Testing > ------- > > With the fix in place, vmware-tools.iso attach to a vmware guest VM works as > expected. > > > Thanks, > > Venkata Siva Vijayendra Bhamidipati > >