fabrics: only print success message with verbose level

The canonical behavior of shell tools is not to print anything on
success.

The problem with the connect command here it will always print the
device on success. This mixes with the --dump-config command which also
writes to stdout and makes the output unusable without additional
processing for a config file.

Signed-off-by: Daniel Wagner <dwagner@suse.de>

Author: igaw

fabrics.c

@@ -1084,7 +1084,8 @@ int nvmf_connect(const char *desc, int argc, char **argv)
 	};
 
 	c = lookup_ctrl(h, &trcfg);
-	if (c && nvme_ctrl_get_name(c) && !cfg.duplicate_connect) {
+	if (c && nvme_ctrl_get_name(c) && !cfg.duplicate_connect &&
+	    log_level >= LOG_INFO) {
 		fprintf(stderr, "already connected\n");
 		errno = EALREADY;
 		goto out_free;
@@ -1109,7 +1110,7 @@ int nvmf_connect(const char *desc, int argc, char **argv)
 			nvme_strerror(errno));
 	else {
 		errno = 0;
-		if (flags != -EINVAL)
+		if (flags != -EINVAL && log_level >= LOG_INFO)
 			nvme_show_connect_msg(c, flags);
 	}