[pinpoint] Register culprit finder workflow to temporal worker
This CL also does some minor clean up of a few comments.
Change-Id: Id725987b9f994964639e6b47e2bf544a5e598cda
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/862900
Auto-Submit: Leina Sun <sunxiaodi@google.com>
Reviewed-by: Kaylee Lubick <kjlubick@google.com>
Commit-Queue: Leina Sun <sunxiaodi@google.com>
diff --git a/pinpoint/go/workflows/worker/main.go b/pinpoint/go/workflows/worker/main.go
index 623d9be..9ac6001 100644
--- a/pinpoint/go/workflows/worker/main.go
+++ b/pinpoint/go/workflows/worker/main.go
@@ -80,13 +80,14 @@
w.RegisterActivity(internal.PostBugCommentActivity)
w.RegisterWorkflowWithOptions(internal.PostBugCommentWorkflow, workflow.RegisterOptions{Name: workflows.BugUpdate})
- // TODO(b/322203189) - Remove Catapult workflows and activities once the backwards UI compatibility is no longer needed
- // and thus the catapult package is deprecated.
+ // TODO(b/322203189) - Remove Catapult workflows and activities once the backwards
+ // UIcompatibility is no longer needed and thus the catapult package is deprecated.
w.RegisterActivity(catapult.FetchTaskActivity)
w.RegisterActivity(catapult.FetchCommitActivity)
w.RegisterActivity(catapult.WriteBisectToCatapultActivity)
w.RegisterWorkflowWithOptions(catapult.CatapultBisectWorkflow, workflow.RegisterOptions{Name: workflows.CatapultBisect})
w.RegisterWorkflowWithOptions(catapult.ConvertToCatapultResponseWorkflow, workflow.RegisterOptions{Name: workflows.ConvertToCatapultResponseWorkflow})
+ w.RegisterWorkflowWithOptions(catapult.CulpritFinderWorkflow, workflow.RegisterOptions{Name: workflows.CulpritFinderWorkflow})
err = w.Run(worker.InterruptCh())
if err != nil {
diff --git a/pinpoint/go/workflows/workflows.go b/pinpoint/go/workflows/workflows.go
index a6a3775..d0452b5 100644
--- a/pinpoint/go/workflows/workflows.go
+++ b/pinpoint/go/workflows/workflows.go
@@ -18,7 +18,6 @@
// Those are used to invoke the workflows. This is meant to decouple the
// souce code dependencies such that the client doesn't need to link with
// the actual implementation.
-// TODO(b/326352379): introduce a specific type to encapsulate these workflow names
const (
Bisect = "perf.bisect"
BuildChrome = "perf.build_chrome"
