----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12700/#review23353 -----------------------------------------------------------
Ship it! LGTM. You don't have to go through reviewboard once you are a committer. please commit this. - Prasanna Santhanam On July 17, 2013, 6:10 p.m., sailaja mada wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12700/ > ----------------------------------------------------------- > > (Updated July 17, 2013, 6:10 p.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-3596 > > > Repository: cloudstack-git > > > Description > ------- > > Deploy VM request is sent without providing the Domain id . Cloudstack > mandates to provide domain id along with account name. > > Fix is to include domain id to resolve script failure (Execute cmd: > deployvirtualmachine failed, due to: errorCode: 431, errorText:Account must > be specified with domainId parameter) > > > Diffs > ----- > > test/integration/component/test_accounts.py 65c0c6f > > Diff: https://reviews.apache.org/r/12700/diff/ > > > Testing > ------- > > Tested with the change. Now test is passed. > > > Thanks, > > sailaja mada > >
