Merge pull request #6 from jdoss/fix_nfs_firewall

Fix whitespace, nfs service name and firewalld task.
jdoss · Jan 19, 2018 · 71f11df · 71f11df
2 parents e473cc4 + 317338c
commit 71f11df
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/setup_workstation.yml b/setup_workstation.yml
@@ -20,4 +20,4 @@
     - include_tasks: tasks/harden.yml
 
   handlers:
-    - include_tasks: handlers/handlers.yml
+    - include_tasks: handlers/main.yml
diff --git a/tasks/services.yml b/tasks/services.yml
@@ -22,7 +22,7 @@
 
 - name: Enable nfs-server
   service:
-    name: nfs-server
+    name: nfs
     enabled: yes
   when: enable_vagrant_nfs
 

diff --git a/tasks/system.yml b/tasks/system.yml
@@ -29,19 +29,19 @@
     virt_pool:
       command: destroy
       name: default
-    when: ansible_libvirt_pools.default.path is defined 
+    when: ansible_libvirt_pools.default.path is defined
 
   - name: Set default pool to inactive
     virt_pool:
       state: inactive
       name: default
-    when: ansible_libvirt_pools.default.path is defined 
+    when: ansible_libvirt_pools.default.path is defined
 
   - name: Undefine default pool
     virt_pool:
       command: undefine
       name: default
-    when: ansible_libvirt_pools.default.path is defined 
+    when: ansible_libvirt_pools.default.path is defined
 
   - name: Define default pool from template
     virt_pool:
@@ -76,8 +76,10 @@
 
   - name: Allow connections to NFS in the firewall
     firewalld:
+      zone: FedoraWorkstation
       service: "{{ item }}"
       permanent: true
+      immediate: true
       state: enabled
     with_items:
     - nfs