Add more debug logs in perf_issuetracker to keep track of the status of list_issues. Bug: b/395663633 Change-Id: Iad765c24eec4e6eb5118818ca78743c438098523 Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/960494 Auto-Submit: Maggie Dong <jiaxindong@google.com> Reviewed-by: Wenbin Zhang <wenbinzhang@google.com> Commit-Queue: Wenbin Zhang <wenbinzhang@google.com>
diff --git a/perf/go/frontend/api/triageApi.go b/perf/go/frontend/api/triageApi.go index 6dd0d3a..4b6b32b 100644 --- a/perf/go/frontend/api/triageApi.go +++ b/perf/go/frontend/api/triageApi.go
@@ -309,11 +309,12 @@ return } + sklog.Debugf("[SkiaTriage] Fetched and returned ListIssuesResponse: %s", resp) + if err := json.NewEncoder(w).Encode(ListIssuesResponse{Issues: resp}); err != nil { httputils.ReportError(w, err, "Failed to write bug id to ListIssuesResponse.", http.StatusInternalServerError) return } - sklog.Debugf("[SkiaTriage] Fetched and returned ListIssuesResponse.") } // For each trace name, mark it as invalidated in the anomalystore's tests cache.
diff --git a/perf/go/issuetracker/BUILD.bazel b/perf/go/issuetracker/BUILD.bazel index feb634a..aaa86b3 100644 --- a/perf/go/issuetracker/BUILD.bazel +++ b/perf/go/issuetracker/BUILD.bazel
@@ -9,6 +9,7 @@ "//go/issuetracker/v1:issuetracker", "//go/secret", "//go/skerr", + "//go/sklog", "//perf/go/config", "@org_golang_google_api//option", "@org_golang_x_oauth2//google",
diff --git a/perf/go/issuetracker/issuetracker.go b/perf/go/issuetracker/issuetracker.go index 6ef1e0b..3981568 100644 --- a/perf/go/issuetracker/issuetracker.go +++ b/perf/go/issuetracker/issuetracker.go
@@ -14,6 +14,7 @@ "go.skia.org/infra/go/secret" "go.skia.org/infra/go/skerr" + "go.skia.org/infra/go/sklog" issuetracker "go.skia.org/infra/go/issuetracker/v1" ) @@ -70,6 +71,8 @@ query := strings.Join(slice, " | ") query = "id: " + query + + sklog.Debugf("[Perf_issuetracker] Start sending list issues request to v1 issuetracker with query: %s", query) requestBodyStr, err := json.Marshal(query) if err != nil { return nil, skerr.Wrapf(err, "Failed to create chrome perf request.") @@ -79,5 +82,7 @@ return nil, skerr.Wrapf(err, "Failed to find issue with request. ") } + sklog.Debugf("[Perf_issuetracker] list issues response received from v1 issuetracker: %s", resp.Issues) + return resp.Issues, nil }
diff --git a/perf/modules/existing-bug-dialog-sk/existing-bug-dialog-sk.ts b/perf/modules/existing-bug-dialog-sk/existing-bug-dialog-sk.ts index 6686a66..b26e5a6 100644 --- a/perf/modules/existing-bug-dialog-sk/existing-bug-dialog-sk.ts +++ b/perf/modules/existing-bug-dialog-sk/existing-bug-dialog-sk.ts
@@ -272,8 +272,11 @@ .then((json) => { const issueList: Issue[] = json.issues; this.bugIdTitleMap = {}; + console.info('Issue list length ' + issueList.length); issueList.forEach((issue) => { const issueid = issue.issueId ? Number(issue.issueId) : 0; + console.info('Issue id: ' + issueid); + console.log('issue title: ' + issue.issueState?.title); if (this._associatedBugIds.has(issueid)) { this.bugIdTitleMap[issueid] = issue.issueState?.title ? issue.issueState!.title : ''; }