add diagnostic for data mover exposer

Signed-off-by: Lyndon-Li <[email protected]>
reasonerjt · Dec 4, 2024 · b607259 · b607259
1 parent abbfac0
commit b607259
Show file tree

Hide file tree

Showing 11 changed files with 996 additions and 36 deletions.
diff --git a/changelogs/unreleased/8482-Lyndon-Li b/changelogs/unreleased/8482-Lyndon-Li
@@ -0,0 +1 @@
+Fix issue #8125, add diagnostic info for data mover exposers when expose timeout
diff --git a/pkg/exposer/csi_snapshot.go b/pkg/exposer/csi_snapshot.go
@@ -355,7 +355,7 @@ func (e *csiSnapshotExposer) DiagnoseExpose(ctx context.Context, ownerObject cor
 	if vs != nil {
 		diag += csi.DiagnoseVS(vs)
 
-		if vs.Status.BoundVolumeSnapshotContentName != nil && *vs.Status.BoundVolumeSnapshotContentName != "" {
+		if vs.Status != nil && vs.Status.BoundVolumeSnapshotContentName != nil && *vs.Status.BoundVolumeSnapshotContentName != "" {
 			if vsc, err := e.csiSnapshotClient.VolumeSnapshotContents().Get(ctx, *vs.Status.BoundVolumeSnapshotContentName, metav1.GetOptions{}); err != nil {
 				diag += fmt.Sprintf("error getting backup vsc %s, err: %v\n", *vs.Status.BoundVolumeSnapshotContentName, err)
 			} else {
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix issue #8125, add diagnostic info for data mover exposers when expose timeout