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



server/src/com/cloud/upgrade/dao/Upgrade410to420.java
<https://reviews.apache.org/r/10336/#comment39319>

    This would by default allow all egress traffic right? How can these rules 
applied to SRX after mgmt server upgrade?


- Sheng Yang


On April 9, 2013, 6:12 a.m., Jayapal Reddy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10336/
> -----------------------------------------------------------
> 
> (Updated April 9, 2013, 6:12 a.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
> 
>

Reply via email to