diff --git a/docs/statusreport-controller.md b/docs/statusreport-controller.md index b529d92f6..28305d423 100644 --- a/docs/statusreport-controller.md +++ b/docs/statusreport-controller.md @@ -56,7 +56,7 @@ flowchart TD does_commitStatus_exist{Does commitStatus exist
on github already?} create_new_commitStatus_on_gh(Create new commitStatus on github
if PR is not from forked repo) does_comment_exist(Does a comment exist for snapshot and scenario?) - update_existing_comment(Update the existing comment for
snapshot and scenario

if PR is not from forked repo) + update_existing_comment(Update the existing comment for
snapshot and scenario
) create_new_comment(Create a new comment for
snapshot and scenario
) collect_commit_info_gl(Collect commit projectID, repo-url and SHA from Snapshot) diff --git a/status/reporter_github.go b/status/reporter_github.go index 9aa372d65..22f1bead5 100644 --- a/status/reporter_github.go +++ b/status/reporter_github.go @@ -456,7 +456,7 @@ func (csu *CommitStatusUpdater) UpdateStatus(ctx context.Context, report TestRep "snapshot.NameSpace", csu.snapshot.Namespace, "snapshot.Name", csu.snapshot.Name, "scenarioName", report.ScenarioName) } } else { - csu.logger.Info("Won't create/update commitStatus since there is access limimation for different source and target Repo Owner", + csu.logger.Info("Won't create/update commitStatus since there is access limitation for different source and target Repo Owner", "snapshot.NameSpace", csu.snapshot.Namespace, "snapshot.Name", csu.snapshot.Name, "sourceRepoOwner", sourceRepoOwner, "targetRepoOwner", csu.owner) } // Create a comment when integration test is neither pending nor inprogress since comment for pending/inprogress is less meaningful and there is commitStatus for all statuses diff --git a/status/reporter_github_test.go b/status/reporter_github_test.go index 5814e6f04..397a6a4af 100644 --- a/status/reporter_github_test.go +++ b/status/reporter_github_test.go @@ -607,7 +607,7 @@ var _ = Describe("GitHubReporter", func() { Summary: "Integration test for snapshot snapshot-sample and scenario scenario2 is pending", } Expect(reporter.ReportStatus(context.TODO(), testReport)).To(Succeed()) - expectedLogEntry := "Won't create/update commitStatus since there is access limimation for different source and target Repo Owner" + expectedLogEntry := "Won't create/update commitStatus since there is access limitation for different source and target Repo Owner" Expect(buf.String()).Should(ContainSubstring(expectedLogEntry)) }) })