diff mbox series

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

Message ID 20200925133905.54030-1-paolo.pisati@canonical.com
State New
Headers show
Series [autotest-client-tests] UBUNTU: SAUCE: ubuntu_qrt_apparmor: add python3-notify2 and python3-psutil deps on Groovy+ | expand

Commit Message

Paolo Pisati Sept. 25, 2020, 1:39 p.m. UTC
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@canonical.com>
---
 ubuntu_qrt_apparmor/ubuntu_qrt_apparmor.py | 2 ++
 1 file changed, 2 insertions(+)

Comments

Colin Ian King Sept. 25, 2020, 1:45 p.m. UTC | #1
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@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@canonical.com>
Po-Hsu Lin Sept. 28, 2020, 4:32 a.m. UTC | #2
Applied and pushed.

Thanks
Sam
diff mbox series

Patch

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)