]> git.apps.os.sepia.ceph.com Git - teuthology.git/commit
teuthology: use next(iter(..)) for accessing first element in a view
authorKefu Chai <kchai@redhat.com>
Fri, 3 Apr 2020 11:24:16 +0000 (19:24 +0800)
committerKefu Chai <kchai@redhat.com>
Fri, 3 Apr 2020 11:28:45 +0000 (19:28 +0800)
commit1d1df8952f986461b0476871f47949b10afcdcbc
treedc9667d9cd9196e278203f70edc9b59f1ad91b43
parent8191bbbb461341ca5deb2182bd65a79e3d19d1a3
teuthology: use next(iter(..)) for accessing first element in a view

in python2, dict.values() and dict.keys() return lists. but in python3,
they return views, which cannot be indexed directly using an integer
index.

there are three use cases when we access these views in python3:

1. get the first element
2. get all the elements and then *might* want to access them by index
3. get the first element assuming there is only a single element in
   the view
4. iterate thru the view

in the 1st case, we cannot assume the number of elements, so to be
python3 compatible, we should use `next(iter(a_dict))` instead.

in the 2nd case, in this change, the view is materialized using
`list(a_dict)`.

in the 3rd case, we can just continue using the short hand of
```py
(first_element,) = a_dict.keys()
```
to unpack the view. this works in both python2 and python3.

in the 4th case, the existing code works in both python2 and python3, as
both list and view can be iterated using `iter`, and `len` works as
well.

Signed-off-by: Kefu Chai <kchai@redhat.com>
teuthology/lock/ops.py
teuthology/openstack/test/openstack-integration.py
teuthology/task/hadoop.py