[GitHub] cloudstack pull request: CLOUDSTACK-8630: Fix for missing Firewall...

2015-07-14 Thread pavanb018
Github user pavanb018 closed the pull request at: https://github.com/apache/cloudstack/pull/581 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack pull request: CLOUDSTACK-8630: Fix for missing Firewall...

2015-07-14 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/581#issuecomment-121195598 Committed to ACS master: commit 03e8176baba43f0394d530e64e7f96d6c841e9b2 Author: pavan018 Date: Mon Jul 13 18:11:24 2015 +0530 Passing fi

[GitHub] cloudstack pull request: CLOUDSTACK-8630: Fix for missing Firewall...

2015-07-14 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/581#issuecomment-121171768 Thanks for the quick fix, @pavanb018 Now it LGTM :+1: Please proceed with the merge. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-8630: Fix for missing Firewall...

2015-07-14 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/581#issuecomment-121164744 Created an ACS ticket describing the changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo