Merge branch 'release/4.7.0'

alvistack · Mar 12, 2021 · d3c2445 · d3c2445
2 parents 9ac9001 + 8d5cf6c
commit d3c2445
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 22 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,9 +1,16 @@
 # Ansible Role for Podman
 
-## 4.7.0 - TBC
+## 4.8.0 - TBC
 
 ### Major Changes
 
+## 4.7.0 - 2021-03-13
+
+### Major Changes
+
+  - Bugfix [ansible-lint `namespace`](https://github.com/ansible-community/ansible-lint/pull/1451)
+  - Bugfix [ansible-lint `no-handler`](https://github.com/ansible-community/ansible-lint/pull/1402)
+  - Bugfix [ansible-lint `unnamed-task`](https://github.com/ansible-community/ansible-lint/pull/1413)
   - Simplify Python dependency with system packages
   - Support RHEL 8 with Molecule
   - Support RHEL 7 with Molecule

diff --git a/ansible-galaxy-requirements.yml b/ansible-galaxy-requirements.yml
@@ -17,7 +17,7 @@
 collections:
   - name: community.general
     src: https://galaxy.ansible.com
-    version: ">=1.0.0,<2.0.0"
+    version: ">=2.0.0,<3.0.0"
 
   - name: community.kubernetes
     src: https://galaxy.ansible.com

diff --git a/handlers/main.yml b/handlers/main.yml
@@ -14,6 +14,20 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+- name: podman | download archive
+  vars:
+    ansible_python_interpreter: "python3"
+  get_url:
+    url: "{{ item.url }}"
+    dest: "{{ item.dest }}"
+    checksum: "{{ item.checksum }}"
+    mode: "{{ item.mode | default('0644') }}"
+    timeout: 300
+  loop:
+    - "{{ podman_download }}"
+  register: result
+  until: result is succeeded
+
 - name: podman | systemctl daemon-reload
   systemd:
     daemon_reload: true

diff --git a/meta/main.yml b/meta/main.yml
@@ -15,6 +15,7 @@
 # limitations under the License.
 
 galaxy_info:
+  namespace: alvistack
   role_name: podman
   author: "Wong Hoi Sing Edison <[email protected]>"
   description: Ansible Role for Podman Installation

diff --git a/molecule/default/converge.yml b/molecule/default/converge.yml
@@ -18,22 +18,27 @@
   remote_user: root
   become: true
   tasks:
-    - include_role:
+    - name: include role
+      include_role:
         name: cni_plugins
       tags: cni_plugins
 
-    - include_role:
+    - name: include role
+      include_role:
         name: containers_common
       tags: containers_common
 
-    - include_role:
+    - name: include role
+      include_role:
         name: runc
       tags: runc
 
-    - include_role:
+    - name: include role
+      include_role:
         name: conmon
       tags: conmon
 
-    - include_role:
+    - name: include role
+      include_role:
         name: podman
       tags: podman
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -60,22 +60,11 @@
     - { dest: "/usr/local/bin" }
     - { dest: "/usr/share/containers" }
     - { dest: "{{ podman_download.dest | dirname }}" }
-  register: file_result
+  notify:
+    - podman | download archive
 
-- name: download archive
-  vars:
-    ansible_python_interpreter: "python3"
-  get_url:
-    url: "{{ item.url }}"
-    dest: "{{ item.dest }}"
-    checksum: "{{ item.checksum }}"
-    mode: "{{ item.mode | default('0644') }}"
-    timeout: 300
-  loop:
-    - "{{ podman_download }}"
-  when: file_result is changed
-  register: result
-  until: result is succeeded
+- name: flush handlers
+  meta: flush_handlers
 
 - name: unarchive package
   shell: |