You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by pritisarap12 <gi...@git.apache.org> on 2015/06/11 10:37:06 UTC

[GitHub] cloudstack pull request: UDSTACK-8549:Update test_concurrent_snaps...

GitHub user pritisarap12 opened a pull request:

    https://github.com/apache/cloudstack/pull/386

    UDSTACK-8549:Update test_concurrent_snapshots_limits.py asesrt statement

    Update test_concurrent_snapshots_limits.py to check if the output(config value) is list or not only if it is not Null

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pritisarap12/cloudstack snapshot_cunc

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/386.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 #386
    
----
commit c7afddd210bbc070baea4f83613942b0ed4ac9a4
Author: pritisarap12 <pr...@clogeny.com>
Date:   2015-06-11T08:20:26Z

    UDSTACK-8549-Update test_concurrent_snapshots_limits.py asesrt statement

----


---
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: UDSTACK-8549:Update test_concurrent_snaps...

Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:

    https://github.com/apache/cloudstack/pull/386#issuecomment-112740085
  
    @pritisarap12 can you ammend the message to say "CLOUDSTACK-..." instead of "UDSTACK-..." :)


---
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-8549:Update test_concurrent_sn...

Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:

    https://github.com/apache/cloudstack/pull/386#issuecomment-121829533
  
    LGTM, really old so merging now


---
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-8549:Update test_concurrent_sn...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/386


---
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.
---