Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/#review20001 --- Commit 807ad0a39ab272a4af2de3156b0b367570acccea in branch refs/heads/master from Nitin Mehta nitin.me...@citrix.com [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=807ad0a ] CLOUDSTACK-741 - Making storage disable thresholds configurable at zone level. - ASF Subversion and Git Services On April 19, 2013, 4:07 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 19, 2013, 4:07 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/#review20026 --- Commit 807ad0a39ab272a4af2de3156b0b367570acccea in branch refs/heads/internallb from Nitin Mehta nitin.me...@citrix.com [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=807ad0a ] CLOUDSTACK-741 - Making storage disable thresholds configurable at zone level. - ASF Subversion and Git Services On April 19, 2013, 4:07 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 19, 2013, 4:07 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/#review19852 --- Commit deaf9106ca557a938edf25bee65cf6b4eb3ac03f in branch refs/heads/master from Harikrishna Patnala harikrishna.patn...@citrix.com [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=deaf910 ] CLOUDSTACK-741: Granular Global Parameters and adding fixes for CLOUDSTACK-2176, CLOUDSTACK-2198, CLOUDSTACK-2200 Adding the zone, cluster, account level parameters The parameters at scope (zone/cluster/pool/account) can be updated by updateConfiguration API with additional parameter zoneid/clusterid/accountid/storagepoolid Whenever these scoped parameters are used in CS they get value from the corresponding details table if not defined get value from global parameter. Same with the listConfiguration API with additional parameter zoneid/clusterid/accountid/storagepoolid - ASF Subversion and Git Services On April 19, 2013, 4:07 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 19, 2013, 4:07 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/#review19857 --- Commit deaf9106ca557a938edf25bee65cf6b4eb3ac03f in branch refs/heads/marvin_refactor from Harikrishna Patnala harikrishna.patn...@citrix.com [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=deaf910 ] CLOUDSTACK-741: Granular Global Parameters and adding fixes for CLOUDSTACK-2176, CLOUDSTACK-2198, CLOUDSTACK-2200 Adding the zone, cluster, account level parameters The parameters at scope (zone/cluster/pool/account) can be updated by updateConfiguration API with additional parameter zoneid/clusterid/accountid/storagepoolid Whenever these scoped parameters are used in CS they get value from the corresponding details table if not defined get value from global parameter. Same with the listConfiguration API with additional parameter zoneid/clusterid/accountid/storagepoolid - ASF Subversion and Git Services On April 19, 2013, 4:07 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 19, 2013, 4:07 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/#review19380 --- api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java https://reviews.apache.org/r/10548/#comment40094 Dont have to add the scope id ? server/src/com/cloud/server/ConfigurationServerImpl.java https://reviews.apache.org/r/10548/#comment40095 This is not going to be maintainable code. So many if's...Please put the switch statement now so that in future everyone uses it - Nitin Mehta On April 17, 2013, 11:09 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 17, 2013, 11:09 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 18, 2013, 8:16 p.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Changes --- updated patch with suggested changes Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs (updated) - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 19, 2013, 4:07 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs (updated) - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION engine/api/src/org/apache/cloudstack/storage/datastore/db/StoragePoolDetailsDao.java be71670 server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java c4da1ab server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/src/com/cloud/storage/dao/StoragePoolDetailsDaoImpl.java 0d797ed server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 17, 2013, 11:09 a.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Changes --- updated patch with suggested changes. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs (updated) - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/src/org/apache/cloudstack/api/response/ConfigurationResponse.java 95b8af2 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION server/src/com/cloud/configuration/Config.java 2993966 server/src/com/cloud/configuration/ConfigurationManager.java afaf0d6 server/src/com/cloud/configuration/ConfigurationManagerImpl.java 4fc2db7 server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b server/src/com/cloud/server/ManagementServerImpl.java 98f789a server/test/com/cloud/vpc/MockConfigurationManagerImpl.java 9443815 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION server/src/com/cloud/configuration/Config.java e69ea19 server/src/com/cloud/configuration/ConfigurationManager.java d7faf19 server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 server/src/com/cloud/configuration/ConfigurationParameterScope.java PRE-CREATION server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b server/src/com/cloud/server/ManagementServerImpl.java 5ed0c92 server/test/com/cloud/vpc/MockConfigurationManagerImpl.java a03e361 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala
Re: Review Request: CLOUDSTACK-741: Granular Global Parameters
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10548/ --- (Updated April 16, 2013, 2:26 p.m.) Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin Mehta. Description --- CLOUDSTACK-741: Granular Global Parameters Modified UpdateConfiguration API and ListConfiguration API by adding additional parameter scope and resource id (zone/cluster/pool/account). Scope decides the parameter which level we can update the value and id is the corresponding id based on the scope. This scope is defined in the config.java file where we are creating groups for each level(zone/cluster/pool/account). Currently submitted the patch for the zone level parameter use.external.dns. Included unit tests and marvin tests. This addresses bug CLOUDSTACK-741. Diffs - api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java aabfd4a api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java ffeb586 api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java PRE-CREATION api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java PRE-CREATION server/src/com/cloud/configuration/Config.java e69ea19 server/src/com/cloud/configuration/ConfigurationManager.java d7faf19 server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 server/src/com/cloud/configuration/ConfigurationParameterScope.java PRE-CREATION server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ab91059 server/src/com/cloud/server/ConfigurationServer.java f441b1f server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b server/src/com/cloud/server/ManagementServerImpl.java 5ed0c92 server/test/com/cloud/vpc/MockConfigurationManagerImpl.java a03e361 test/integration/smoke/test_UpdateCfg.py PRE-CREATION Diff: https://reviews.apache.org/r/10548/diff/ Testing --- Tested locally - tested the modified UpdateConfiguration API and ListConfiguration API by providing the zone scope and zone id - tested the modified parameter to take the updated value at zone level(after updation). Thanks, Harikrishna Patnala