add comments for switch of syncUnboundClaim

This commit is contained in:
carlory 2023-11-02 18:29:53 +08:00
parent d038b65fee
commit ae24846c48

View File

@ -346,7 +346,7 @@ func (ctrl *PersistentVolumeController) syncUnboundClaim(ctx context.Context, cl
}
if volume == nil {
logger.V(4).Info("Synchronizing unbound PersistentVolumeClaim, no volume found", "PVC", klog.KObj(claim))
// No PV could be found
// No PV could be found. Try to provision one if possible.
// OBSERVATION: pvc is "Pending", will retry
logger.V(4).Info("Attempting to assign storage class to unbound PersistentVolumeClaim", "PVC", klog.KObj(claim))
@ -363,10 +363,15 @@ func (ctrl *PersistentVolumeController) syncUnboundClaim(ctx context.Context, cl
switch {
case delayBinding && !storagehelpers.IsDelayBindingProvisioning(claim):
// Scheduler does not observe any pod using this claim.
if err = ctrl.emitEventForUnboundDelayBindingClaim(claim); err != nil {
return err
}
case storagehelpers.GetPersistentVolumeClaimClass(claim) != "":
// The provisionClaim function may start a new asynchronous operation to provision a volume,
// or the operation is already running. The claim will be updated in the asynchronous operation,
// so the branch should be returned directly and the bind operation is expected to continue in
// the next sync loop.
if err = ctrl.provisionClaim(ctx, claim); err != nil {
return err
}