]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/commitdiff
remove kv store support
authorGuillaume Abrioux <gabrioux@redhat.com>
Mon, 26 Nov 2018 13:54:02 +0000 (14:54 +0100)
committermergify[bot] <mergify[bot]@users.noreply.github.com>
Fri, 30 Nov 2018 13:45:12 +0000 (13:45 +0000)
the next stable release will drop this feature.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
16 files changed:
group_vars/all.yml.sample
group_vars/rhcs.yml.sample
roles/ceph-config/tasks/main.yml
roles/ceph-container-common/tasks/main.yml
roles/ceph-defaults/defaults/main.yml
roles/ceph-mds/templates/ceph-mds.service.j2
roles/ceph-mgr/templates/ceph-mgr.service.j2
roles/ceph-mon/tasks/ceph_keys.yml
roles/ceph-mon/tasks/docker/main.yml
roles/ceph-mon/tasks/docker/start_docker_monitor.yml
roles/ceph-mon/templates/ceph-mon.service.j2
roles/ceph-nfs/templates/ceph-nfs.service.j2
roles/ceph-osd/tasks/ceph_disk_cli_options_facts.yml
roles/ceph-osd/templates/ceph-osd-run.sh.j2
roles/ceph-rbd-mirror/templates/ceph-rbd-mirror.service.j2
roles/ceph-rgw/templates/ceph-radosgw.service.j2

index 66d307711ee8c1978f3c3920b71f9dfb0dc3fcab..e4d59daa8c53b3d15cd731c265633065695fb5d8 100644 (file)
@@ -546,15 +546,6 @@ dummy:
 #container_binary:
 
 
-############
-# KV store #
-############
-#containerized_deployment_with_kv: false
-#mon_containerized_default_ceph_conf_with_kv: false
-#kv_type: etcd
-#kv_endpoint: 127.0.0.1
-#kv_port: 2379
-
 
 # this is only here for usage with the rolling_update.yml playbook
 # do not ever change this here
index dda71ce94f9f873d4a983f719991b169a6433fba..989696a7929cd25be461ad2917b135ea09cdadab 100644 (file)
@@ -546,15 +546,6 @@ ceph_docker_registry: "registry.access.redhat.com/rhceph/"
 #container_binary:
 
 
-############
-# KV store #
-############
-#containerized_deployment_with_kv: false
-#mon_containerized_default_ceph_conf_with_kv: false
-#kv_type: etcd
-#kv_endpoint: 127.0.0.1
-#kv_port: 2379
-
 
 # this is only here for usage with the rolling_update.yml playbook
 # do not ever change this here
index e6b7c101d1a11207908785f5160c4287cb13f02f..3b87f44156fafc3062630ef25c77b0f2e766cc93 100644 (file)
   when:
     - not containerized_deployment|bool
 
-# ceph-container-common
-# only create fetch directory when:
-# we are not populating kv_store with default ceph.conf AND host is a mon
-# OR
-# we are not population kv_store with default ceph.conf AND there at least 1 nfs in nfs group AND host is the first nfs
 - block:
   - name: create a local fetch directory if it does not exist
     file:
     run_once: true
     when:
       - (cephx or generate_fsid)
-      - (not mon_containerized_default_ceph_conf_with_kv and
-          (inventory_hostname in groups.get(mon_group_name, []))) or
-        (not mon_containerized_default_ceph_conf_with_kv and
-          ((groups.get(nfs_group_name, []) | length > 0)
-            and (inventory_hostname == groups.get(nfs_group_name, [])[0])))
+      - ((inventory_hostname in groups.get(mon_group_name, [])) or
+        (groups.get(nfs_group_name, []) | length > 0) and inventory_hostname == groups.get(nfs_group_name, [])[0])
 
   - name: generate cluster uuid
     shell: python -c 'import uuid; print(str(uuid.uuid4()))' | tee {{ fetch_directory }}/ceph_cluster_uuid.conf
index d7faa28c2d27a4a45497bbe19ac7219ce61e69a5..e2e688294b7a7e0457fb1e1cbd9f59d01f29720d 100644 (file)
 - name: include checks.yml
   include_tasks: checks.yml
   when:
-    - (not containerized_deployment_with_kv and
-       ((inventory_hostname in groups.get(mon_group_name, [])) or
-        (inventory_hostname in groups.get(nfs_group_name, [])) or
-        (inventory_hostname in groups.get(osd_group_name, []))))
+    - (inventory_hostname in groups.get(mon_group_name, []) or
+      inventory_hostname in groups.get(nfs_group_name, []) or
+      inventory_hostname in groups.get(osd_group_name, []))
     - not ceph_current_status.get('rc', 1) == 0
     - not rolling_update | default(false)
 
index 188f6116a22ed03eeab88ae0a084b44ee398904a..d1b4ef2f7fd4d927c3af6446bc6f7af539dd7650 100644 (file)
@@ -538,15 +538,6 @@ containerized_deployment: False
 container_binary:
 
 
-############
-# KV store #
-############
-containerized_deployment_with_kv: false
-mon_containerized_default_ceph_conf_with_kv: false
-kv_type: etcd
-kv_endpoint: 127.0.0.1
-kv_port: 2379
-
 
 # this is only here for usage with the rolling_update.yml playbook
 # do not ever change this here
index a81097ec872e42e1575cb917d7eb10c932873f5d..38c4b1691e61796f96952766d52f525715145e37 100644 (file)
@@ -13,15 +13,9 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
   {% else -%}
   --cpu-quota={{ ceph_mds_docker_cpu_limit * 100000 }} \
   {% endif -%}
-  {% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/run/ceph:/var/run/ceph:z \
-  {% else -%}
-  -e KV_TYPE={{kv_type}} \
-  -e KV_IP={{kv_endpoint}} \
-  -e KV_PORT={{kv_port}} \
-  {% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
   -e CLUSTER={{ cluster }} \
   -e CEPH_DAEMON=MDS \
index e47162086f74ed9d872d0784ac2e0dd59c540bf0..889862051e57d0b4913a0b475a3045978990ed86 100644 (file)
@@ -13,15 +13,9 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
   {% else -%}
   --cpu-quota={{ ceph_mgr_docker_cpu_limit * 100000 }} \
   {% endif -%}
-  {% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/run/ceph:/var/run/ceph:z \
-  {% else -%}
-  -e KV_TYPE={{kv_type}} \
-  -e KV_IP={{kv_endpoint}} \
-  -e KV_PORT={{kv_port}} \
-  {% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
   -e CLUSTER={{ cluster }} \
   -e CEPH_DAEMON=MGR \
index 186d385f65a0721fb6fcc7b4ca979225316550d2..30661b4327e526fa70b9dfcfe17a0b11cf32168a 100644 (file)
     - cephx
     - ansible_version.major == 2 and ansible_version.minor < 4
 
-- name: test if initial mon keyring is in mon kv store
-  command: ceph --cluster {{ cluster }} config-key get initial_mon_keyring
-  changed_when: false
-  ignore_errors: true
-  check_mode: no
-  run_once: true
-  failed_when: false
-  register: is_initial_mon_keyring_in_kv
-
-- name: put initial mon keyring in mon kv store
-  command: ceph --cluster {{ cluster }} config-key put initial_mon_keyring {{ monitor_keyring.stdout }}
-  changed_when: false
-  check_mode: no
-  run_once: true
-  when:
-    - is_initial_mon_keyring_in_kv.rc != 0
-    - cephx
-
 - name: create ceph mgr keyring(s) when mon is not containerized
   ceph_key:
     name: "mgr.{{ hostvars[item]['ansible_hostname'] }}"
index abe6323edc996f989d319b859f7617ead8bcd649..28092966f6fd35213382e5240e7d71408b711112 100644 (file)
@@ -1,7 +1,6 @@
 ---
 - name: include copy_configs.yml
   include_tasks: copy_configs.yml
-  when: not containerized_deployment_with_kv
 
 - name: include start_docker_monitor.yml
   include_tasks: start_docker_monitor.yml
   command: "{{ docker_exec_cmd }} ceph --admin-daemon /var/run/ceph/{{ cluster }}-mon.{{ monitor_name }}.asok add_bootstrap_peer_hint {{ _current_monitor_address }}"
   changed_when: false
   failed_when: false
-  when:
-    - not containerized_deployment_with_kv
 
 - name: include fetch_configs.yml
   include_tasks: fetch_configs.yml
   run_once: true
-  when: not containerized_deployment_with_kv
 
 - block:
   - name: create ceph mgr keyring(s) when mon is containerized
@@ -39,7 +35,6 @@
       - cephx
       - containerized_deployment
       - groups.get(mgr_group_name, []) | length > 0
-      - not containerized_deployment_with_kv
     with_items: "{{ groups.get(mgr_group_name, []) }}"
 
   - name: fetch ceph mgr key(s)
index 3645d1057d318a978ab3f4f85cb9d4102e8a537f..9719ed1f1c0dc6053837a199f20ae5802668d0c7 100644 (file)
@@ -1,45 +1,4 @@
 ---
-- name: populate kv_store with default ceph.conf
-  docker_container:
-    name: populate-kv-store
-    image: "{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
-    command: populate_kvstore
-    network: host
-    env:
-      KV_TYPE: "{{kv_type}}"
-      KV_IP: "{{kv_endpoint}}"
-      KV_PORT: "{{kv_port}}"
-  run_once: true
-  when:
-    - inventory_hostname == groups[mon_group_name][0]
-    - containerized_deployment_with_kv
-    - mon_containerized_default_ceph_conf_with_kv
-
-- name: populate kv_store with custom ceph.conf
-  docker_container:
-    name: populate-kv-store
-    image: "{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
-    command: populate_kvstore
-    network: host
-    env:
-      KV_TYPE: "{{kv_type}}"
-      KV_IP: "{{kv_endpoint}}"
-      KV_PORT: "{{kv_port}}"
-    volumes:
-      - /etc/ceph/"{{ cluster }}".conf:/etc/ceph/ceph.defaults
-  run_once: true
-  when:
-    - inventory_hostname == groups[mon_group_name][0]
-    - containerized_deployment_with_kv
-    - not mon_containerized_default_ceph_conf_with_kv
-
-- name: delete populate-kv-store docker
-  docker_container:
-    name: populate-kv-store
-    state: absent
-    image: "{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
-  when: containerized_deployment_with_kv
-
 - name: generate systemd unit file
   become: true
   template:
index ddd4203a6c531cb7fe09af0f5307df2d15634985..92e4e55ccabaeaab97d7e26ef836d78aa64095b7 100644 (file)
@@ -13,15 +13,9 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --name ceph-mon-%i \
 {% else -%}
   --cpu-quota={{ ceph_mon_docker_cpu_limit * 100000 }} \
 {% endif -%}
-{% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/run/ceph:/var/run/ceph:z \
-{% else -%}
-  -e KV_TYPE={{kv_type}} \
-  -e KV_IP={{kv_endpoint}}\
-  -e KV_PORT={{kv_port}} \
-{% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
 {% if mon_docker_privileged -%}
   --privileged \
index b77ab47109ba482ccdce570868e248247d87f8bb..82cc61759e4c1c6be48eebabe1b8a4ac64deafec 100644 (file)
@@ -8,7 +8,6 @@ EnvironmentFile=-/etc/environment
 ExecStartPre=-/usr/bin/{{ container_binary }} rm ceph-nfs-%i
 ExecStartPre=/usr/bin/mkdir -p /etc/ceph /etc/ganesha /var/lib/nfs/ganesha
 ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
-  {% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/lib/nfs/ganesha:/var/lib/nfs/ganesha:z \
@@ -18,11 +17,6 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
   --privileged \
   -v /var/run/dbus/system_bus_socket:/var/run/dbus/system_bus_socket \
   {% endif -%}
-  {% else -%}
-  -e KV_TYPE={{kv_type}} \
-  -e KV_IP={{kv_endpoint}}\
-  -e KV_PORT={{kv_port}} \
-  {% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
   -e CLUSTER={{ cluster }} \
   -e CEPH_DAEMON=NFS \
index d29a034eba66e1ebe25acf894bb4c0336e9e6aaa..5c77a58bf6f23b40ebe956e000bc6ae92995ba19 100644 (file)
     - dmcrypt
     - not containerized_deployment
 
-- name: set_fact docker_env_args '-e KV_TYPE={{ kv_type }} -e KV_IP={{ kv_endpoint }} -e KV_PORT={{ kv_port }}'
-  set_fact:
-    docker_env_args: -e KV_TYPE={{ kv_type }} -e KV_IP={{ kv_endpoint }} -e KV_PORT={{ kv_port }}
-  when:
-    - containerized_deployment_with_kv
-
 - name: set_fact docker_env_args '-e OSD_BLUESTORE=0 -e OSD_FILESTORE=1 -e OSD_DMCRYPT=0'
   set_fact:
     docker_env_args: -e OSD_BLUESTORE=0 -e OSD_FILESTORE=1 -e OSD_DMCRYPT=0
index e123f38052e845572df037677c63678f54544af4..9f3f6ac46b326188725df301e92f953354df0a92 100644 (file)
@@ -87,11 +87,6 @@ expose_partitions "$1"
   {% if ansible_distribution == 'Ubuntu' -%}
   --security-opt apparmor:unconfined \
   {% endif -%}
-  {% if containerized_deployment_with_kv -%}
-  -e KV_TYPE={{ kv_type }} \
-  -e KV_IP={{ kv_endpoint }} \
-  -e KV_PORT={{ kv_port }} \
-  {% endif -%}
   {% if osd_objectstore == 'filestore' and not dmcrypt -%}
   -e OSD_FILESTORE=1 \
   -e OSD_DMCRYPT=0 \
index a5d3af767cd5daeec62e01d47d848003f0faf5a3..a25633c0a5e24eb6222089c1a909b530d22fbe64 100644 (file)
@@ -13,15 +13,9 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
   {% else -%}
   --cpu-quota={{ ceph_rbd_mirror_docker_cpu_limit * 100000 }} \
   {% endif -%}
-  {% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/run/ceph:/var/run/ceph:z \
-  {% else -%}
-  -e KV_TYPE={{kv_type}} \
-  -e KV_IP={{kv_endpoint}} \
-  -e KV_PORT={{kv_port}} \
-  {% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
   -e CLUSTER={{ cluster }} \
   -e CEPH_DAEMON=RBD_MIRROR \
index 89efcb730fc59e999389d5d98535cdda3cbd3903..0d7944c1eeb0a771eb8334bc58f1c206314898c9 100644 (file)
@@ -13,15 +13,9 @@ ExecStart=/usr/bin/{{ container_binary }} run --rm --net=host \
   {% else -%}
   --cpu-quota={{ ceph_rgw_docker_cpu_limit * 100000 }} \
   {% endif -%}
-  {% if not containerized_deployment_with_kv -%}
   -v /var/lib/ceph:/var/lib/ceph:z \
   -v /etc/ceph:/etc/ceph:z \
   -v /var/run/ceph:/var/run/ceph:z \
-  {% else -%}
-  -e KV_TYPE={{ kv_type }} \
-  -e KV_IP={{ kv_endpoint }} \
-  -e KV_PORT={{ kv_port }} \
-  {% endif -%}
   -v /etc/localtime:/etc/localtime:ro \
   -e CEPH_DAEMON=RGW \
   -e CLUSTER={{ cluster }} \