Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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 remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151798750
LGTM, test runs fine:
```
=== TestName: test_deploy_vm | Status : SUCCESS ===
=== TestName: test_deploy_vm_multiple | Status : SUCCESS ===
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151755463
Ping @remibergsma @DaanHoogland
Any time to test this one today?
Cheers,
Wilder
---
If your project is set up for it, you can reply to
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151556191
@remibergsma Jenkins happy :)
https://builds.apache.org/job/cloudstack-pull-analysis/1008/
---
If your project is set up for it, you can reply to this em
GitHub user borisroman reopened a pull request:
https://github.com/apache/cloudstack/pull/979
[4.6] CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic"
setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup.
This PR adds that functionality
Github user borisroman closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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 user borisroman reopened a pull request:
https://github.com/apache/cloudstack/pull/979
[4.6] CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic"
setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup.
This PR adds that functionality
Github user borisroman closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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 user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151476883
@borisroman If you make Jenkins happy, I'll try to run this test as well ;-)
---
If your project is set up for it, you can reply to this email and have your
repl
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151467948
Ping @borisroman @remibergsma @karuturi @DaanHoogland
This PR LGTM :+1:
```
[root@cs3 integration]# nosetests --with-marvin
--marvin-c
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151460148
Hi @borisroman ,
I just finish the tests: All looking fine!
```
[root@cs3 integration]# nosetests --with-marvin
--marvin-config=/data/sh
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151210234
I have talked to @borisroman and he explained me that this PR depends on
the fixes on PR #860 from @karuturi . I will merge both and run the tests again.
12 matches
Mail list logo
