systemctl: option to list units by state

This allows to show only units with specified LOAD or SUB or ACTIVE state.
This commit is contained in:
Maciej Wereski 2013-07-18 13:24:12 +02:00 committed by Lennart Poettering
parent 44bc6e1fe0
commit 9b9b3d36b8
2 changed files with 48 additions and 35 deletions

View File

@ -94,25 +94,29 @@ along with systemd; If not, see <http://www.gnu.org/licenses/>.
<listitem>
<para>The argument should be a comma-separated list of unit
types such as <option>service</option> and
<option>socket</option>, or unit load states such as
<option>loaded</option> and <option>masked</option>
(types and states can be mixed).</para>
<option>socket</option>.
</para>
<para>If one of the arguments is a unit type, when listing
units, limit display to certain unit types. Otherwise, units
of all types will be shown.</para>
<para>If one of the arguments is a unit load state, when
listing units, limit display to certain unit
types. Otherwise, units of in all load states will be
shown.</para>
<para>As a special case, if one of the arguments is
<option>help</option>, a list of allowed values will be
printed and the program will exit.</para>
</listitem>
</varlistentry>
<varlistentry>
<term><option>--state=</option></term>
<listitem>
<para>Argument should be a comma-separated list of unit LOAD
or SUB or ACTIVE states. When listing units show only those
with specified LOAD or SUB or ACTIVE state.</para>
</listitem>
</varlistentry>
<varlistentry>
<term><option>-p</option></term>
<term><option>--property=</option></term>
@ -165,15 +169,6 @@ along with systemd; If not, see <http://www.gnu.org/licenses/>.
</listitem>
</varlistentry>
<varlistentry>
<term><option>--failed</option></term>
<listitem>
<para>When listing units, show only failed units. Do not
confuse with <option>--fail</option>.</para>
</listitem>
</varlistentry>
<varlistentry>
<term><option>-l</option></term>
<term><option>--full</option></term>

View File

@ -69,7 +69,7 @@
#include "fileio.h"
static char **arg_types = NULL;
static char **arg_load_states = NULL;
static char **arg_states = NULL;
static char **arg_properties = NULL;
static bool arg_all = false;
static enum dependency {
@ -93,7 +93,6 @@ static bool arg_quiet = false;
static bool arg_full = false;
static int arg_force = 0;
static bool arg_ask_password = true;
static bool arg_failed = false;
static bool arg_runtime = false;
static char **arg_wall = NULL;
static const char *arg_kill_who = NULL;
@ -301,12 +300,11 @@ static int compare_unit_info(const void *a, const void *b) {
static bool output_show_unit(const struct unit_info *u) {
const char *dot;
if (arg_failed)
return streq(u->active_state, "failed");
if (!strv_isempty(arg_states))
return strv_contains(arg_states, u->load_state) || strv_contains(arg_states, u->sub_state) || strv_contains(arg_states, u->active_state);
return (!arg_types || ((dot = strrchr(u->id, '.')) &&
strv_find(arg_types, dot+1))) &&
(!arg_load_states || strv_find(arg_load_states, u->load_state)) &&
(arg_all || !(streq(u->active_state, "inactive")
|| u->following[0]) || u->job_id > 0);
}
@ -4705,12 +4703,12 @@ static int systemctl_help(void) {
" -h --help Show this help\n"
" --version Show package version\n"
" -t --type=TYPE List only units of a particular type\n"
" --state=STATE Show only units with particular LOAD or SUB or ACTIVE state\n"
" -p --property=NAME Show only properties by this name\n"
" -a --all Show all loaded units/properties, including dead/empty\n"
" ones. To list all units installed on the system, use\n"
" the 'list-unit-files' command instead.\n"
" --reverse Show reverse dependencies with 'list-dependencies'\n"
" --failed Show only failed units\n"
" -l --full Don't ellipsize unit names on output\n"
" --fail When queueing a new job, fail if conflicting jobs are\n"
" pending\n"
@ -4896,13 +4894,6 @@ static int help_types(void) {
puts(t);
}
puts("\nAvailable unit load states: ");
for(i = 0; i < _UNIT_LOAD_STATE_MAX; i++) {
t = unit_load_state_to_string(i);
if (t)
puts(t);
}
return 0;
}
@ -4931,7 +4922,8 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
ARG_FAILED,
ARG_RUNTIME,
ARG_FORCE,
ARG_PLAIN
ARG_PLAIN,
ARG_STATE
};
static const struct option options[] = {
@ -4970,6 +4962,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
{ "lines", required_argument, NULL, 'n' },
{ "output", required_argument, NULL, 'o' },
{ "plain", no_argument, NULL, ARG_PLAIN },
{ "state", required_argument, NULL, ARG_STATE },
{ NULL, 0, NULL, 0 }
};
@ -5014,8 +5007,12 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
continue;
}
/* It's much nicer to use --state= for
* load states, but let's support this
* in --types= too for compatibility
* with old versions */
if (unit_load_state_from_string(optarg) >= 0) {
if (strv_push(&arg_load_states, type))
if (strv_push(&arg_states, type) < 0)
return log_oom();
type = NULL;
continue;
@ -5047,7 +5044,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
if (!prop)
return log_oom();
if (strv_push(&arg_properties, prop)) {
if (strv_push(&arg_properties, prop) < 0) {
free(prop);
return log_oom();
}
@ -5131,7 +5128,9 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
break;
case ARG_FAILED:
arg_failed = true;
if (strv_extend(&arg_states, "failed") < 0)
return log_oom();
break;
case 'q':
@ -5201,6 +5200,25 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
arg_plain = true;
break;
case ARG_STATE: {
char *word, *state;
size_t size;
FOREACH_WORD_SEPARATOR(word, size, optarg, ",", state) {
char *s;
s = strndup(word, size);
if (!s)
return log_oom();
if (strv_push(&arg_states, s) < 0) {
free(s);
return log_oom();
}
}
break;
}
case '?':
return -EINVAL;
@ -6257,7 +6275,7 @@ finish:
dbus_shutdown();
strv_free(arg_types);
strv_free(arg_load_states);
strv_free(arg_states);
strv_free(arg_properties);
pager_close();