----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/ -----------------------------------------------------------
(Updated June 15, 2014, 9:40 p.m.) Review request for cloudstack. Changes ------- Please review Repository: cloudstack-git Description ------- This is the review request for the core changes related with #17790 that has only the new plugin codes. Diffs (updated) ----- api/src/com/cloud/event/EventTypes.java 39ef710 api/src/com/cloud/user/AccountService.java 7e37b38 api/src/com/cloud/user/DomainService.java 4c1f93d api/src/org/apache/cloudstack/api/ApiConstants.java fdb4558 api/src/org/apache/cloudstack/api/BaseCmd.java f6f21ae api/src/org/apache/cloudstack/api/command/admin/account/CreateAccountCmd.java 50d67d9 api/src/org/apache/cloudstack/api/command/admin/account/DeleteAccountCmd.java 5754ec5 api/src/org/apache/cloudstack/api/command/admin/account/DisableAccountCmd.java 3e5e1d3 api/src/org/apache/cloudstack/api/command/admin/account/EnableAccountCmd.java f30c985 api/src/org/apache/cloudstack/api/command/admin/account/LockAccountCmd.java 3c185e4 api/src/org/apache/cloudstack/api/command/admin/account/UpdateAccountCmd.java a7ce74a api/src/org/apache/cloudstack/api/command/admin/domain/CreateDomainCmd.java 312c9ee api/src/org/apache/cloudstack/api/command/admin/domain/DeleteDomainCmd.java a6d2b0b api/src/org/apache/cloudstack/api/command/admin/domain/UpdateDomainCmd.java 409a84d api/src/org/apache/cloudstack/api/command/admin/region/AddRegionCmd.java f6743ba api/src/org/apache/cloudstack/api/command/admin/region/UpdateRegionCmd.java b08cbbb api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java 51e218d api/src/org/apache/cloudstack/api/command/admin/user/DeleteUserCmd.java 08ba521 api/src/org/apache/cloudstack/api/command/admin/user/DisableUserCmd.java c6e09ef api/src/org/apache/cloudstack/api/command/admin/user/EnableUserCmd.java d69eccf api/src/org/apache/cloudstack/api/command/admin/user/LockUserCmd.java 69623d0 api/src/org/apache/cloudstack/api/command/admin/user/RegisterCmd.java 2090d21 api/src/org/apache/cloudstack/api/command/admin/user/UpdateUserCmd.java cf5d355 api/src/org/apache/cloudstack/api/response/RegionResponse.java 6c74fa6 api/src/org/apache/cloudstack/region/Region.java df64e44 api/src/org/apache/cloudstack/region/RegionService.java afefcc7 api/test/org/apache/cloudstack/api/command/test/RegionCmdTest.java 10c3d85 client/pom.xml d8dbde7 client/tomcatconf/commands.properties.in 45debe4 engine/schema/resources/META-INF/cloudstack/core/spring-engine-schema-core-daos-context.xml 489b37d engine/schema/src/com/cloud/user/AccountVO.java 0f5a044 engine/schema/src/org/apache/cloudstack/region/RegionVO.java 608bd2b plugins/network-elements/juniper-contrail/test/org/apache/cloudstack/network/contrail/management/MockAccountManager.java 957f708 plugins/pom.xml 9b391b8 plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapCreateAccountCmd.java 626bb8f plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapImportUsersCmd.java 887ad00 server/resources/META-INF/cloudstack/core/spring-server-core-managers-context.xml fc1c7e2 server/src/com/cloud/api/ApiResponseHelper.java 38f2f0b server/src/com/cloud/api/dispatch/ParamProcessWorker.java e9bdd8b server/src/com/cloud/event/ActionEventUtils.java 28e5680 server/src/com/cloud/projects/ProjectManagerImpl.java d10c059 server/src/com/cloud/user/AccountManager.java 03bf842 server/src/com/cloud/user/AccountManagerImpl.java 2070ee6 server/src/com/cloud/user/DomainManager.java f72b18a server/src/com/cloud/user/DomainManagerImpl.java fbbe0c2 server/src/org/apache/cloudstack/region/RegionManager.java 6f25481 server/src/org/apache/cloudstack/region/RegionManagerImpl.java 8910714 server/src/org/apache/cloudstack/region/RegionServiceImpl.java 98cf500 server/test/com/cloud/user/MockAccountManagerImpl.java f373cba server/test/com/cloud/user/MockDomainManagerImpl.java 7dddefb server/test/org/apache/cloudstack/region/RegionManagerTest.java d7bc537 setup/db/db/schema-440to450.sql 2bd5386 ui/scripts/regions.js 66dae8c Diff: https://reviews.apache.org/r/20099/diff/ Testing ------- 1. Successfully tested real time synchronization as soon as resources are created/deleted/modified in one region. 2. Successfully tested full scans to synchronize resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough
