Re: [PATCH v2] dpkg-base: Warn about unmounting problems

[Jan Kiszka <jan.kiszka@siemens.com>]

On 02.10.19 12:56, Baurzhan Ismagulov wrote:
> From: Baurzhan Ismagulov <ibr@ilbers.de>
> 
> If a problem occurs, log umount's error messages. Wait for the processes to
> terminate and retry. If that doesn't help, warn the user about retrying
> every 10 s to avoid clobbering CI logs.
> 
> Signed-off-by: Baurzhan Ismagulov <ibr@ilbers.de>
> ---
> Changes in v2:
> - Warn after some time.
> 
>   meta/classes/dpkg-base.bbclass | 9 ++++++++-
>   1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/classes/dpkg-base.bbclass b/meta/classes/dpkg-base.bbclass
> index 2e8a0d1..006d8b1 100644
> --- a/meta/classes/dpkg-base.bbclass
> +++ b/meta/classes/dpkg-base.bbclass
> @@ -1,5 +1,6 @@
>   # This software is a part of ISAR.
>   # Copyright (C) 2017-2019 Siemens AG
> +# Copyright (C) 2019 ilbers GmbH
>   #
>   # SPDX-License-Identifier: MIT
>   
> @@ -78,8 +79,14 @@ dpkg_do_mounts() {
>   }
>   
>   dpkg_undo_mounts() {
> -    while ! sudo umount ${BUILDROOT} 2>/dev/null; do
> +    i=1
> +    while ! sudo umount ${BUILDROOT}; do
>           sleep 0.1
> +        i=`expr $i + 1`
> +        if [ $i -gt 100 ]; then
> +            bbwarn "${BUILDROOT}: Couldn't unmount, retrying..."
> +            i=1
> +        fi
>       done
>       sudo rmdir ${BUILDROOT}
>   }
> 

Looks good to me.

Thanks,
Jan

-- 
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux