Merge pull request #132 from l1ll1/master

separate making of files systes from exporting so we can reuse make_f…

roles/nfs-server/tasks/main.yml

@@ -4,5 +4,4 @@
   sudo: true
   when: ansible_os_family == "Debian"
 
-- include: mkFilesystem.yml 
 - include: startServer.yml

roles/nfs-server/tasks/startServer.yml

 --- 
 - name: "Starting rpcbind"
-  service: "name=rpcbind state=restarted"
+  service: "name=rpcbind state=started"
   sudo: true
   when: ansible_os_family == "RedHat"
 
 - name: "Start the Server"
-  service: "name=nfs state=restarted"
+  service: "name=nfs state=started"
   sudo: true
   when: ansible_os_family == "RedHat"
 
 - name: "Start the Server"
-  service: "name=nfs-kernel-server state=restarted"
+  service: "name=nfs-kernel-server state=started"
   sudo: true
   when: ansible_os_family == "Debian"

roles/syncExports/tasks/addExports.yml

 ---
-- name: "Create exports if necessary"
-  file: dest={{ item.src }} state=directory mode=755 owner=root group=root
-  sudo: true
-  with_items: exportList
-
 - name: "Templating /etc/exports"
   copy: src=files/etcExports dest=/etc/exports owner=root group=root mode=644
   sudo: true
+  register: exports
 
 - name: "Start the Server"
   service: "name=nfs state=restarted"
   sudo: true
-  when: ansible_os_family == "RedHat"
+  when: ansible_os_family == "RedHat" and exports.changed
 
 - name: "Start the Server"
   service: "name=nfs-kernel-server state=restarted"
   sudo: true
-  when: ansible_os_family == "Debian"
+  when: ansible_os_family == "Debian" and exports.changed
 
 - name : "Pause ... clients sometimes have errors"
   command: sleep 60
   notify: "Reload exports"
+  when: exports.changed