Refactor apt-migrate-to-trixie.yml: clean up comments, improve task definitions, and ensure proper permissions for backup files

apt-migrate-to-trixie.yml

@@ -1,5 +1,4 @@
 ---
-
 # ---
 # deb822 ist das neue Konfigurationsformats für APT-Quellen (Repositories).
 # Es basiert auf der Debian Control Syntax nach RFC 822 – daher der Name
@@ -35,13 +34,12 @@
 
   pre_tasks:
     - name: Sicherstellen, dass wir Debian sind
-      assert:
+      ansible.builtin.assert:
         that:
           - ansible_facts['os_family'] == "Debian"
         fail_msg: "Dieses Playbook ist nur für Debian geeignet."
 
   tasks:
-
     - name: Keyring für Debian-Archive sicherstellen (falls Signed-By genutzt)
       ansible.builtin.apt:
         name: debian-archive-keyring
@@ -54,7 +52,13 @@
         dest: /etc/apt/sources.list.before-trixie
         remote_src: true
         force: false
-      ignore_errors: true
+        owner: root
+        group: root
+        mode: "0644"
+      failed_when:
+        - backup_result.failed
+        - "'No such file or directory' not in backup_result.msg"
+      register: backup_result
 
     - name: Alte /etc/apt/sources.list deaktivieren (leere Kommentar-Datei)
       ansible.builtin.copy:
@@ -90,6 +94,7 @@
         group: root
         mode: "0644"
       when: enable_backports
+
     - name: Backports-Repo entfernen wenn deaktiviert
       ansible.builtin.file:
         path: /etc/apt/sources.list.d/backports.sources
@@ -107,7 +112,7 @@
 
     - name: APT-Cache aktualisieren
       ansible.builtin.apt:
-        update_cache: yes
+        update_cache: true
         cache_valid_time: "{{ apt_cache_valid_time }}"
 
     - name: Verifikation - zeigen, ob Suites auf trixie stehen
@@ -118,4 +123,3 @@
     - name: Ausgabe anzeigen (nur Info)
       ansible.builtin.debug:
         msg: "{{ apt_policy.stdout.split('\n') | select('search', 'trixie') | list | join('\n') }}"
-