----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8813/#review15262 -----------------------------------------------------------
Looks like the code for this feature is still under debate, so unable to commit the documentation until that is resolved. See CLOUDSTACK-637. Some questions about what the default policy should be for egress traffic. The decision might not match what is currently in this documentation. docs/en-US/firewall-rules.xml <https://reviews.apache.org/r/8813/#comment32936> There's an extra word "where" in this sentence. - Jessica Tomechak On Jan. 4, 2013, 8:43 a.m., Radhika PC wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8813/ > ----------------------------------------------------------- > > (Updated Jan. 4, 2013, 8:43 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 388bf7e > docs/en-US/images/egress-firewall-rule.png PRE-CREATION > docs/en-US/ip-forwarding-firewalling.xml 61aa6ad > > Diff: https://reviews.apache.org/r/8813/diff/ > > > Testing > ------- > > Publican builds cleanly. Patch cleanly applies. > > > Thanks, > > Radhika PC > >
