From f0b18841041fcf1c704221472934308b8154847d Mon Sep 17 00:00:00 2001 From: Aleksandr Didenko Date: Tue, 22 Nov 2016 17:58:46 +0100 Subject: [PATCH] Set defaults for ansible_ssh_user When setting permission for containers download/upload dir we're using `ansible_ssh_user`. But if playbook is executed without user being explicitly set `ansible_ssh_user` may be undefined. In such situations dir ownership will default to `ansible_user_id` Closes: #644 --- roles/download/tasks/main.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/download/tasks/main.yml b/roles/download/tasks/main.yml index 66e635935..273e6817c 100644 --- a/roles/download/tasks/main.yml +++ b/roles/download/tasks/main.yml @@ -47,7 +47,7 @@ download_delegate: "{% if download_localhost %}localhost{% else %}{{groups['kube-master'][0]}}{% endif %}" - name: Create dest directory for saved/loaded container images - file: path="{{local_release_dir}}/containers" state=directory recurse=yes mode=0755 owner={{ansible_ssh_user}} + file: path="{{local_release_dir}}/containers" state=directory recurse=yes mode=0755 owner={{ansible_ssh_user|default(ansible_user_id)}} when: "{{ download.enabled|bool and download.container|bool }}" # This is required for the download_localhost delegate to work smooth with CoreOS cluster nodes