From edc9cd124dcf3d13cdf1989a221369efc7d45f40 Mon Sep 17 00:00:00 2001 From: Logan V Date: Thu, 19 Jan 2017 13:48:05 -0600 Subject: [PATCH] Fix fastest mirror fact set Aside from the ansible bug workaround not working, an issue was introduced by I5a018c88dde73564d15e15535d080adc5415e663 which causes the fastest mirror fact set output to render incorrectly. It should use stdout_lines instead of stdout, because running the "list" filter on stdout (a string) causes the string to be split into a char-by-char list of the string, rather than a single element list containing the whole string. This removes the call to the list filter and uses the list in stdout_lines that already exists. Unfortunately this does not work around the ansible bug but it is cleaner than reverting to the original fact set so I would prefer to fix the updated implementation instead of reverting I5a018c88dde73564d15e15535d080adc5415e663. Change-Id: Ic8eb28f83618f6caa8d5cf4094563f07e302a29f --- tests/roles/bootstrap-host/tasks/prepare_aio_config.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml b/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml index 4c66d91fbb..bb32b1e183 100644 --- a/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml +++ b/tests/roles/bootstrap-host/tasks/prepare_aio_config.yml @@ -91,7 +91,7 @@ - name: Set repo_build_pip_extra_indexes fact set_fact: - repo_build_pip_extra_indexes: "{{ fastest_wheel_mirror.stdout | list }}" + repo_build_pip_extra_indexes: "{{ fastest_wheel_mirror.stdout_lines }}" when: not pip_conf_file.stat.exists - name: Check whether the host has a git cache