incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kishan Kavala" <Kishan.Kav...@citrix.com>
Subject Re: Review Request: Removed the concept of owner region
Date Wed, 13 Mar 2013 17:37:07 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9905/
-----------------------------------------------------------

(Updated March 13, 2013, 5:37 p.m.)


Review request for cloudstack, David Nalley and Chip Childers.


Description
-------

CLOUDSTACK-1636: Removed the concept of owner region.
    Removed region_id from user/account and domain tables.
    Removed forwarding of api calls to owner region.
    Removed api_key and secret_key from region table.
    Included related DB upgrade changes.


This addresses bug CLOUDSTACK-1636.


Diffs
-----

  api/src/com/cloud/domain/Domain.java f8277c2 
  api/src/com/cloud/user/Account.java e65e017 
  api/src/com/cloud/user/AccountService.java f9c6ddb 
  api/src/com/cloud/user/DomainService.java 1f03019 
  api/src/com/cloud/user/User.java 7d80c43 
  api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 4121651 
  api/src/org/apache/cloudstack/api/command/admin/domain/CreateDomainCmd.java 5eae486 
  api/src/org/apache/cloudstack/api/command/admin/region/AddRegionCmd.java 2036670 
  api/src/org/apache/cloudstack/api/command/admin/region/UpdateRegionCmd.java 16693b6 
  api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 6ea8d9b 
  api/src/org/apache/cloudstack/region/Region.java 7f0aeea 
  api/src/org/apache/cloudstack/region/RegionService.java 8679ca9 
  api/test/org/apache/cloudstack/api/command/test/RegionCmdTest.java 01cd33b 
  core/src/com/cloud/user/AccountVO.java fd37c77 
  core/src/com/cloud/user/UserVO.java de7f31b 
  server/src/com/cloud/domain/DomainVO.java 346e19f 
  server/src/com/cloud/projects/ProjectManagerImpl.java 33feb5d 
  server/src/com/cloud/upgrade/dao/Upgrade40to41.java 9268764 
  server/src/com/cloud/user/AccountManager.java 4b3a601 
  server/src/com/cloud/user/AccountManagerImpl.java 069ed2b 
  server/src/com/cloud/user/DomainManager.java f268f6e 
  server/src/com/cloud/user/DomainManagerImpl.java 9f0ad53 
  server/src/org/apache/cloudstack/region/RegionManager.java 4cbd664 
  server/src/org/apache/cloudstack/region/RegionManagerImpl.java 6b7bc16 
  server/src/org/apache/cloudstack/region/RegionServiceImpl.java 0662c32 
  server/src/org/apache/cloudstack/region/RegionVO.java 0c36db2 
  server/src/org/apache/cloudstack/region/RegionsApiUtil.java 2ace4f9 
  server/test/com/cloud/user/MockAccountManagerImpl.java b637c2a 
  server/test/com/cloud/user/MockDomainManagerImpl.java bc92965 
  server/test/org/apache/cloudstack/region/RegionManagerTest.java 330f0b4 
  setup/db/db/schema-40to410.sql 822eae3 

Diff: https://reviews.apache.org/r/9905/diff/


Testing
-------

Tested account/user/domain api operations.
Tested deploydb and upgrade changed.


Thanks,

Kishan Kavala


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message