Merge pull request #231 from ventsislav-georgiev/use-status-when-miss…

…ing-conclusion feat(workflows): use status when missing conclusion
promhippie · Jul 13, 2023 · 6b98320 · 6b98320
2 parents d3cef39 + dfc0eae
commit 6b98320
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/pkg/exporter/workflow.go b/pkg/exporter/workflow.go
@@ -124,10 +124,15 @@ func (c *WorkflowCollector) Collect(ch chan<- prometheus.Metric) {
 			strconv.Itoa(record.GetRunAttempt()),
 		}
 
+		status := record.GetConclusion()
+		if status == "" {
+			status = record.GetStatus()
+		}
+
 		ch <- prometheus.MustNewConstMetric(
 			c.Status,
 			prometheus.GaugeValue,
-			conclusionToGauge(record.GetConclusion()),
+			statusToGauge(status),
 			labels...,
 		)
 
@@ -270,7 +275,7 @@ func (c *WorkflowCollector) pagedRepoWorkflows(ctx context.Context, owner, name
 	return workflows, nil
 }
 
-func conclusionToGauge(conclusion string) float64 {
+func statusToGauge(conclusion string) float64 {
 	switch conclusion {
 	case "completed":
 		return 1.0