You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by ustcweizhou <gi...@git.apache.org> on 2015/09/10 08:04:21 UTC
[GitHub] cloudstack pull request: CLOUDSTACK-5863: Add unit tests for volum...
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/796
CLOUDSTACK-5863: Add unit tests for volume snapshot
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ustcweizhou/cloudstack unit-tests-for-snapshot
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/796.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #796
----
commit 007f5b842d7b0c8d200b562f48ad610b1324ca16
Author: Wei Zhou <w....@tech.leaseweb.com>
Date: 2015-09-09T12:48:31Z
CLOUDSTACK-5863: Add unit tests for create/delete/revert volume snapshot
commit e586d11a1b666e61cc43d75313f9b9d3c3672f8f
Author: Wei Zhou <w....@tech.leaseweb.com>
Date: 2015-09-09T12:49:07Z
CLOUDSTACK-5863: Add unit tests for take volume snapshot
commit c15729e71afb582d300c608f6b230af15a7d1958
Author: Wei Zhou <w....@tech.leaseweb.com>
Date: 2015-09-09T12:57:58Z
Fix coverity scan 1323172
----
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-5863: Add unit tests for volum...
Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/796#issuecomment-139134869
great, we need more;)
LGTM
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-5863: Add unit tests for volum...
Posted by karuturi <gi...@git.apache.org>.
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/796#issuecomment-139215606
:+1:
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-5863: Add unit tests for volum...
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/796#issuecomment-139489439
LGTM -> Test run fine and increase coverage.
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request: CLOUDSTACK-5863: Add unit tests for volum...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/796
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---