fix: do not update parent & path if graph cannot be formed

flanksource · May 14, 2024 · 268a0a9 · 268a0a9
1 parent 9760964
commit 268a0a9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/db/update.go b/db/update.go
@@ -627,6 +627,12 @@ func extractConfigsAndChangesFromResults(ctx api.ScrapeContext, scrapeStartTime
 				if existing == nil || existing.ID == "" {
 					newConfigs = append(newConfigs, ci)
 				} else {
+					// In case, we are not able to derive the path & parent_id
+					// by forming a tree, we need to use the existing one
+					// otherwise they'll be updated to empty values
+					ci.ParentID = existing.ParentID
+					ci.Path = existing.Path
+
 					configsToUpdate = append(configsToUpdate, &updateConfigArgs{
 						Result:   result,
 						Existing: existing,

diff --git a/scrapers/cron.go b/scrapers/cron.go
@@ -144,6 +144,7 @@ func scheduleScraperJob(sc api.ScrapeContext) error {
 		Schedule:     schedule,
 		Singleton:    true,
 		JobHistory:   true,
+		RunNow:       true,
 		Retention:    job.RetentionBalanced,
 		ResourceID:   sc.ScrapeConfig().GetPersistedID().String(),
 		ResourceType: job.ResourceTypeScraper,