Refactored native-backups-cleanup and native-disc-space-check. \n Removed hardcoded values and used parameters instead.

roles/native-backups-cleanup/README.md

@@ -9,3 +9,6 @@ It may be neccessary to install gcc seperat to use psutil
 ```bash
   sudo pacman -S gcc
 ```
+
+## further information
+- https://stackoverflow.com/questions/48929553/get-hard-disk-size-in-python

roles/native-backups-cleanup/files/backups-cleanup.py

@@ -1,15 +1,19 @@
-# @see https://stackoverflow.com/questions/48929553/get-hard-disk-size-in-python
 import psutil
 import shutil
 import os
-backup_disk_path = "{{backup_disk_path}}"
-backups_folder_path = os.path.join(backup_disk_path, "Backups/")
+import argparse
+
+# Validating arguments
+parser.add_argument('--maximum-backup-size-percent', type=int, dest='maximum_backup_size_percent',required=True, choices=range(0,100), help="The directory from which the data should be encrypted.")
+parser.add_argument('--backups-folder-path',type=str,dest='backups_folder_path',required=True, help="The folder in which the backups are stored")
+args = parser.parse_args()
+
 deleted = True
-while psutil.disk_usage(backup_disk_path).percent > int({{size_percent_maximum_backup}}) and deleted:
+while psutil.disk_usage(args.backups_folder_path).percent > args.maximum_backup_size_percent and deleted:
     deleted = False
-    print("%d %% of disk %s are used. Freeing space..." % (psutil.disk_usage(backup_disk_path).percent,backup_disk_path))
-    for primary_directory in os.listdir(backups_folder_path):
-        primary_directory = os.path.join(backups_folder_path, primary_directory)
+    print("%d %% of disk %s are used. Freeing space..." % (psutil.disk_usage(args.backups_folder_path).percent,args.backups_folder_path))
+    for primary_directory in os.listdir(args.backups_folder_path):
+        primary_directory = os.path.join(args.backups_folder_path, primary_directory)
         for application_directory in os.listdir(primary_directory):
             application_directory = os.path.join(primary_directory, application_directory)
             versions_directory = os.path.join(application_directory, "versions/")
@@ -22,4 +26,4 @@ while psutil.disk_usage(backup_disk_path).percent > int({{size_percent_maximum_b
                 deleted = True
 if not deleted:
     print("All versions had been deleted!")
-print("Cleaning up finished: %d %% of disk %s are used." % (psutil.disk_usage(backup_disk_path).percent,backup_disk_path))
+print("Cleaning up finished: %d %% of disk %s are used." % (psutil.disk_usage(args.backups_folder_path).percent,args.backups_folder_path))

roles/native-backups-cleanup/tasks/main.yml

@@ -9,7 +9,9 @@
     mode: 0755
 
 - name: create backups-cleanup.py
-  template: src=backups-cleanup.py.j2 dest={{docker_backups_cleanup}}backups-cleanup.py
+  copy: 
+    src:  backups-cleanup.py
+    dest: {{docker_backups_cleanup}}backups-cleanup.py
 
 - name: create backups-cleanup.service
   template: src=backups-cleanup.service.j2 dest=/etc/systemd/system/backups-cleanup.service

roles/native-backups-cleanup/templates/backups-cleanup.service.j2

@@ -4,4 +4,4 @@ OnFailure=systemd-email@%n.service
 
 [Service]
 Type=oneshot
-ExecStart=/usr/bin/python {{docker_backups_cleanup}}/backups-cleanup.py
+ExecStart=/usr/bin/python {{docker_backups_cleanup}}/backups-cleanup.py --backups-folder-path {{backups_folder_path}} --maximum-backup-size-percent {{size_percent_maximum_backup}}