Hello community,
here is the log from the commit of package transactional-update for
openSUSE:Factory checked in at 2020-09-25 16:17:23
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/transactional-update (Old)
and /work/SRC/openSUSE:Factory/.transactional-update.new.4249 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "transactional-update"
Fri Sep 25 16:17:23 2020 rev:60 rq: version:2.24.1
Changes:
--------
---
/work/SRC/openSUSE:Factory/transactional-update/transactional-update.changes
2020-09-24 16:14:04.572888547 +0200
+++
/work/SRC/openSUSE:Factory/.transactional-update.new.4249/transactional-update.changes
2020-09-25 16:17:24.427139892 +0200
@@ -2,20 +1,0 @@
-Mon Sep 21 16:17:12 UTC 2020 - Ignaz Forster <[email protected]>
-
-- Version 2.25.1
- - Fix inotify watcher setup
- - Use log_{info,error} for more messages to avoid messing up Salt logs
-
--------------------------------------------------------------------
-Mon Sep 21 14:09:41 UTC 2020 - Ignaz Forster <[email protected]>
-
-- Version 2.25
- - Reduce number of overlays:
- Instead of using transparent overlays for all previous layers only add the
- previous snapshot's overlay; this will greatly reduce the number of
- overlays while still making sure that /etc changes in the running system
- will be visible in the next snapshot
- - When using --drop-if-no-change synchronize potential /etc changes with
- running system
- - Exclude all non-root-fs mounts from inotify watcher
-
--------------------------------------------------------------------
Old:
----
transactional-update-2.25.1.tar.gz
New:
----
transactional-update-2.24.1.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ transactional-update.spec ++++++
--- /var/tmp/diff_new_pack.cpQd3G/_old 2020-09-25 16:17:25.115140501 +0200
+++ /var/tmp/diff_new_pack.cpQd3G/_new 2020-09-25 16:17:25.119140504 +0200
@@ -17,7 +17,7 @@
Name: transactional-update
-Version: 2.25.1
+Version: 2.24.1
Release: 0
Summary: Transactional Updates with btrfs and snapshots
License: GPL-2.0-or-later
++++++ transactional-update-2.25.1.tar.gz -> transactional-update-2.24.1.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/transactional-update-2.25.1/NEWS
new/transactional-update-2.24.1/NEWS
--- old/transactional-update-2.25.1/NEWS 2020-09-21 18:16:14.000000000
+0200
+++ new/transactional-update-2.24.1/NEWS 2020-08-12 12:04:41.000000000
+0200
@@ -2,20 +2,6 @@
Copyright (C) 2016-2019 Thorsten Kukuk et al.
-Version 2.25.1
-* Fix inotify watcher setup
-* Use log_{info,error} for more messages to avoid messing up Salt logs
-
-Version 2.25
-* Reduce number of overlays:
- Instead of using transparent overlays for all previous layers only add the
- previous snapshot's overlay; this will greatly reduce the number of overlays
- while still making sure that /etc changes in the running system will be
- visible in the next snapshot
-* When using --drop-if-no-change synchronize potential /etc changes with
- running system
-* Exclude all non-root-fs mounts from inotify watcher
-
Version 2.24.1
* SELinux: adjust labels for etc, fstab and grub.cfg
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/transactional-update-2.25.1/configure.ac
new/transactional-update-2.24.1/configure.ac
--- old/transactional-update-2.25.1/configure.ac 2020-09-21
18:16:14.000000000 +0200
+++ new/transactional-update-2.24.1/configure.ac 2020-08-12
12:04:41.000000000 +0200
@@ -1,5 +1,5 @@
dnl Process this file with autoconf to produce a configure script.
-AC_INIT(transactional-update, 2.25.1)
+AC_INIT(transactional-update, 2.24.1)
AM_INIT_AUTOMAKE
AC_PREFIX_DEFAULT(/usr)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/transactional-update-2.25.1/doc/transactional-update.xml
new/transactional-update-2.24.1/doc/transactional-update.xml
--- old/transactional-update-2.25.1/doc/transactional-update.xml
2020-09-21 18:16:14.000000000 +0200
+++ new/transactional-update-2.24.1/doc/transactional-update.xml
2020-08-12 12:04:41.000000000 +0200
@@ -446,36 +446,21 @@
<filename
class='directory'>/var/lib/overlay/<snapshotnum>/etc</filename>.
</para>
<para>
- Each snapshot will have one associated overlay directory. On creating
- a new snapshot the previous snapshot's
- <filename class='directory'>/etc</filename> state will be sychronized
- into the new snapshot and used as a base. The overlay directories of
- the current and the new snapshot are then mounted using overlay
- stacking, i.e. the new snapshot's overlay will be mounted as the
- <option>upperdir</option> and the current snapshot's overlay as
- <option>lowerdir</option>. This way changes applied to
- <filename class='directory'>/etc</filename> after the snapshot was
- taken, but before the reboot takes place, will still be visible to the
- new snapshot (Exception: If the file has been modified both in the
- current and the new snapshot, then the file state of the new snapshot
- will be visible).
- </para>
- <para>
- If the <option>--continue</option> is used multiple times to extend a
- new snapshot while the system has not been rebooted, and if that
- snapshot is based on the currently active system, then the
- synchronization will only run for the first snapshot; the additional
- snapshot layers will be added to <option>lowerdir</option>. Again
- this is to make sure that changes to the running system will still be
- visible after booting into the new system.
+ Each snapshot will have one associated overlay directory. The overlay
+ directories are mounted using overlay stacking, i.e. the current
+ snapshot will mount its associated overlay as the
+ <option>upperdir</option> and the previous snapshot's overlays as
+ <option>lowerdir</option>. That means that the current snapshot will
+ see all the changes accumulated in the previous snapshots, but older
+ snapshots will not see changes applied to newer snapshots.
</para>
<para>
Let's have a look at an example <filename>fstab</filename> entry:
- <programlisting>overlay /etc overlay
defaults,upperdir=/sysroot/var/lib/overlay/82/etc,lowerdir=/sysroot/var/lib/overlay/81/etc:/sysroot/var/lib/overlay/76/etc:/sysroot/etc,workdir=/sysroot/var/lib/overlay/work-etc,x-systemd.requires-mounts-for=/var,x-systemd.requires-mounts-for=/var/lib/overlay,x-systemd.requires-mounts-for=/sysroot/var,x-systemd.requires-mounts-for=/sysroot/var/lib/overlay,x-initrd.mount
0 0</programlisting>
+ <programlisting>overlay /etc overlay
defaults,upperdir=/sysroot/var/lib/overlay/83/etc,lowerdir=/sysroot/var/lib/overlay/81/etc:/sysroot/var/lib/overlay/80/etc:/sysroot/var/lib/overlay/77/etc:/sysroot/var/lib/overlay/76/etc:/sysroot/etc,workdir=/sysroot/var/lib/overlay/work-etc,x-systemd.requires-mounts-for=/var,x-systemd.requires-mounts-for=/var/lib/overlay,x-systemd.requires-mounts-for=/sysroot/var,x-systemd.requires-mounts-for=/sysroot/var/lib/overlay,x-initrd.mount
0 0</programlisting>
</para>
<itemizedlist>
<listitem><para>
- We are currently in snapshot <emphasis>82</emphasis> as indicated by the
+ We are currently in snapshot <emphasis>83</emphasis> as indicated by the
<option>upperdir</option> directory. This can be confirmed by typing
<command>snapper list</command> or
<command>btrfs subvolume get-default /</command>. All changes to
@@ -483,18 +468,12 @@
directory.
</para></listitem>
<listitem><para>
- <option>lowerdir</option> contains two numbered overlay directories.
- The later directory with number <emphasis>76</emphasis> indicates the
- snapshot which was used as a base. This snapshot's
- <filename class='directory'>/etc</filename> state was also synchronized
- into the read-only root file system of snapshot 82. As the
- <option>lowerdir</option> contains a second entry with number
- <emphasis>81</emphasis> it means that the <option>--continue</option>
- has been used before the system was rebooted. Gaps in the number such
- as seen here may indicate that those snapshots were discarded or a
- rollback to snapshot 76 was performed. The lowest snapshot is always
- <filename class='directory'>/sysroot/etc</filename>, containing the
- root file system's contents.
+ <option>lowerdir</option> contains several overlays from previous
+ snapshots. In this case it seem the snapshots were not created
+ consecutively - maybe there were a few rollbacks in between or a
+ snapshot wasn't applied by rebooting into it. The lowest snapshot is
+ always <filename class='directory'>/sysroot/etc</filename>, containing
+ the root file system's contents.
</para></listitem>
<listitem><para>
As <filename class='directory'>/etc</filename> is mounted by
@@ -505,8 +484,16 @@
</para></listitem>
</itemizedlist>
<para>
- Overlays no longer referenced by any snapshots will be deleted during
- the <command>transactional-update cleanup</command> run.
+ To prevent the <filename class='directory'>/etc</filename> overlays
+ from growing infinitely they will be synced into the root file system if
+ no snapshot is referencing them any more. So please be aware that the
+ root file system's <filename class='directory'>/etc</filename> state by
+ itself does neither contain the original package contents nor the
+ snapshot's current state. Only the snapshot plus all overlay layers
+ represents the state of <filename class='directory'>/etc</filename>
+ correctly. Overlays no longer needed because they have been merged into
+ all snapshots will be deleted during the <command>transactional-update
+ cleanup</command> run.
</para>
</section>
<!--
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/transactional-update-2.25.1/man/transactional-update.8.xml
new/transactional-update-2.24.1/man/transactional-update.8.xml
--- old/transactional-update-2.25.1/man/transactional-update.8.xml
2020-09-21 18:16:14.000000000 +0200
+++ new/transactional-update-2.24.1/man/transactional-update.8.xml
2020-08-12 12:04:41.000000000 +0200
@@ -104,11 +104,11 @@
<filename class='directory'>/var/lib/overlay</filename>. As configuration files
may be modified after a snapshot was created and before a reboot is performed
(e.g. via configuration management software) the overlay file system will use
-multiple lower layers, i.e. configuration file changes applied to the currently
-running system will be visible in the new system, but not vice versa. To keep
-the number of layers at a minimum the
-<filename class='directory'>/etc</filename> state of the previous snapshot is
-synchronized into the new root file system; unused overlays will be
+multiple lower layers, i.e. changes in older layers will be visible in newer
+snapshots, but not vice versa. To keep the number of layers at a minimum and
the
+amount of data stored in the overlays low the
+<filename class='directory'>/etc</filename> state of the oldest still available
+snapshot is synchronized into the new root file system; unused overlays will be
removed at a later time (see the <option>cleanup</option> option).
</para>
<warning>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/transactional-update-2.25.1/sbin/transactional-update.in
new/transactional-update-2.24.1/sbin/transactional-update.in
--- old/transactional-update-2.25.1/sbin/transactional-update.in
2020-09-21 18:16:14.000000000 +0200
+++ new/transactional-update-2.24.1/sbin/transactional-update.in
2020-08-12 12:04:41.000000000 +0200
@@ -82,7 +82,6 @@
SNAPPER_NO_DBUS=""
TMPFILE=""
ETC_OVERLAY_DIR=""
-ETC_OVERLAY_DIR_PREV=""
ETC_OVERLAY_WORK_DIR=""
INOTIFY_PID=""
@@ -171,14 +170,14 @@
log_info() {
echo `date "+%Y-%m-%d %H:%M"` "$@" >> ${LOGFILE}
if [ ${VERBOSITY} -ge 2 ]; then
- echo -e "$@"
+ echo "$@"
fi
}
log_error() {
TELEM_PAYLOAD="${TELEM_PAYLOAD}\nmessage=$@"
echo `date "+%Y-%m-%d %H:%M"` "$@" >> ${LOGFILE}
- echo -e "$@" 1>&4
+ echo "$@" 1>&4
}
bashlock() {
@@ -265,11 +264,11 @@
done
if [ -n "${SNAPSHOT_ID}" ] ; then
- log_info "Removing snapshot #${SNAPSHOT_ID}..."
+ log_error "Removing snapshot #${SNAPSHOT_ID}..."
snapper ${SNAPPER_NO_DBUS} delete ${SNAPSHOT_ID} |& tee -a ${LOGFILE}
fi
if [ -n "${ETC_OVERLAY_DIR}" ] ; then
- log_info "Removing overlay directory ${ETC_OVERLAY_DIR%/*}..."
+ log_error "Removing overlay directory ${ETC_OVERLAY_DIR%/*}..."
if findmnt "${ETC_OVERLAY_DIR}" >/dev/null; then
umount "${ETC_OVERLAY_DIR}"
fi
@@ -291,13 +290,6 @@
kill ${INOTIFY_PID}
log_info "The snapshot does not contain any changed files."
INOTIFY_PID=""
-
- # Even if the snapshot itself did not contain any changes,
- # /etc may do so. Changes in /etc may be applied immediately,
- # so merge them back into the running system.
- log_info "Merging overlay directory ${SNAPSHOT_DIR}/etc into /etc..."
- rsync --archive --inplace --xattrs --acls --exclude 'fstab' --delete
--quiet "${SNAPSHOT_DIR}/etc/" /etc
-
quit 0
fi
@@ -439,9 +431,10 @@
local option lowerdir overlay_id
fstab=""
+ fstab_active_lowerdirs=()
fstab_lowerdirs=()
fstab_upper=""
- fstab_previous_snapshot=""
+ fstab_oldest_active_snapshot=""
fstab_otheroptions=""
# Check for /etc overlay
@@ -462,8 +455,18 @@
for lowerdir in `echo ${option#*=} | tr ':' ' '`; do
overlay_id=`echo "${lowerdir#/sysroot}" | sed -n
's#^/var/lib/overlay/\([[:digit:]]\+\)/etc$#\1#p'`
- if [ -z "${fstab_previous_snapshot}" -a -n "${overlay_id}"
]; then
-
fstab_previous_snapshot="/.snapshots/${overlay_id}/snapshot"
+ # Special handling for old overlay location
+ if [ "${lowerdir}" = "/sysroot/var/lib/overlay/etc" ]; then
+ if [ "${fstab_lowerdirs[*]}" =
"${fstab_active_lowerdirs[*]}" ]; then
+ fstab_active_lowerdirs+=("${lowerdir}")
+ fi
+ # Add common directories and directories of existing
snapshots
+ elif [ -z "${overlay_id}" -o -e
"/.snapshots/${overlay_id}/snapshot" ]; then
+ if [ -n "${overlay_id}" ]; then
+
fstab_oldest_active_snapshot="/.snapshots/${overlay_id}/snapshot"
+ fstab_active_lowerdirs=("${fstab_lowerdirs[@]}")
+ fi
+ fstab_active_lowerdirs+=("${lowerdir}")
fi
fstab_lowerdirs+=("${lowerdir}")
done
@@ -477,6 +480,7 @@
;;
esac
done
+
fstab_oldest_active_snapshot="${fstab_oldest_active_snapshot:-/.snapshots/${CURRENT_SNAPSHOT_ID}/snapshot}"
}
# Replace all /sysroot prefixes from /etc fstab entry; for the overlays in /var
@@ -498,11 +502,16 @@
if [ ${#fstab_lowerdirs[@]} -le 1 ]; then
return
fi
- log_info "Syncing /etc of previous snapshot $1 as base into new snapshot
$2"
+ log_info "Syncing /etc of oldest snapshot $1 as base into new snapshot $2"
mount_opts="defaults,"
mount_opts+="lowerdir="
- mount_opts+="${fstab_upper#/sysroot}:"
+ # Special case: If the current in-use snapshot is the oldest snapshot
+ # already it will be added as a lowerdir, so don't mount it for
+ # syncing (it may change at any time anyway).
+ if [ "${fstab_upper}" != "${current_upper}" ]; then
+ mount_opts+="${fstab_upper#/sysroot}:"
+ fi
mount_opts+="`parse_lowerdirs_for_mount $1`"
mount -t overlay overlay "${ETC_OVERLAY_DIR}" -o ${mount_opts}
if [ $? -ne 0 ]; then
@@ -510,7 +519,7 @@
quit 1
fi
- rsync --archive --inplace --xattrs --acls --delete --quiet
"${ETC_OVERLAY_DIR}" "$2"
+ rsync --archive --inplace --xattrs --filter='-x security.selinux' --acls
--delete --quiet "${ETC_OVERLAY_DIR}" "$2"
if [ $? -ne 0 ]; then
log_error "ERROR: syncing $1 into snapshot $2 failed!"
quit 1
@@ -518,17 +527,6 @@
umount "${ETC_OVERLAY_DIR}"
}
-# Return the optimized list of mount points
-root_mount_points() {
- local prev=0
- for mount in $(findmnt --noheadings --submounts --target / --output TARGET
--raw | sort | tail -n +2); do
- if [ "${prev}/" != "${mount:0:$((${#prev}+1))}" ]; then
- echo "${mount}"
- prev="${mount}"
- fi
- done
-}
-
ORIG_ARGS=("$@")
while [ 1 ]; do
@@ -736,7 +734,7 @@
if [ -z "${TA_UPDATE_TMPFILE}" ]; then
bashlock
if [ $? -ne 0 ]; then
- log_error "Couldn't get lock, is another instance already running?"
+ echo "Couldn't get lock, is another instance already running?"
exit 1
fi
self_update "${ORIG_ARGS[@]}"
@@ -759,7 +757,7 @@
SNAPPER_VERSION=`snapper --version | head -1 | cut -d ' ' -f 2`
if [ -n "${BASE_SNAPSHOT_ID}" -a $(zypper --terse versioncmp $SNAPPER_VERSION
0.8.4) -lt 0 ]; then
- log_error "ERROR: snapper >= 0.8.4 required for --continue option!"
+ echo "ERROR: snapper >= 0.8.4 required for --continue option!"
log_info "transactional-update finished"
telem_finish 1
exit 1
@@ -858,7 +856,7 @@
NEED_REBOOT_WARNING=1
fi
if [ ${NEED_REBOOT_WARNING} -eq 1 ]; then
- log_error "Please reboot to finish rollback!"
+ echo "Please reboot to finish rollback!"
fi
telem_finish 0
exit 0
@@ -1000,14 +998,10 @@
get_etc_overlay_from /
current_upper="${fstab_upper}"
- get_etc_overlay_from "/.snapshots/${BASE_SNAPSHOT_ID}/snapshot"
- # Copy the previous snapshot's /etc state into new snapshot (if it
- # hasn't been synced in a previous layer already)
- if [ -n "${fstab_previous_snapshot}" ] && [[ ${fstab_lowerdirs[@]} !=
*"${current_upper}"* ]]; then
- sync_etc "${fstab_previous_snapshot}" "${SNAPSHOT_DIR}"
- get_etc_overlay_from "/.snapshots/${BASE_SNAPSHOT_ID}/snapshot"
- fi
+ # Copy the contents of oldest /etc layer into snapshot
+ sync_etc "${fstab_oldest_active_snapshot}" "${SNAPSHOT_DIR}"
+ get_etc_overlay_from "/.snapshots/${BASE_SNAPSHOT_ID}/snapshot"
# Special handling /etc/fstab{.sys}: dracut needs the *current* version
# of the files in the snapshot, as the overlay is not yet mounted
@@ -1031,19 +1025,10 @@
gawk -i inplace '$3 == "overlay" && $4 ~
"=/sysroot/var/lib/overlay/" { $4 =
$4",x-systemd.requires-mounts-for=/var,x-systemd.requires-mounts-for=/var/lib/overlay,x-systemd.requires-mounts-for=/sysroot/var,x-systemd.requires-mounts-for=/sysroot/var/lib/overlay,x-initrd.mount";
} 1' "${SNAPSHOT_DIR}/etc/fstab"
fi
- # Check whether the new snapshot is based on the currently running
- # one - if so, then keep the overlay stack to not loose changes
- # applied until reboot
- if [[ ${fstab_lowerdirs[@]} == *"${current_upper}"* ]]; then
- lowerdirs="${fstab_upper}:$(echo ${fstab_lowerdirs[@]} | tr ' '
':')"
- else
- lowerdirs="${fstab_upper}:/sysroot/etc"
- fi
-
# Update fstab{,.sys} entries for /etc overlays
fstab_line="overlay /etc overlay ${fstab_otheroptions}"
fstab_line+="upperdir=/sysroot${ETC_OVERLAY_DIR},"
- fstab_line+="lowerdir=${lowerdirs},"
+ fstab_line+="lowerdir=${fstab_upper}:`echo ${fstab_active_lowerdirs[@]}
| tr ' ' ':'`,"
fstab_line+="workdir=/sysroot/var/lib/overlay/work-etc,"
fstab_line+="x-systemd.requires-mounts-for=/var,"
fstab_line+="x-systemd.requires-mounts-for=/var/lib/overlay,"
@@ -1112,16 +1097,11 @@
if [ ${ETC_IS_OVERLAY} -eq 1 ]; then
lowerdir="${fstab_upper///sysroot\/var//var}:"
- # Is the snapshot based on the currently running system?
- if [[ ${fstab_lowerdirs[@]} == *"${current_upper}"* ]]; then
- lowerdir+="$(parse_lowerdirs_for_mount
"/.snapshots/${BASE_SNAPSHOT_ID}/snapshot")"
- # Mounting an already used overlay directory again is not
- # supported by overlayfs - use the old mount point instead
- lowerdir="${lowerdir/${current_upper#/sysroot}*/\/etc}"
- else
- # Use the (synced) /etc directory of the new snapshot as base
- lowerdir+="/.snapshots/${SNAPSHOT_ID}/snapshot/etc"
- fi
+ lowerdir+="`parse_lowerdirs_for_mount
"/.snapshots/${BASE_SNAPSHOT_ID}/snapshot"`"
+ # Check whether the current upper directory is part of the snapshot's
lower
+ # directory stack; if so use reuse /etc directly instead, as mounting
+ # the same upper directory multiple times is not supported by overlayfs
+ lowerdir="${lowerdir/${current_upper#/sysroot}*/\/etc}"
etc_opts="defaults,"
etc_opts+="lowerdir=${lowerdir},"
@@ -1191,8 +1171,8 @@
if [ $DROP_IF_NO_CHANGE -eq 1 ]; then
#TODO: Handle directories with special characters (space, newline,
apostrophe etc)
- for snapmount in $(root_mount_points); do
- INOTIFY_EXCLUDES="${INOTIFY_EXCLUDES} @${SNAPSHOT_DIR}${snapmount}"
+ for snapmount in `findmnt --noheadings --submounts --target
"${SNAPSHOT_DIR}" --output TARGET --raw | grep "${SNAPSHOT_DIR}"`; do
+ INOTIFY_EXCLUDES="${INOTIFY_EXCLUDES} @${snapmount}"
done
INOTIFY_PID=$( ( inotifywait -r -e modify -e move -e create -e delete
-e attrib "${SNAPSHOT_DIR}" ${INOTIFY_EXCLUDES} 2>&1 & echo "PID: $!" ) | (
grep -m1 '^PID: ' | cut -d ' ' -f 2 && grep -q "Watches established" ) )
fi
@@ -1260,7 +1240,7 @@
else
log_error "ERROR: zypper ${ZYPPER_ARG} on ${MOUNT_DIR} failed with
exit code ${RETVAL}!"
if [ -n "${ZYPPER_NONINTERACTIVE}" ]; then
- log_error "Use '--interactive' for manual problem resolution."
+ echo "Use '--interactive' for manual problem resolution."
fi
EXITCODE=1
fi
@@ -1309,7 +1289,7 @@
fi
if [ ${RUN_SHELL} -eq 1 ]; then
- log_info "Opening chroot in snapshot ${SNAPSHOT_ID}, continue with
'exit'"
+ echo "Opening chroot in snapshot ${SNAPSHOT_ID}, continue with 'exit'"
env PS1="transactional update # " chroot ${MOUNT_DIR} bash 2>&4
fi
@@ -1385,7 +1365,8 @@
if [ ${EXITCODE} -ne 0 ]; then
quit ${EXITCODE}
elif [ $REBOOT_AFTERWARDS -eq 0 ]; then
- log_error "\nPlease reboot your machine to activate the changes and
avoid data loss."
+ echo
+ echo "Please reboot your machine to activate the changes and avoid data
loss."
touch "${NEEDS_RESTARTING_FILE}"
fi
@@ -1419,8 +1400,8 @@
reboot_autodetect
;;
esac
- log_error "The system couldn't be rebooted using method
'${REBOOT_METHOD}'. Please reboot the system"
- log_error "manually."
+ echo "The system couldn't be rebooted using method '${REBOOT_METHOD}'.
Please reboot the system"
+ echo "manually."
fi
fi
