airavata git commit: Moving composer update after permissions update

[ma...@apache.org]

Repository: airavata
Updated Branches:
  refs/heads/master f6f2adc5a -> da4a57069


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/master
Commit: da4a57069c14f2d579bc4db89ba6e26858dc473d
Parents: f6f2adc
Author: Marcus Christie <ma...@iu.edu>
Authored: Mon Jun 12 15:38:34 2017 -0400
Committer: Marcus Christie <ma...@iu.edu>
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 }}