[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create Storage_p...

2016-06-01 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/876 ### ACS CI BVT Run **Sumarry:** Build Number 104 Hypervisor xenserver NetworkType Advanced Passed=70 Failed=0 Skipped=3 _Link to logs Folder (se

[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create...

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/876#issuecomment-216194398 @SudharmaJain sorry but can you again please rebase against latest master and push -f, update on status of your PR I would suggest you can add a marvin test th

[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create...

2016-04-21 Thread SudharmaJain
Github user SudharmaJain commented on the pull request: https://github.com/apache/cloudstack/pull/876#issuecomment-212794906 @alexandrelimassantana This method is about a query execution on database. Could you suggest how can i add a unit test for it? --- If your project is set up f

[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create...

2016-04-20 Thread SudharmaJain
Github user SudharmaJain commented on the pull request: https://github.com/apache/cloudstack/pull/876#issuecomment-212725838 Rebased with master. --- 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 ha

[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create...

2016-02-21 Thread alexandrelimassantana
Github user alexandrelimassantana commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/876#discussion_r53570532 --- Diff: server/src/com/cloud/resource/ResourceManagerImpl.java --- @@ -2378,6 +2378,23 @@ public boolean maintenanceFailed(final long hostI

[GitHub] cloudstack pull request: CLOUDSTACK-8865: Adding SR doesn't create...

2015-09-22 Thread SudharmaJain
GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/876 CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry… … for disabled host This causes VM deployment failure on the host that was disabled while adding the stor