Merge pull request #3291 from edx/jibsheet/tools-jenkins-cleanups

Jibsheet/tools jenkins cleanups

docker/build/tools_jenkins/Dockerfile

+FROM edxops/precise-common:latest
+MAINTAINER edxops
+
+USER root
+RUN apt-get update
+
+ADD . /edx/app/edx_ansible/edx_ansible
+WORKDIR /edx/app/edx_ansible/edx_ansible/docker/plays
+COPY docker/build/tools_jenkins/ansible_overrides.yml /
+RUN PYTHONUNBUFFERED=1 /edx/app/edx_ansible/venvs/edx_ansible/bin/ansible-playbook -v jenkins_tools.yml -i '127.0.0.1,' -c local -e@/ansible_overrides.yml -vv

docker/build/tools_jenkins/ansible_overrides.yml

+---
+DOCKER_TLD: "edx"
+

docker/plays/jenkins_tools.yml

+- name: Deploy the tools jenkins
+  hosts: all
+  sudo: True
+  gather_facts: True
+  vars:
+    serial_count: 1
+  serial: "{{ serial_count }}"
+  roles:
+    - common_vars
+    - docker
+    - tools_jenkins

playbooks/roles/tools_jenkins/tasks/main.yml

@@ -16,6 +16,7 @@
 - name: Get the list of jpi files
   shell: "ls {{ jenkins_home }}/plugins/*.jpi"
   register: jpi_files
+  ignore_errors: true
 
 - name: Set the permission on jpi files
   file: 
@@ -24,5 +25,6 @@
     group: "{{ jenkins_group }}"
     mode: 0644
   with_items: jpi_files.stdout_lines
+  when: jpi_files
   notify:
-    - restart Jenkins
\ No newline at end of file
+    - restart Jenkins