mobile: downgrade MM launch failure warning severity to debug
MM won't always be present, and if it's not, your logs will fill up with warnings about MM not being able to be launched. And when running with systemd, you'll get a different class of errors like: <warn> error poking ModemManager: GDBus.Error:org.freedesktop.systemd1.LoadFailed: Unit dbus-org.freedesktop.ModemManager1.service failed to load: No such file or directory. See system logs and 'systemctl status dbus-org.freedesktop.ModemManager1.service' for details. and I'm tired of chasing and special-casing all the launch-failed errors that D-Bus and systemd use. Plus, we have dynamic log level changing via the D-Bus interface so if people need to debug this, just chaning the log level will tell you what's wrong.
This commit is contained in:
@@ -595,10 +595,10 @@ modem_manager_1_poke_cb (GDBusConnection *connection,
|
||||
&& !g_error_matches (error, G_DBUS_ERROR, G_DBUS_ERROR_SPAWN_FAILED)
|
||||
&& !g_error_matches (error, G_DBUS_ERROR, G_DBUS_ERROR_TIMEOUT)
|
||||
&& !g_error_matches (error, G_DBUS_ERROR, G_DBUS_ERROR_SPAWN_SERVICE_NOT_FOUND)) {
|
||||
nm_log_warn (LOGD_MB, "error poking ModemManager: %s", error->message);
|
||||
nm_log_dbg (LOGD_MB, "error poking ModemManager: %s", error->message);
|
||||
}
|
||||
|
||||
g_error_free (error);
|
||||
|
||||
/* Setup timeout to relaunch */
|
||||
schedule_modem_manager_1_relaunch (self, MODEM_POKE_INTERVAL);
|
||||
} else
|
||||
|
Reference in New Issue
Block a user