Skip to content

Commit 1a0452e

Browse files
committed
More elegant UUID extraction avoiding case distinction.
... based on suggestion by @mbuechse. Signed-off-by: Kurt Garloff <[email protected]>
1 parent 7a4dafb commit 1a0452e

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

Tests/iaas/key-manager/check-for-key-manager.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,11 +88,8 @@ def _delete_secret(conn: openstack.connection.Connection, secret: openstack.key_
8888
each time checking whether it was effective. But that's three delete
8989
plus list calls and very ugly.)
9090
"""
91-
uuid_part = secret.id.rfind('/') + 1
92-
if uuid_part != 0:
93-
conn.key_manager.delete_secret(secret.id[uuid_part:])
94-
else:
95-
conn.key_manager.delete_secret(secret.id)
91+
uuid = secret.id.rsplit('/', 1)[-1]
92+
conn.key_manager.delete_secret(uuid)
9693

9794

9895
def check_key_manager_permissions(conn: openstack.connection.Connection) -> None:

0 commit comments

Comments
 (0)