----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22725 -----------------------------------------------------------
Commit 8526a6f8f9130a757a2e54e968965052dcc054e4 in branch refs/heads/master-6-17-stable from Prasanna Santhanam [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=8526a6f ] CLOUDSTACK-3184: integration test for the updateVirtualMachine updateVirutalMachine fails if no args are passed. this test also further verifies that the displayname is changed when vm is started up successfully. Signed-off-by: Prasanna Santhanam <t...@apache.org> (cherry picked from commit 9f0c9ee20c1a65d427702a330798512e7297e322) - ASF Subversion and Git Services On July 3, 2013, 11:21 a.m., Sanjay Tripathi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12243/ > ----------------------------------------------------------- > > (Updated July 3, 2013, 11:21 a.m.) > > > Review request for cloudstack, Devdeep Singh and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-3184 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-3184: updateVirtualMachine api gives java NPE. > > This bug is not reproducible from CloudStack UI. Steps to reproduce this bug: > > 1. Deploy a VM without passing any parameters in Name and Description (last > step in deploy VM wizard). > 2. Stop this VM. > 3. Fire updateVIrtualMachine API without passing "displayName" parameter, > you'll see NPE in MS logs. Do not try with UI because UI will pass > displayName paramter with blank value. > > There isn't any null check for displayName parameter and it is null because > initially it was not set during the VM deployment and afterwards we are not > passing displayName paramter in the API. > > So added null check for displayName in if condition. > > > Diffs > ----- > > server/src/com/cloud/vm/UserVmManagerImpl.java a59fa5b > > Diff: https://reviews.apache.org/r/12243/diff/ > > > Testing > ------- > > Same as Steps mentioned in the description. > > > Thanks, > > Sanjay Tripathi > >
