New role 'capture-performance-data' failing when pymysql is not installed

[Fernando Royo]

New role 'capture-performance-data' is failing on that jobs where pymysql module is not installed, reporting following error:

TASK [capture-performance-data : Generate statistics]
controller | Traceback (most recent call last):
controller | File "/opt/stack/devstack//tools/get-stats.py", line 14, in <module>
controller | import pymysql
controller | ModuleNotFoundError: No module named 'pymysql'

Some example of jobs error are located in [1] or [2]

[1] https://zuul.opendev.org/t/openstack/build/00701df174b04f4286700ee59865ca84
[2] https://zuul.opendev.org/t/openstack/build/2c567ef1f7bf4b89a8de41dbf8073a32

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/devstack/+/839193

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/devstack/+/839217

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/devstack/+/839217
Committed: https://opendev.org/openstack/devstack/commit/1b601c7b1e8a3ec4816cb827ccd8bf909a05debb
Submitter: "Zuul (22348)"
Branch: master

commit 1b601c7b1e8a3ec4816cb827ccd8bf909a05debb
Author: Dan Smith <email address hidden>
Date: Mon Apr 25 07:47:56 2022 -0700

    Tolerate missing deps in get-stats.py

    In order to run on systems where not all requirements are present,
    we should be tolerant of missing external dependencies, such as
    psutil and pymysql. Print a warning (to stderr) and just leave out
    those stats in that case.

    Also make running the stats collector use ignore_errors:yes to avoid
    failures in the future. I think the stats is not critical enough to
    fail a job for bugs like this.

    Related-Bug: #1970195
    Change-Id: I132b0e1f5033c4f109a8b8cc776c0877574c4a49

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Fernando Royo <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/devstack/+/839193
Reason: Cover by https://review.opendev.org/c/openstack/devstack/+/839217

[Martin Kopec (mkopec)]

It's been decided to tolerate the missing dep by https://review.opendev.org/c/openstack/devstack/+/839217