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 : '';
}
