ansible: bump to ansible 2.12

Add required changes to support ansible 2.12

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

diff --git a/docs/source/index.rst b/docs/source/index.rst
index 6a77ca1809c96fc86f5127aa1cf408559cee3817..58bba1ac758ee013fd759249fe74b7bc9d58669a 100644 (file)
--- a/docs/source/index.rst
+++ b/docs/source/index.rst
@@ -93,7 +93,9 @@ The ``main`` branch should be considered experimental and used with caution.
 
 - ``stable-6.0`` Supports Ceph version ``pacific``. This branch requires Ansible version ``2.10``.
 
-- ``main`` Supports the main branch of Ceph. This branch requires Ansible version ``2.10``.
+- ``stable-7.0`` Supports Ceph version ``quincy``. This branch requires Ansible version ``2.12``.
+
+- ``main`` Supports the main (devel) branch of Ceph. This branch requires Ansible version ``2.12``.
 
 .. NOTE:: ``stable-3.0`` and ``stable-3.1`` branches of ceph-ansible are deprecated and no longer maintained.
 

diff --git a/infrastructure-playbooks/filestore-to-bluestore.yml b/infrastructure-playbooks/filestore-to-bluestore.yml
index 0a8430e53395fc5375a40c77bdefbca914ec83c8..659d54c8badfb252d05610cc1377b9b555fd2a76 100644 (file)
--- a/infrastructure-playbooks/filestore-to-bluestore.yml
+++ b/infrastructure-playbooks/filestore-to-bluestore.yml
@@ -138,7 +138,7 @@
               when: item.1.stdout == 'ceph data'
 
             - name: umount osd data
-              mount:
+              ansible.posix.mount:
                 path: "/var/lib/ceph/osd/{{ cluster }}-{{ (item.0.stdout | from_json).whoami }}"
                 state: unmounted
               with_together:
@@ -147,7 +147,7 @@
               when: item.1.stdout == 'ceph data'
 
             - name: umount osd lockbox
-              mount:
+              ansible.posix.mount:
                 path: "/var/lib/ceph/osd-lockbox/{{ (item.0.stdout | from_json).data.uuid }}"
                 state: unmounted
               with_together:
@@ -441,4 +441,4 @@
           "This node has been skipped because OSDs are either"
           "all bluestore or there's a mix of filestore and bluestore OSDs"
       when:
-        - skip_this_node | bool
\ No newline at end of file
+        - skip_this_node | bool

diff --git a/infrastructure-playbooks/lv-teardown.yml b/infrastructure-playbooks/lv-teardown.yml
index b9ea569cb72afc214f01327674ba8c9daace5239..4d0654c38ca784b84f1241777e130995734fc50f 100644 (file)
--- a/infrastructure-playbooks/lv-teardown.yml
+++ b/infrastructure-playbooks/lv-teardown.yml
@@ -40,7 +40,7 @@
     changed_when: false
 
   - name: tear down any existing osd filesystem
-    mount:
+    ansible.posix.mount:
       path: "{{ item }}"
       state: unmounted
     with_items: "{{ old_osd_filesystems.stdout_lines }}"

diff --git a/infrastructure-playbooks/purge-cluster.yml b/infrastructure-playbooks/purge-cluster.yml
index 109a256977d93dc1e154fc31aafda46acde3dccb..92cf99b7010617ab71e89d0fd77148fe42ef19d9 100644 (file)
--- a/infrastructure-playbooks/purge-cluster.yml
+++ b/infrastructure-playbooks/purge-cluster.yml
@@ -75,7 +75,7 @@
           with_items: "{{ groups[nfs_group_name] }}"
 
         - name: ensure nfs-ganesha mountpoint(s) are unmounted
-          mount:
+          ansible.posix.mount:
             path: "{{ item.split(' ')[1] }}"
             state: unmounted
           with_items:
@@ -434,7 +434,7 @@
         - (containerized_deployment | bool or ceph_volume_present.rc == 0)
 
     - name: umount osd data partition
-      mount:
+      ansible.posix.mount:
         path: "{{ item }}"
         state: unmounted
       with_items: "{{ mounted_osd.stdout_lines }}"
@@ -801,7 +801,7 @@
       listen: "remove data"
 
     - name: umount osd data partition
-      mount:
+      ansible.posix.mount:
         path: "{{ item }}"
         state: unmounted
       with_items: "{{ mounted_osd.stdout_lines }}"

diff --git a/infrastructure-playbooks/shrink-osd.yml b/infrastructure-playbooks/shrink-osd.yml
index 42d96dddd7203a9befde9ede58604ac76ed89a88..b4255f47c0d90400cbbf4272c363a4e422fd5813 100644 (file)
--- a/infrastructure-playbooks/shrink-osd.yml
+++ b/infrastructure-playbooks/shrink-osd.yml
@@ -161,7 +161,7 @@
       delegate_to: "{{ item.0 }}"
 
     - name: umount osd lockbox
-      mount:
+      ansible.posix.mount:
         path: "/var/lib/ceph/osd-lockbox/{{ ceph_osd_data_json[item.2]['data']['uuid'] }}"
         state: absent
       loop: "{{ _osd_hosts }}"
@@ -173,7 +173,7 @@
         - ceph_osd_data_json[item.2]['data']['uuid'] is defined
 
     - name: umount osd data
-      mount:
+      ansible.posix.mount:
         path: "/var/lib/ceph/osd/{{ cluster }}-{{ item.2 }}"
         state: absent
       loop: "{{ _osd_hosts }}"

diff --git a/requirements.txt b/requirements.txt
index 3bf76b93a377aa963944002354fec7cac624d548..4ed60d62d3b86d0b4499a91798a331f0469b11d3 100644 (file)
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,4 +1,4 @@
 # These are Python requirements needed to run ceph-ansible main
-ansible>=2.10,<2.11,!=2.9.10
+ansible-core>=2.12,<2.13
 netaddr
 six

diff --git a/requirements.yml b/requirements.yml
index 53a5fbbcb0d7ece64b0d477421c9b7d9bd273184..8168617c9b258e293098d61373e1ecdf04d954b9 100644 (file)
--- a/requirements.yml
+++ b/requirements.yml
@@ -5,3 +5,6 @@ collections:
     version: 1.2.1
     type: git
   - name: ansible.utils
+    version: '>=2.5.0'
+  - name: community.general
+  - name: ansible.posix

diff --git a/roles/ceph-validate/tasks/check_system.yml b/roles/ceph-validate/tasks/check_system.yml
index dc8cdd6fd92bd9d1e4d4515cc4ab0a1d80c7e756..d404530249c2fb9ffffdcb03e17d385a14e593f7 100644 (file)
--- a/roles/ceph-validate/tasks/check_system.yml
+++ b/roles/ceph-validate/tasks/check_system.yml
@@ -6,8 +6,8 @@
 
 - name: fail on unsupported ansible version
   fail:
-    msg: "Ansible version must be 2.10!"
-  when: ansible_version.minor|int != 10
+    msg: "Ansible version must be 2.12!"
+  when: ansible_version.minor|int != 12
 
 - name: fail on unsupported system
   fail:

diff --git a/tests/functional/setup.yml b/tests/functional/setup.yml
index c72db297c9bc21d11b3eb8362515a26c7b53de06..d06a6e90a56092df1566d741c2f259c4d87d3fb0 100644 (file)
--- a/tests/functional/setup.yml
+++ b/tests/functional/setup.yml
@@ -69,7 +69,7 @@
 
     # mount -o remount doesn't work on RHEL 8 for now
     - name: add mount options to /
-      mount:
+      ansible.posix.mount:
         path: '{{ rootmount.mount }}'
         src: '{{ rootmount.device }}'
         opts: "noatime,nodiratime{% if ansible_facts['os_family'] == 'RedHat' and ansible_facts['distribution_major_version'] | int < 8 %},nobarrier{% endif %}"

diff --git a/tests/requirements.txt b/tests/requirements.txt
index 1ede580c23d6c77bb2c03557f22294bcd5ce629b..de7c25ec65990a9d9a820be4f103de9f7c6ba9f8 100644 (file)
--- a/tests/requirements.txt
+++ b/tests/requirements.txt
@@ -2,7 +2,7 @@
 testinfra
 pytest-xdist
 pytest
-ansible>=2.10,<2.11,!=2.9.10
+ansible-core>=2.12,<2.13,!=2.9.10
 Jinja2>=2.10
 netaddr
 mock

diff --git a/tox-update.ini b/tox-update.ini
index 55eec336e76309f5ff5e492715a9f804d0f9cebd..c02a12b742bbab84e40d1dd0e75305fa9598a793 100644 (file)
--- a/tox-update.ini
+++ b/tox-update.ini
@@ -40,6 +40,13 @@ commands=
 
   ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/tests/functional/setup.yml
 
+#  # use the stable-7.0 branch to deploy an octopus cluster
+#  git clone -b stable-7.0 --single-branch https://github.com/ceph/ceph-ansible.git {envdir}/tmp/ceph-ansible
+#  pip install -r {envdir}/tmp/ceph-ansible/tests/requirements.txt
+#  bash -c 'ANSIBLE_CONFIG={envdir}/tmp/ceph-ansible/ansible.cfg ansible-playbook -vv -i {envdir}/tmp/ceph-ansible/tests/functional/all_daemons{env:CONTAINER_DIR:}/hosts {envdir}/tmp/ceph-ansible/tests/functional/setup.yml'
+#  # configure lvm, we exclude osd2 given this node uses lvm batch scenario (see corresponding inventory host file)
+#  bash -c 'ANSIBLE_CONFIG={envdir}/tmp/ceph-ansible/ansible.cfg ansible-playbook -vv -i {envdir}/tmp/ceph-ansible/tests/functional/all_daemons{env:CONTAINER_DIR:}/hosts {envdir}/tmp/ceph-ansible/tests/functional/lvm_setup.yml --extra-vars "osd_scenario=lvm"'
+
   # configure lvm, we exclude osd2 given this node uses lvm batch scenario (see corresponding inventory host file)
   ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/tests/functional/lvm_setup.yml --limit 'osds:!osd2'
 
@@ -53,6 +60,10 @@ commands=
       ceph_docker_registry_password={env:DOCKER_HUB_PASSWORD} \
   "
 
+#  pip uninstall -y ansible
+#  pip install -r {toxinidir}/tests/requirements.txt
+  ansible-galaxy install -r {toxinidir}/requirements.yml -v
+
   ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/infrastructure-playbooks/rolling_update.yml --extra-vars "\
       ireallymeanit=yes \
       ceph_dev_branch={env:UPDATE_CEPH_DEV_BRANCH:main} \