Skip to content

Commit

Permalink
add better clarification on ceph-ansible current status
Browse files Browse the repository at this point in the history
Given that the project is still maintained for the time being, let's add a better
comment about that.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
  • Loading branch information
guits committed Jun 15, 2022
1 parent 11c0e93 commit 77fc0e1
Show file tree
Hide file tree
Showing 7 changed files with 24 additions and 16 deletions.
4 changes: 3 additions & 1 deletion README.rst
@@ -1,8 +1,10 @@
ceph-ansible -- DEPRECATED --
ceph-ansible
============

**NOTE: cephadm is the new official installer, you should consider migrating to cephadm.**

The project is still maintained for the time being but it is encouraged to migrate to ``cephadm``.

Ansible playbooks for Ceph, the distributed filesystem.

Please refer to our hosted documentation here: https://docs.ceph.com/projects/ceph-ansible/en/latest/
Expand Down
10 changes: 3 additions & 7 deletions site-container.yml.sample
Expand Up @@ -3,16 +3,12 @@
- hosts: localhost
connection: local
tasks:
- name: Warn about ceph-ansible deprecation
debug:
- name: Warn about ceph-ansible current status
fail:
msg: "cephadm is the new official installer. Please, consider migrating.
See https://docs.ceph.com/en/latest/cephadm/install for new deployments
or https://docs.ceph.com/en/latest/cephadm/adoption for migrating existing deployments."
retries: 9
delay: 2
until: false
ignore_errors: true
when: not skip_deprecation_warn | default(false) | bool
when: not yes_i_know | default(false) | bool

- hosts:
- mons
Expand Down
10 changes: 3 additions & 7 deletions site.yml.sample
Expand Up @@ -4,16 +4,12 @@
- hosts: localhost
connection: local
tasks:
- name: Warn about ceph-ansible deprecation
debug:
- name: Warn about ceph-ansible current status
fail:
msg: "cephadm is the new official installer. Please, consider migrating.
See https://docs.ceph.com/en/latest/cephadm/install for new deployments
or https://docs.ceph.com/en/latest/cephadm/adoption for migrating existing deployments."
retries: 9
delay: 2
until: false
ignore_errors: true
when: not skip_deprecation_warn | default(false) | bool
when: not yes_i_know | default(false) | bool

- hosts:
- mons
Expand Down
2 changes: 2 additions & 0 deletions tox-external_clients.ini
Expand Up @@ -41,6 +41,7 @@ commands=

non_container: ansible-playbook -vv -i "localhost," -c local {toxinidir}/tests/functional/dev_setup.yml --extra-vars "dev_setup=True change_dir={changedir} ceph_dev_branch=main ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb" --tags "vagrant_setup"
ansible-playbook -vv -i {changedir}/inventory/hosts {toxinidir}/{env:PLAYBOOK:site.yml.sample} --limit 'all:!clients' --extra-vars "\
yes_i_know=true \
delegate_facts_host={env:DELEGATE_FACTS_HOST:True} \
ceph_dev_branch=main \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -52,6 +53,7 @@ commands=
ansible-playbook -vv -i {changedir}/inventory {toxinidir}/tests/functional/external_clients_admin_key.yml

ansible-playbook -vv -i {changedir}/inventory/external_clients-hosts {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
fsid=40358a87-ab6e-4bdc-83db-1d909147861c \
external_cluster_mon_ips=192.168.31.10,192.168.31.11,192.168.31.12 \
Expand Down
1 change: 1 addition & 0 deletions tox-subset_update.ini
Expand Up @@ -43,6 +43,7 @@ commands=

non_container: ansible-playbook -vv -i "localhost," -c local {toxinidir}/tests/functional/dev_setup.yml --extra-vars "dev_setup=True change_dir={changedir} ceph_dev_branch={env:UPDATE_CEPH_DEV_BRANCH:main} ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb" --tags "vagrant_setup"
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
delegate_facts_host={env:DELEGATE_FACTS_HOST:True} \
ceph_dev_branch={env:UPDATE_CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand Down
1 change: 1 addition & 0 deletions tox-update.ini
Expand Up @@ -52,6 +52,7 @@ commands=

non_container: ansible-playbook -vv -i "localhost," -c local {toxinidir}/tests/functional/dev_setup.yml --extra-vars "dev_setup=True change_dir={changedir} ceph_dev_branch={env:UPDATE_CEPH_DEV_BRANCH:main} ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb" --tags "vagrant_setup"
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
delegate_facts_host={env:DELEGATE_FACTS_HOST:True} \
ceph_dev_branch={env:UPDATE_CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand Down
12 changes: 11 additions & 1 deletion tox.ini
Expand Up @@ -61,6 +61,7 @@ commands=

# set up the cluster again
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars @ceph-override.json --extra-vars "\
yes_i_know=true \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
ceph_docker_registry_auth=True \
Expand All @@ -81,6 +82,7 @@ commands=

# set up the cluster again
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars @ceph-override.json --extra-vars "\
yes_i_know=true \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
ceph_docker_registry_auth=True \
Expand All @@ -104,6 +106,7 @@ commands=

# set up the cluster again
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
"
Expand Down Expand Up @@ -183,6 +186,7 @@ commands=
commands=
ansible-playbook -vv -i {changedir}/hosts-2 --limit mgrs {toxinidir}/tests/functional/setup.yml
ansible-playbook -vv -i {changedir}/hosts-2 --limit mgrs {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -196,6 +200,7 @@ commands=
commands=
ansible-playbook -vv -i {changedir}/hosts-2 --limit mdss {toxinidir}/tests/functional/setup.yml
ansible-playbook -vv -i {changedir}/hosts-2 --limit mdss {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -209,6 +214,7 @@ commands=
commands=
ansible-playbook -vv -i {changedir}/hosts-2 --limit rbdmirrors {toxinidir}/tests/functional/setup.yml
ansible-playbook -vv -i {changedir}/hosts-2 --limit rbdmirrors {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -222,6 +228,7 @@ commands=
commands=
ansible-playbook -vv -i {changedir}/hosts-2 --limit rgws {toxinidir}/tests/functional/setup.yml
ansible-playbook -vv -i {changedir}/hosts-2 --limit rgws {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -242,6 +249,7 @@ commands=
ansible -i localhost, all -c local -b -m iptables -a 'chain=FORWARD protocol=tcp source=192.168.0.0/16 destination=192.168.0.0/16 jump=ACCEPT action=insert rule_num=1 state=absent'
ansible -i localhost, all -c local -b -m iptables -a 'chain=FORWARD protocol=tcp source=192.168.0.0/16 destination=192.168.0.0/16 jump=ACCEPT action=insert rule_num=1 state=present'
ansible-playbook --ssh-common-args='-F {changedir}/secondary/vagrant_ssh_config -o ControlMaster=auto -o ControlPersist=600s -o PreferredAuthentications=publickey' -vv -i {changedir}/secondary/hosts {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
ireallymeanit=yes \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -250,6 +258,7 @@ commands=
ceph_docker_registry_password={env:DOCKER_HUB_PASSWORD} \
"
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --limit rgws --extra-vars "\
yes_i_know=true \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
ceph_docker_registry_auth=True \
Expand Down Expand Up @@ -369,6 +378,7 @@ commands=
ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/tests/functional/setup.yml

ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "\
yes_i_know=true \
deploy_secondary_zones=False \
ceph_dev_branch={env:CEPH_DEV_BRANCH:main} \
ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb \
Expand All @@ -387,7 +397,7 @@ commands=
all_daemons,collocation: py.test --reruns 20 --reruns-delay 3 -n 8 --durations=0 --sudo -v --connection=ansible --ansible-inventory={changedir}/{env:INVENTORY} --ssh-config={changedir}/vagrant_ssh_config {toxinidir}/tests/functional/tests

# handlers/idempotency test
all_daemons,all_in_one,collocation: ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "delegate_facts_host={env:DELEGATE_FACTS_HOST:True} ceph_docker_image_tag={env:CEPH_DOCKER_IMAGE_TAG_BIS:latest-bis-main} ceph_dev_branch={env:CEPH_DEV_BRANCH:main} ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb" --extra-vars @ceph-override.json
all_daemons,all_in_one,collocation: ansible-playbook -vv -i {changedir}/{env:INVENTORY} {toxinidir}/{env:PLAYBOOK:site.yml.sample} --extra-vars "delegate_facts_host={env:DELEGATE_FACTS_HOST:True} ceph_docker_image_tag={env:CEPH_DOCKER_IMAGE_TAG_BIS:latest-bis-main} ceph_dev_branch={env:CEPH_DEV_BRANCH:main} ceph_dev_sha1=6f765e25ab7b138b5c901363389df705244e91bb yes_i_know=true" --extra-vars @ceph-override.json

purge: {[purge]commands}
purge_dashboard: {[purge-dashboard]commands}
Expand Down

0 comments on commit 77fc0e1

Please sign in to comment.