Merge pull request #929 from 14rcole/groupsnapshot-reporting-panic

fix: group snapshot error causing panic
konflux-ci · Nov 13, 2024 · d5085ad · d5085ad
2 parents ff7a734 + 2f7f58f
commit d5085ad
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/internal/controller/buildpipeline/buildpipeline_adapter.go b/internal/controller/buildpipeline/buildpipeline_adapter.go
@@ -246,11 +246,14 @@ func (a *Adapter) notifySnapshotsInGroupAboutFailedBuild(pipelineRun *tektonv1.P
 			a.logger.Error(err, "Failed to fetch Snapshots for component", "component.Name", applicationComponent.Name)
 			return err
 		}
-		latestSnapshot := gitops.SortSnapshots(*allComponentSnapshotsInGroup)[0]
-		err = gitops.AnnotateSnapshot(a.context, &latestSnapshot, gitops.PRGroupCreationAnnotation,
-			buildPLRFailureMsg, a.client)
-		if err != nil {
-			return err
+
+		if len(*allComponentSnapshotsInGroup) > 0 {
+			latestSnapshot := gitops.SortSnapshots(*allComponentSnapshotsInGroup)[0]
+			err = gitops.AnnotateSnapshot(a.context, &latestSnapshot, gitops.PRGroupCreationAnnotation,
+				buildPLRFailureMsg, a.client)
+			if err != nil {
+				return err
+			}
 		}
 	}