[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15156136#comment-15156136
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8865:
--------------------------------------------

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 hostId) {
         }
     
         @Override
    +    public List<HostVO> listAllUpHosts(Type type, Long clusterId, Long 
podId, long dcId) {
    --- End diff --
    
    Hi @SudharmaJain
    
    could you also write an unit test for this new method you created since it 
brings new functionalities to the code?


> Adding SR doesn't create Storage_pool_host_ref entry for disabled host
> ----------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8865
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8865
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>    Affects Versions: 4.5.0
>            Reporter: sudharma jain
>
> When we add Primary Storage into XS cluster which has a host in disabled 
> state the mapping info about each host and each storage pool on 
> storage_pool_host_ref is not created for the disabled host. However from XS 
> side SR is added in the pool elvel so SR can be seen from all hosts. James 
> wants mapping info populated in db.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to