do not fail node read if secret does not exist

if we cannot find the credential id attached to the node,
we should not fail the node. we should just return what is
found and leave it to the user from there. failing the node
results in unreadable nodes, which helps no one.

Signed-off-by: Victoria Jeffrey <vjeffrey@chef.io>

components/nodemanager-service/api/nodes/server/server.go

@@ -208,7 +208,8 @@ func getSecretsAgg(ctx context.Context, secretIds []string, secretsClient secret
 		arrMap := make(map[string]string, 0)
 		s, err := secretsClient.Read(ctx, &secrets.Id{Id: id})
 		if err != nil {
-			return nil, errors.Wrap(err, "GetSecretsAgg error")
+			logrus.Errorf("credential %s not found", id)
+			continue
 		}
 
 		for _, kv := range s.Data {