Skip to content

Commit

Permalink
Remove ireallymeanit prompt from configure-lvm-volumes play (#339)
Browse files Browse the repository at this point in the history
This play only generates files in /tmp.

Signed-off-by: Christian Berendt <[email protected]>
  • Loading branch information
berendt committed Dec 5, 2023
1 parent 1c9df62 commit 6fca72b
Showing 1 changed file with 1 addition and 19 deletions.
20 changes: 1 addition & 19 deletions playbooks/ceph/configure-lvm-volumes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@
# use an external configuration repo add the stanza to your repo and
# then do: `osism apply configuration` followed by the two commands before.
# 3. Run this configuration playbook for the hosts you wish to configure:
# `osism apply ceph-configure-lvm-volumes -e ireallymeanit=yes`
# `osism apply ceph-configure-lvm-volumes`
# 4. The configuration generated for the hosts can be found on the
# first manager node of your setup in
# /tmp/<inventory_hostname>-ceph-lvm-configuration.yml
Expand All @@ -94,25 +94,7 @@
vars:
_ceph_configure_lvm_config_path: "/tmp/"
_ceph_configure_lvm_config_file: "ceph-lvm-configuration.yml"
vars_prompt:
- name: ireallymeanit
prompt: >
Are you sure you want to configure Ceph LVM?
Do you want to continue?
default: 'no'
private: false
tasks:
- name: Exit playbook, if user did not mean to configure LVM
ansible.builtin.fail:
msg: >-
Exiting ceph-configure-lvm-volumes playbook,
LVM was NOT configured.
To configure Ceph LVM the node,
either say 'yes' on the prompt
or use `-e ireallymeanit=yes` on the command line when
invoking the playbook
when: ireallymeanit != 'yes'

- name: Get extra vars for Ceph configuration
run_once: true
ansible.builtin.include_vars:
Expand Down

0 comments on commit 6fca72b

Please sign in to comment.