Merge pull request #7 from iangreenleaf/debian-support

Debian support

handlers/main.yml

@@ -2,10 +2,10 @@
 ---
 - name: Restart Samba
   service:
-    name: smb
+    name: "{{ samba_service }}"
     state: restarted
 
 - name: Restart WinBind
   service:
-    name: nmb
+    name: "{{ winbind_service }}"
     state: restarted

meta/main.yml

@@ -9,6 +9,12 @@ galaxy_info:
   - name: EL
     versions:
       - 7
+  - name: Ubuntu
+    versions:
+      - all
+  - name: Debian
+    versions:
+      - all
   categories:
     - system
     - networking

tasks/main.yml

@@ -1,17 +1,24 @@
 # File: roles/samba/tasks/main.yml
 ---
 
+- include_vars: "{{ item }}"
+  with_first_found:
+   - "os_{{ ansible_distribution }}.yml"
+   - "os_{{ ansible_os_family }}.yml"
+   - "default.yml"
+
 - name: Install Samba packages
-  yum:
+  action: "{{ ansible_pkg_mgr }} pkg={{ item }} state=installed"
-    pkg: "{{ item }}"
-    state: installed
   with_items:
-    - libsemanage-python
     - samba-common
     - samba
     - samba-client
   tags: samba
 
+- name: Install SELinux package
+  action: "{{ ansible_pkg_mgr }} pkg=libsemanage-python state=installed"
+  when: ansible_selinux
+
 - name: Make sure SELinux boolean settings are correct
   seboolean:
     name: "{{ item }}"
@@ -20,7 +27,7 @@
   with_items:
     - samba_enable_home_dirs
     - samba_export_all_rw
-  tags: samba
+  when: ansible_selinux
 
 - name: Create Samba shares root directory
   file:
@@ -71,14 +78,14 @@
 
 - name: Start Samba service
   service:
-    name: smb
+    name: "{{ samba_service }}"
     state: started
     enabled: yes
   tags: samba
 
 - name: Start WindBind service
   service:
-    name: nmb
+    name: "{{ winbind_service }}"
     state: started
     enabled: yes
   tags: samba

vars/default.yml

@@ -0,0 +1,4 @@
+---
+
+samba_service: smbd
+winbind_service: nmbd

vars/os_CentOS.yml

@@ -0,0 +1,4 @@
+---
+
+samba_service: smb
+winbind_service: nmb

vars/os_Debian.yml

@@ -0,0 +1,4 @@
+---
+
+samba_service: smbd
+winbind_service: nmbd