You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Jessica Tomechak <je...@gmail.com> on 2013/03/22 04:27:17 UTC

[ACS41][Patch Request] CLOUDSTACK-1028 Fix documentation related to new egress rules feature

I've made a commit to master that should be added to 4.1, please:

b1e61cc94a6a4e3cf89a5fc345b54bfa36c6d4ec

The section "Firewall Rules" seems to have been deleted when the new
"Egress Firewall Rules" section was added for CLOUDSTACK-1028. The first
section is about ingress rules, so I'm pretty sure we still need it. These
are sub-sections of the larger section "IP Forwarding and Firewalling".

In addition, the intro paragraph of "IP Forwarding and Firewalling" wasn't
updated when "Egress Firewall Rules" was added, and it contained a
statement that contradicts the new egress default policy (blocked).

Thanks,
Jessica T.

Re: [ACS41][Patch Request] CLOUDSTACK-1028 Fix documentation related to new egress rules feature

Posted by Chip Childers <ch...@sungard.com>.
On Thu, Mar 21, 2013 at 08:31:24PM -0700, Jessica Tomechak wrote:
> OK Chip!
> 
> On Thu, Mar 21, 2013 at 8:28 PM, Chip Childers <ch...@sungard.com>wrote:
> 
> > Docs can be committed directly. Please do so.
> >
> > On Mar 21, 2013, at 11:27 PM, Jessica Tomechak
> > <je...@gmail.com> wrote:
> >
> > > I've made a commit to master that should be added to 4.1, please:
> > >
> > > b1e61cc94a6a4e3cf89a5fc345b54bfa36c6d4ec
> > >
> > > The section "Firewall Rules" seems to have been deleted when the new
> > > "Egress Firewall Rules" section was added for CLOUDSTACK-1028. The first
> > > section is about ingress rules, so I'm pretty sure we still need it.
> > These
> > > are sub-sections of the larger section "IP Forwarding and Firewalling".
> > >
> > > In addition, the intro paragraph of "IP Forwarding and Firewalling"
> > wasn't
> > > updated when "Egress Firewall Rules" was added, and it contained a
> > > statement that contradicts the new egress default policy (blocked).
> > >
> > > Thanks,
> > > Jessica T.
> >

I see you applied this.  Thanks!

Re: [ACS41][Patch Request] CLOUDSTACK-1028 Fix documentation related to new egress rules feature

Posted by Jessica Tomechak <je...@gmail.com>.
OK Chip!

On Thu, Mar 21, 2013 at 8:28 PM, Chip Childers <ch...@sungard.com>wrote:

> Docs can be committed directly. Please do so.
>
> On Mar 21, 2013, at 11:27 PM, Jessica Tomechak
> <je...@gmail.com> wrote:
>
> > I've made a commit to master that should be added to 4.1, please:
> >
> > b1e61cc94a6a4e3cf89a5fc345b54bfa36c6d4ec
> >
> > The section "Firewall Rules" seems to have been deleted when the new
> > "Egress Firewall Rules" section was added for CLOUDSTACK-1028. The first
> > section is about ingress rules, so I'm pretty sure we still need it.
> These
> > are sub-sections of the larger section "IP Forwarding and Firewalling".
> >
> > In addition, the intro paragraph of "IP Forwarding and Firewalling"
> wasn't
> > updated when "Egress Firewall Rules" was added, and it contained a
> > statement that contradicts the new egress default policy (blocked).
> >
> > Thanks,
> > Jessica T.
>

Re: [ACS41][Patch Request] CLOUDSTACK-1028 Fix documentation related to new egress rules feature

Posted by Chip Childers <ch...@sungard.com>.
Docs can be committed directly. Please do so.

On Mar 21, 2013, at 11:27 PM, Jessica Tomechak
<je...@gmail.com> wrote:

> I've made a commit to master that should be added to 4.1, please:
>
> b1e61cc94a6a4e3cf89a5fc345b54bfa36c6d4ec
>
> The section "Firewall Rules" seems to have been deleted when the new
> "Egress Firewall Rules" section was added for CLOUDSTACK-1028. The first
> section is about ingress rules, so I'm pretty sure we still need it. These
> are sub-sections of the larger section "IP Forwarding and Firewalling".
>
> In addition, the intro paragraph of "IP Forwarding and Firewalling" wasn't
> updated when "Egress Firewall Rules" was added, and it contained a
> statement that contradicts the new egress default policy (blocked).
>
> Thanks,
> Jessica T.