ceph helm: broken check-mon bash script

Bug #1796313 reported by yann degat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-helm
Fix Released
Undecided
anticw

Bug Description

the `openstack-helm-infra/ceph-mon/templates/bin/mon/_check.sh.tpl`

is broken with an undefined var:

```
if [ -S "${SOCKDIR}/${SBASE}.${MON_NAME}.${SSUFFIX}" ]; then
   MON_STATE=$(ceph -f json-pretty --connect-timeout 1 --admin-daemon "${sock}" mon_status|grep state|sed 's/.*://;s/[^a-z]//g')
 ...
  fi
```

${sock} is undefined.
it should be defined to the `${SOCKDIR}/${SBASE}.${MON_NAME}.${SSUFFIX}`
value

Changed in openstack-helm:
assignee: nobody → yann degat (yann-degat-z)
assignee: yann degat (yann-degat-z) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-helm-infra (master)

Fix proposed to branch: master
Review: https://review.openstack.org/612059

Changed in openstack-helm:
assignee: nobody → Matthew Heler (mheler)
status: New → In Progress
Revision history for this message
Matthew Heler (mheler) wrote :
Changed in openstack-helm:
assignee: Matthew Heler (mheler) → anticw (anticw)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-helm-infra (master)

Reviewed: https://review.openstack.org/612059
Committed: https://git.openstack.org/cgit/openstack/openstack-helm-infra/commit/?id=154fcd894f5385ca43a4229d5ad738f093853e77
Submitter: Zuul
Branch: master

commit 154fcd894f5385ca43a4229d5ad738f093853e77
Author: Matthew Heler <email address hidden>
Date: Sat Oct 20 10:03:57 2018 -0500

    Use the correct socket file for the Ceph mon check.

    Change-Id: If8c40c3c0501b78db88d3a7f33bf3838c0e60199
    Closes-Bug: 1796313

Changed in openstack-helm:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.