[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-09-14 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1666#discussion_r78733924 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py --- @@ -145,40 +145,41 @@ def add_rule(self, cidr):

[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-09-14 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1666#discussion_r78733866 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py --- @@ -145,40 +145,41 @@ def add_rule(self, cidr):

[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-09-14 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1666#discussion_r78733780 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py --- @@ -145,40 +145,41 @@ def add_rule(self, cidr):

[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-09-01 Thread murali-reddy
Github user murali-reddy commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1666#discussion_r77119259 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py --- @@ -145,40 +145,41 @@ def add_rule(self, cidr):

[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-08-31 Thread jayapalu
Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1666#discussion_r77116561 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/configure.py --- @@ -145,40 +145,41 @@ def add_rule(self, cidr):

[GitHub] cloudstack pull request #1666: CLOUDSTACK-9480: Egress Firewall: Incorrect u...

2016-08-30 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1666 CLOUDSTACK-9480: Egress Firewall: Incorrect use of Allow/Deny for ICMP When 'default egress policy' is set to 'allow' in the network offering, any egress rule that is added will 'deny'