> On Feb. 14, 2013, 2:59 p.m., Jayapal Reddy wrote: > > docs/en-US/egress-firewall-rule.xml, line 40 > > <https://reviews.apache.org/r/8813/diff/2/?file=256012#file256012line40> > > > > TCP, UDP, ICMP and ALL protocol. > > Please add the 'ALL' also. > > Protocol 'ALL' is allow all the traffic.
ALL ~ or all the three already mentioned. - Radhika ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8813/#review16581 ----------------------------------------------------------- On Feb. 14, 2013, 4:41 p.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8813/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2013, 4:41 p.m.) > > > Review request for cloudstack, David Nalley, Chip Childers, and Jessica > Tomechak. > > > Description > ------- > > Documentation on egress firewall rule support > > > Diffs > ----- > > docs/en-US/creating-network-offerings.xml df39242 > 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 > >
