🌱 GitLab: Add RepoHost tag to OpenCensus view (#3320)

* Add RepoHost tag to OpenCensus view

Signed-off-by: Raghav Kaul <[email protected]>

* placate godot

Signed-off-by: Raghav Kaul <[email protected]>

* don't panic

Signed-off-by: Raghav Kaul <[email protected]>

* init new logger and emit warning

Signed-off-by: Raghav Kaul <[email protected]>

---------

Signed-off-by: Raghav Kaul <[email protected]>

Author: raghavkaul

checker/check_runner.go

@@ -96,14 +96,21 @@ func (r *Runner) Run(ctx context.Context, c Check) CheckResult {
 				fmt.Sprintf("requiredType: %s not supported by check %s", fmt.Sprint(unsupported), r.CheckName)))
 	}
 
+	l := NewLogger()
 	ctx, err := tag.New(ctx, tag.Upsert(stats.CheckName, r.CheckName))
 	if err != nil {
-		panic(err)
+		l.Warn(&LogMessage{Text: fmt.Sprintf("tag.New: %v", err)})
 	}
+
+	ctx, err = tag.New(ctx, tag.Upsert(stats.RepoHost, r.CheckRequest.Repo.Host()))
+	if err != nil {
+		l.Warn(&LogMessage{Text: fmt.Sprintf("tag.New: %v", err)})
+	}
+
 	startTime := time.Now()
 
 	var res CheckResult
-	l := NewLogger()
+	l = NewLogger()
 	for retriesRemaining := checkRetries; retriesRemaining > 0; retriesRemaining-- {
 		checkRequest := r.CheckRequest
 		checkRequest.Ctx = ctx

stats/tags.go

@@ -23,4 +23,6 @@ var (
 	ErrorName = tag.MustNewKey("errorName")
 	// RequestTag is the tag key for the request type.
 	RequestTag = tag.MustNewKey("requestTag")
+	// RepoHost is the tag key for the host of the repository being scanned.
+	RepoHost = tag.MustNewKey("repoHost")
 )

stats/views.go

@@ -26,7 +26,7 @@ var (
 		Name:        "CheckRuntime",
 		Description: "CPU runtime stats per check",
 		Measure:     CheckRuntimeInSec,
-		TagKeys:     []tag.Key{CheckName},
+		TagKeys:     []tag.Key{CheckName, RepoHost},
 		//nolint:gomnd
 		Aggregation: view.Distribution(
 			0,
@@ -51,7 +51,7 @@ var (
 		Name:        "CheckErrorCount",
 		Description: "Error count by type per check",
 		Measure:     CheckErrors,
-		TagKeys:     []tag.Key{CheckName, ErrorName},
+		TagKeys:     []tag.Key{CheckName, RepoHost, ErrorName},
 		Aggregation: view.Count(),
 	}
 
@@ -60,7 +60,7 @@ var (
 		Name:        "OutgoingHTTPRequests",
 		Description: "HTTPRequests made per check",
 		Measure:     HTTPRequests,
-		TagKeys:     []tag.Key{CheckName, RequestTag},
+		TagKeys:     []tag.Key{CheckName, RepoHost, RequestTag},
 		Aggregation: view.Count(),
 	}
 )