[libvirt] [PATCH v2 3/6] Drop support for Upstart init scripts

Andrea Bolognani abologna at redhat.com
Fri Mar 15 17:27:40 UTC 2019


Not a single one of the platforms we target still uses Upstart, and
the Upstart project itself has been abandoned for several years now.

Signed-off-by: Andrea Bolognani <abologna at redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange at redhat.com>
---
 m4/virt-init-script.m4      |  7 +-----
 src/Makefile.am             | 29 -----------------------
 src/remote/Makefile.inc.am  |  2 --
 src/remote/libvirtd.upstart | 47 -------------------------------------
 4 files changed, 1 insertion(+), 84 deletions(-)
 delete mode 100644 src/remote/libvirtd.upstart

diff --git a/m4/virt-init-script.m4 b/m4/virt-init-script.m4
index 757fd10541..7094e29f46 100644
--- a/m4/virt-init-script.m4
+++ b/m4/virt-init-script.m4
@@ -20,7 +20,7 @@ dnl
 AC_DEFUN([LIBVIRT_ARG_INIT_SCRIPT],[
     LIBVIRT_ARG_WITH([INIT_SCRIPT],
                      [Style of init script to install: redhat, systemd,
-                      systemd+redhat, upstart, check, none], [check])
+                      systemd+redhat, check, none], [check])
 ])
 
 AC_DEFUN([LIBVIRT_CHECK_INIT_SCRIPT],[
@@ -28,7 +28,6 @@ AC_DEFUN([LIBVIRT_CHECK_INIT_SCRIPT],[
 
     init_redhat=no
     init_systemd=no
-    init_upstart=no
 
     if test "$with_init_script" = check && test "$cross_compiling" = yes; then
         with_init_script=none
@@ -51,9 +50,6 @@ AC_DEFUN([LIBVIRT_CHECK_INIT_SCRIPT],[
         [systemd],[
             init_systemd=yes
         ],
-        [upstart],[
-            init_upstart=yes
-        ],
         [redhat],[
             init_redhat=yes
         ],
@@ -64,7 +60,6 @@ AC_DEFUN([LIBVIRT_CHECK_INIT_SCRIPT],[
     )
 
     AM_CONDITIONAL([LIBVIRT_INIT_SCRIPT_RED_HAT], test "$init_redhat" = "yes")
-    AM_CONDITIONAL([LIBVIRT_INIT_SCRIPT_UPSTART], test "$init_upstart" = "yes")
     AM_CONDITIONAL([LIBVIRT_INIT_SCRIPT_SYSTEMD], test "$init_systemd" = "yes")
 
     AC_MSG_RESULT($with_init_script)
diff --git a/src/Makefile.am b/src/Makefile.am
index 5dc9990c17..94a9a7c27d 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -85,7 +85,6 @@ PODFILES =
 MANINFILES =
 SYSTEMD_UNIT_FILES_IN =
 SYSVINIT_FILES_IN =
-UPSTART_FILES =
 SYSCONF_FILES =
 sbin_PROGRAMS =
 man8_MANS =
@@ -872,34 +871,6 @@ endif LIBVIRT_INIT_SCRIPT_SYSTEMD
 endif WITH_LIBVIRTD
 
 
-EXTRA_DIST += $(UPSTART_FILES)
-
-if WITH_LIBVIRTD
-if LIBVIRT_INIT_SCRIPT_UPSTART
-
-install-upstart: install-sysconfig
-	$(MKDIR_P) $(DESTDIR)$(sysconfdir)/event.d
-	for f in $(UPSTART_FILES:%.upstart=%); \
-	do \
-	  tgt=`basename $$f` ; \
-	  $(INSTALL_SCRIPT) $(srcdir)/$$f.upstart \
-	      $(DESTDIR)$(sysconfdir)/event.d/$$tgt ; \
-	done
-
-uninstall-upstart: uninstall-sysconfig
-	for f in $(UPSTART_FILES:%.upstart=%); \
-	do \
-	  tgt=`basename $$f` ; \
-	  rm -f $(DESTDIR)$(sysconfdir)/event.d/$$tgt ; \
-	done
-	rmdir $(DESTDIR)$(sysconfdir)/event.d || :
-
-INSTALL_DATA_LOCAL += install-upstart
-UNINSTALL_LOCAL += uninstall-upstart
-endif LIBVIRT_INIT_SCRIPT_UPSTART
-endif WITH_LIBVIRTD
-
-
 EXTRA_DIST += dtrace2systemtap.pl
 
 
diff --git a/src/remote/Makefile.inc.am b/src/remote/Makefile.inc.am
index 468a3f5d97..bc1d7f462f 100644
--- a/src/remote/Makefile.inc.am
+++ b/src/remote/Makefile.inc.am
@@ -54,8 +54,6 @@ SYSTEMD_UNIT_FILES_IN += \
 	remote/virt-guest-shutdown.target.in \
 	$(NULL)
 
-UPSTART_FILES += remote/libvirtd.upstart
-
 REMOTE_PROTOCOL = $(srcdir)/remote/remote_protocol.x
 LXC_PROTOCOL = $(srcdir)/remote/lxc_protocol.x
 QEMU_PROTOCOL = $(srcdir)/remote/qemu_protocol.x
diff --git a/src/remote/libvirtd.upstart b/src/remote/libvirtd.upstart
deleted file mode 100644
index e620f41450..0000000000
--- a/src/remote/libvirtd.upstart
+++ /dev/null
@@ -1,47 +0,0 @@
-# libvirtd upstart job
-#
-# XXX wait for rc to get all dependent initscripts started
-# from sysv libvirtd initscript: Required-Start: $network messagebus
-start on stopped rc RUNLEVEL=[345]
-stop on runlevel [!345]
-
-respawn
-
-script
-    LIBVIRTD_CONFIG=
-    LIBVIRTD_ARGS=
-    KRB5_KTNAME=/etc/libvirt/krb5.tab
-
-    if [ -f /etc/sysconfig/libvirtd ]; then
-        . /etc/sysconfig/libvirtd
-    fi
-
-    export QEMU_AUDIO_DRV
-    export SDL_AUDIODRIVER
-    export KRB5_KTNAME
-
-    LIBVIRTD_CONFIG_ARGS=
-    if [ -n "$LIBVIRTD_CONFIG" ]; then
-        LIBVIRTD_CONFIG_ARGS="--config $LIBVIRTD_CONFIG"
-    fi
-
-    # DAEMON_COREFILE_LIMIT from /etc/sysconfig/libvirtd is not handled
-    # automatically
-    if [ -n "$DAEMON_COREFILE_LIMIT" ]; then
-        ulimit -c "$DAEMON_COREFILE_LIMIT"
-    fi
-
-    # LIBVIRTD_NOFILES_LIMIT from /etc/sysconfig/libvirtd is not handled
-    # automatically
-    if [ -n "$LIBVIRTD_NOFILES_LIMIT" ]; then
-        ulimit -n "$LIBVIRTD_NOFILES_LIMIT"
-    fi
-    mkdir -p /var/cache/libvirt
-    rm -rf /var/cache/libvirt/*
-
-    exec /usr/sbin/libvirtd $LIBVIRTD_CONFIG_ARGS $LIBVIRTD_ARGS
-end script
-
-post-stop script
-    rm -rf /var/cache/libvirt/*
-end script
-- 
2.20.1




More information about the libvir-list mailing list