Skip to content

Commit

Permalink
docker-common: Fix bug when updating config
Browse files Browse the repository at this point in the history
in containerized deployment, if you try to update your `ceph.conf` file
it won't be actually updated on your nodes because it is overwritten by
the copy of the file which is present in your fetch directory.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
  • Loading branch information
guits committed Aug 1, 2017
1 parent a7b011f commit 1f3d713
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
6 changes: 0 additions & 6 deletions roles/ceph-docker-common/tasks/create_configs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,6 @@
mode: "0644"
config_overrides: "{{ ceph_conf_overrides }}"
config_type: ini
when:
- (not mon_containerized_default_ceph_conf_with_kv and
(inventory_hostname in groups.get(mon_group_name, []) or inventory_hostname in groups.get(osd_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])))
notify:
- restart ceph mons
- restart ceph osds
Expand Down
1 change: 0 additions & 1 deletion roles/ceph-docker-common/tasks/fetch_configs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
- name: set config and keys paths
set_fact:
ceph_config_keys:
- "{{ ceph_conf_key_directory }}/{{ cluster }}.conf"
- "{{ ceph_conf_key_directory }}/{{ cluster }}.client.admin.keyring"
- "{{ ceph_conf_key_directory }}/monmap-{{ cluster }}"
- "{{ ceph_conf_key_directory }}/{{ cluster }}.mon.keyring"
Expand Down

0 comments on commit 1f3d713

Please sign in to comment.