From c2ca5999235569689bf7a15d432834887bdbfa79 Mon Sep 17 00:00:00 2001
From: Stephen Taylor <stephen.taylor.1@att.com>
Date: Tue, 20 Jul 2021 15:30:19 -0600
Subject: [PATCH] [ceph-osd] Mount /var/crash inside ceph-osd pods

This change adds /var/crash as a host-path volume mount for
ceph-osd pods in order to facilitate core dump capture when
ceph-osd daemons crash.

Change-Id: Ie517c64e08b11504f71d7d570394fbdb2ac8e54e
---
 ceph-osd/Chart.yaml                   |  2 +-
 ceph-osd/templates/daemonset-osd.yaml | 13 +++++++++++++
 releasenotes/notes/ceph-osd.yaml      |  1 +
 3 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/ceph-osd/Chart.yaml b/ceph-osd/Chart.yaml
index dbf096fd5..4525dd12b 100644
--- a/ceph-osd/Chart.yaml
+++ b/ceph-osd/Chart.yaml
@@ -15,6 +15,6 @@ apiVersion: v1
 appVersion: v1.0.0
 description: OpenStack-Helm Ceph OSD
 name: ceph-osd
-version: 0.1.25
+version: 0.1.26
 home: https://github.com/ceph/ceph
 ...
diff --git a/ceph-osd/templates/daemonset-osd.yaml b/ceph-osd/templates/daemonset-osd.yaml
index 6dbab0dd1..d188d769a 100644
--- a/ceph-osd/templates/daemonset-osd.yaml
+++ b/ceph-osd/templates/daemonset-osd.yaml
@@ -136,6 +136,9 @@ spec:
             - name: pod-var-lib-ceph-tmp
               mountPath: /var/lib/ceph/tmp
               readOnly: false
+            - name: pod-var-crash
+              mountPath: /var/crash
+              readOnly: false
         - name: ceph-log-ownership
 {{ tuple $envAll "ceph_osd" | include "helm-toolkit.snippets.image" | indent 10 }}
 {{ dict "envAll" $envAll "application" "osd" "container" "ceph_log_ownership" | include "helm-toolkit.snippets.kubernetes_container_security_context" | indent 10 }}
@@ -278,6 +281,9 @@ spec:
             - name: pod-var-log
               mountPath: /var/log/ceph
               readOnly: false
+            - name: pod-var-crash
+              mountPath: /var/crash
+              readOnly: false
       containers:
         - name: log-runner
 {{ tuple $envAll "ceph_osd" | include "helm-toolkit.snippets.image" | indent 10 }}
@@ -441,6 +447,9 @@ spec:
             - name: pod-var-log
               mountPath: /var/log/ceph
               readOnly: false
+            - name: pod-var-crash
+              mountPath: /var/crash
+              readOnly: false
       volumes:
         - name: pod-tmp
           emptyDir: {}
@@ -472,6 +481,10 @@ spec:
           hostPath:
             path: /var/lib/openstack-helm/ceph/var-tmp
             type: DirectoryOrCreate
+        - name: pod-var-crash
+          hostPath:
+            path: /var/crash
+            type: DirectoryOrCreate
         - name: pod-var-log
           emptyDir: {}
         - name: ceph-osd-bin
diff --git a/releasenotes/notes/ceph-osd.yaml b/releasenotes/notes/ceph-osd.yaml
index 277ee0bbb..c2e4b0cc3 100644
--- a/releasenotes/notes/ceph-osd.yaml
+++ b/releasenotes/notes/ceph-osd.yaml
@@ -26,4 +26,5 @@ ceph-osd:
   - 0.1.23 Use full image ref for docker official images
   - 0.1.24 Ceph OSD Init Improvements
   - 0.1.25 Export crash dumps when Ceph daemons crash
+  - 0.1.26 Mount /var/crash inside ceph-osd pods
 ...