mirror of https://github.com/ceph/ceph-ansible.git
94cdef2757
If the SOCKET variable isn't defined in the script then the test
command won't fail because the return code is 0
$ test -S
$ echo $?
0
There multiple issues in that script:
- The default SOCKET value isn't defined.
- Update the wget parameters because the command is doing a loop.
We now use the same option than curl.
- The check_rest function doesn't test the radosgw at all due to
a wrong test command (test against a string) and always returns 0.
This needs to use the DOCKER_EXEC variable in order to execute the
command.
$ test 'wget http://192.168.100.11:8080'
$ echo $?
0
Resolves: #3926
Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>
(cherry picked from commit
|
||
---|---|---|
.. | ||
restart_mds_daemon.sh.j2 | ||
restart_mgr_daemon.sh.j2 | ||
restart_mon_daemon.sh.j2 | ||
restart_nfs_daemon.sh.j2 | ||
restart_osd_daemon.sh.j2 | ||
restart_rbd_mirror_daemon.sh.j2 | ||
restart_rbd_target_api.sh.j2 | ||
restart_rbd_target_gw.sh.j2 | ||
restart_rgw_daemon.sh.j2 | ||
restart_tcmu_runner.sh.j2 |