Fixes

playbooks/roles/edx_notes_api/defaults/main.yml

@@ -21,17 +21,16 @@ EDX_NOTES_API_MYSQL_DB_NAME: "{{ EDX_NOTES_API_DATASTORE_NAME }}"
 EDX_NOTES_API_MYSQL_DB_PASS: secret
 EDX_NOTES_API_MYSQL_HOST: localhost
 EDX_NOTES_API_ELASTICSEARCH_URL: http://127.0.0.1:9200
-EDX_NOTES_API_NEWRELIC_APPNAME: "{{ COMMON_ENVIRONMENT }}_{{ COMMON_DEPLOYMENT }}_notes"
+EDX_NOTES_API_NEWRELIC_APPNAME: "{{ COMMON_ENVIRONMENT }}-{{ COMMON_DEPLOYMENT }}-notes"
 # Change these values!!
 EDX_NOTES_API_SECRET_KEY: "i^,9%i=e=y/Nlpb=Mkx!j&,HD9d/17F][9P,FLdwM2+G6|]BEU"
-<<<<<<< HEAD
+EDX_NOTES_API_CLIENT_ID: edx_notes_id
+EDX_NOTES_API_CLIENT_SECRET: edx_notes_secret
 EDX_NOTES_API_CLIENT_ID: edx-notes-id
 EDX_NOTES_API_CLIENT_SECRET: edx-notes-secret
-EDX_NOTES_API_GIT_SSH_KEY: !!null
-=======
 EDX_NOTES_API_CLIENT_ID: edx_notes_id
 EDX_NOTES_API_CLIENT_SECRET: edx_notes_secret
->>>>>>> Potentially risky dashes to underscores
+EDX_NOTES_API_GIT_SSH_KEY: !null
 
 EDX_NOTES_API_DATABASES:
   # rw user
@@ -46,12 +45,13 @@ EDX_NOTES_API_ALLOWED_HOSTS:
   - localhost
 
 EDX_NOTES_API_REPOS:
-  - PROTOCOL: "{{COMMON_GIT_PROTOCOL}}"
-    DOMAIN: "{{COMMON_GIT_MIRROR}}"
-    PATH: "{{COMMON_GIT_PATH}}"
-    REPO: "{{ edx_notes_api_service_name }}.git"
-    VERSION: e0d/migration-overrides
-    DESTINATION: "{{edx_notes_api_code_dir}}"
+  # - PROTOCOL: "{{ COMMON_GIT_PROTOCOL }}"
+  - PROTOCOL: ssh
+    DOMAIN: "{{ COMMON_GIT_MIRROR }}"
+    PATH: "{{ COMMON_GIT_PATH }}"
+    REPO: edx-notes-api.git
+    VERSION: max/dashes2underscores
+    DESTINATION: "{{ edx_notes_api_code_dir }}"
     SSH_KEY: "{{ EDX_NOTES_API_GIT_SSH_KEY }}"
 
 #
@@ -85,9 +85,9 @@ edx_notes_api_service_config:
 # vars are namespace with the module name.
 #
 edx_notes_api_service_name: edx_notes_api
-edx_notes_api_app_dir: "{{ COMMON_APP_DIR }}/{{ edx_notes_api_service_name }}"
+edx_notes_api_user: "{{ edx_notes_api_service_name }}"
 edx_notes_api_home: "{{ COMMON_APP_DIR }}/{{ edx_notes_api_service_name }}"
-edx_notes_api_python_path: "{{ edx_notes_api_code_dir }}/{{ edx_notes_api_service_name }}"
+edx_notes_api_code_dir: "{{ edx_notes_api_home }}/{{ edx_notes_api_service_name }}"
 edx_notes_api_conf_dir: "{{ edx_notes_api_home }}"
 
 edx_notes_api_gunicorn_host: "127.0.0.1"

playbooks/roles/edx_notes_api/meta/main.yml

@@ -23,7 +23,7 @@ dependencies:
   - role: edx_service
     edx_service_name: "{{ edx_notes_api_service_name }}"
     edx_service_config: "{{ edx_notes_api_service_config }}"
-    edx_service_repos: "{{ edx_notes_api_repos }}"
+    edx_service_repos: "{{ EDX_NOTES_API_REPOS }}"
     edx_service_user: "{{ edx_notes_api_user }}"
     edx_service_home: "{{ edx_notes_api_home }}"
   - supervisor

playbooks/roles/edx_notes_api/tasks/main.yml

@@ -47,8 +47,8 @@
 - name: install application requirements
   pip: >
     requirements="{{ edx_notes_api_requirements_base }}/{{ item }}"
-    virtualenv="{{ edx_notes_api_home }}/venvs" state=present
-    extra_args="--exists-action w"
+    virtualenv="{{ edx_notes_api_home }}/venvs/{{ edx_notes_api_service_name }}"
+    state=present extra_args="--exists-action w"
   sudo_user: "{{ edx_notes_api_user }}"
   with_items: edx_notes_api_requirements
 
@@ -65,8 +65,8 @@
 
 - name: write out gunicorn.py
   template: >
-    src=edx/app/edx-notes-api/{{ edx_notes_api_service_name }}-gunicorn.py.j2
-    dest={{ edx_notes_api_app_dir }}/{{ edx_notes_api_service_name }}-gunicorn.py
+    src=edx/app/edx_notes_api/edx_notes_api_gunicorn.py.j2
+    dest={{ edx_notes_api_home }}/{{ edx_notes_api_service_name }}_gunicorn.py
     mode=0650 owner={{ supervisor_user }} group={{ common_web_user }}
   tags:
     - deploy
@@ -108,7 +108,7 @@
 - name: create symlinks from the venv bin dir
   file: >
     src="{{ edx_notes_api_home }}/venvs/{{ edx_notes_api_service_name }}/bin/{{ item }}"
-    dest="{{ COMMON_BIN_DIR }}/{{ item.split('.')[0] }}.{{ edx_notes_api_role_name }}"
+    dest="{{ COMMON_BIN_DIR }}/{{ item.split('.')[0] }}.{{ edx_notes_api_service_name }}"
     state=link
   with_items:
   - python

playbooks/roles/edx_notes_api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2

 [program:{{ edx_notes_api_service_name }}]
 
 {% if COMMON_ENABLE_NEWRELIC_APP %}
+<<<<<<< Updated upstream:playbooks/roles/edx_notes_api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
+{% set edx_notes_api_venv_bin = edx_notes_api_home/venvs/edx_notes_api_service_name/bin %}
+=======
+<<<<<<< Updated upstream:playbooks/roles/edx-notes-api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
+=======
+{% set edx_notes_api_venv_bin = edx_notes_api_home + '/venvs/' + edx_notes_api_service_name + '/bin' %}
+>>>>>>> Stashed changes:playbooks/roles/edx_notes_api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
+>>>>>>> Stashed changes:playbooks/roles/edx-notes-api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
 {% set executable = edx_notes_api_venv_bin + '/newrelic-admin run-program ' + edx_notes_api_venv_bin + '/gunicorn' %}
 {% else %}
 {% set executable = edx_notes_api_venv_bin + '/gunicorn' %}
 {% endif %}
 
+<<<<<<< Updated upstream:playbooks/roles/edx_notes_api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
+command={{ executable }} -c {{ edx_notes_api_home }}/edx-notes-api-gunicorn.py {{ EDX_NOTES_API_GUNICORN_WORKERS_EXTRA }} {{ edx_notes_api_wsgi }}
+=======
+<<<<<<< Updated upstream:playbooks/roles/edx-notes-api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
 command={{ executable }} -c {{ edx_notes_api_app_dir }}/edx-notes-api-gunicorn.py {{ EDX_NOTES_API_GUNICORN_WORKERS_EXTRA }} {{ edx_notes_api_wsgi }}
+=======
+command={{ executable }} -c {{ edx_notes_api_home }}/edx_notes_api_gunicorn.py {{ EDX_NOTES_API_GUNICORN_WORKERS_EXTRA }} {{ edx_notes_api_wsgi }}
+>>>>>>> Stashed changes:playbooks/roles/edx_notes_api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
+>>>>>>> Stashed changes:playbooks/roles/edx-notes-api/templates/edx/app/supervisor/conf.d.available/edx_notes_api.conf.j2
 
 user={{ common_web_user }}
 directory={{ edx_notes_api_code_dir }}

playbooks/roles/edx_service/tasks/main.yml

@@ -23,6 +23,7 @@
 # file.  The including role should add the following
 # depency definition.
 #
+# TODO: FIX THIS
 # dependencies:
 #   - { role: edx_service, edx_service_name: "hotg" }
 #
@@ -81,22 +82,6 @@
   when: ansible_distribution in common_redhat_variants
 
 
-- name: check if git repo exists before pruning
-  stat: path={{ item.DESTINATION }}/.git
-  register: git_dir_exists
-  with_items: edx_service_repos
-
-- name: git prune before checking out
-  shell: cd {{ item.0.DESTINATION }} && git remote prune origin
-  sudo_user: "{{ edx_service_name }}"
-  when: >
-    (item.1.stat.exists) and 
-    (item.1.stat.isdir is defined) and
-    item.1.stat.isdir
-  with_together:
-    - edx_service_repos
-    - git_dir_exists.results
-
 - name: validate git protocol
   fail: msg='REPOS.PROTOCOL must be "https" or "ssh"'
   when: (item.PROTOCOL != "https") and (item.PROTOCOL != "ssh")
@@ -104,7 +89,7 @@
 
 - name: install read-only ssh key
   copy: >
-    dest="{{ edx_service_home }}.ssh/{{ item.REPO }}"
+    dest="{{ edx_service_home }}/.ssh/{{ item.REPO }}"
     content="{{ item.SSH_KEY }}" owner={{ edx_service_user }}
     group={{ edx_service_user }} mode=0600
   when: item.PROTOCOL == "ssh"
@@ -113,9 +98,9 @@
 - name: checkout code over ssh
   git: >
     repo=git@{{ item.DOMAIN }}:{{ item.PATH }}/{{ item.REPO }}
-    dest={{ item.DESTINATION }}  version={{ item.VERSION }}
-    accept_hostkey=yes key_file={{ edx_service_home }}.ssh/{{ item.REPO }}
-  sudo_user: "{{ edx_role_name }}"
+    dest={{ item.DESTINATION }}  version={{ item.VERSION }} depth=1
+    accept_hostkey=yes key_file={{ edx_service_home }}/.ssh/{{ item.REPO }}
+  sudo_user: "{{ edx_service_user }}"
   register: code_checkout
   when: item.PROTOCOL == "ssh"
   with_items: edx_service_repos
@@ -124,7 +109,7 @@
   git: >
     repo=https://{{ item.DOMAIN }}/{{ item.PATH }}/{{ item.REPO }}
     dest={{ item.DESTINATION }}  version={{ item.VERSION }}
-  sudo_user: "{{ edx_role_name }}"
+  sudo_user: "{{ edx_service_user }}"
   register: code_checkout
   when: item.PROTOCOL == "https"
   with_items: edx_service_repos

playbooks/roles/nginx/templates/edx/app/nginx/sites-available/edx-notes-api.j2 → playbooks/roles/nginx/templates/edx/app/nginx/sites-available/edx_notes_api.j2

-upstream {{ edx_notes_api_role_name_safe }}_app_server {
+upstream {{ edx_notes_api_service_name }}_app_server {
     {% for host in nginx_edx_notes_api_gunicorn_hosts %}
         server {{ host }}:{{ edx_notes_api_gunicorn_port }} fail_timeout=0;
     {% endfor %}
@@ -20,7 +20,7 @@ location @proxy_to_app {
     proxy_set_header Host $http_host;
 
     proxy_redirect off;
-    proxy_pass http://{{ edx_notes_api_role_name_safe }}_app_server;
+    proxy_pass http://{{ edx_notes_api_service_name }}_app_server;
   }
 }