Merge pull request #84 from edx/jbau/minor-git-improvements

Jbau/minor git improvements

playbooks/roles/lms/tasks/main.yml

@@ -75,7 +75,7 @@
   - cms
 
 - name: git checkout edx-platform repo into $app_base_dir
-  git: dest={{platform_code_dir}} repo={{lms_source_repo}}
+  git: dest={{platform_code_dir}} repo={{lms_source_repo}} version={{lms_version}}
   environment:
     GIT_SSH: /tmp/git_ssh.sh
   tags:
@@ -83,6 +83,14 @@
   - cms
   - install
   
+- name: sets permissions on platform code dir
+  file: path={{platform_code_dir}} state=directory owner=www-data group=www-data mode=755
+  tags:
+  - lms
+  - cms
+  - install
+
+
 ## Install the debian package requirements system-wide
 - name: store remote apt_repos list for ansible use
   command: cat {{ apt_sources_file }}

playbooks/roles/lms/vars/main.yml

@@ -4,15 +4,16 @@
 lms_auth_config: {}
 lms_env_config: {}
 lms_source_repo: git@github.com:edx/edx-platform.git
+lms_version: 'HEAD'
 
 apt_sources_file:  "{{ platform_code_dir }}/requirements/system/{{os_name}}/apt-repos.txt"
 apt_packages_file: "{{ platform_code_dir }}/requirements/system/{{os_name}}/apt-packages.txt"
 
-local_requirements_file:  "{{ platform_code_dir }}/requirements/local.txt"
-post_requirements_file:   "{{ platform_code_dir }}/requirements/post.txt"
-base_requirements_file:   "{{ platform_code_dir }}/requirements/base.txt"
-github_requirements_file: "{{ platform_code_dir }}/requirements/github.txt"
-repo_requirements_file:   "{{ platform_code_dir }}/requirements/repo.txt"
+local_requirements_file:  "{{ platform_code_dir }}/requirements/edx/local.txt"
+post_requirements_file:   "{{ platform_code_dir }}/requirements/edx/post.txt"
+base_requirements_file:   "{{ platform_code_dir }}/requirements/edx/base.txt"
+github_requirements_file: "{{ platform_code_dir }}/requirements/edx/github.txt"
+repo_requirements_file:   "{{ platform_code_dir }}/requirements/edx/repo.txt"
 
 lms_debian_pkgs:
   - apparmor-utils