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-mon | |
| 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-mon')
| -rwxr-xr-x | docker/ceph/ceph-mon/start.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/docker/ceph/ceph-mon/start.sh b/docker/ceph/ceph-mon/start.sh index 3afaa3ad4..a8661d449 100755 --- a/docker/ceph/ceph-mon/start.sh +++ b/docker/ceph/ceph-mon/start.sh @@ -21,7 +21,7 @@ set_configs # of the KOLLA_BOOTSTRAP variable being set, including empty. if [[ "${!KOLLA_BOOTSTRAP[@]}" ]]; then # Lookup our fsid from the ceph.conf - FSID="$(awk '/^fsid/ {print $3; exit}' /etc/ceph/ceph.conf)" + FSID=$(awk '/^fsid/ {print $3; exit}' /etc/ceph/ceph.conf) # Generating initial keyrings and monmap ceph-authtool --create-keyring "${KEYRING_MON}" --gen-key -n mon. --cap mon 'allow *' |
