Alexandros Kosiaris has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/405703 )

Change subject: ircecho: Remove support for sysvinit script
......................................................................


ircecho: Remove support for sysvinit script

No longer needed as we are on debian jessie+ which supports systemd.

Change-Id: Ic62fe63668ae815929016c4a5917136f730820fc
---
M modules/ircecho/manifests/init.pp
D modules/ircecho/templates/initscripts/ircecho.sysvinit.erb
2 files changed, 2 insertions(+), 145 deletions(-)

Approvals:
  Alexandros Kosiaris: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ircecho/manifests/init.pp 
b/modules/ircecho/manifests/init.pp
index bd7e9d4..b0b225f 100644
--- a/modules/ircecho/manifests/init.pp
+++ b/modules/ircecho/manifests/init.pp
@@ -33,10 +33,9 @@
         notify  => Service['ircecho'],
     }
 
-    base::service_unit { 'ircecho':
+    systemd::service { 'ircecho':
         ensure         => $ensure,
-        systemd        => systemd_template('ircecho'),
-        sysvinit       => sysvinit_template('ircecho'),
+        content        => systemd_template('ircecho'),
         require        => File['/usr/local/bin/ircecho'],
         service_params => {
             hasrestart => true,
diff --git a/modules/ircecho/templates/initscripts/ircecho.sysvinit.erb 
b/modules/ircecho/templates/initscripts/ircecho.sysvinit.erb
deleted file mode 100644
index 228834f..0000000
--- a/modules/ircecho/templates/initscripts/ircecho.sysvinit.erb
+++ /dev/null
@@ -1,142 +0,0 @@
-#! /bin/sh
-### BEGIN INIT INFO
-# Provides:          irc-echoer
-# Required-Start:    $remote_fs $syslog
-# Required-Stop:     $remote_fs $syslog
-# Default-Start:     2 3 4 5
-# Default-Stop:      0 1 6
-# Short-Description: Input to IRC echoer
-# Description:       Input to IRC echoer
-### END INIT INFO
-
-# Author: Ryan Lane <rl...@wikimedia.org>
-#
-# Do NOT "set -e"
-
-PATH=/sbin:/usr/sbin:/bin:/usr/bin
-DESC="Input to IRC echoer"
-DAEMON="/usr/local/bin/ircecho"
-USER="nobody"
-SCRIPTNAME="/etc/init.d/ircecho"
-NAME="ircecho"
-
-# Exit if the package is not installed
-[ -x "$DAEMON" ] || exit 0
-
-# Read configuration variable file if it is present
-[ -r /etc/default/$NAME ] && . /etc/default/$NAME
-
-# Load the VERBOSE setting and other rcS variables
-. /lib/init/vars.sh
-
-# Define LSB log_* functions.
-# Depend on lsb-base (>= 3.0-6) to ensure that this file is present.
-. /lib/lsb/init-functions
-
-#
-# Function that starts the daemon/service
-#
-do_start()
-{
-       # Return
-       #   0 if daemon has been started
-       #   1 if daemon was already running
-       #   2 if daemon could not be started
-       start-stop-daemon --start -m --pidfile=/var/run/ircecho.pid -b --quiet 
-c $USER --exec $DAEMON --test -- --infile=$INFILE $CHANS $NICK $SERVER > 
/dev/null \
-               || return 1
-       start-stop-daemon --start -m --pidfile=/var/run/ircecho.pid -b --quiet 
-c $USER --exec $DAEMON -- --infile=$INFILE $CHANS $NICK $SERVER \
-               || return 2
-       # Add code here, if necessary, that waits for the process to be ready
-       # to handle requests from services started subsequently which depend
-       # on this one.  As a last resort, sleep for some time.
-}
-
-#
-# Function that stops the daemon/service
-#
-do_stop()
-{
-       # Return
-       #   0 if daemon has been stopped
-       #   1 if daemon was already stopped
-       #   2 if daemon could not be stopped
-       #   other if a failure occurred
-       start-stop-daemon --stop --pidfile=/var/run/ircecho.pid --signal 9 
--quiet
-       RETVAL="$?"
-       [ "$RETVAL" = 2 ] && return 2
-       return "$RETVAL"
-}
-
-#
-# Function that sends a SIGHUP to the daemon/service
-#
-do_reload() {
-       #
-       # If the daemon can reload its configuration without
-       # restarting (for example, when it is sent a SIGHUP),
-       # then implement that here.
-       #
-       do_stop
-       do_start
-       return 0
-}
-
-case "$1" in
-  start)
-       [ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
-       do_start
-       case "$?" in
-               0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
-               2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
-       esac
-       ;;
-  stop)
-       [ "$VERBOSE" != no ] && log_daemon_msg "Stopping $DESC" "$NAME"
-       do_stop
-       case "$?" in
-               0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
-               2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
-       esac
-       ;;
-  status)
-       status_of_proc "$DAEMON" "$NAME" && exit 0 || exit $?
-       ;;
-  #reload|force-reload)
-       #
-       # If do_reload() is not implemented then leave this commented out
-       # and leave 'force-reload' as an alias for 'restart'.
-       #
-       #log_daemon_msg "Reloading $DESC" "$NAME"
-       #do_reload
-       #log_end_msg $?
-       #;;
-  restart|force-reload)
-       #
-       # If the "reload" option is implemented then remove the
-       # 'force-reload' alias
-       #
-       log_daemon_msg "Restarting $DESC" "$NAME"
-       do_stop
-       case "$?" in
-         0|1)
-               do_start
-               case "$?" in
-                       0) log_end_msg 0 ;;
-                       1) log_end_msg 1 ;; # Old process is still running
-                       *) log_end_msg 1 ;; # Failed to start
-               esac
-               ;;
-         *)
-               # Failed to stop
-               log_end_msg 1
-               ;;
-       esac
-       ;;
-  *)
-       #echo "Usage: $SCRIPTNAME {start|stop|restart|reload|force-reload}" >&2
-       echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
-       exit 3
-       ;;
-esac
-
-:

-- 
To view, visit https://gerrit.wikimedia.org/r/405703
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic62fe63668ae815929016c4a5917136f730820fc
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Paladox <thomasmulhall...@yahoo.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to