Compare commits

..

3 Commits

Author SHA1 Message Date
Lennart Poettering 4c39820562
Merge pull request #15298 from keszybz/show-property-value
systemctl --property-value as shortcut for --property --value
2020-04-04 11:22:59 +02:00
Zbigniew Jędrzejewski-Szmek 5522e628f5 test: use show -P in places
Not all places are converted on purpose, to also test the other option
name.
2020-04-03 18:24:21 +02:00
Zbigniew Jędrzejewski-Szmek 5292c24047 systemctl: add -P as short for --value --property= 2020-04-03 18:24:20 +02:00
6 changed files with 42 additions and 29 deletions

View File

@ -1502,6 +1502,17 @@ Jan 12 10:46:45 example.com bluetoothd[8900]: gatt-time-server: Input/output err
</listitem> </listitem>
</varlistentry> </varlistentry>
<varlistentry>
<term><option>-P</option></term>
<listitem>
<para>Equivalent to <option>--value</option> <option>--property=</option>, i.e. shows the
value of the property without the property name or <literal>=</literal>. Note that using
<option>-P</option> once will also affect all properties listed with
<option>-p</option>/<option>--property=</option>.</para>
</listitem>
</varlistentry>
<varlistentry> <varlistentry>
<term><option>-a</option></term> <term><option>-a</option></term>
<term><option>--all</option></term> <term><option>--all</option></term>
@ -1627,9 +1638,8 @@ Jan 12 10:46:45 example.com bluetoothd[8900]: gatt-time-server: Input/output err
<term><option>--value</option></term> <term><option>--value</option></term>
<listitem> <listitem>
<para>When printing properties with <command>show</command>, <para>When printing properties with <command>show</command>, only print the value, and skip the
only print the value, and skip the property name and property name and <literal>=</literal>. Also see option <option>-P</option> above.</para>
<literal>=</literal>.</para>
</listitem> </listitem>
</varlistentry> </varlistentry>

View File

@ -7966,6 +7966,7 @@ static int systemctl_help(void) {
" --state=STATE List units with particular LOAD or SUB or ACTIVE state\n" " --state=STATE List units with particular LOAD or SUB or ACTIVE state\n"
" --failed Shorcut for --state=failed\n" " --failed Shorcut for --state=failed\n"
" -p --property=NAME Show only properties by this name\n" " -p --property=NAME Show only properties by this name\n"
" -P NAME Equivalent to --value --property=NAME\n"
" -a --all Show all properties/all units currently in memory,\n" " -a --all Show all properties/all units currently in memory,\n"
" including dead/empty ones. To list all units installed\n" " including dead/empty ones. To list all units installed\n"
" on the system, use 'list-unit-files' instead.\n" " on the system, use 'list-unit-files' instead.\n"
@ -8334,7 +8335,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
/* we default to allowing interactive authorization only in systemctl (not in the legacy commands) */ /* we default to allowing interactive authorization only in systemctl (not in the legacy commands) */
arg_ask_password = true; arg_ask_password = true;
while ((c = getopt_long(argc, argv, "ht:p:alqfs:H:M:n:o:iTr.::", options, NULL)) >= 0) while ((c = getopt_long(argc, argv, "ht:p:P:alqfs:H:M:n:o:iTr.::", options, NULL)) >= 0)
switch (c) { switch (c) {
@ -8389,6 +8390,10 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
break; break;
} }
case 'P':
arg_value = true;
_fallthrough_;
case 'p': case 'p':
/* Make sure that if the empty property list was specified, we won't show any /* Make sure that if the empty property list was specified, we won't show any
properties. */ properties. */
@ -8413,9 +8418,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
} }
} }
/* If the user asked for a particular /* If the user asked for a particular property, show it, even if it is empty. */
* property, show it to him, even if it is
* empty. */
arg_all = true; arg_all = true;
break; break;

View File

@ -13,11 +13,11 @@ EOF
systemctl daemon-reload systemctl daemon-reload
[[ "$(systemctl show --value -p DefaultLimitNOFILESoft)" = "10000" ]] [[ "$(systemctl show -P DefaultLimitNOFILESoft)" = "10000" ]]
[[ "$(systemctl show --value -p DefaultLimitNOFILE)" = "16384" ]] [[ "$(systemctl show -P DefaultLimitNOFILE)" = "16384" ]]
[[ "$(systemctl show --value -p LimitNOFILESoft testsuite-05.service)" = "10000" ]] [[ "$(systemctl show -P LimitNOFILESoft testsuite-05.service)" = "10000" ]]
[[ "$(systemctl show --value -p LimitNOFILE testsuite-05.service)" = "16384" ]] [[ "$(systemctl show -P LimitNOFILE testsuite-05.service)" = "16384" ]]
systemd-run --wait -t bash -c '[[ "$(ulimit -n -S)" = "10000" ]]' systemd-run --wait -t bash -c '[[ "$(ulimit -n -S)" = "10000" ]]'
systemd-run --wait -t bash -c '[[ "$(ulimit -n -H)" = "16384" ]]' systemd-run --wait -t bash -c '[[ "$(ulimit -n -H)" = "16384" ]]'

View File

@ -5,7 +5,7 @@ set -o pipefail
systemd-analyze log-level debug systemd-analyze log-level debug
systemd-analyze log-target console systemd-analyze log-target console
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Start a test process inside of our own cgroup # Start a test process inside of our own cgroup
sleep infinity & sleep infinity &
@ -14,45 +14,45 @@ disown
# Start a test process outside of our own cgroup # Start a test process outside of our own cgroup
systemd-run -p DynamicUser=1 --unit=test20-sleep.service /bin/sleep infinity systemd-run -p DynamicUser=1 --unit=test20-sleep.service /bin/sleep infinity
EXTERNALPID=`systemctl show -p MainPID --value test20-sleep.service` EXTERNALPID=`systemctl show -P MainPID test20-sleep.service`
# Update our own main PID to the external test PID, this should work # Update our own main PID to the external test PID, this should work
systemd-notify MAINPID=$EXTERNALPID systemd-notify MAINPID=$EXTERNALPID
test `systemctl show -p MainPID --value testsuite-20.service` -eq $EXTERNALPID test `systemctl show -P MainPID testsuite-20.service` -eq $EXTERNALPID
# Update our own main PID to the internal test PID, this should work, too # Update our own main PID to the internal test PID, this should work, too
systemd-notify MAINPID=$INTERNALPID systemd-notify MAINPID=$INTERNALPID
test `systemctl show -p MainPID --value testsuite-20.service` -eq $INTERNALPID test `systemctl show -P MainPID testsuite-20.service` -eq $INTERNALPID
# Update it back to our own PID, this should also work # Update it back to our own PID, this should also work
systemd-notify MAINPID=$$ systemd-notify MAINPID=$$
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Try to set it to PID 1, which it should ignore, because that's the manager # Try to set it to PID 1, which it should ignore, because that's the manager
systemd-notify MAINPID=1 systemd-notify MAINPID=1
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Try to set it to PID 0, which is invalid and should be ignored # Try to set it to PID 0, which is invalid and should be ignored
systemd-notify MAINPID=0 systemd-notify MAINPID=0
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Try to set it to a valid but non-existing PID, which should be ignored. (Note # Try to set it to a valid but non-existing PID, which should be ignored. (Note
# that we set the PID to a value well above any known /proc/sys/kernel/pid_max, # that we set the PID to a value well above any known /proc/sys/kernel/pid_max,
# which means we can be pretty sure it doesn't exist by coincidence) # which means we can be pretty sure it doesn't exist by coincidence)
systemd-notify MAINPID=1073741824 systemd-notify MAINPID=1073741824
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Change it again to the external PID, without privileges this time. This should be ignored, because the PID is from outside of our cgroup and we lack privileges. # Change it again to the external PID, without privileges this time. This should be ignored, because the PID is from outside of our cgroup and we lack privileges.
systemd-notify --uid=1000 MAINPID=$EXTERNALPID systemd-notify --uid=1000 MAINPID=$EXTERNALPID
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
# Change it again to the internal PID, without privileges this time. This should work, as the process is on our cgroup, and that's enough even if we lack privileges. # Change it again to the internal PID, without privileges this time. This should work, as the process is on our cgroup, and that's enough even if we lack privileges.
systemd-notify --uid=1000 MAINPID=$INTERNALPID systemd-notify --uid=1000 MAINPID=$INTERNALPID
test `systemctl show -p MainPID --value testsuite-20.service` -eq $INTERNALPID test `systemctl show -P MainPID testsuite-20.service` -eq $INTERNALPID
# Update it back to our own PID, this should also work # Update it back to our own PID, this should also work
systemd-notify --uid=1000 MAINPID=$$ systemd-notify --uid=1000 MAINPID=$$
test `systemctl show -p MainPID --value testsuite-20.service` -eq $$ test `systemctl show -P MainPID testsuite-20.service` -eq $$
cat >/tmp/test20-mainpid.sh <<EOF cat >/tmp/test20-mainpid.sh <<EOF
#!/usr/bin/env bash #!/usr/bin/env bash
@ -76,7 +76,7 @@ EOF
chmod +x /tmp/test20-mainpid.sh chmod +x /tmp/test20-mainpid.sh
systemd-run --unit=test20-mainpidsh.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh -p PIDFile=/run/mainpidsh/pid /tmp/test20-mainpid.sh systemd-run --unit=test20-mainpidsh.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh -p PIDFile=/run/mainpidsh/pid /tmp/test20-mainpid.sh
test `systemctl show -p MainPID --value test20-mainpidsh.service` -eq `cat /run/mainpidsh/pid` test `systemctl show -P MainPID test20-mainpidsh.service` -eq `cat /run/mainpidsh/pid`
cat >/tmp/test20-mainpid2.sh <<EOF cat >/tmp/test20-mainpid2.sh <<EOF
#!/usr/bin/env bash #!/usr/bin/env bash
@ -101,7 +101,7 @@ EOF
chmod +x /tmp/test20-mainpid2.sh chmod +x /tmp/test20-mainpid2.sh
systemd-run --unit=test20-mainpidsh2.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh2 -p PIDFile=/run/mainpidsh2/pid /tmp/test20-mainpid2.sh systemd-run --unit=test20-mainpidsh2.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh2 -p PIDFile=/run/mainpidsh2/pid /tmp/test20-mainpid2.sh
test `systemctl show -p MainPID --value test20-mainpidsh2.service` -eq `cat /run/mainpidsh2/pid` test `systemctl show -P MainPID test20-mainpidsh2.service` -eq `cat /run/mainpidsh2/pid`
cat >/dev/shm/test20-mainpid3.sh <<EOF cat >/dev/shm/test20-mainpid3.sh <<EOF
#!/usr/bin/env bash #!/usr/bin/env bash
@ -130,7 +130,7 @@ chmod 755 /dev/shm/test20-mainpid3.sh
! systemd-run --unit=test20-mainpidsh3.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh3 -p PIDFile=/run/mainpidsh3/pid -p DynamicUser=1 -p TimeoutStartSec=2s /dev/shm/test20-mainpid3.sh ! systemd-run --unit=test20-mainpidsh3.service -p StandardOutput=tty -p StandardError=tty -p Type=forking -p RuntimeDirectory=mainpidsh3 -p PIDFile=/run/mainpidsh3/pid -p DynamicUser=1 -p TimeoutStartSec=2s /dev/shm/test20-mainpid3.sh
# Test that this failed due to timeout, and not some other error # Test that this failed due to timeout, and not some other error
test `systemctl show -p Result --value test20-mainpidsh3.service` = timeout test `systemctl show -P Result test20-mainpidsh3.service` = timeout
systemd-analyze log-level info systemd-analyze log-level info

View File

@ -23,12 +23,12 @@ if test -f /sys/fs/cgroup/system.slice/testsuite-32.service/memory.oom.group; th
echo f >/proc/sysrq-trigger echo f >/proc/sysrq-trigger
while : ; do while : ; do
STATE=`systemctl show -p ActiveState --value oomtest.service` STATE=`systemctl show -P ActiveState oomtest.service`
[ "$STATE" = "failed" ] && break [ "$STATE" = "failed" ] && break
sleep .5 sleep .5
done done
RESULT=`systemctl show -p Result --value oomtest.service` RESULT=`systemctl show -P Result oomtest.service`
test "$RESULT" = "oom-kill" test "$RESULT" = "oom-kill"
systemd-analyze log-level info systemd-analyze log-level info

View File

@ -12,10 +12,10 @@ systemd-analyze log-target console
! systemd-run --unit=one -p Type=oneshot -p Restart=on-failure /bin/bash -c "exit 1" ! systemd-run --unit=one -p Type=oneshot -p Restart=on-failure /bin/bash -c "exit 1"
for ((secs=0; secs<$MAX_SECS; secs++)); do for ((secs=0; secs<$MAX_SECS; secs++)); do
[[ "$(systemctl show one.service -p NRestarts --value)" -le 0 ]] || break [[ "$(systemctl show one.service -P NRestarts)" -le 0 ]] || break
sleep 1 sleep 1
done done
if [[ "$(systemctl show one.service -p NRestarts --value)" -le 0 ]]; then if [[ "$(systemctl show one.service -P NRestarts)" -le 0 ]]; then
exit 1 exit 1
fi fi