Skip to content

Commit 1f9725d

Browse files
kaworuaanm
authored andcommitted
cmd/client: load the full config
Signed-off-by: Alexandre Perrin <[email protected]>
1 parent 35932b7 commit 1f9725d

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

cmd/client.go

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -63,16 +63,16 @@ var (
6363
)
6464

6565
func runClient() {
66-
flakeCfg, err := loadConfig(config)
66+
cfg, err := loadConfig(config)
6767
if err != nil {
6868
panic(err)
6969
}
7070

71-
triggerRegexp := regexp.MustCompile(flakeCfg.JenkinsConfig.RegexTrigger)
71+
triggerRegexp := regexp.MustCompile(cfg.FlakeTracker.JenkinsConfig.RegexTrigger)
7272

7373
useCache := false
7474

75-
if len(flakeCfg.JenkinsConfig.StableJobNames) != len(flakeCfg.JenkinsConfig.PRJobNames) {
75+
if len(cfg.FlakeTracker.JenkinsConfig.StableJobNames) != len(cfg.FlakeTracker.JenkinsConfig.PRJobNames) {
7676
panic(fmt.Sprintf("%s jobs and PR jobs should have the same length", baseBranch))
7777
}
7878
var (
@@ -129,7 +129,7 @@ func runClient() {
129129
// GH Issues
130130
fmt.Printf("Getting Issues from GH\n")
131131

132-
issueKnownFlakes, err = ghClient.GetFlakeIssues(globalCtx, orgName, repoName, github.IssueCreator, flakeCfg.IssueTracker.IssueLabels)
132+
issueKnownFlakes, err = ghClient.GetFlakeIssues(globalCtx, orgName, repoName, github.IssueCreator, cfg.FlakeTracker.IssueTracker.IssueLabels)
133133
if err != nil {
134134
panic(err)
135135
}
@@ -144,7 +144,7 @@ func runClient() {
144144
jobNameToJenkinsFails = map[string]jenkins.JenkinsFailures{}
145145
}
146146

147-
jc, err := jenkins.NewJenkinsClient(globalCtx, flakeCfg.JenkinsConfig.JenkinsURL, false)
147+
jc, err := jenkins.NewJenkinsClient(globalCtx, cfg.FlakeTracker.JenkinsConfig.JenkinsURL, false)
148148
if err != nil {
149149
panic(err)
150150
}
@@ -156,24 +156,27 @@ func runClient() {
156156
default:
157157
}
158158

159-
err := ghClient.TriagePRFailures(globalCtx, jc, flakeCfg, prNumber, urlFails, issueKnownFlakes, jobNameToJenkinsFails, triggerRegexp)
159+
err := ghClient.TriagePRFailures(globalCtx, jc, cfg.FlakeTracker, prNumber, urlFails, issueKnownFlakes, jobNameToJenkinsFails, triggerRegexp)
160160
if err != nil {
161161
panic(err)
162162
}
163163
}
164164
}
165165

166-
func loadConfig(cfgFile string) (*github.FlakeConfig, error) {
166+
func loadConfig(cfgFile string) (*github.PRBlockerConfig, error) {
167167
b, err := ioutil.ReadFile(cfgFile)
168168
if err != nil {
169169
return nil, err
170170
}
171171

172-
var cfg github.FlakeConfig
172+
var cfg github.PRBlockerConfig
173173
err = yaml.Unmarshal(b, &cfg)
174174
if err != nil {
175175
return nil, err
176176
}
177+
if cfg.FlakeTracker == nil {
178+
cfg.FlakeTracker = &github.FlakeConfig{}
179+
}
177180

178181
return &cfg, nil
179182
}

0 commit comments

Comments
 (0)