Compare commits

..

4 Commits

Author SHA1 Message Date
Zbigniew Jędrzejewski-Szmek 7fd7dab90f
Merge pull request #13554 from keur/systemctl_status_timer
systemctl: Add timer activation to status
2019-11-08 14:19:40 +01:00
Zbigniew Jędrzejewski-Szmek bc9aa96bbd systemctl: simplify printing of Triggers/TriggeredBy 2019-11-05 22:01:46 +01:00
Kevin Kuehler 0d588deae2 systemctl: Align all status outputs to TriggeredBy 2019-11-05 21:51:31 +01:00
Kevin Kuehler 965c5d1d5a systemctl: Add TriggeredBy and Triggers to status
For all units that aren't timers, if it is activated by another unit,
add the triggering unit under the  "TriggeredBy:" header. If a unit can
trigger other units, print the units it triggers other the "Triggers:"
header.
2019-11-05 21:51:31 +01:00
2 changed files with 77 additions and 43 deletions

1
TODO
View File

@ -1113,7 +1113,6 @@ External:
- <command> <verb> -<TAB> should complete options, but currently does not - <command> <verb> -<TAB> should complete options, but currently does not
- systemctl add-wants,add-requires - systemctl add-wants,add-requires
* systemctl status foo.service should say that it is trigger by foo.timer
* systemctl status should know about 'systemd-analyze calendar ... --iterations=' * systemctl status should know about 'systemd-analyze calendar ... --iterations='
* If timer has just OnInactiveSec=..., it should fire after a specified time * If timer has just OnInactiveSec=..., it should fire after a specified time
after being started. after being started.

View File

@ -4050,6 +4050,9 @@ typedef struct UnitStatusInfo {
char **dropin_paths; char **dropin_paths;
char **triggered_by;
char **triggers;
const char *load_error; const char *load_error;
const char *result; const char *result;
@ -4134,6 +4137,8 @@ static void unit_status_info_free(UnitStatusInfo *info) {
strv_free(info->documentation); strv_free(info->documentation);
strv_free(info->dropin_paths); strv_free(info->dropin_paths);
strv_free(info->triggered_by);
strv_free(info->triggers);
strv_free(info->listen); strv_free(info->listen);
while ((c = info->conditions)) { while ((c = info->conditions)) {
@ -4147,6 +4152,17 @@ static void unit_status_info_free(UnitStatusInfo *info) {
} }
} }
static void format_active_state(const char *active_state, const char **active_on, const char **active_off) {
if (streq_ptr(active_state, "failed")) {
*active_on = ansi_highlight_red();
*active_off = ansi_normal();
} else if (STRPTR_IN_SET(active_state, "active", "reloading")) {
*active_on = ansi_highlight_green();
*active_off = ansi_normal();
} else
*active_on = *active_off = "";
}
static void print_status_info( static void print_status_info(
sd_bus *bus, sd_bus *bus,
UnitStatusInfo *i, UnitStatusInfo *i,
@ -4166,14 +4182,7 @@ static void print_status_info(
/* This shows pretty information about a unit. See /* This shows pretty information about a unit. See
* print_property() for a low-level property printer */ * print_property() for a low-level property printer */
if (streq_ptr(i->active_state, "failed")) { format_active_state(i->active_state, &active_on, &active_off);
active_on = ansi_highlight_red();
active_off = ansi_normal();
} else if (STRPTR_IN_SET(i->active_state, "active", "reloading")) {
active_on = ansi_highlight_green();
active_off = ansi_normal();
} else
active_on = active_off = "";
printf("%s%s%s %s", active_on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), active_off, strna(i->id)); printf("%s%s%s %s", active_on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), active_off, strna(i->id));
@ -4196,24 +4205,24 @@ static void print_status_info(
path = formatted_path; path = formatted_path;
if (!isempty(i->load_error)) if (!isempty(i->load_error))
printf(" Loaded: %s%s%s (Reason: %s)\n", printf(" Loaded: %s%s%s (Reason: %s)\n",
on, strna(i->load_state), off, i->load_error); on, strna(i->load_state), off, i->load_error);
else if (path && !isempty(i->unit_file_state) && !isempty(i->unit_file_preset) && else if (path && !isempty(i->unit_file_state) && !isempty(i->unit_file_preset) &&
!STR_IN_SET(i->unit_file_state, "generated", "transient")) !STR_IN_SET(i->unit_file_state, "generated", "transient"))
printf(" Loaded: %s%s%s (%s; %s; vendor preset: %s)\n", printf(" Loaded: %s%s%s (%s; %s; vendor preset: %s)\n",
on, strna(i->load_state), off, path, i->unit_file_state, i->unit_file_preset); on, strna(i->load_state), off, path, i->unit_file_state, i->unit_file_preset);
else if (path && !isempty(i->unit_file_state)) else if (path && !isempty(i->unit_file_state))
printf(" Loaded: %s%s%s (%s; %s)\n", printf(" Loaded: %s%s%s (%s; %s)\n",
on, strna(i->load_state), off, path, i->unit_file_state); on, strna(i->load_state), off, path, i->unit_file_state);
else if (path) else if (path)
printf(" Loaded: %s%s%s (%s)\n", printf(" Loaded: %s%s%s (%s)\n",
on, strna(i->load_state), off, path); on, strna(i->load_state), off, path);
else else
printf(" Loaded: %s%s%s\n", printf(" Loaded: %s%s%s\n",
on, strna(i->load_state), off); on, strna(i->load_state), off);
if (i->transient) if (i->transient)
printf("Transient: yes\n"); printf(" Transient: yes\n");
if (!strv_isempty(i->dropin_paths)) { if (!strv_isempty(i->dropin_paths)) {
_cleanup_free_ char *dir = NULL; _cleanup_free_ char *dir = NULL;
@ -4225,8 +4234,8 @@ static void print_status_info(
const char *df; const char *df;
if (!dir || last) { if (!dir || last) {
printf(dir ? " " : printf(dir ? " " :
" Drop-In: "); " Drop-In: ");
dir = mfree(dir); dir = mfree(dir);
@ -4237,7 +4246,7 @@ static void print_status_info(
} }
printf("%s\n" printf("%s\n"
" %s", dir, " %s", dir,
special_glyph(SPECIAL_GLYPH_TREE_RIGHT)); special_glyph(SPECIAL_GLYPH_TREE_RIGHT));
} }
@ -4254,10 +4263,10 @@ static void print_status_info(
ss = streq_ptr(i->active_state, i->sub_state) ? NULL : i->sub_state; ss = streq_ptr(i->active_state, i->sub_state) ? NULL : i->sub_state;
if (ss) if (ss)
printf(" Active: %s%s (%s)%s", printf(" Active: %s%s (%s)%s",
active_on, strna(i->active_state), ss, active_off); active_on, strna(i->active_state), ss, active_off);
else else
printf(" Active: %s%s%s", printf(" Active: %s%s%s",
active_on, strna(i->active_state), active_off); active_on, strna(i->active_state), active_off);
if (!isempty(i->result) && !streq(i->result, "success")) if (!isempty(i->result) && !streq(i->result, "success"))
@ -4278,6 +4287,18 @@ static void print_status_info(
else else
printf("\n"); printf("\n");
STRV_FOREACH(t, i->triggered_by) {
UnitActiveState state = _UNIT_ACTIVE_STATE_INVALID;
(void) get_state_one_unit(bus, *t, &state);
format_active_state(unit_active_state_to_string(state), &on, &off);
printf("%s %s%s%s %s\n",
t == i->triggered_by ? "TriggeredBy:" : " ",
on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), off,
*t);
}
if (endswith(i->id, ".timer")) { if (endswith(i->id, ".timer")) {
char tstamp1[FORMAT_TIMESTAMP_RELATIVE_MAX], char tstamp1[FORMAT_TIMESTAMP_RELATIVE_MAX],
tstamp2[FORMAT_TIMESTAMP_MAX]; tstamp2[FORMAT_TIMESTAMP_MAX];
@ -4286,7 +4307,7 @@ static void print_status_info(
i->next_elapse_monotonic}; i->next_elapse_monotonic};
usec_t next_elapse; usec_t next_elapse;
printf(" Trigger: "); printf(" Trigger: ");
dual_timestamp_get(&nw); dual_timestamp_get(&nw);
next_elapse = calc_next_elapse(&nw, &next); next_elapse = calc_next_elapse(&nw, &next);
@ -4299,6 +4320,18 @@ static void print_status_info(
printf("n/a\n"); printf("n/a\n");
} }
STRV_FOREACH(t, i->triggers) {
UnitActiveState state = _UNIT_ACTIVE_STATE_INVALID;
(void) get_state_one_unit(bus, *t, &state);
format_active_state(unit_active_state_to_string(state), &on, &off);
printf("%s %s%s%s %s\n",
t == i->triggers ? " Triggers:" : " ",
on, special_glyph(SPECIAL_GLYPH_BLACK_CIRCLE), off,
*t);
}
if (!i->condition_result && i->condition_timestamp > 0) { if (!i->condition_result && i->condition_timestamp > 0) {
UnitCondition *c; UnitCondition *c;
int n = 0; int n = 0;
@ -4306,7 +4339,7 @@ static void print_status_info(
s1 = format_timestamp_relative(since1, sizeof(since1), i->condition_timestamp); s1 = format_timestamp_relative(since1, sizeof(since1), i->condition_timestamp);
s2 = format_timestamp(since2, sizeof(since2), i->condition_timestamp); s2 = format_timestamp(since2, sizeof(since2), i->condition_timestamp);
printf("Condition: start %scondition failed%s at %s%s%s\n", printf(" Condition: start %scondition failed%s at %s%s%s\n",
ansi_highlight_yellow(), ansi_normal(), ansi_highlight_yellow(), ansi_normal(),
s2, s1 ? "; " : "", strempty(s1)); s2, s1 ? "; " : "", strempty(s1));
@ -4316,7 +4349,7 @@ static void print_status_info(
LIST_FOREACH(conditions, c, i->conditions) LIST_FOREACH(conditions, c, i->conditions)
if (c->tristate < 0) if (c->tristate < 0)
printf(" %s %s=%s%s%s was not met\n", printf(" %s %s=%s%s%s was not met\n",
--n ? special_glyph(SPECIAL_GLYPH_TREE_BRANCH) : special_glyph(SPECIAL_GLYPH_TREE_RIGHT), --n ? special_glyph(SPECIAL_GLYPH_TREE_BRANCH) : special_glyph(SPECIAL_GLYPH_TREE_RIGHT),
c->name, c->name,
c->trigger ? "|" : "", c->trigger ? "|" : "",
@ -4328,24 +4361,24 @@ static void print_status_info(
s1 = format_timestamp_relative(since1, sizeof(since1), i->assert_timestamp); s1 = format_timestamp_relative(since1, sizeof(since1), i->assert_timestamp);
s2 = format_timestamp(since2, sizeof(since2), i->assert_timestamp); s2 = format_timestamp(since2, sizeof(since2), i->assert_timestamp);
printf(" Assert: start %sassertion failed%s at %s%s%s\n", printf(" Assert: start %sassertion failed%s at %s%s%s\n",
ansi_highlight_red(), ansi_normal(), ansi_highlight_red(), ansi_normal(),
s2, s1 ? "; " : "", strempty(s1)); s2, s1 ? "; " : "", strempty(s1));
if (i->failed_assert_trigger) if (i->failed_assert_trigger)
printf(" none of the trigger assertions were met\n"); printf(" none of the trigger assertions were met\n");
else if (i->failed_assert) else if (i->failed_assert)
printf(" %s=%s%s was not met\n", printf(" %s=%s%s was not met\n",
i->failed_assert, i->failed_assert,
i->failed_assert_negate ? "!" : "", i->failed_assert_negate ? "!" : "",
i->failed_assert_parameter); i->failed_assert_parameter);
} }
if (i->sysfs_path) if (i->sysfs_path)
printf(" Device: %s\n", i->sysfs_path); printf(" Device: %s\n", i->sysfs_path);
if (i->where) if (i->where)
printf(" Where: %s\n", i->where); printf(" Where: %s\n", i->where);
if (i->what) if (i->what)
printf(" What: %s\n", i->what); printf(" What: %s\n", i->what);
STRV_FOREACH(t, i->documentation) { STRV_FOREACH(t, i->documentation) {
_cleanup_free_ char *formatted = NULL; _cleanup_free_ char *formatted = NULL;
@ -4356,16 +4389,16 @@ static void print_status_info(
else else
q = *t; q = *t;
printf(" %*s %s\n", 9, t == i->documentation ? "Docs:" : "", q); printf(" %*s %s\n", 9, t == i->documentation ? "Docs:" : "", q);
} }
STRV_FOREACH_PAIR(t, t2, i->listen) STRV_FOREACH_PAIR(t, t2, i->listen)
printf(" %*s %s (%s)\n", 9, t == i->listen ? "Listen:" : "", *t2, *t); printf(" %*s %s (%s)\n", 9, t == i->listen ? "Listen:" : "", *t2, *t);
if (i->accept) { if (i->accept) {
printf(" Accepted: %u; Connected: %u;", i->n_accepted, i->n_connections); printf(" Accepted: %u; Connected: %u;", i->n_accepted, i->n_connections);
if (i->n_refused) if (i->n_refused)
printf(" Refused: %u", i->n_refused); printf(" Refused: %u", i->n_refused);
printf("\n"); printf("\n");
} }
@ -4383,7 +4416,7 @@ static void print_status_info(
continue; continue;
argv = strv_join(p->argv, " "); argv = strv_join(p->argv, " ");
printf(" Process: "PID_FMT" %s=%s ", p->pid, p->name, strna(argv)); printf(" Process: "PID_FMT" %s=%s ", p->pid, p->name, strna(argv));
good = is_clean_exit(p->code, p->status, EXIT_CLEAN_DAEMON, NULL); good = is_clean_exit(p->code, p->status, EXIT_CLEAN_DAEMON, NULL);
if (!good) { if (!good) {
@ -4420,7 +4453,7 @@ static void print_status_info(
if (i->main_pid > 0 || i->control_pid > 0) { if (i->main_pid > 0 || i->control_pid > 0) {
if (i->main_pid > 0) { if (i->main_pid > 0) {
printf(" Main PID: "PID_FMT, i->main_pid); printf(" Main PID: "PID_FMT, i->main_pid);
if (i->running) { if (i->running) {
@ -4472,14 +4505,14 @@ static void print_status_info(
} }
if (i->status_text) if (i->status_text)
printf(" Status: \"%s\"\n", i->status_text); printf(" Status: \"%s\"\n", i->status_text);
if (i->status_errno > 0) if (i->status_errno > 0)
printf(" Error: %i (%s)\n", i->status_errno, strerror_safe(i->status_errno)); printf(" Error: %i (%s)\n", i->status_errno, strerror_safe(i->status_errno));
if (i->ip_ingress_bytes != (uint64_t) -1 && i->ip_egress_bytes != (uint64_t) -1) { if (i->ip_ingress_bytes != (uint64_t) -1 && i->ip_egress_bytes != (uint64_t) -1) {
char buf_in[FORMAT_BYTES_MAX], buf_out[FORMAT_BYTES_MAX]; char buf_in[FORMAT_BYTES_MAX], buf_out[FORMAT_BYTES_MAX];
printf(" IP: %s in, %s out\n", printf(" IP: %s in, %s out\n",
format_bytes(buf_in, sizeof(buf_in), i->ip_ingress_bytes), format_bytes(buf_in, sizeof(buf_in), i->ip_ingress_bytes),
format_bytes(buf_out, sizeof(buf_out), i->ip_egress_bytes)); format_bytes(buf_out, sizeof(buf_out), i->ip_egress_bytes));
} }
@ -4487,13 +4520,13 @@ static void print_status_info(
if (i->io_read_bytes != UINT64_MAX && i->io_write_bytes != UINT64_MAX) { if (i->io_read_bytes != UINT64_MAX && i->io_write_bytes != UINT64_MAX) {
char buf_in[FORMAT_BYTES_MAX], buf_out[FORMAT_BYTES_MAX]; char buf_in[FORMAT_BYTES_MAX], buf_out[FORMAT_BYTES_MAX];
printf(" IO: %s read, %s written\n", printf(" IO: %s read, %s written\n",
format_bytes(buf_in, sizeof(buf_in), i->io_read_bytes), format_bytes(buf_in, sizeof(buf_in), i->io_read_bytes),
format_bytes(buf_out, sizeof(buf_out), i->io_write_bytes)); format_bytes(buf_out, sizeof(buf_out), i->io_write_bytes));
} }
if (i->tasks_current != (uint64_t) -1) { if (i->tasks_current != (uint64_t) -1) {
printf(" Tasks: %" PRIu64, i->tasks_current); printf(" Tasks: %" PRIu64, i->tasks_current);
if (i->tasks_max != (uint64_t) -1) if (i->tasks_max != (uint64_t) -1)
printf(" (limit: %" PRIu64 ")\n", i->tasks_max); printf(" (limit: %" PRIu64 ")\n", i->tasks_max);
@ -4504,7 +4537,7 @@ static void print_status_info(
if (i->memory_current != (uint64_t) -1) { if (i->memory_current != (uint64_t) -1) {
char buf[FORMAT_BYTES_MAX]; char buf[FORMAT_BYTES_MAX];
printf(" Memory: %s", format_bytes(buf, sizeof(buf), i->memory_current)); printf(" Memory: %s", format_bytes(buf, sizeof(buf), i->memory_current));
if (i->memory_min > 0 || i->memory_low > 0 || if (i->memory_min > 0 || i->memory_low > 0 ||
i->memory_high != CGROUP_LIMIT_MAX || i->memory_max != CGROUP_LIMIT_MAX || i->memory_high != CGROUP_LIMIT_MAX || i->memory_max != CGROUP_LIMIT_MAX ||
@ -4544,7 +4577,7 @@ static void print_status_info(
if (i->cpu_usage_nsec != (uint64_t) -1) { if (i->cpu_usage_nsec != (uint64_t) -1) {
char buf[FORMAT_TIMESPAN_MAX]; char buf[FORMAT_TIMESPAN_MAX];
printf(" CPU: %s\n", format_timespan(buf, sizeof(buf), i->cpu_usage_nsec / NSEC_PER_USEC, USEC_PER_MSEC)); printf(" CPU: %s\n", format_timespan(buf, sizeof(buf), i->cpu_usage_nsec / NSEC_PER_USEC, USEC_PER_MSEC));
} }
if (i->control_group) { if (i->control_group) {
@ -4552,7 +4585,7 @@ static void print_status_info(
static const char prefix[] = " "; static const char prefix[] = " ";
unsigned c; unsigned c;
printf(" CGroup: %s\n", i->control_group); printf(" CGroup: %s\n", i->control_group);
c = columns(); c = columns();
if (c > sizeof(prefix) - 1) if (c > sizeof(prefix) - 1)
@ -5510,6 +5543,8 @@ static int show_one(
{ "DropInPaths", "as", NULL, offsetof(UnitStatusInfo, dropin_paths) }, { "DropInPaths", "as", NULL, offsetof(UnitStatusInfo, dropin_paths) },
{ "LoadError", "(ss)", map_load_error, offsetof(UnitStatusInfo, load_error) }, { "LoadError", "(ss)", map_load_error, offsetof(UnitStatusInfo, load_error) },
{ "Result", "s", NULL, offsetof(UnitStatusInfo, result) }, { "Result", "s", NULL, offsetof(UnitStatusInfo, result) },
{ "TriggeredBy", "as", NULL, offsetof(UnitStatusInfo, triggered_by) },
{ "Triggers", "as", NULL, offsetof(UnitStatusInfo, triggers) },
{ "InactiveExitTimestamp", "t", NULL, offsetof(UnitStatusInfo, inactive_exit_timestamp) }, { "InactiveExitTimestamp", "t", NULL, offsetof(UnitStatusInfo, inactive_exit_timestamp) },
{ "InactiveExitTimestampMonotonic", "t", NULL, offsetof(UnitStatusInfo, inactive_exit_timestamp_monotonic) }, { "InactiveExitTimestampMonotonic", "t", NULL, offsetof(UnitStatusInfo, inactive_exit_timestamp_monotonic) },
{ "ActiveEnterTimestamp", "t", NULL, offsetof(UnitStatusInfo, active_enter_timestamp) }, { "ActiveEnterTimestamp", "t", NULL, offsetof(UnitStatusInfo, active_enter_timestamp) },