[GitHub] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-11 Thread gauravaradhye
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] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-09 Thread bhaisaab
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] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-09 Thread bhaisaab
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] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-09 Thread asfgit
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] cloudstack pull request: CLOUDSTACK-8299: Adding additional test c...

2015-03-09 Thread gauravaradhye
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.