Skip to content
  • Iustin Pop's avatar
    Fix issue in LUClusterVerifyGroup with multi-group clusters · 350506c6
    Iustin Pop authored
    
    
    In case LUClusterVerifyGroup is run on a group which doesn't contain
    the master node, the following could happen:
    
    - master node is selected due to the explicit check
    - if the order of nodes in the 'absent_nodes' list is such that the
      master node is the first in it, then we'll select (again) the master
      node
    - passing duplicate nodes to RPC calls will break due to RPC
      internals; this should be fixed separately, but in the meantime we
      just refrain from passing such duplicates
    
    This patch should not change the semantics of the code, since it
    wasn't guaranteed even before that we find a vm_capable node.
    
    Signed-off-by: default avatarIustin Pop <iustin@google.com>
    Reviewed-by: default avatarBernardo Dal Seno <bdalseno@google.com>
    350506c6