----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8813/#review16581 -----------------------------------------------------------
docs/en-US/egress-firewall-rule.xml <https://reviews.apache.org/r/8813/#comment35168> TCP, UDP, ICMP and ALL protocol. Please add the 'ALL' also. Protocol 'ALL' is allow all the traffic. - Jayapal Reddy On Feb. 6, 2013, 6:26 a.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8813/ > ----------------------------------------------------------- > > (Updated Feb. 6, 2013, 6:26 a.m.) > > > Review request for cloudstack, David Nalley, Chip Childers, and Jessica > Tomechak. > > > Description > ------- > > Documentation on egress firewall rule support > > > Diffs > ----- > > docs/en-US/add-ingress-egress-rules.xml 2490cec > docs/en-US/egress-firewall-rule.xml PRE-CREATION > docs/en-US/firewall-rules.xml 01d072b > docs/en-US/images/egress-firewall-rule.png PRE-CREATION > docs/en-US/ip-forwarding-firewalling.xml c154b07 > > Diff: https://reviews.apache.org/r/8813/diff/ > > > Testing > ------- > > Publican builds cleanly. Patch cleanly applies. > > > Thanks, > > Radhika PC > >