• Iustin Pop's avatar
    LURemoveNode safety in face of wrong node list · 9bb31ea8
    Iustin Pop authored
    
    
    LURemoveNode runs under the BGL, which means we're guaranteed that the
    list of nodes as retrieved in CheckPrereq is still valid in
    BuildHooksEnv. However, we can make Ganeti handle failures in case the
    locking is broken (or the node list has been modified otherwise) easily,
    which is better than crashing hard in such a case.
    
    This will also fix issue 79, even though that is due to an out-of-tree
    patch.
    Signed-off-by: default avatarIustin Pop <iustin@google.com>
    Reviewed-by: default avatarGuido Trotter <ultrotter@google.com>
    9bb31ea8
cmdlib.py 253 KB