From 659f2c60b55ebda2350d8a4f20db76d1394fcb90 Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Wed, 28 Nov 2018 20:53:10 +0100 Subject: [PATCH] validate: change default value for `radosgw_address` change default value of `radosgw_address` to keep consistency with `monitor_address`. Moreover, `ceph-validate` checks if the value is '0.0.0.0' to determine if it has to run `check_eth_rgw.yml`. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1600227 Signed-off-by: Guillaume Abrioux (cherry picked from commit e4869ac8bd574af56952f02b1c8f63ecae0d5d86) --- group_vars/all.yml.sample | 2 +- group_vars/rhcs.yml.sample | 2 +- plugins/actions/validate.py | 2 +- roles/ceph-defaults/defaults/main.yml | 2 +- roles/ceph-docker-common/tasks/check_mandatory_vars.yml | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/group_vars/all.yml.sample b/group_vars/all.yml.sample index 80a55c1c0..18d2e23cc 100644 --- a/group_vars/all.yml.sample +++ b/group_vars/all.yml.sample @@ -404,7 +404,7 @@ dummy: # Eg. If you want to specify for each radosgw node which address the radosgw will bind to you can set it in your **inventory host file** by using 'radosgw_address' variable. # Preference will go to radosgw_address if both radosgw_address and radosgw_interface are defined. #radosgw_interface: interface -#radosgw_address: address +#radosgw_address: 0.0.0.0 #radosgw_address_block: subnet #radosgw_keystone_ssl: false # activate this when using keystone PKI keys # Rados Gateway options diff --git a/group_vars/rhcs.yml.sample b/group_vars/rhcs.yml.sample index 6a8a3a5a6..8031d0279 100644 --- a/group_vars/rhcs.yml.sample +++ b/group_vars/rhcs.yml.sample @@ -404,7 +404,7 @@ ceph_rhcs_version: 3 # Eg. If you want to specify for each radosgw node which address the radosgw will bind to you can set it in your **inventory host file** by using 'radosgw_address' variable. # Preference will go to radosgw_address if both radosgw_address and radosgw_interface are defined. #radosgw_interface: interface -#radosgw_address: address +#radosgw_address: 0.0.0.0 #radosgw_address_block: subnet #radosgw_keystone_ssl: false # activate this when using keystone PKI keys # Rados Gateway options diff --git a/plugins/actions/validate.py b/plugins/actions/validate.py index 620358eae..c4ca4ac24 100644 --- a/plugins/actions/validate.py +++ b/plugins/actions/validate.py @@ -180,7 +180,7 @@ def validate_rados_options(value): Either radosgw_interface, radosgw_address or radosgw_address_block must be defined. """ - radosgw_address_given = notario_store["radosgw_address"] != "address" + radosgw_address_given = notario_store["radosgw_address"] != "0.0.0.0" radosgw_address_block_given = notario_store["radosgw_address_block"] != "subnet" radosgw_interface_given = notario_store["radosgw_interface"] != "interface" diff --git a/roles/ceph-defaults/defaults/main.yml b/roles/ceph-defaults/defaults/main.yml index e6e535f34..807b3bc4a 100644 --- a/roles/ceph-defaults/defaults/main.yml +++ b/roles/ceph-defaults/defaults/main.yml @@ -396,7 +396,7 @@ radosgw_frontend_options: "{{ radosgw_civetweb_options if radosgw_frontend_type # Eg. If you want to specify for each radosgw node which address the radosgw will bind to you can set it in your **inventory host file** by using 'radosgw_address' variable. # Preference will go to radosgw_address if both radosgw_address and radosgw_interface are defined. radosgw_interface: interface -radosgw_address: address +radosgw_address: 0.0.0.0 radosgw_address_block: subnet radosgw_keystone_ssl: false # activate this when using keystone PKI keys # Rados Gateway options diff --git a/roles/ceph-docker-common/tasks/check_mandatory_vars.yml b/roles/ceph-docker-common/tasks/check_mandatory_vars.yml index 011433b20..0114157bd 100644 --- a/roles/ceph-docker-common/tasks/check_mandatory_vars.yml +++ b/roles/ceph-docker-common/tasks/check_mandatory_vars.yml @@ -14,5 +14,5 @@ when: - rgw_group_name in group_names - radosgw_interface == 'interface' - - radosgw_address == 'address' + - radosgw_address == '0.0.0.0' - radosgw_address_block == 'subnet' -- 2.39.5