cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alena Prokharchyk" <alena.prokharc...@citrix.com>
Subject Re: Review Request 19578: CLOUDSTACK-6253: Optimizing VR alerts getting algorithm
Date Tue, 25 Mar 2014 16:53:28 GMT


> On March 24, 2014, 10:03 p.m., Alena Prokharchyk wrote:
> > Also if you are hardcoding the state to search by, to Running in your method, there
is no need to define it in the method signature:
> > 
> >  public List<DomainRouterVO> listByRunningStateAndManagementServer(State state,
long mgmtSrvrId) { - remove it from the signature
> > 308	
> >         SearchCriteria<DomainRouterVO> sc = RunningSearchByManagementServerId.create();
> > 309	
> >         sc.setParameters("state", State.Running); - here is the hard coding
> 
> Harikrishna Patnala wrote:
>     Thank you Alena, I'll make this a generic method to listbyStateAndManagementServer
so that it can be used for any state.

If you make it a generic method, please change the name of it from "listByRunningStateAndManagementServer"
to "listByStateAndManagementServer" :)


- Alena


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


On March 25, 2014, 6:24 a.m., Harikrishna Patnala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19578/
> -----------------------------------------------------------
> 
> (Updated March 25, 2014, 6:24 a.m.)
> 
> 
> Review request for cloudstack, Jayapal Reddy, Murali Reddy, and Sheng Yang.
> 
> 
> Bugs: CLOUDSTACK-6253
>     https://issues.apache.org/jira/browse/CLOUDSTACK-6253
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> CLOUDSTACK-6253: Optimizing VR alerts getting algorithm 
> In addition to this a new configuration parameter is added router.alerts.check.interval
defaulted to 30minutes to check for alerts in Virtual Router
> 
> 
> Diffs
> -----
> 
>   core/src/com/cloud/agent/api/GetRouterAlertsAnswer.java 06a7a7a 
>   core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java e13fbf6

>   engine/schema/src/com/cloud/vm/dao/DomainRouterDao.java e3f75fa 
>   engine/schema/src/com/cloud/vm/dao/DomainRouterDaoImpl.java 6b62f56 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManager.java e3597ac 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java c692491

>   systemvm/patches/debian/config/opt/cloud/bin/getRouterAlerts.sh e5e8abe 
>   test/integration/component/test_VirtualRouter_alerts.py PRE-CREATION 
>   test/integration/smoke/test_VirtualRouter_alerts.py 2333d84 
> 
> Diff: https://reviews.apache.org/r/19578/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>


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