Merge pull request #291 from agross/develop

Remove enterprise repos before adding new repos

tasks/main.yml

@@ -132,6 +132,10 @@
     name: os-prober
     state: absent
 
+- ansible.builtin.import_tasks: remove_enterprise_repos.yml
+  when:
+    - "'pve-no-subscription' in pve_repository_line"
+
 - name: Add Proxmox repository
   apt_repository:
     repo: "{{ pve_repository_line }}"
@@ -195,14 +199,7 @@
   until: _proxmox_install is succeeded
 
 - block:
-    - name: Remove automatically installed PVE Enterprise repo configuration
-      apt_repository:
-        repo: "{{ item }}"
-        filename: pve-enterprise
-        state: absent
-      with_items:
-        - "deb https://enterprise.proxmox.com/debian {{ ansible_distribution_release }} pve-enterprise"
-        - "deb https://enterprise.proxmox.com/debian/pve {{ ansible_distribution_release }} pve-enterprise"
+    - ansible.builtin.import_tasks: remove_enterprise_repos.yml
 
     - name: Remove subscription check wrapper function in web UI
       ansible.builtin.lineinfile:

tasks/remove_enterprise_repos.yml

@@ -0,0 +1,10 @@
+- name: Remove automatically installed PVE Enterprise repo configuration
+  ansible.builtin.apt_repository:
+    repo: "{{ item }}"
+    filename: pve-enterprise
+    state: absent
+    update_cache: false
+  with_items:
+    - "deb https://enterprise.proxmox.com/debian {{ ansible_distribution_release }} pve-enterprise"
+    - "deb https://enterprise.proxmox.com/debian/pve {{ ansible_distribution_release }} pve-enterprise"
+    - "deb https://enterprise.proxmox.com/debian/ceph-quincy {{ ansible_distribution_release }} enterprise"