Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1219
---
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 borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/1219#issuecomment-164179455
LGTM :+1:
Deployed with Ceph.
![screenshot from 2015-12-12 20 44
13](https://cloud.githubusercontent.com/assets/5996146/11763591/2c88476a-a111-
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1219#issuecomment-164125989
LGTM, I could still do all my deployments using Marvin. So, didn't test
this feature but did prove the existing functionality still works.
---
If your project
GitHub user DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/1219
CLOUDSTACK-9139 make zwps default when defined on zone level
When a marvin definition contains a primary storage definition at zone
level it will default to zone scope. A test run is under