Merge pull request #1408 from kaytu-io/fix-steampipe-upgrade

fix: remove hard code url
opengovern · Aug 2, 2024 · f99211a · f99211a
2 parents 1eb29ec + 7bf892b
commit f99211a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pkg/describe/server.go b/pkg/describe/server.go
@@ -826,6 +826,10 @@ func (h HttpServer) ReEvaluateComplianceJob(ctx echo.Context) error {
 		NextJobParameters: &jp,
 		Status:            model2.JobSequencerWaitingForDependencies,
 	})
+	if err != nil {
+		h.Scheduler.logger.Error("failed to create job sequencer", zap.Error(err))
+		return err
+	}
 
 	return ctx.NoContent(http.StatusOK)
 }

diff --git a/services/migrator/job/git.go b/services/migrator/job/git.go
@@ -25,14 +25,11 @@ func GitClone(conf config.MigratorConfig, logger *zap.Logger) (string, error) {
 		UserRole: api.AdminRole,
 	}, models.MetadataKeyAnalyticsGitURL)
 
-	logger.Info("url before metadata", zap.String("url", gitConfig.AnalyticsGitURL))
 	if err == nil && len(value.GetValue().(string)) > 0 {
 		gitConfig.AnalyticsGitURL = value.GetValue().(string)
 	} else if err != nil {
 		logger.Error("failed to get analytics git url from metadata", zap.Error(err))
 	}
-	logger.Info("url after metadata", zap.String("url", gitConfig.AnalyticsGitURL))
-	gitConfig.AnalyticsGitURL = "https://github.com/kaytu-io/configz-deprecated"
 
 	logger.Info("using git repo", zap.String("url", gitConfig.AnalyticsGitURL))