Use blocks for macvlan tasks for each distribution (#8918)

For the code readability, this adds blocks for each distribution.

roles/network_plugin/macvlan/tasks/main.yml

@@ -27,62 +27,65 @@
   notify: Macvlan | restart network
   when: ansible_os_family in ["Debian"]
 
-- name: Macvlan | Install macvlan script on centos
+- block:
-  copy:
+  - name: Macvlan | Install macvlan script on centos
-    src: "{{ item }}"
+    copy:
-    dest: /etc/sysconfig/network-scripts/
+      src: "{{ item }}"
-    owner: root
+      dest: /etc/sysconfig/network-scripts/
-    group: root
+      owner: root
-    mode: "0755"
+      group: root
-  with_fileglob:
+      mode: "0755"
+    with_fileglob:
     - files/*
-  when: ansible_os_family == "RedHat"
 
-- name: Macvlan | Install post-up script on centos
+  - name: Macvlan | Install post-up script on centos
-  copy:
+    copy:
-    src: "files/ifup-local"
+      src: "files/ifup-local"
-    dest: /sbin/
+      dest: /sbin/
-    owner: root
+      owner: root
-    group: root
+      group: root
-    mode: "0755"
+      mode: "0755"
-  when: ansible_os_family == "RedHat" and enable_nat_default_gateway
+    when: enable_nat_default_gateway
 
-- name: Macvlan | Install network gateway interface on centos
+  - name: Macvlan | Install network gateway interface on centos
-  template:
+    template:
-    src: "{{ item.src }}.j2"
+      src: "{{ item.src }}.j2"
-    dest: "/etc/sysconfig/network-scripts/{{ item.dst }}"
+      dest: "/etc/sysconfig/network-scripts/{{ item.dst }}"
-    mode: 0644
+      mode: 0644
-  with_items:
+    with_items:
     - {src: centos-network-macvlan.cfg, dst: ifcfg-mac0 }
     - {src: centos-routes-macvlan.cfg, dst: route-mac0 }
     - {src: centos-postup-macvlan.cfg, dst: post-up-mac0 }
-  notify: Macvlan | restart network
+    notify: Macvlan | restart network
+
   when: ansible_os_family == "RedHat"
 
-- name: Macvlan | Install service nat via gateway on Flatcar Container Linux
+- block:
-  template:
+  - name: Macvlan | Install service nat via gateway on Flatcar Container Linux
-    src: coreos-service-nat_ouside.j2
+    template:
-    dest: /etc/systemd/system/enable_nat_ouside.service
+      src: coreos-service-nat_ouside.j2
-    mode: 0644
+      dest: /etc/systemd/system/enable_nat_ouside.service
-  when: ansible_os_family in ["Flatcar", "Flatcar Container Linux by Kinvolk"] and enable_nat_default_gateway
+      mode: 0644
+    when: enable_nat_default_gateway
 
-- name: Macvlan | Enable service nat via gateway on Flatcar Container Linux
+  - name: Macvlan | Enable service nat via gateway on Flatcar Container Linux
-  command: "{{ item }}"
+    command: "{{ item }}"
-  with_items:
+    with_items:
     - systemctl daemon-reload
     - systemctl enable enable_nat_ouside.service
-  when: ansible_os_family in ["Flatcar", "Flatcar Container Linux by Kinvolk"] and enable_nat_default_gateway
+    when: enable_nat_default_gateway
 
-- name: Macvlan | Install network gateway interface on Flatcar Container Linux
+  - name: Macvlan | Install network gateway interface on Flatcar Container Linux
-  template:
+    template:
-    src: "{{ item.src }}.j2"
+      src: "{{ item.src }}.j2"
-    dest: "/etc/systemd/network/{{ item.dst }}"
+      dest: "/etc/systemd/network/{{ item.dst }}"
-    mode: 0644
+      mode: 0644
-  with_items:
+    with_items:
     - {src: coreos-device-macvlan.cfg, dst: macvlan.netdev }
     - {src: coreos-interface-macvlan.cfg, dst: output.network }
     - {src: coreos-network-macvlan.cfg, dst: macvlan.network }
-  notify: Macvlan | restart network
+    notify: Macvlan | restart network
+
   when: ansible_os_family in ["Flatcar", "Flatcar Container Linux by Kinvolk"]
 
 - name: Macvlan | Install cni definition for Macvlan