diff options
| author | Tony Asleson <tasleson@redhat.com> | 2019-10-14 14:33:16 -0500 |
|---|---|---|
| committer | Tony Asleson <tasleson@redhat.com> | 2019-10-30 10:38:40 -0500 |
| commit | 508d1808b07f204ca0ce71d4d505ded82f30d0ce (patch) | |
| tree | 28e82560b95c0deaf74a3607b35a4bf6d629364a | |
| parent | f91df163e21b283e652bc830b5af49a7bb3cf3fb (diff) | |
| download | lvm2-508d1808b07f204ca0ce71d4d505ded82f30d0ce.tar.gz | |
lvmdbustest.py: Use local data instead of fetching
Avoid making more dbus calls to get information we already have. This
also avoids us getting an error where a dbus object representation is
being deleted by another process while we are trying to gather information
about it across the wire.
| -rwxr-xr-x | test/dbus/lvmdbustest.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/test/dbus/lvmdbustest.py b/test/dbus/lvmdbustest.py index 711b2476b..522e3ca13 100755 --- a/test/dbus/lvmdbustest.py +++ b/test/dbus/lvmdbustest.py @@ -88,8 +88,12 @@ def _root_pv_name(res, pv_name): vg_name = pv_name.split('/')[2] for v in res[VG_INT]: if v.Vg.Name == vg_name: - pv = ClientProxy(bus, v.Vg.Pvs[0], interfaces=(PV_INT, )) - return _root_pv_name(res, pv.Pv.Name) + for pv in res[PV_INT]: + if pv.object_path in v.Vg.Pvs: + return _root_pv_name(res, pv.Pv.Name) + return None + + def _prune_lvs(res, interface, vg_object_path): lvs = [lv for lv in res[interface] if lv.LvCommon.Vg == vg_object_path] res[interface] = lvs |