change to script user: cv4pve (#251)

Co-authored-by: Michael Grote <michael.grote@posteo.de>
Reviewed-on: mg/ansible#251
Co-authored-by: mg <michael.grote@posteo.de>
Co-committed-by: mg <michael.grote@posteo.de>

roles/mgrote.cv4pve-autosnap/README.md

@@ -5,20 +5,8 @@ Installiert [cv4pve-autosnap](https://github.com/Corsinvest/cv4pve-autosnap).
 Legt einen cronjob und das Logfile an.
 
 ### getestet auf
-- [x] ProxMox 6.1
+- [x] ProxMox 6*
+- [x] ProxMox 7*
 
 ### Variablen + Defaults
-##### Version
-  ``cv4pve_version: "v1.9.3"``
-##### Cron Minute
-  ``cv4pve_cron_minute: "39"``
-##### Cron Stunde
-  ``cv4pve_cron_hour: "5"``
-##### API-Token (vorher erstellen)
-  ``cv4pve_api_token: "XXXXXXXXXXXXXXXXXXXXXX"``
-##### API-User (vorher erstellen)
-  ``cv4pve_api_user: "root@pam!test2"``
-##### VMs
-  ``cv4pve_vmid: all``
-##### Anzahl Snpshots zum aufheben
-  ``cv4pve_keep_snapshots: 3``
+- see [defaults](./defaults/main.yml)

roles/mgrote.cv4pve-autosnap/defaults/main.yml

@@ -1,8 +1,16 @@
 ---
-  cv4pve_version: "v1.9.3"
+  # when should cv4pve be run
   cv4pve_cron_minute: "39"
   cv4pve_cron_hour: "5"
+  # proxmox api-token and user
   cv4pve_api_token: "XXXXXXXXXXXXXXXXXXXXXX"
   cv4pve_api_user: "root@pam!test2"
+  # which vm to snapshot
   cv4pve_vmid: all
+  # how many snapshots to keep
   cv4pve_keep_snapshots: 3
+  # under which user the script is run
+  cv4pve_user_group: cv4pve
+  cv4pve_user: cv4pve
+  # logfile path
+  cv4pve_logfile: /var/log/cv4pve-autosnap.log

roles/mgrote.cv4pve-autosnap/tasks/main.yml

@@ -1,27 +1,34 @@
 ---
+  - name: include user tasks
+    include_tasks: user.yml
+
   - name: create directories
     become: yes
     ansible.builtin.file:
       path: "{{ item }}"
       state: directory
+      owner: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
     loop:
       - '/tmp/cv4pve'
       - '/usr/local/bin/cv4pve'
 
   - name: download archives
     become: yes
-    ansible.builtin.get_url: ## hier variable für version
+    ansible.builtin.get_url:
       url: "{{ cv4pve_dl_link }}"
       dest: /tmp/cv4pve/cv4pve-autosnap-linux-x64.zip
       mode: '0775'
 
-  - name: Extract archives
+  - name: extract archives
     become: yes
     ansible.builtin.unarchive:
       src: /tmp/cv4pve/cv4pve-autosnap-linux-x64.zip
       dest: /usr/local/bin/cv4pve
       remote_src: yes
       mode: a+x
+      owner: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
 
   - name: copy bash-script
     become: yes
@@ -29,6 +36,8 @@
       src: "cv4pve-script.sh"
       dest: "/usr/local/bin/cv4pve/cv4pve-script.sh"
       mode: a+x
+      owner: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
 
   - name: create cronjob
     become: yes
@@ -38,14 +47,15 @@
       job: "/usr/local/bin/cv4pve/cv4pve-script.sh"
       minute: "{{ cv4pve_cron_minute }}"
       hour: "{{ cv4pve_cron_hour }}"
+      user: "{{ cv4pve_user }}"
 
-  - name: Create log
+  - name: create log
     become: true
     ansible.builtin.file:
       path: /var/log/cv4pve-autosnap.log
       state: touch
-      owner: root
-      group: root
+      owner: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
       mode: 0640
       access_time: preserve
       modification_time: preserve
@@ -55,6 +65,6 @@
     ansible.builtin.template:
       src: logrotate_cv4pve
       dest: /etc/logrotate.d/cv4pve-autosnap
-      owner: root
-      group: root
+      owner: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
       mode: 0644

roles/mgrote.cv4pve-autosnap/tasks/user.yml

@@ -0,0 +1,18 @@
+---
+  - name: ensure group exists
+    become: true
+    ansible.builtin.group:
+      name: "{{ cv4pve_user_group }}"
+      state: present
+    when:
+      - cv4pve_user_group is defined
+
+  - name: ensure user exists
+    become: true
+    ansible.builtin.user:
+      name: "{{ cv4pve_user }}"
+      group: "{{ cv4pve_user_group }}"
+      shell: /usr/sbin/nologin
+    when:
+      - cv4pve_user_group is defined
+      - cv4pve_user is defined

roles/mgrote.cv4pve-autosnap/templates/cv4pve-script.sh

@@ -1,24 +1,7 @@
 #!/bin/bash
 {{ file_header | default () }}
-LOCKDIR=${HOME}/.cache
-function exlock() {
-    exec {lock_fd}>${LOCKDIR}/$(basename $0).lock
-    flock -nx "$lock_fd"
-    if [[ $? == 1 ]]; then
-        exit 1
-    fi
-}
-function unlock() {
-    rm "${LOCKDIR}/$(basename $0).lock"
-    [[ -n $1 ]] && exit $1
-    exit
-}
 
-exlock
 (
-
 /usr/local/bin/cv4pve/cv4pve-autosnap --host=127.0.0.1 --api-token {{ cv4pve_api_user }}={{ cv4pve_api_token }} --vmid="{{ cv4pve_vmid }}" snap --label='daily' --keep="{{ cv4pve_keep_snapshots }}" --state  | ts '%Y-%m-%d - %H-%M-%S'
 /usr/local/bin/cv4pve/cv4pve-autosnap --host=127.0.0.1 --api-token {{ cv4pve_api_user }}={{ cv4pve_api_token }} --vmid="all" status  | ts '%Y-%m-%d - %H-%M-%S'
-) >> /var/log/cv4pve-autosnap.log 2>&1
-
-unlock
+) >> {{ cv4pve_logfile }} 2>&1

roles/mgrote.cv4pve-autosnap/templates/logrotate_cv4pve

@@ -1,5 +1,5 @@
 {{ file_header | default () }}
-/var/log/cv4pve-autosnap.log {
+{{ cv4pve_logfile }} {
   su root root
   create 0640 root root
   rotate 4