open ebs zfs

ansible/roles/zfs_repo_install/tasks/main.yml

@@ -23,7 +23,7 @@
     ansible.builtin.uri:
       dest: "/tmp/{{ zfs_key_id }}"
       url: https://archzfs.com/archzfs.gpg
-    creates: "/tmp/{{ zfs_key_id }}"
+      creates: "/tmp/{{ zfs_key_id }}"
 
   - name: install and lsign zfs key
     ansible.builtin.shell:
@@ -41,34 +41,35 @@
     community.general.pacman:
       name: "{{ zfs_packages }}"
       state: latest
-
-  - name: set zfs module parameters
-    ansible.builtin.template:
-      src: zfs.conf.j2
-      dest: /etc/modprobe.d/zfs.conf
-      owner: root
-      group: root
-      mode: 0664
-
-  - name: load zfs module
-    community.general.modprobe:
-      name: zfs
-      state: present
-
-  - name: enable zfs services
-    become: true
-    ansible.builtin.service:
-      name: "{{ item }}"
-      state: started
-      enabled: yes
-    loop:
-      - zfs-import-cache.service
-      - zfs-mount.service
-      - zfs.target
+    register: zfs_installed
 
   - name: reboot system
     ansible.builtin.reboot:
       post_reboot_delay: 90
+    when: zfs_installed is changed
+
+- name: set zfs module parameters
+  ansible.builtin.template:
+    src: zfs.conf.j2
+    dest: /etc/modprobe.d/zfs.conf
+    owner: root
+    group: root
+    mode: 0664
+
+- name: load zfs module
+  community.general.modprobe:
+    name: zfs
+    state: present
+
+- name: enable zfs services
+  ansible.builtin.service:
+    name: "{{ item }}"
+    state: started
+    enabled: yes
+  loop:
+    - zfs-import-cache.service
+    - zfs-mount.service
+    - zfs.target
 
 - name: gather existing zpool facts
   community.general.zpool_facts: 
@@ -97,19 +98,14 @@
         -O canmount=off \
         -O devices=off \
         -O compression={{ item.compression }} \
+        -O reservation=none \
+        -O refreservation=none \
+        -O recordsize={{ item.recordsize }} \
         {{ item.name }} {{ item.type }} {{ item.disks }} 
   when: item.name not in ( ansible_zfs_pools | selectattr("name") | list )
   with_items:
     "{{ zfs_pools }}"
 
-- name: gather existing zfs dataset facts
-  community.general.zfs_facts:
-    name: "{{ item.name }}"
-    recurse: true
-    type: filesystem
-  with_items:
-    "{{ zfs_pools }}"
-
 - name: create zfs datasets
   loop: "{{ zfs_pools | subelements('datasets') }}"
   community.general.zfs:
@@ -118,16 +114,16 @@
     extra_zfs_properties:
       canmount: off
       mountpoint: none
-      primarycache: none
+      primarycache: metadata
       secondarycache: none
       reservation: none
       refreservation: none
       dedup: off
       encryption: off
+      compression: off
       volmode: dev
       devices: off
       atime: off
-  when: item.1.name not in ( ansible_zfs_datasets | selectattr("name") | list )