From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] squid init v3.1: Bugfix release
Date: Mon, 16 May 2016 16:14:56 +0100
Message-ID: <1463411696.18591.283.camel@ipfire.org>
In-Reply-To: <1463408471-1292-1-git-send-email-matthias.fischer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2645011303854712368=="
List-Id: <development.lists.ipfire.org>

--===============2645011303854712368==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Don't comment anything out. Either we need a line or we don't.

On Mon, 2016-05-16 at 16:21 +0200, Matthias Fischer wrote:
> - Fixed and adjusted non-working 'while'-loop for stopping still running
>   '(squid-1)'-processes.
> 
> - Commented 'wait' after 'killproc ...'-block: until now, no troubles found.
> 
> - Cosmetic tuning.
> 
> Best,
> Matthias
> 
> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
> ---
>  src/initscripts/init.d/squid | 21 ++++++++++++++-------
>  1 file changed, 14 insertions(+), 7 deletions(-)
> 
> diff --git a/src/initscripts/init.d/squid b/src/initscripts/init.d/squid
> index 47bf182..e5c9b85 100644
> --- a/src/initscripts/init.d/squid
> +++ b/src/initscripts/init.d/squid
> @@ -95,9 +95,9 @@ case "$1" in
>  			iptables -t nat -F SQUID
>  		if [ -e /var/run/squid.pid ]; then
>  			boot_mesg -n "Shutting down Squid Proxy Server...\n"
> -			boot_mesg "(this may take up to a few minutes)."
> +			boot_mesg "(this may take up to a few minutes)"
>  			/usr/sbin/squid -k shutdown >/dev/null 2>&1
> -			evaluate_retval
> +#			evaluate_retval
>  
>  			# Stop squidGuard, updxlrator, squidclamav
>  			# and redirect_wrappers.
> @@ -107,31 +107,38 @@ case "$1" in
>  			killproc /usr/sbin/redirect_wrapper >/dev/null &
>  
>  			# Wait until all redirectors have been stopped.
> -			wait
> +#			wait
>  
>  			# If some squid processes are still running, wait up
> to 360 seconds
>  			# before we go on to kill all and delete entire cache
> structure.
> +
>  			n=0
> -			while [ /usr/sbin/squid -k check > /dev/null 2>&1 ]
> && [ $n -lt 360 ]; do
> -				sleep 1
> -				n=$(( ${n} + 1 ))
> +			while /usr/sbin/squid -k check > /dev/null 2>&1 && [
> $n -lt 360 ]; do
> +				sleep 2
> +				n=$(( ${n} + 2 ))
> +				echo -n .
>  			done
> +			echo ""
> +			echo "Done."
> +			echo "Shutdown time:" "$n" "seconds"
>  
>  			# If (squid-1) is still running after 360 seconds,
>  			# kill all squid processes and delete damaged cache
> structure.
>  			if ( pgrep -fl "(squid-1)" > /dev/null 2>&1 ); then
>  				killproc /usr/sbin/squid >/dev/null
>  				rm -rf /var/log/cache/*
> +				echo ""
>  				boot_mesg -n "You should not be reading this
> warning.\n"
>  				boot_mesg -n "Some squid-processes had to be
> killed after 360 seconds,\n"
>  				boot_mesg -n "so index file and cache
> contents had to be deleted.\n"
>  				boot_mesg -n "Therefore, the complete cache
> structure will be rebuild\n"
>  				boot_mesg "during next start."
>  				echo_warning
> +				echo ""
>  			else
>  				boot_mesg "All squid processes exited
> normally."
>  				echo_ok
> -				boot_mesg ""
> +				echo ""
>  			fi
>  		fi
>  

--===============2645011303854712368==
Content-Type: application/pgp-signature
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="signature.asc"
MIME-Version: 1.0

LS0tLS1CRUdJTiBQR1AgU0lHTkFUVVJFLS0tLS0KVmVyc2lvbjogR251UEcgdjIKCmlRSWNCQUFC
Q2dBR0JRSlhPZVB4QUFvSkVJQjU4UDl2a0FrSEZvY1FBSmNDSEdhRWZBV0QrbzlxQWY4b05BZEwK
RDBKZTRsL1lmbEdPalp2a1ZDdmZCUFhjcmpYbHhzTUdTSFBKYWQzR0lSK0s1Y1RRbWxqeTBhWjNK
V1Y3V1ZlNwpYaFNxVjg4NncyU1RrSHZuSVh2TEFLaXFkcjd1K3VCdGtKcFVXSmY4alprSXZoT3Jx
TlZjY2ZTWGV0TEN3YlZQCmFDUCtmVDROaFNKR2VReWlNMWxFdUR4NzhPeGR1cE1LaTlwUlpPUjVJ
SFBrN0hxVVpJeXA1VGlGYWRGdDlOWisKYWFQSTYrTnRFUVdSRlJoWVNSbjE5NE5iVXpTZUMwWnVP
QTN6UFMreGRhUFpscDBIOC91cXVtaDlQVUYxd0hHMwpNOXBsbHhDZFVEdWc1ZEhVbjhzVGFRMnpn
MTlXOUpqS2VFc0RMTFVxRkVoSUlibko0YXRINzhueFJWdmFxZXNCCjZ4dGo2ejF0QzA2cHVNMHpa
Z0VxS00yNlQxMWlLYUlXcUMvbjlPa1BVU0N3NnpPaDFpNnhZcm9JME92YnhzbzcKWm9EUGsycFA3
alNvV2g4ZTc4WTl6dzFQRi8vS3lKRUFMMVFQYWx4dGxiVDdQTGxpTlFHdktFUzJJaCtnMXBmdAp2
T3FCcU05ZEllRXVsWWdlV0IreWx0SjhvczllM3pWOFh6OFd6RmVuMnRCSkNpU3c3YVZKS2o4UGhs
dVlkOTdXCi9nTkxMTUNIM0d0eGRKeWNYYWZUc25FRFBEQUV2UjZrYlQ1T09tN2szN3g5Tk9JLysv
b1gvd2tCQ2NhNk9ub0gKR1c1b0g4dk41a1hlaDVRMVJYdFlib0RFblBIbGZlWFIxZTlJODM0ZlVi
ODhKL1hBcTRtVTdtanRhK0x0OWhMNgo0aHRRYnB6RDlVbEQ1UDZOM05MOAo9MUFLcgotLS0tLUVO
RCBQR1AgU0lHTkFUVVJFLS0tLS0K

--===============2645011303854712368==--