cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jayapal Reddy" <jayapalreddy.ur...@citrix.com>
Subject Re: Review Request: Changes for Egress firewall rules feature support in SRX
Date Wed, 10 Apr 2013 12:47:08 GMT


> On April 8, 2013, 11:25 p.m., Sheng Yang wrote:
> > plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java,
line 830
> > <https://reviews.apache.org/r/10336/diff/1/?file=278647#file278647line830>
> >
> >     What's these trafficType and guestVlan for? Didn't see them in the scope.
> 
> Jayapal Reddy wrote:
>     1. The traffic type is for identifying the rule type whether it is Egress/Ingress.
>     2. Guest Vlan is used for crating unique egress firewall rule name.
> 
> Sheng Yang wrote:
>     I meant, I didn't see the reference of them anywhere in the code.

Those un used variables. I removed those.


- Jayapal


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


On April 10, 2013, 12:46 p.m., Jayapal Reddy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10336/
> -----------------------------------------------------------
> 
> (Updated April 10, 2013, 12:46 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Sheng Yang, and Murali Reddy.
> 
> 
> Description
> -------
> 
> Added egress firewall rules support for SRX device.
> Supported networks:
> 1. Advanced Isolated networks.
> 
> 
> This addresses bug CLOUDSTACK-779.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/agent/api/to/FirewallRuleTO.java 7f77936 
>   plugins/network-elements/juniper-srx/src/com/cloud/network/element/JuniperSRXExternalFirewallElement.java
af0912a 
>   plugins/network-elements/juniper-srx/src/com/cloud/network/resource/JuniperSrxResource.java
8482168 
>   scripts/network/juniper/application-add.xml 6603850 
>   scripts/network/juniper/security-policy-add.xml 632a17d 
>   server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java 1fc32d0 
>   server/src/com/cloud/upgrade/dao/Upgrade410to420.java f39038f 
> 
> Diff: https://reviews.apache.org/r/10336/diff/
> 
> 
> Testing
> -------
> 
> Unit Testing done.
> 
> 
> Thanks,
> 
> Jayapal Reddy
> 
>


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