added samba share role

ansible/roles/setup_samba/defaults/main.yml

@@ -1,7 +1,10 @@
 ---
 samba_group: smbgroup
 samba_user: smbuser
+users:
+  - name: cianhatton
 shares:
   - /share
   - /share/public_files
-  - /share/private_files
+  - /share/private_files
+  - /share/cian_files

ansible/roles/setup_samba/files/shares.conf

@@ -1,28 +1,9 @@
-[Public Files]
-path = /share/public_files
-# the user that we're going to force this to be shared as
-force user = smbuser
-force group = smbgroup
-# permissions for new files
-create mask = 0664
-force create mode = 0664
-# permissions for new directories
-directory mask = 0775
-force directory mode = 0775
-public = yes
+[Cian Private Files]
+comment= My Private Files
+path = /share/cian_files
+valid users = cianhatton
+public = no
 writable = yes
-
-[Private Files]
-path = /share/private_files
-# the user that we're going to force this to be shared as
-force user = smbuser
-force group = smbgroup
-# permissions for new files
-create mask = 0664
-force create mode = 0664
-# permissions for new directories
-directory mask = 0775
-force directory mode = 0775
-public = yes
-writable = no
-
+browsable = yes
+printable = no
+create mask = 0765

ansible/roles/setup_samba/files/smb.conf

@@ -1,7 +1,7 @@
 [global]
 server string = QNAP
 # name of workgroup for the share
-workgroup = HOMELAB
+workgroup = WORKGROUP
 security = user
 # allow anyone to access
 map to guest = Bad User

ansible/roles/setup_samba/handlers/main.yml

@@ -2,4 +2,9 @@
 - name: restart-samba
   ansible.builtin.service:
     name: smbd
-    state: restarted
+    state: restarted
+
+- name: reload-samba
+  ansible.builtin.service:
+    name: smbd
+    state: reloaded

ansible/roles/setup_samba/tasks/main.yml

@@ -12,7 +12,7 @@
     state: present
     system: true
   notify:
-    - restart-samba
+    - reload-samba
 
 - name: Ensure samba user
   ansible.builtin.user:
@@ -22,7 +22,14 @@
     shell: /bin/false
     create_home: false
   notify:
-    - restart-samba
+    - reload-samba
+
+- name: Add the User(s) to Samba group
+  user:
+    name: "{{ item.name }}"
+    groups: "{{ samba_group }}"
+    append: yes
+  with_items: "{{users}}"
 
 - name: Ensure Share directories
   ansible.builtin.file:
@@ -34,7 +41,7 @@
     mode: 'u+rw,g+rw'
   with_items: "{{shares}}"
   notify:
-    - restart-samba
+    - reload-samba
 
 - name: Copy smb conf files
   copy:
@@ -44,4 +51,4 @@
     - smb.conf
     - shares.conf
   notify:
-    - restart-samba
+    - reload-samba