fix: ignore before orphaning

Author: adityathebe

db/update.go

@@ -299,6 +299,16 @@ func extractChanges(ctx api.ScrapeContext, result *v1.ScrapeResult, ci *models.C
 			continue
 		}
 
+		if exclude, err := shouldExcludeChange(ctx, result, changeResult); err != nil {
+			ctx.JobHistory().AddError(fmt.Sprintf("error running change exclusion: %v", err))
+		} else if exclude {
+			changeSummary.AddIgnored(changeResult.ChangeType)
+			if logExclusions {
+				ctx.Logger.V(3).Infof("excluded change: %v", changeResult)
+			}
+			continue
+		}
+
 		if changeResult.ConfigID != "" {
 			if _, ok := OrphanCache.Get(changeResult.ConfigID); ok {
 				changeSummary.AddOrphaned(changeResult.ChangeType, changeResult.ConfigID)
@@ -313,16 +323,6 @@ func extractChanges(ctx api.ScrapeContext, result *v1.ScrapeResult, ci *models.C
 			}
 		}
 
-		if exclude, err := shouldExcludeChange(ctx, result, changeResult); err != nil {
-			ctx.JobHistory().AddError(fmt.Sprintf("error running change exclusion: %v", err))
-		} else if exclude {
-			changeSummary.AddIgnored(changeResult.ChangeType)
-			if logExclusions {
-				ctx.Logger.V(3).Infof("excluded change: %v", changeResult)
-			}
-			continue
-		}
-
 		if changeResult.Action == v1.Delete {
 			if err := deleteChangeHandler(ctx, changeResult); err != nil {
 				return nil, nil, changeSummary, fmt.Errorf("failed to delete config from change: %w", err)