Skip to content

Commit 403fbb1

Browse files
committed
Fix trueorfalse call in tempest use accounts check
The lib/tempest variable definition for TEMPEST_USE_TEST_ACCOUNTS was incorrectly calling the trueorfalse function by passing the variable's value to the function instead of the variable's name. This was causing trueorfalse the default value of false to always be returned even when specifying the option as true in the localrc. (well assuming True or it's variants wasn't an actual defined variable with a value that would return true) This commit fixes this issue by properly using the trueorfalse function. Change-Id: I8cefb58f49dcd2cb2def8a5071d0892af520e7f7
1 parent d66638f commit 403fbb1

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

lib/tempest

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ TEMPEST_HAS_ADMIN=$(trueorfalse True TEMPEST_HAS_ADMIN)
8989

9090
# Credential provider configuration option variables
9191
TEMPEST_ALLOW_TENANT_ISOLATION=${TEMPEST_ALLOW_TENANT_ISOLATION:-$TEMPEST_HAS_ADMIN}
92-
TEMPEST_USE_TEST_ACCOUNTS=$(trueorfalse False $TEMPEST_USE_TEST_ACCOUNTS)
92+
TEMPEST_USE_TEST_ACCOUNTS=$(trueorfalse False TEMPEST_USE_TEST_ACCOUNTS)
9393

9494
# The number of workers tempest is expected to be run with. This is used for
9595
# generating a accounts.yaml for running with test-accounts. This is also the

0 commit comments

Comments
 (0)