Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22726 --- Commit 6b3257781173f3596114c5b19366e72ffb46 in branch refs/head

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22725 --- Commit 8526a6f8f9130a757a2e54e968965052dcc054e4 in branch refs/head

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22724 --- Commit b99355698823bf7fecfa4fddcc63e5b0e76c4931 in branch refs/heads

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22723 --- Commit 614d42dda26f4b1f688f65f597116b2bb5b3d5a7 in branch refs/heads

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread Prasanna Santhanam
> On July 3, 2013, 2:09 p.m., Prasanna Santhanam wrote: > > 31df729008c748e6eafe49ff86e1da618013c551 for the fix. > > > > Added an integration test to back up the API > > issue:9f0c9ee20c1a65d427702a330798512e7297e322 Test passes after the fix. Please close the review as submitted. $ nosetest

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22718 --- Ship it! 31df729008c748e6eafe49ff86e1da618013c551 for the fix. Add

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22716 --- Commit 9f0c9ee20c1a65d427702a330798512e7297e322 in branch refs/heads

Re: Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/#review22717 --- Commit 31df729008c748e6eafe49ff86e1da618013c551 in branch refs/heads

Review Request 12243: CLOUDSTACK-3184: updateVirtualMachine api gives java NPE.

2013-07-03 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12243/ --- Review request for cloudstack, Devdeep Singh and Prasanna Santhanam. Bugs: CLOU