Skip to content

Commit

Permalink
core: bump ansible version
Browse files Browse the repository at this point in the history
We should consider bumping ansible version for future releases, so let's
start testing against ansible 2.10

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 839fac8)
  • Loading branch information
guits committed May 13, 2022
1 parent 71cf666 commit 9842b10
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 9 deletions.
4 changes: 2 additions & 2 deletions ceph-ansible.spec.in
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ Obsoletes: ceph-iscsi-ansible <= 1.5

BuildArch: noarch

BuildRequires: ansible >= 2.9
Requires: ansible >= 2.9
BuildRequires: ansible >= 2.10
Requires: ansible >= 2.10

%if 0%{?rhel} == 7
BuildRequires: python2-devel
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# These are Python requirements needed to run ceph-ansible master
ansible>=2.9,<2.10,!=2.9.10
ansible>=3,<4
netaddr
six
4 changes: 2 additions & 2 deletions roles/ceph-validate/tasks/check_system.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@

- name: fail on unsupported ansible version
fail:
msg: "Ansible version must be 2.9!"
when: ansible_version.minor|int != 9
msg: "Ansible version must be 2.10!"
when: ansible_version.minor|int != 10

- name: fail on unsupported system
fail:
Expand Down
3 changes: 2 additions & 1 deletion tests/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@
testinfra>=3,<4
pytest-xdist==1.28.0
pytest>=4.6,<5.0
ansible>=2.9,<2.10,!=2.9.10
ansible>=3,<4
Jinja2>=2.10
netaddr
mock
jmespath
pytest-rerunfailures<9.0
pytest-cov
six
2 changes: 1 addition & 1 deletion tox-subset_update.ini
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ setenv=
UPDATE_CEPH_DEV_BRANCH = master
UPDATE_CEPH_DEV_SHA1 = latest
ROLLING_UPDATE = True
deps= -r{toxinidir}/tests/requirements.txt
changedir={toxinidir}/tests/functional/subset_update{env:CONTAINER_DIR:}
commands=
bash {toxinidir}/tests/scripts/vagrant_up.sh --no-provision {posargs:--provider=virtualbox}
Expand All @@ -55,6 +54,7 @@ commands=
ceph_docker_registry_password={env:DOCKER_HUB_PASSWORD} \
"'

pip install -r {toxinidir}/tests/requirements.txt
# upgrade mons
# mon1
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/infrastructure-playbooks/rolling_update.yml --limit mon1 --tags=mons --extra-vars "\
Expand Down
3 changes: 1 addition & 2 deletions tox-update.ini
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ setenv=
CEPH_DOCKER_IMAGE_TAG = latest-octopus
UPDATE_CEPH_DOCKER_IMAGE_TAG = latest-pacific
ROLLING_UPDATE = True
deps= -r{toxinidir}/tests/requirements.txt
changedir={toxinidir}/tests/functional/all_daemons{env:CONTAINER_DIR:}
commands=
bash {toxinidir}/tests/scripts/vagrant_up.sh --no-provision {posargs:--provider=virtualbox}
Expand Down Expand Up @@ -62,7 +61,7 @@ commands=
ceph_nfs_rgw_access_key=fake_access_key \
ceph_nfs_rgw_secret_key=fake_secret_key \
"'

pip install -r {toxinidir}/tests/requirements.txt
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/infrastructure-playbooks/rolling_update.yml --extra-vars "\
ireallymeanit=yes \
ceph_docker_registry_auth=True \
Expand Down

0 comments on commit 9842b10

Please sign in to comment.