implemented max-attempts

This commit is contained in:
Kevin Veen-Birkenbach 2023-12-14 03:21:19 +01:00
parent 5a91ecf8e4
commit c9b8b3a19f
3 changed files with 16 additions and 9 deletions

View File

@ -4,6 +4,6 @@ OnFailure=systemd-notifier@%n.service cleanup-failed-docker-backups.service
[Service] [Service]
Type=oneshot Type=oneshot
ExecStartPre=/bin/sh -c '/usr/bin/python {{ path_system_maintenance_service_freezer_script }} freeze "{{ system_maintenance_services | reject('equalto', "backup-docker-to-local") | join(',') }}"' ExecStartPre=/bin/sh -c '/usr/bin/python {{ path_system_maintenance_service_freezer_script }} freeze "{{ system_maintenance_services }}" --ignore "backup-docker-to-local,backup-remote-to-local,backup-data-to-usb" --max_attempts 600'
ExecStart=/usr/bin/python {{backup_docker_to_local_folder}}backup-docker-to-local.py ExecStart=/usr/bin/python {{backup_docker_to_local_folder}}backup-docker-to-local.py
ExecStartPost=/bin/sh -c 'systemctl start system-maintenance-service-defrost.service' ExecStartPost=/bin/sh -c 'systemctl start system-maintenance-service-defrost.service'

View File

@ -4,6 +4,6 @@ OnFailure=systemd-notifier@%n.service cleanup-failed-docker-backups.service
[Service] [Service]
Type=oneshot Type=oneshot
ExecStartPre=/bin/sh -c '/usr/bin/python {{ path_system_maintenance_service_freezer_script }} freeze "{{ system_maintenance_services | reject('equalto', "backup-remote-to-local") | join(',') }}"' ExecStartPre=/bin/sh -c '/usr/bin/python {{ path_system_maintenance_service_freezer_script }} freeze "{{ system_maintenance_services }}" --ignore "backup-docker-to-local,backup-remote-to-local,backup-data-to-usb" --max_attempts 600'
ExecStart=/usr/bin/bash {{docker_backup_remote_to_local_folder}}backups-remote-to-local.sh ExecStart=/usr/bin/bash {{docker_backup_remote_to_local_folder}}backups-remote-to-local.sh
ExecStartPost=/bin/sh -c 'systemctl start system-maintenance-service-defrost.service' ExecStartPost=/bin/sh -c 'systemctl start system-maintenance-service-defrost.service'

View File

@ -22,17 +22,21 @@ def check_service_active(service_name):
service_status = result.stdout.decode('utf-8').strip() service_status = result.stdout.decode('utf-8').strip()
return service_status in ['active', 'activating'] return service_status in ['active', 'activating']
def freeze(services_to_wait_for, ignored_services): def freeze(services_to_wait_for, ignored_services, max_attempts):
# Filter services that exist and are not in the ignored list # Filter services that exist and are not in the ignored list
for service in services_to_wait_for: for service in services_to_wait_for:
print(f"\nFreezing: {service}") print(f"\nFreezing: {service}")
if service in ignored_services: if service in ignored_services:
print(f"{service} will be ignored.") print(f"{service} will be ignored.")
else: else:
attempt=0
break_time_sec=5
while check_service_active(service): while check_service_active(service):
print(f"Waiting for 5 seconds for {service} to stop...") attempt += 1
time.sleep(5) print(f"({attempt}/{max_attempts}) Waiting for {break_time_sec} seconds for {service} to stop...")
time.sleep(break_time_sec)
if attempt > max_attempts:
raise Exception(f"Error: Maximum attempts ({max_attempts}) reached. Exit.")
# Stop and disable the corresponding timer, if it exists # Stop and disable the corresponding timer, if it exists
if service_file_exists(service,"timer"): if service_file_exists(service,"timer"):
@ -60,12 +64,12 @@ def defrost(services_to_wait_for, ignored_services):
print(f"Skipped.") print(f"Skipped.")
print("\nAll required services are started.") print("\nAll required services are started.")
def main(services_to_wait_for, ignored_services, action): def main(services_to_wait_for, ignored_services, action, max_attempts):
print(f"Services to wait for: {services_to_wait_for}") print(f"Services to wait for: {services_to_wait_for}")
print(f"Services to ignore: {ignored_services}") print(f"Services to ignore: {ignored_services}")
if action == 'freeze': if action == 'freeze':
print("Freezing services."); print("Freezing services.");
freeze(services_to_wait_for, ignored_services) freeze(services_to_wait_for, ignored_services, max_attempts)
elif action == 'defrost': elif action == 'defrost':
print("Unfreezing services."); print("Unfreezing services.");
defrost(services_to_wait_for, ignored_services) defrost(services_to_wait_for, ignored_services)
@ -77,7 +81,10 @@ if __name__ == "__main__":
parser.add_argument('action', choices=['freeze', 'defrost'], help='Action to perform: freeze or defrost services.') parser.add_argument('action', choices=['freeze', 'defrost'], help='Action to perform: freeze or defrost services.')
parser.add_argument('services', help='Comma-separated list of services to apply the action to') parser.add_argument('services', help='Comma-separated list of services to apply the action to')
parser.add_argument('--ignore', help='Comma-separated list of services to ignore in the action', default='') parser.add_argument('--ignore', help='Comma-separated list of services to ignore in the action', default='')
parser.add_argument('--max_attempts', type=int, default=60, help='Maximum number of attempts for freezing services')
args = parser.parse_args() args = parser.parse_args()
services_to_wait_for = args.services.split(',') services_to_wait_for = args.services.split(',')
ignored_services = args.ignore.split(',') if args.ignore else [] ignored_services = args.ignore.split(',') if args.ignore else []
main(services_to_wait_for, ignored_services,args.action) max_attempts = args.max_attempts
main(services_to_wait_for, ignored_services,args.action,max_attempts)