diff --git a/tools/flaky-test-reporter/github_issue_test.go b/tools/flaky-test-reporter/github_issue_test.go index 5a2c2be23a..f2a1a7600e 100644 --- a/tools/flaky-test-reporter/github_issue_test.go +++ b/tools/flaky-test-reporter/github_issue_test.go @@ -80,7 +80,7 @@ func createNewIssue(fgi *GithubIssue, title, body, testStat string) (*github.Iss } func createRepoData(passed, flaky, failed, notenoughdata int, startTime int64) *RepoData { - config := &JobConfig{ + config := JobConfig{ Repo: fakeRepo, } tss := map[string]*TestStat{} diff --git a/tools/flaky-test-reporter/main.go b/tools/flaky-test-reporter/main.go index b2b5d48344..6e77b044a1 100644 --- a/tools/flaky-test-reporter/main.go +++ b/tools/flaky-test-reporter/main.go @@ -64,7 +64,7 @@ func main() { for _, jc := range jobConfigs { log.Printf("collecting results for repo '%s'\n", jc.Repo) - rd, err := collectTestResultsForRepo(&jc) + rd, err := collectTestResultsForRepo(jc) if nil != err { log.Fatalf("Error collecting results for repo '%s': %v", jc.Repo, err) } diff --git a/tools/flaky-test-reporter/result.go b/tools/flaky-test-reporter/result.go index 63d60204a2..6bcaea2834 100644 --- a/tools/flaky-test-reporter/result.go +++ b/tools/flaky-test-reporter/result.go @@ -41,7 +41,7 @@ const ( // RepoData struct contains all configurations and test results for a repo type RepoData struct { - Config *JobConfig + Config JobConfig TestStats map[string]*TestStat // key is test full name BuildIDs []int // all build IDs scanned in this run LastBuildStartTime *int64 // timestamp, determines how fresh the data is @@ -169,7 +169,7 @@ func getCombinedResultsForBuild(build *prow.Build) ([]*junit.TestSuites, error) // collectTestResultsForRepo collects test results, build IDs from all builds, // as well as LastBuildStartTime, and stores them in RepoData -func collectTestResultsForRepo(jc *JobConfig) (*RepoData, error) { +func collectTestResultsForRepo(jc JobConfig) (*RepoData, error) { rd := &RepoData{Config: jc} job := prow.NewJob(jc.Name, jc.Type, jc.Repo, 0) if !job.PathExists() {