ACK: [PATCH] [autotest-client-tests] UBUNTU: SAUCE: ubuntu_qrt_apparmor: add python3-notify2 and python3-psutil deps on Groovy+

[Colin Ian King]

On 25/09/2020 14:39, Paolo Pisati wrote:
> Add a couple of missing python3 dependencies:
> 
> ...
> 08:49:13 ERROR| [stderr] Traceback (most recent call last):
> 08:49:13 ERROR| [stderr] File "../aa-notify", line 38, in <module>
> 08:49:13 ERROR| [stderr] import notify2
> 08:49:13 ERROR| [stderr] ModuleNotFoundError: No module named 'notify2'
> ...
> 
> and
> 
> ...
> 09:07:34 ERROR| [stderr] import psutils
> 09:07:34 ERROR| [stderr] ModuleNotFoundError: No module named 'psutils'
> ...
> 
> Signed-off-by: Paolo Pisati <paolo.pisati at canonical.com>
> ---
>  ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py b/ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py
> index f735ef47..58e5e993 100644
> --- a/ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py
> +++ b/ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py
> @@ -52,6 +52,8 @@ class ubuntu_qrt_apparmor(test.test):
>          else:
>              pkgs.append('pyflakes3')
>              pkgs.append('python3-pexpect')
> +            pkgs.append('python3-notify2')
> +            pkgs.append('python3-psutil')
>  
>          cmd = 'DEBIAN_FRONTEND=noninteractive apt-get install --yes --force-yes ' + ' '.join(pkgs)
>          self.results = utils.system_output(cmd, retain_output=True)
> 

This non-[precise|trusty|xenial|bionic|focal] code path looks OK to me.

Thanks Paolo.

Acked-by: Colin Ian King <colin.king at canonical.com>