Repository: airavata Updated Branches: refs/heads/develop f0f6b3404 -> 280068981
Moving composer update after permissions update Some files get created in app storage by the httpd server process and are owned by 'apache'. When that happens the composer update fails since it runs as user 'pga'. Moving the composer update after the permissions update, which gives 'pga' ownership of app storage files, fixes this problem. Project: http://git-wip-us.apache.org/repos/asf/airavata/repo Commit: http://git-wip-us.apache.org/repos/asf/airavata/commit/da4a5706 Tree: http://git-wip-us.apache.org/repos/asf/airavata/tree/da4a5706 Diff: http://git-wip-us.apache.org/repos/asf/airavata/diff/da4a5706 Branch: refs/heads/develop Commit: da4a57069c14f2d579bc4db89ba6e26858dc473d Parents: f6f2adc Author: Marcus Christie <[email protected]> Authored: Mon Jun 12 15:38:34 2017 -0400 Committer: Marcus Christie <[email protected]> Committed: Mon Jun 12 15:38:34 2017 -0400 ---------------------------------------------------------------------- dev-tools/ansible/roles/pga/tasks/main.yml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/airavata/blob/da4a5706/dev-tools/ansible/roles/pga/tasks/main.yml ---------------------------------------------------------------------- diff --git a/dev-tools/ansible/roles/pga/tasks/main.yml b/dev-tools/ansible/roles/pga/tasks/main.yml index f510b70..954fa0d 100644 --- a/dev-tools/ansible/roles/pga/tasks/main.yml +++ b/dev-tools/ansible/roles/pga/tasks/main.yml @@ -86,11 +86,6 @@ - restart httpd when: ansible_os_family == "RedHat" -- name: Run composer update - composer: command=update working_dir="{{ doc_root_dir }}" - become: yes - become_user: "{{user}}" - # step 6: Change pga configurations - name: Copy pga config file template: src=pga_config.php.j2 dest="{{ doc_root_dir }}/app/config/pga_config.php" owner="{{user}}" group="{{group}}" @@ -114,6 +109,11 @@ file: path="{{ doc_root_dir }}/app/storage" state=directory mode=0777 owner="{{user}}" group="{{group}}" recurse=yes become: yes +- name: Run composer update + composer: command=update working_dir="{{ doc_root_dir }}" + become: yes + become_user: "{{user}}" + # For SELinux, need to apply file contexts, but I'm not sure why since the contexts were created before the files/directories - name: run restorecon on those directories command: restorecon -R {{ doc_root_dir }} {{ user_data_dir }}
