mirror of https://github.com/ceph/ceph-ansible.git
purge-docker: remove redundant task
The `remove_packages` prompt is redundant to the `ireallymeanit` prompt since it does exactly the same thing. I guess the only goal of this task was to make a break to warn user about `--skip-tags=with_pkg` feature. This warning should be part of the first prompt. Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>pull/2489/head
parent
ca572a11f1
commit
e32a177af8
|
@ -11,13 +11,9 @@
|
||||||
|
|
||||||
vars_prompt:
|
vars_prompt:
|
||||||
- name: ireallymeanit
|
- name: ireallymeanit
|
||||||
prompt: Are you sure you want to purge the cluster?
|
|
||||||
default: 'no'
|
|
||||||
private: no
|
|
||||||
|
|
||||||
- name: remove_packages
|
|
||||||
prompt: >
|
prompt: >
|
||||||
If --skip-tags=with_pkg is not set docker packages
|
Are you sure you want to purge the cluster?
|
||||||
|
Note that if with_pkg is not set docker packages
|
||||||
and more will be uninstalled from non-atomic hosts.
|
and more will be uninstalled from non-atomic hosts.
|
||||||
Do you want to continue?
|
Do you want to continue?
|
||||||
default: 'no'
|
default: 'no'
|
||||||
|
@ -33,16 +29,6 @@
|
||||||
invoking the playbook"
|
invoking the playbook"
|
||||||
when: ireallymeanit != 'yes'
|
when: ireallymeanit != 'yes'
|
||||||
|
|
||||||
- name: exit playbook, if user did not mean to remove packages
|
|
||||||
fail:
|
|
||||||
msg: >
|
|
||||||
"Exiting purge-docker-cluster playbook. No packages were removed.
|
|
||||||
To skip removing packages use --skip-tag=with_pkg. To continue
|
|
||||||
with removing packages, do not specify --skip-tag=with_pkg and
|
|
||||||
either say 'yes' on the prompt or use `-e remove_packages=yes`
|
|
||||||
on the command line when invoking the playbook"
|
|
||||||
when: remove_packages != 'yes'
|
|
||||||
|
|
||||||
- name: set ceph_docker_registry value if not set
|
- name: set ceph_docker_registry value if not set
|
||||||
set_fact:
|
set_fact:
|
||||||
ceph_docker_registry: "docker.io"
|
ceph_docker_registry: "docker.io"
|
||||||
|
|
Loading…
Reference in New Issue