Use GetByPrefix to find nodes when approving/rejecting them (#3681)

Rather than requiring an exact node id, we can accept a shortened
version of it and use getbyprefix. This will complain appropriately if
there is more than one node returned by the prefix.

pkg/node/manager/node_manager.go

@@ -190,9 +190,9 @@ func (n *NodeManager) Delete(ctx context.Context, nodeID string) error {
 // reason for the approval (for audit). The return values denote success and any
 // failure of the operation as a human readable string.
 func (n *NodeManager) Approve(ctx context.Context, nodeID string, reason string) (bool, string) {
-	info, err := n.nodeInfo.Get(ctx, nodeID)
+	info, err := n.nodeInfo.GetByPrefix(ctx, nodeID)
 	if err != nil {
-		return false, "node not found"
+		return false, err.Error()
 	}
 
 	if info.Approval == models.NodeApprovals.APPROVED {
@@ -213,9 +213,9 @@ func (n *NodeManager) Approve(ctx context.Context, nodeID string, reason string)
 // reason for the rejection (for audit). The return values denote success and any
 // failure of the operation as a human readable string.
 func (n *NodeManager) Reject(ctx context.Context, nodeID string, reason string) (bool, string) {
-	info, err := n.nodeInfo.Get(ctx, nodeID)
+	info, err := n.nodeInfo.GetByPrefix(ctx, nodeID)
 	if err != nil {
-		return false, "node not found"
+		return false, err.Error()
 	}
 
 	if info.Approval == models.NodeApprovals.REJECTED {