You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Abhinandan Prateek <ap...@apache.org> on 2013/05/01 08:21:21 UTC
Re: Review Request: CLOUDSTACK-741 global granular parameters and
CLOUDSTACK-2036 router template Id
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10862/#review19991
-----------------------------------------------------------
Ship it!
Ship It!
- Abhinandan Prateek
On April 30, 2013, 1:42 p.m., Harikrishna Patnala wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10862/
> -----------------------------------------------------------
>
> (Updated April 30, 2013, 1:42 p.m.)
>
>
> Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
>
>
> Description
> -------
>
> CLOUDSTACK-741: Granular Global Parameters Added parameters
> to cluster level
> cluster.storage.allocated.capacity.notificationthreshold
> cluster.storage.capacity.notificationthreshold
>
> CLOUDSTACK-2036
> global parameter for Router Template ID functionality added
> We use 5 parameters to set the router template name for each hypervisor
>
> Moved cpu.overprovisioning.factor and mem.overprovisioning.factor to be able to update/list using update/list configuration API at cluster level
>
>
> This addresses bugs CLOUDSTACK-2036 and CLOUDSTACK-741.
>
>
> Diffs
> -----
>
> server/src/com/cloud/alert/AlertManagerImpl.java 6839d44
> server/src/com/cloud/configuration/Config.java af6adcf
> server/src/com/cloud/configuration/ConfigurationManagerImpl.java 068a586
> server/src/com/cloud/dc/ClusterDetailsDaoImpl.java 4c85918
> server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 4c7bc75
> server/src/com/cloud/storage/dao/VMTemplateDao.java 3f1fe93
> server/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 3b37f24
> test/integration/smoke/test_global_settings.py ec18cea
>
> Diff: https://reviews.apache.org/r/10862/diff/
>
>
> Testing
> -------
>
> Tested locally
>
>
> Thanks,
>
> Harikrishna Patnala
>
>
Re: Review Request: CLOUDSTACK-741 global granular parameters and
CLOUDSTACK-2036 router template Id
Posted by Abhinandan Prateek <ap...@apache.org>.
> On May 1, 2013, 6:21 a.m., Abhinandan Prateek wrote:
> > Ship It!
Waiting for nitin's review.
- Abhinandan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10862/#review19991
-----------------------------------------------------------
On April 30, 2013, 1:42 p.m., Harikrishna Patnala wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10862/
> -----------------------------------------------------------
>
> (Updated April 30, 2013, 1:42 p.m.)
>
>
> Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
>
>
> Description
> -------
>
> CLOUDSTACK-741: Granular Global Parameters Added parameters
> to cluster level
> cluster.storage.allocated.capacity.notificationthreshold
> cluster.storage.capacity.notificationthreshold
>
> CLOUDSTACK-2036
> global parameter for Router Template ID functionality added
> We use 5 parameters to set the router template name for each hypervisor
>
> Moved cpu.overprovisioning.factor and mem.overprovisioning.factor to be able to update/list using update/list configuration API at cluster level
>
>
> This addresses bugs CLOUDSTACK-2036 and CLOUDSTACK-741.
>
>
> Diffs
> -----
>
> server/src/com/cloud/alert/AlertManagerImpl.java 6839d44
> server/src/com/cloud/configuration/Config.java af6adcf
> server/src/com/cloud/configuration/ConfigurationManagerImpl.java 068a586
> server/src/com/cloud/dc/ClusterDetailsDaoImpl.java 4c85918
> server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 4c7bc75
> server/src/com/cloud/storage/dao/VMTemplateDao.java 3f1fe93
> server/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 3b37f24
> test/integration/smoke/test_global_settings.py ec18cea
>
> Diff: https://reviews.apache.org/r/10862/diff/
>
>
> Testing
> -------
>
> Tested locally
>
>
> Thanks,
>
> Harikrishna Patnala
>
>