[GitHub] cloudstack pull request: Vpcr Marvin test and some fixes for vprc

2015-07-02 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/558 Vpcr Marvin test and some fixes for vprc A combined commit because otherwise it would not make sense. If I commit the fixes alone there is not test to verify them If I commit the

[GitHub] cloudstack pull request: Allow test to run on tiny linux templates...

2015-06-10 Thread isoutham
Github user isoutham commented on the pull request: https://github.com/apache/cloudstack/pull/381#issuecomment-110892103 Hi, Yes merge it please. Test 03 does not fail to me. Might be an idea to increase the retry count for people running slow ivm systems. I will look

[GitHub] cloudstack pull request: Allow test to run on tiny linux templates...

2015-06-10 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/381 Allow test to run on tiny linux templates using busybox When running in test environments containing netted-nested hypervisors, the centos template is a little "heavy". This tes

[GitHub] cloudstack pull request: Pep8 changes that got lost with the stran...

2015-05-26 Thread isoutham
Github user isoutham commented on the pull request: https://github.com/apache/cloudstack/pull/304#issuecomment-105500276 Jenkins fails on --- 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

[GitHub] cloudstack pull request: Pep8 changes that got lost with the stran...

2015-05-26 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/304 Pep8 changes that got lost with the strange double push request probl… pep8 changes and a couple of minor fixes to the test/systemvm/python unit tests. The unit tests are just a

[GitHub] cloudstack pull request: Pep8 and some unit tests

2015-05-19 Thread isoutham
Github user isoutham commented on the pull request: https://github.com/apache/cloudstack/pull/265#issuecomment-103648791 It looks like I have somehow managed to put the second PR in the first (was a seperate branch) not quite sure what I did here. Ignore the first and

[GitHub] cloudstack pull request: Pep8 and some unit tests

2015-05-19 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/265 Pep8 and some unit tests Fixes up a bunch of pep8 failure (trying to get rid of some jenkins failures) Also the first steps to a unit testing framework for the python code in the systemvm

[GitHub] cloudstack pull request: Allow forward to fix port forwarding rule...

2015-05-19 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/262 Allow forward to fix port forwarding rules Fixes a missing iptables rule that would prevent port forwarding from working. Tested against (with hardware) smoke tests. You can merge