fix

Signed-off-by: Valeriy Khorunzhin <valeriy.khorunzhin@flant.com>

Author: Valeriy Khorunzhin

images/virtualization-artifact/pkg/controller/cvi/internal/source/object_ref.go

@@ -104,7 +104,7 @@ func (ds ObjectRefDataSource) Sync(ctx context.Context, cvi *virtv2.ClusterVirtu
 		}
 
 		if vi == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VI object ref source %s is nil", cvi.Spec.DataSource.ObjectRef.Name)
 		}
 
 		if vi.Spec.Storage == virtv2.StorageKubernetes || vi.Spec.Storage == virtv2.StoragePersistentVolumeClaim {
@@ -118,7 +118,7 @@ func (ds ObjectRefDataSource) Sync(ctx context.Context, cvi *virtv2.ClusterVirtu
 		}
 
 		if vd == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VD object ref source %s is nil", cvi.Spec.DataSource.ObjectRef.Name)
 		}
 
 		return ds.vdSyncer.Sync(ctx, cvi, vd, cb)
@@ -131,7 +131,7 @@ func (ds ObjectRefDataSource) Sync(ctx context.Context, cvi *virtv2.ClusterVirtu
 		}
 
 		if vdSnapshot == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VDSnapshot object ref %s is nil", vdSnapshotKey)
 		}
 
 		return ds.vdSnapshotSyncer.Sync(ctx, cvi, vdSnapshot, cb)

images/virtualization-artifact/pkg/controller/vd/internal/source/object_ref.go

@@ -72,7 +72,7 @@ func (ds ObjectRefDataSource) Sync(ctx context.Context, vd *virtv2.VirtualDisk)
 		}
 
 		if vi == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VI object ref source %s is nil", vd.Spec.DataSource.ObjectRef.Name)
 		}
 
 		switch vi.Spec.Storage {

images/virtualization-artifact/pkg/controller/vi/internal/source/object_ref.go

@@ -110,7 +110,7 @@ func (ds ObjectRefDataSource) StoreToPVC(ctx context.Context, vi *virtv2.Virtual
 		}
 
 		if viRef == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VI object ref %s is nil", viKey)
 		}
 
 		if viRef.Spec.Storage == virtv2.StorageKubernetes || viRef.Spec.Storage == virtv2.StoragePersistentVolumeClaim {
@@ -124,7 +124,7 @@ func (ds ObjectRefDataSource) StoreToPVC(ctx context.Context, vi *virtv2.Virtual
 		}
 
 		if vd == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VD object ref %s is nil", vdKey)
 		}
 
 		return ds.vdSyncer.StoreToPVC(ctx, vi, vd, cb)
@@ -317,7 +317,7 @@ func (ds ObjectRefDataSource) StoreToDVCR(ctx context.Context, vi *virtv2.Virtua
 		}
 
 		if viRef == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VI object ref source %s is nil", vi.Spec.DataSource.ObjectRef.Name)
 		}
 
 		if viRef.Spec.Storage == virtv2.StorageKubernetes || viRef.Spec.Storage == virtv2.StoragePersistentVolumeClaim {
@@ -331,7 +331,7 @@ func (ds ObjectRefDataSource) StoreToDVCR(ctx context.Context, vi *virtv2.Virtua
 		}
 
 		if vd == nil {
-			return reconcile.Result{}, nil
+			return reconcile.Result{}, fmt.Errorf("VD object ref %s is nil", viKey)
 		}
 
 		return ds.vdSyncer.StoreToDVCR(ctx, vi, vd, cb)