diff options
| author | Zhao Lei <zhaolei@cn.fujitsu.com> | 2015-09-25 20:36:31 +0800 |
|---|---|---|
| committer | Zhao Lei <zhaolei@cn.fujitsu.com> | 2015-09-28 17:50:36 +0800 |
| commit | c62ddbfa977fbf27bbf1241bf20e9e1835212197 (patch) | |
| tree | a9126c9a32946ad1bbc3782d217bb37fde3520a0 /docker/ceph/ceph-osd | |
| parent | 0f297ed0ee9e9a2eedf8c846e200321fd0070072 (diff) | |
| download | kolla-c62ddbfa977fbf27bbf1241bf20e9e1835212197.tar.gz kolla-c62ddbfa977fbf27bbf1241bf20e9e1835212197.zip | |
Remove quotes from subshell call in bash script
Always no quotes for $() statement.
We don't need quotes to hold blanks in result:
# i=$(echo 1 2 3)
# echo $i
1 2 3
#
These quotes can make something wrong in some case:
# i=$(echo '!')
#
# i="$(echo '!')"
-bash: !: event not found
#
No real problem for current code, only to use a better code style.
Change-Id: I06520f4b11ea6b56c4e73049b6f38bfc62fe1392
Closes-Bug: #1499716
Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Diffstat (limited to 'docker/ceph/ceph-osd')
| -rwxr-xr-x | docker/ceph/ceph-osd/start.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/docker/ceph/ceph-osd/start.sh b/docker/ceph/ceph-osd/start.sh index 84549ae8c..501d44bef 100755 --- a/docker/ceph/ceph-osd/start.sh +++ b/docker/ceph/ceph-osd/start.sh @@ -21,10 +21,10 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then partprobe || true # We look up the appropriate device path with partition. - OSD_PARTITION="$(ls ${OSD_DEV}* | egrep ${OSD_DEV}p?1)" + OSD_PARTITION=$(ls "${OSD_DEV}"* | egrep "${OSD_DEV}p?1") JOURNAL_PARTITION="${OSD_PARTITION%?}2" - OSD_ID="$(ceph osd create)" + OSD_ID=$(ceph osd create) OSD_DIR="/var/lib/ceph/osd/ceph-${OSD_ID}" mkdir -p "${OSD_DIR}" mkfs.xfs -f "${OSD_PARTITION}" @@ -47,7 +47,7 @@ if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then fi # We look up the appropriate journal since we cannot rely on symlinks -JOURNAL_PARTITION="$(ls ${OSD_DEV}* | egrep ${OSD_DEV}p?2)" +JOURNAL_PARTITION=$(ls "${OSD_DEV}"* | egrep "${OSD_DEV}p?2") OSD_DIR="/var/lib/ceph/osd/ceph-${OSD_ID}" CMD="/usr/bin/ceph-osd" ARGS="-f -d -i ${OSD_ID} --osd-journal ${JOURNAL_PARTITION} -k ${OSD_DIR}/keyring" |