Skip to content

Commit a9f9a90

Browse files
mrunalpagnisl-technicore
authored andcommitted
FSS controller nil pointer reference causing panic and CPO restarts
1 parent f60fa59 commit a9f9a90

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/csi/driver/fss_controller.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ func (d *FSSControllerDriver) getOrCreateExport(ctx context.Context, err error,
398398

399399
provisionedExport := &fss.Export{}
400400
if exportSummary != nil {
401-
provisionedExport = &fss.Export{Id: exportSummary.ExportSetId}
401+
provisionedExport = &fss.Export{Id: exportSummary.Id}
402402
} else {
403403
// Creating new export
404404
provisionedExport, err = provisionExport(ctx, log, d.client, filesystemOCID, exportSetId, storageClassParameters)
@@ -641,9 +641,9 @@ func (d *FSSControllerDriver) DeleteVolume(ctx context.Context, req *csi.DeleteV
641641
metrics.SendMetricData(d.metricPusher, metrics.FSSDelete, time.Since(startTime).Seconds(), dimensionsMap)
642642
metrics.SendMetricData(d.metricPusher, metrics.FssAllDelete, time.Since(startTime).Seconds(), dimensionsMap)
643643
return nil, status.Errorf(codes.Internal, "failed to delete filesystem, volumeId: %s ERROR: %v", volumeId, err.Error())
644-
} else {
645-
log.Info("File system does not exist.")
646644
}
645+
log.Info("File system does not exist.")
646+
return &csi.DeleteVolumeResponse{}, nil
647647
}
648648

649649
compartmentID := *fileSystem.CompartmentId

0 commit comments

Comments
 (0)