This review [1] is wrong, it builds on a wrong presumption that [2] does something it doesn't. Based on that [3] is just a workaround, but it doesn't fix the original issue.
The original issue is, that python-tempestconf doesn't take into account that tempest_roles might be a list, therefore this condition [4] is wrong.
Give me some time, I'll send a review to fix that within today.
This review [1] is wrong, it builds on a wrong presumption that [2] does something it doesn't. Based on that [3] is just a workaround, but it doesn't fix the original issue.
The original issue is, that python-tempestconf doesn't take into account that tempest_roles might be a list, therefore this condition [4] is wrong.
Give me some time, I'll send a review to fix that within today.
[1] https:/ /review. opendev. org/#/c/ 660036/ /review. opendev. org/#/c/ 638617/ /review. opendev. org/#/c/ 660649/ /opendev. org/openstack/ python- tempestconf/ src/commit/ e0d5dd6725f9a21 090cae69a12bf08 7004a17e13/ config_ tempest/ users.py# L76
[2] https:/
[3] https:/
[4] https:/