ifreload: more diagnostic messages (bnc#882311)

client/ifreload.c

@@ -225,12 +225,18 @@ ni_do_ifreload(int argc, char **argv)
 
 			/* skip unused devices without config */
 			if (!ni_ifcheck_worker_config_exists(w) &&
-			    !ni_ifcheck_device_configured(dev))
+			    !ni_ifcheck_device_configured(dev)) {
+				ni_info("skipping %s interface: no configuration exists and"
+					"device is not configured by wicked", w->name);
 				continue;
+			}
 
-			/* skip if config changed somehow */
-			if (ni_ifcheck_worker_config_matches(w))
+			/* skip if config has not been changed */
+			if (ni_ifcheck_worker_config_matches(w)) {
+				ni_info("skipping %s interface: "
+					"configuration unchanged", w->name);
 				continue;
+			}
 
 			/* Mark persistend when requested */
 			if (opt_persistent)
@@ -240,17 +246,26 @@ ni_do_ifreload(int argc, char **argv)
 			ni_ifworker_array_append(&marked, w);
 
 			/* Do not ifdown non-existing device */
-			if (!dev)
+			if (!dev) {
+				ni_info("skipping ifdown operation for %s interface: "
+					"non-existing device", w->name);
 				continue;
+			}
 
 			/* Persistent do not go down but up only */
-			if (ni_ifcheck_device_is_persistent(dev))
+			if (ni_ifcheck_device_is_persistent(dev)) {
+				ni_info("skipping ifdown operation for %s interface: "
+					"persistent device", w->name);
 				continue;
+			}
 
 			/* Decide how much down we go */
 			if (ni_ifcheck_worker_config_exists(w)) {
-				if (!ni_ifcheck_device_configured(dev))
+				if (!ni_ifcheck_device_configured(dev)) {
+					ni_info("skipping ifdown operation for %s interface: "
+						"device is not configured by wicked", w->name);
 					continue;
+				}
 				w->target_range.min = NI_FSM_STATE_NONE;
 				w->target_range.max = NI_FSM_STATE_DEVICE_READY;
 				nmarked++;

src/fsm.c

@@ -1985,7 +1985,7 @@ ni_fsm_get_matching_workers(ni_fsm_t *fsm, ni_ifmatcher_t *match, ni_ifworker_ar
 		/* skipping ifworkers of interfaces not configured in the past */
 		if (ni_string_empty(w->config.origin) && match->require_configured) {
 			ni_info("skipping %s interface: "
-				"not configured yet", w->name);
+				"device is not configured by wicked yet", w->name);
 			continue;
 		}
 		/* skipping ifworkers of interfaces in the persistent mode */