----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12853/#review23685 -----------------------------------------------------------
Review 12853 PASSED the build test The url of build cloudstack-master-with-patch #22 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/22/ - Jenkins Cloudstack.org On July 23, 2013, 11:06 a.m., Sanjay Tripathi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12853/ > ----------------------------------------------------------- > > (Updated July 23, 2013, 11:06 a.m.) > > > Review request for cloudstack, Devdeep Singh and Prasanna Santhanam. > > > Bugs: CLOUDSTACK-3688 > > > Repository: cloudstack-git > > > Description > ------- > > > CLOUDSTACK-3688: Test cases > test_accounts.TestDomainForceRemove.test_forceDeleteDomain, not found the > domain to delete and failed. > > This issue is in CS product and not in the test_script. > The problem is coming because in case of deleteDomain with cleanup = true, > CS is not allowing deletion of domain if there are any account under that > domain needs clean up; though these accounts are removed and admin can't > see them in the listaccounts. > So CS should not restrict the deleteDomain in case of accounts needs > cleanup. > > > Diffs > ----- > > server/src/com/cloud/user/DomainManagerImpl.java 1117ff0 > test/integration/component/test_accounts.py 3c284bd > > Diff: https://reviews.apache.org/r/12853/diff/ > > > Testing > ------- > > Verified marvin test on my local cloudstack setup. > > > Thanks, > > Sanjay Tripathi > >
