Browse Source

Merge pull request #1938 from gravitl/mkasun/gra-989-delete_node

when deleting node from UI/API add node to zombie list
dcarns 2 years ago
parent
commit
0d99b507d6
1 changed files with 2 additions and 1 deletions
  1. 2 1
      logic/nodes.go

+ 2 - 1
logic/nodes.go

@@ -82,7 +82,7 @@ func UpdateNode(currentNode *models.Node, newNode *models.Node) error {
 	return fmt.Errorf("failed to update node " + currentNode.ID.String() + ", cannot change ID.")
 	return fmt.Errorf("failed to update node " + currentNode.ID.String() + ", cannot change ID.")
 }
 }
 
 
-// DeleteNode - marks node for deletion if called by UI or deletes node if called by node
+// DeleteNode - marks node for deletion (and adds to zombie list) if called by UI or deletes node if called by node
 func DeleteNode(node *models.Node, purge bool) error {
 func DeleteNode(node *models.Node, purge bool) error {
 	node.Action = models.NODE_DELETE
 	node.Action = models.NODE_DELETE
 	if !purge {
 	if !purge {
@@ -91,6 +91,7 @@ func DeleteNode(node *models.Node, purge bool) error {
 		if err := UpdateNode(node, &newnode); err != nil {
 		if err := UpdateNode(node, &newnode); err != nil {
 			return err
 			return err
 		}
 		}
+		newZombie <- node.ID
 		return nil
 		return nil
 	}
 	}
 	host, err := GetHost(node.HostID.String())
 	host, err := GetHost(node.HostID.String())