skip some tasks when the edx-platfor checkout hasn't changed

playbooks/roles/edxapp/tasks/deploy.yml

@@ -30,14 +30,14 @@
 # Do A Checkout
 - name: edxapp | checkout edx-platform repo into {{edxapp_code_dir}}
   git: dest={{edxapp_code_dir}} repo={{edx_platform_repo}} version={{edx_platform_commit}}
-  register: edx_platform_checkout
+  register: chkout
   sudo_user: "{{ edxapp_user }}"
   tags:
   - deploy
 
 - name: edxapp | git clean after checking out edx-platform
   shell: cd {{edxapp_code_dir}} && git clean -xdf
-  when: edx_platform_checkout.changed
+  when: chkout.changed
   sudo_user: "{{ edxapp_user }}"
   tags:
   - deploy
@@ -55,6 +55,7 @@
     /usr/bin/md5sum {{ " ".join(edxapp_all_req_files) }} 2>/dev/null > /var/tmp/edxapp.req.new
   sudo_user: "{{ edxapp_user }}"
   ignore_errors: true
+  when: chkout.changed
   tags:
   - deploy
 
@@ -72,7 +73,7 @@
     /bin/sed -i -e 's/github\.com/{{ COMMON_GIT_MIRROR }}/g' {{ item }}
   with_items: edxapp_all_req_files
   sudo_user: "{{ edxapp_user }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 
@@ -84,7 +85,7 @@
       executable=/bin/bash
   environment: "{{ edxapp_environment }}"
   sudo_user: "{{ edxapp_user }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 - name: edxapp | bundle install
@@ -94,7 +95,7 @@
       executable=/bin/bash
   sudo_user: "{{ edxapp_user }}"
   environment: "{{ edxapp_environment }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 # Node play that need to be run after platform updates.
@@ -102,7 +103,7 @@
   shell: npm install chdir={{ edxapp_code_dir }}
   sudo_user: "{{ edxapp_user }}"
   environment: "{{ edxapp_environment }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 
@@ -115,7 +116,7 @@
     extra_args="-i {{ edxapp_pypi_local_mirror }}"
   sudo_user: "{{ edxapp_user }}"
   environment: "{{ edxapp_environment }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 # Install the python modules into {{ edxapp_venv_dir }}
@@ -128,7 +129,7 @@
     chdir={{ edxapp_code_dir }}
   environment: "{{ edxapp_environment }}"
   sudo_user: "{{ edxapp_user }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 # Install the python post requirements into {{ edxapp_venv_dir }}
@@ -140,7 +141,7 @@
     extra_args="-i {{ edxapp_pypi_local_mirror }}"
   sudo_user: "{{ edxapp_user }}"
   environment: "{{ edxapp_environment }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 # Install the final python modules into {{ edxapp_venv_dir }}
@@ -154,7 +155,7 @@
   - "{{ github_requirements_file }}"
   - "{{ local_requirements_file }}"
   sudo_user: "{{ edxapp_user }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 
@@ -170,12 +171,13 @@
   - "{{ sandbox_post_requirements }}"
   when: install_sandbox_reqs_into_regular_venv
   sudo_user: "{{ edxapp_user }}"
-  when: not inst.stat.exists or new.stat.md5 != inst.stat.md5
+  when: chkout.changed and not inst.stat.exists or new.stat.md5 != inst.stat.md5
   tags: deploy
 
 - name: edxapp | create checksum for installed requirements
   shell: cp /var/tmp/edxapp.req.new /var/tmp/edxapp.req.installed
   sudo_user: "{{ edxapp_user }}"
+  when: chkout.changed
   tags: deploy
 
 

playbooks/roles/edxapp/tasks/service_variant_config.yml

@@ -59,7 +59,7 @@
     executable=/bin/bash
     chdir={{ edxapp_code_dir }}
   sudo_user: "{{ edxapp_user }}"
-  when: celery_worker is not defined
+  when: chkout.changed and celery_worker is not defined
   with_items: service_variants_enabled
   environment: "{{ edxapp_environment }}"
   tags:
@@ -68,13 +68,13 @@
 
 - name: edxapp | syncdb and migrate
   shell: sudo -u {{ edxapp_user }} SERVICE_VARIANT=lms {{ edxapp_venv_bin}}/django-admin.py syncdb --migrate --noinput --settings=lms.envs.aws --pythonpath={{ edxapp_code_dir }}
-  when: migrate_db is defined and migrate_db|lower == "yes"
+  when: chkout and migrate_db is defined and migrate_db|lower == "yes"
   tags:
   - deploy
 
 - name: edxapp | db migrate
   shell: sudo -u {{ edxapp_user }} SERVICE_VARIANT=lms {{ edxapp_venv_bin }}/django-admin.py migrate --noinput --settings=lms.envs.aws --pythonpath={{ edxapp_code_dir }}
-  when: migrate_only is defined and migrate_only|lower == "yes"
+  when: chkout.changed and migrate_only is defined and migrate_only|lower == "yes"
   tags:
   - deploy