[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2016-05-31 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/899 ### ACS CI BVT Run **Sumarry:** Build Number 100 Hypervisor xenserver NetworkType Advanced Passed=72 Failed=1 Skipped=3 _Link to logs Folder (se

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/899#issuecomment-216195030 @yvsubhash please rebase against latest master and push -f, update on status of your PR; please squash changes into single commit tag:easypr --- If your proj

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/899#issuecomment-150838585 @yvsubhash Please squash the commits and rebase with current master, thanks! --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-06 Thread remibergsma
Github user remibergsma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/899#discussion_r41290357 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java --- @@ -544,6 +546,9 @@ public

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/899#issuecomment-145154881 @yvsubhash Please update the title of the PR to be more descriptive. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-01 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/899#discussion_r40903392 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java --- @@ -544,6 +546,9 @@ public Answer

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-10-01 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/899#issuecomment-144683655 Hi @yvsubhash Is there any integration test to cover this change? What would be the steps for us to test it? Cheers, Wilder --- If yo

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-8921

2015-09-29 Thread yvsubhash
GitHub user yvsubhash opened a pull request: https://github.com/apache/cloudstack/pull/899 BUG-ID:CLOUDSTACK-8921 Summary: CLOUDSTACK-8921 snapshot_store_ref table should store actual size of back snapshot in secondary storage Calling SR scan to make sure size is update