Github user gauravaradhye commented on the pull request:
https://github.com/apache/cloudstack/pull/102#issuecomment-78430462
No, this is not intended for 4.5. I have not tested on 4.5.
Regards,
Gaurav
On Tue, Mar 10, 2015 at 10:22 AM, Rohit Yadav
wrote:
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/102#issuecomment-77995716
@gauravaradhye your patch fails for 4.5, do you want to add test on 4.5
branch?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/102#issuecomment-77995623
LGTM, merging.
---
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 fe
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/102
---
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 is
GitHub user gauravaradhye opened a pull request:
https://github.com/apache/cloudstack/pull/102
CLOUDSTACK-8299: Adding additional test case to test ingress rules with two
security groups together
This is in addition to pull request at
https://github.com/apache/cloudstack/pull/95.