e2e,failures: Include links to the failed prow jobs for each test lane in the failure board

Having the failed job links included makes these metrics more actionable
as the relevant SIGs can quickly check the failures.

Signed-off-by: Brian Carey <[email protected]>

Author: brianmcarey

pkg/sigretests/main.go

@@ -30,6 +30,7 @@ type SigRetests struct {
 	SigStorageSuccess  int
 	SigOperatorSuccess int
 	FailedJobNames     []string
+	FailedJobURLs      []string
 	SuccessJobNames    []string
 }
 
@@ -55,7 +56,7 @@ func filterJobs(node *html.Node) (jobs []job) {
 				for _, href := range node.FirstChild.Attr {
 					if strings.Contains(href.Val, "e2e") {
 						e2eJob.failure = true
-						e2eJob.buildURL = href.Val
+						e2eJob.buildURL = fmt.Sprintf("https://prow.ci.kubevirt.io/%s", href.Val)
 						buildLogUrl := strings.Split(href.Val, "/")
 						e2eJob.jobName = buildLogUrl[len(buildLogUrl)-2]
 						e2eJob.buildNumber = buildLogUrl[len(buildLogUrl)-1]
@@ -160,6 +161,7 @@ func getJobsForLatestCommit(org string, repo string, prNumber string) (jobsLastC
 func sortJobNamesOnResult(job job, sigRetests SigRetests) (jobCounts SigRetests) {
 	if job.failure == true {
 		sigRetests.FailedJobNames = append(sigRetests.FailedJobNames, job.jobName)
+		sigRetests.FailedJobURLs = append(sigRetests.FailedJobURLs, job.buildURL)
 	} else {
 		sigRetests.SuccessJobNames = append(sigRetests.SuccessJobNames, job.jobName)
 	}

pkg/stats/stats.go

@@ -5,6 +5,7 @@ import (
 	"math"
 	"slices"
 	"strconv"
+	"strings"
 	"time"
 
 	"github.com/kubevirt/ci-health/pkg/chatops"
@@ -256,6 +257,7 @@ func (h *Handler) mergedPRsProcessor(results *types.Results) (*types.Results, er
 func (h *Handler) sigRetestsProcessor(results *types.Results) (*types.Results, error) {
 	currentTime, err := time.Parse(constants.DateFormat, results.EndDate)
 	var failedJobNames []string
+	var failedJobURLs []string
 	var successJobNames []string
 	if err != nil {
 		return results, err
@@ -290,6 +292,7 @@ func (h *Handler) sigRetestsProcessor(results *types.Results) (*types.Results, e
 			})
 		failedJobNames = slices.Concat(failedJobNames, jobsPerSIG.FailedJobNames)
 		successJobNames = slices.Concat(successJobNames, jobsPerSIG.SuccessJobNames)
+		failedJobURLs = slices.Concat(failedJobURLs, jobsPerSIG.FailedJobURLs)
 	}
 	sortedFailedJobs := types.SortByMostFailed(countFailedJobs(failedJobNames))
 	for i, job := range sortedFailedJobs {
@@ -298,6 +301,12 @@ func (h *Handler) sigRetestsProcessor(results *types.Results) (*types.Results, e
 				sortedFailedJobs[i].SuccesCount++
 			}
 		}
+		for _, failedJobURL := range failedJobURLs {
+			jobNameInURL := strings.Split(failedJobURL, "/")[len(strings.Split(failedJobURL, "/"))-2]
+			if job.JobName == jobNameInURL {
+				sortedFailedJobs[i].FailureURLs = append(sortedFailedJobs[i].FailureURLs, failedJobURL)
+			}
+		}
 	}
 	dataItem.FailedJobLeaderBoard = sortedFailedJobs
 	results.Data[constants.SIGRetests] = dataItem

pkg/types/main.go

@@ -215,6 +215,7 @@ type FailedJob struct {
 	JobName      string
 	FailureCount int
 	SuccesCount  int
+	FailureURLs  []string
 }
 
 type FailedJobs []FailedJob
@@ -227,7 +228,7 @@ func SortByMostFailed(countFailedJobs map[string]int) FailedJobs {
 	fl := make(FailedJobs, len(countFailedJobs))
 	i := 0
 	for j, f := range countFailedJobs {
-		fl[i] = FailedJob{j, f, 0}
+		fl[i] = FailedJob{j, f, 0, nil}
 		i++
 	}
 	sort.Sort(sort.Reverse(fl))