Skip to content

Commit ca2c74c

Browse files
[test_operator] Add default for horizontest admin credentials
Recently, the defaults for the admin_password and admin_username parameters were removed due to having out-of-date values. However, the current state of the code is leaving them undefined, which causes different errors. This PR aims to set the defaults to empty string to fix both issues at once. Signed-off-by: Katarina Strenkova <kstrenko@redhat.com>
1 parent d73b30a commit ca2c74c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

roles/test_operator/defaults/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -308,8 +308,8 @@ cifmw_test_operator_horizontest_config:
308308
storageClass: "{{ cifmw_test_operator_storage_class }}"
309309
privileged: "{{ cifmw_test_operator_privileged }}"
310310
containerImage: "{{ stage_vars_dict.cifmw_test_operator_horizontest_image }}:{{ stage_vars_dict.cifmw_test_operator_horizontest_image_tag }}"
311-
adminUsername: "{{ stage_vars_dict.cifmw_test_operator_horizontest_admin_username }}"
312-
adminPassword: "{{ stage_vars_dict.cifmw_test_operator_horizontest_admin_password | string }}"
311+
adminUsername: "{{ stage_vars_dict.cifmw_test_operator_horizontest_admin_username | default('') }}"
312+
adminPassword: "{{ stage_vars_dict.cifmw_test_operator_horizontest_admin_password | default('') | string }}"
313313
dashboardUrl: "{{ stage_vars_dict.cifmw_test_operator_horizontest_dashboard_url }}"
314314
authUrl: "{{ stage_vars_dict.cifmw_test_operator_horizontest_auth_url }}"
315315
repoUrl: "{{ stage_vars_dict.cifmw_test_operator_horizontest_repo_url }}"

0 commit comments

Comments
 (0)