From 2d4161de632e6a2b18bef2be0802819b64d05ea7 Mon Sep 17 00:00:00 2001
From: Paul Bourke <paul.bourke@oracle.com>
Date: Tue, 5 Jul 2016 16:06:40 +0100
Subject: [PATCH] Improve task names for Ceph

The task names while bootstrapping Ceph OSDs and cache OSDs were
identical making them harder to distinguish in the Ansible output.

TrivialFix

Change-Id: I0228221a09bdaaba46b6280b6e1be9920ad13598
---
 ansible/roles/ceph/tasks/bootstrap_osds.yml | 8 ++++----
 ansible/roles/ceph/tasks/start_osds.yml     | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ansible/roles/ceph/tasks/bootstrap_osds.yml b/ansible/roles/ceph/tasks/bootstrap_osds.yml
index f5067ae8b9..47a9459c87 100644
--- a/ansible/roles/ceph/tasks/bootstrap_osds.yml
+++ b/ansible/roles/ceph/tasks/bootstrap_osds.yml
@@ -1,5 +1,5 @@
 ---
-- name: Looking up disks to bootstrap for Ceph
+- name: Looking up disks to bootstrap for Ceph OSDs
   command: docker exec -t kolla_toolbox /usr/bin/ansible localhost
     -m find_disks
     -a "partition_name='KOLLA_CEPH_OSD_BOOTSTRAP' match_mode='prefix'"
@@ -7,11 +7,11 @@
   changed_when: "{{ osd_lookup.stdout.find('localhost | SUCCESS => ') != -1 and (osd_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).changed }}"
   failed_when: osd_lookup.stdout.split()[2] != 'SUCCESS'
 
-- name: Reading data from variable
+- name: Parsing disk info for Ceph OSDs
   set_fact:
     osds_bootstrap: "{{ (osd_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).disks|from_json }}"
 
-- name: Looking up disks to bootstrap for Ceph
+- name: Looking up disks to bootstrap for Ceph Cache OSDs
   command: docker exec -t kolla_toolbox /usr/bin/ansible localhost
     -m find_disks
     -a "partition_name='KOLLA_CEPH_OSD_CACHE_BOOTSTRAP' match_mode='prefix'"
@@ -19,7 +19,7 @@
   changed_when: "{{ osd_cache_lookup.stdout.find('localhost | SUCCESS => ') != -1 and (osd_cache_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).changed }}"
   failed_when: osd_cache_lookup.stdout.split()[2] != 'SUCCESS'
 
-- name: Reading data from variable
+- name: Parsing disk info for Ceph Cache OSDs
   set_fact:
     osds_cache_bootstrap: "{{ (osd_cache_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).disks|from_json }}"
 
diff --git a/ansible/roles/ceph/tasks/start_osds.yml b/ansible/roles/ceph/tasks/start_osds.yml
index 96080775ad..0f4a9a76ef 100644
--- a/ansible/roles/ceph/tasks/start_osds.yml
+++ b/ansible/roles/ceph/tasks/start_osds.yml
@@ -7,7 +7,7 @@
   changed_when: "{{ osd_lookup.stdout.find('localhost | SUCCESS => ') != -1 and (osd_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).changed }}"
   failed_when: osd_lookup.stdout.split()[2] != 'SUCCESS'
 
-- name: Reading data from variable
+- name: Parsing disk info for Ceph OSDs
   set_fact:
     osds: "{{ (osd_lookup.stdout.split('localhost | SUCCESS => ')[1]|from_json).disks|from_json }}"
 
-- 
GitLab