rpk: collect crash_reports in debug bundle

Same as startup_log, we will only collect the
directory if it's present

src/go/rpk/pkg/cli/debug/bundle/bundle.go

@@ -249,6 +249,9 @@ COMMON FILES
  - Broker metrics: The broker's Prometheus metrics, fetched through its
    admin API (/metrics and /public_metrics).
 
+ - Crash information: Both startup_log and crash_reports will be collected if
+   present in the configured data directory.
+
 BARE-METAL
 
  - Kernel: The kernel logs ring buffer (syslog) and parameters (sysctl).

src/go/rpk/pkg/cli/debug/bundle/bundle_k8s_linux.go

@@ -70,6 +70,7 @@ func executeK8SBundle(ctx context.Context, bp bundleParams) error {
 		saveCmdLine(ps),
 		saveConfig(ps, bp.yActual),
 		saveControllerLogDir(ps, bp.y, bp.controllerLogLimitBytes),
+		saveCrashReports(ps, bp.y),
 		saveDataDirStructure(ps, bp.y),
 		saveDiskUsage(ctx, ps, bp.y),
 		saveInterrupts(ps),

src/go/rpk/pkg/cli/debug/bundle/bundle_linux.go

@@ -135,6 +135,7 @@ func executeBundle(ctx context.Context, bp bundleParams) error {
 		saveCmdLine(ps),
 		saveConfig(ps, bp.y),
 		saveControllerLogDir(ps, bp.y, bp.controllerLogLimitBytes),
+		saveCrashReports(ps, bp.y),
 		saveDNSData(ctx, ps),
 		saveDataDirStructure(ps, bp.y),
 		saveDiskUsage(ctx, ps, bp.y),
@@ -1047,6 +1048,27 @@ func saveStartupLog(ps *stepParams, y *config.RedpandaYaml) step {
 	}
 }
 
+func saveCrashReports(ps *stepParams, y *config.RedpandaYaml) step {
+	return func() error {
+		if y.Redpanda.Directory == "" {
+			return fmt.Errorf("failed to save crash_reports: 'redpanda.data_directory' is empty on the provided configuration file")
+		}
+		crashReportDir := filepath.Join(y.Redpanda.Directory, "crash_reports")
+		exists, err := afero.Exists(ps.fs, crashReportDir)
+		if err != nil {
+			return fmt.Errorf("failed to save crash_reports: unable to check existence of the crash_reports directory")
+		}
+		if !exists {
+			return fmt.Errorf("skipping crash_reports collection: directory %q does not exists", crashReportDir)
+		}
+		err = writeDirToZip(ps, crashReportDir, "crash_reports", nil)
+		if err != nil {
+			return fmt.Errorf("failed to save crash_reports: %v", err)
+		}
+		return nil
+	}
+}
+
 func walkDir(root string, files map[string]*fileInfo) error {
 	return filepath.WalkDir(
 		root,

tests/rptest/tests/rpk_debug_bundle_test.py

@@ -70,6 +70,13 @@ def test_debug_bundle(self):
                 continue
             if re.match(r".* error querying .*\.ntp\..* i\/o timeout", l):
                 self.logger.error(f"Non-fatal transitory NTP error: {l}")
+            if re.match(
+                    r".*skipping\s+(startup_log collection|crash_reports collection):\s*(unable to find file|directory).*",
+                    l):
+                # this tests runs a development container, it will not have a
+                # startup_log and we don't expect a crash_reports dir to be
+                # in the data_directory as the container is new.
+                continue
             else:
                 self.logger.error(f"Bad output line: {l}")
                 filtered_errors.append(l)