kubernetes-sigs / gcp-filestore-csi-driver

The Google Cloud Filestore Container Storage Interface (CSI) Plugin.
Apache License 2.0
90 stars 73 forks source link

Change error reporting to Temporary while Creating Volume #925

Open riteshghorse opened 1 month ago

riteshghorse commented 1 month ago

When CreateVolume is called, the csi-provisioner expects either "final" or "temporary" errors. If we report final error when Filestore instance creation could be ongoing or it is already created, then a PVC deletion before successful volume creation could lead to resource leakage as DeleteVolume won't happen.

This is already done for gcp-compute-persistent-disk-csi-driver.

PRs:

riteshghorse commented 1 month ago

/assign

leiyiz commented 1 week ago

From #924: handling the metrics for new Temporary error types. This will be done in separate PR.