]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mgr/telemetry: fix device id splitting when anonymizing serial 37302/head
authorYaarit Hatuka <yaarit@redhat.com>
Thu, 27 Aug 2020 03:04:34 +0000 (23:04 -0400)
committerKiefer Chang <kiefer.chang@suse.com>
Tue, 22 Sep 2020 07:33:09 +0000 (15:33 +0800)
commit4bdd4a819f8950fa2db7140452744cafb96a9433
treed0066b1ede7c6f8ee365499b46a14632f2b2e769
parentc0f11d9169f8dd691a9d9a5ba862bed7e312d259
mgr/telemetry: fix device id splitting when anonymizing serial

Anonymizing the serial number in the device id string fails in rare
cases where 'vendor' and 'model' are missing from the device id
string. Ideally, device id is generated (in blkdev.cc) as
'vendor_model_serial', in case all fields were successfully retrieved
from the device. In cases where they were not, device id can also be
generated as 'model_serial' or 'serial'. Splitting by '_' fails in the
latter case (since 'serial' is the only element in the string).

In order to anonymize serial numbers in smartctl reports we now rely
on the serial number value as retrieved from the raw smartctl report
itself (as opposed to the one in device id). That's in order to avoid
possible inconsistencies between the serial retrieved from device id and
the one in the report.

Fixes: https://tracker.ceph.com/issues/46977
Signed-off-by: Yaarit Hatuka <yaarit@redhat.com>
(cherry picked from commit e5099a7b58bcf39d80beb908c192c3bf639db1a4)
src/pybind/mgr/telemetry/module.py