From a279757b204971cfde11588e6242a8d17118a04d77bc7fcc29a1cbd16bb20c71 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Thu, 8 May 2025 10:57:17 +0200 Subject: [PATCH 01/11] Staging: include also changed directories Projects which are not using submodules as package directories still need to be handled in stagings. --- common/git_utils.go | 1 + 1 file changed, 1 insertion(+) diff --git a/common/git_utils.go b/common/git_utils.go index a7e76e2..720173d 100644 --- a/common/git_utils.go +++ b/common/git_utils.go @@ -904,6 +904,7 @@ func (e *GitHandlerImpl) GitSubmoduleList(gitPath, commitId string) (submoduleLi for _, te := range tree.items { if te.isTree() { trees[p+te.name+"/"] = te.hash + submoduleList[p+te.name] = te.hash } else if te.isSubmodule() { submoduleList[p+te.name] = te.hash } -- 2.51.1 From d8034ee04ca27a97aa4cd0b958b19c2489747220cebc0dd1950cb57bc1599b6a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Mon, 1 Sep 2025 17:23:41 +0200 Subject: [PATCH 02/11] obs_staging: fix results compare Esp. with multiple repositories: * j counter was not reset * wrong limit of results used (from reference not stage project) * package name compare using == was always wrong (and unneeded here) --- obs-staging-bot/main.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index fd14c37..c589f34 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -225,7 +225,6 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B slices.SortFunc(results, PackageBuildStatusSorter) slices.SortFunc(ref, PackageBuildStatusSorter) - j := 0 SomeSuccess = false for i := 0; i < len(results); i++ { res, ok := common.ObsBuildStatusDetails[results[i].Code] @@ -240,10 +239,11 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B if !res.Success { // not failed if reference project also failed for same package here - for ; j < len(results) && strings.Compare(results[i].Package, ref[j].Package) < 0; j++ { + j := 0 + for ; j < len(ref) && strings.Compare(results[i].Package, ref[j].Package) != 0; j++ { } - if j < len(results) && results[i].Package == ref[j].Package { + if j < len(ref) && results[i].Package == ref[j].Package { refRes, ok := common.ObsBuildStatusDetails[ref[j].Code] if !ok { common.LogInfo("unknown ref package result code:", ref[j].Code, "package:", ref[j].Package) @@ -324,7 +324,6 @@ func GenerateObsPrjMeta(git common.Git, gitea common.Gitea, pr *models.PullReque } // generate new project with paths pointinig back to original repos - // disable publishing meta.Name = stagingPrj meta.Description = fmt.Sprintf(`Pull request build job PR#%d to branch %s of %s/%s`, -- 2.51.1 From e269650599879a5fed488c81f0abb827954a408fd0ce5b1a076e63c1aebfd907 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Thu, 11 Sep 2025 10:07:20 +0200 Subject: [PATCH 03/11] workflow-pr: don't crash when no submodule got changed eg. config changes only --- workflow-pr/pr_processor.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/workflow-pr/pr_processor.go b/workflow-pr/pr_processor.go index 6128c95..b100f75 100644 --- a/workflow-pr/pr_processor.go +++ b/workflow-pr/pr_processor.go @@ -460,10 +460,10 @@ func (pr *PRProcessor) Process(req *models.PullRequest) error { continue } } - } - if err = pr.UpdatePrjGitPR(prset); err != nil { - return err + if err = pr.UpdatePrjGitPR(prset); err != nil { + return err + } } } -- 2.51.1 From 9a90fb705f163c3fd42795bdaa3c16df73ef146e4c51114258c0e24c00465885 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Tue, 23 Sep 2025 09:09:23 +0200 Subject: [PATCH 04/11] obs-staging: fix onlybuild for subdirectories The parameter only takes a package name, but not a full subdirectory string. So we hand over only the last part behind a / here --- obs-staging-bot/main.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index c589f34..28ba0d9 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -338,7 +338,10 @@ func GenerateObsPrjMeta(git common.Git, gitea common.Gitea, pr *models.PullReque urlPkg := make([]string, 0, len(modifiedOrNew)) for _, pkg := range modifiedOrNew { - urlPkg = append(urlPkg, "onlybuild="+url.QueryEscape(pkg)) + // FIXME: skip manifest subdirectories itself + // strip any leading directory name and just hand over last directory as package name + onlybuilds := strings.Split(pkg, "/") + urlPkg = append(urlPkg, "onlybuild="+url.QueryEscape(onlybuilds[len(onlybuilds)-1])) } meta.ScmSync = pr.Head.Repo.CloneURL + "?" + strings.Join(urlPkg, "&") + "#" + pr.Head.Sha if len(meta.ScmSync) >= 65535 { -- 2.51.1 From 797f9f1c9c42b83278865bf713c593455be68a96417de507b9f8495173968dfb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Tue, 23 Sep 2025 16:55:06 +0200 Subject: [PATCH 05/11] obs-staging-bot: fix wrong acceptance Any pull request with multiple packages was always successful, just when one package was successful build. Dropping the "someSuccess" handling for now since it is currently broken and I got also complaints that we anyway never should accept broken submissions. Even when the failure did exist before. This would anyway break an mainteannce/patchinfo build. We may want to add a config option to allow the again under given circumstances. --- obs-staging-bot/main.go | 29 +++++++++++------------------ 1 file changed, 11 insertions(+), 18 deletions(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index 28ba0d9..6eb8303 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -171,23 +171,23 @@ func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatus slices.SortFunc(refProject.Result, BuildResultSorter) common.LogDebug("comparing results", len(project.Result), "vs. ref", len(refProject.Result)) - SomeSuccess := false for i := 0; i < len(project.Result); i++ { common.LogDebug("searching for", project.Result[i].Repository, "/", project.Result[i].Arch) j := 0 found: - for ; j < len(refProject.Result); j++ { + for j = 0; j < len(refProject.Result); j++ { if project.Result[i].Repository != refProject.Result[j].Repository || project.Result[i].Arch != refProject.Result[j].Arch { continue } common.LogDebug(" found match for @ idx:", j) - res, success := ProcessRepoBuildStatus(project.Result[i].Status, refProject.Result[j].Status) + res := ProcessRepoBuildStatus(project.Result[i].Status, refProject.Result[j].Status) switch res { case BuildStatusSummarySuccess: - SomeSuccess = SomeSuccess || success break found + case BuildStatusSummaryFailed: + return BuildStatusSummaryFailed default: return res } @@ -201,14 +201,10 @@ func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatus } } - if SomeSuccess { - return BuildStatusSummarySuccess - } - - return BuildStatusSummaryFailed + return BuildStatusSummarySuccess } -func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status BuildStatusSummary, SomeSuccess bool) { +func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status BuildStatusSummary) { PackageBuildStatusSorter := func(a, b *common.PackageBuildStatus) int { return strings.Compare(a.Package, b.Package) } @@ -225,16 +221,15 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B slices.SortFunc(results, PackageBuildStatusSorter) slices.SortFunc(ref, PackageBuildStatusSorter) - SomeSuccess = false for i := 0; i < len(results); i++ { res, ok := common.ObsBuildStatusDetails[results[i].Code] if !ok { common.LogInfo("unknown package result code:", results[i].Code, "for package:", results[i].Package) - return BuildStatusSummaryUnknown, SomeSuccess + return BuildStatusSummaryUnknown } if !res.Finished { - return BuildStatusSummaryBuilding, SomeSuccess + return BuildStatusSummaryBuilding } if !res.Success { @@ -247,7 +242,7 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B refRes, ok := common.ObsBuildStatusDetails[ref[j].Code] if !ok { common.LogInfo("unknown ref package result code:", ref[j].Code, "package:", ref[j].Package) - return BuildStatusSummaryUnknown, SomeSuccess + return BuildStatusSummaryUnknown } if !refRes.Finished { @@ -255,15 +250,13 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B } if refRes.Success { - return BuildStatusSummaryFailed, SomeSuccess + return BuildStatusSummaryFailed } } - } else { - SomeSuccess = true } } - return BuildStatusSummarySuccess, SomeSuccess + return BuildStatusSummarySuccess } func GenerateObsPrjMeta(git common.Git, gitea common.Gitea, pr *models.PullRequest, stagingPrj, buildPrj string, stagingMasterPrj string) (*common.ProjectMeta, error) { -- 2.51.1 From 025c5eebeaf043a759d0f97e98df6604d0d9670dd2f25e5529014a062d82b325 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Tue, 23 Sep 2025 16:57:35 +0200 Subject: [PATCH 06/11] HACK DON'T MERGE As long, as we don't handle _manifest files correctly, we need to ignore the master sub directories. Just hardcoding the names for now to rescue my vacation. --- obs-staging-bot/main.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index 6eb8303..f8943d3 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -819,11 +819,13 @@ func ProcessPullRequest(gitea common.Gitea, org, repo string, id int64) (bool, e if !stagingConfig.RebuildAll { for pkg, headOid := range headSubmodules { if baseOid, exists := baseSubmodules[pkg]; !exists || baseOid != headOid { + if pkg != "rpms" && pkg != "dependencies" { if exists { modifiedPackages = append(modifiedPackages, pkg) } else { newPackages = append(newPackages, pkg) } + } common.LogDebug(pkg, ":", baseOid, "->", headOid) } } -- 2.51.1 From 5aa35169fe3544b7148b260fd696211001a0d8fd4215d54d60ec81851619d5ae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Tue, 14 Oct 2025 16:08:20 +0200 Subject: [PATCH 07/11] Fix build status check when package source is in subdir We must not use the upper directory name as part of the package name. --- obs-staging-bot/main.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index f8943d3..a53e0d2 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -26,6 +26,7 @@ import ( "net/url" "os" "path" + "path/filepath" "regexp" "runtime/debug" "slices" @@ -820,10 +821,11 @@ func ProcessPullRequest(gitea common.Gitea, org, repo string, id int64) (bool, e for pkg, headOid := range headSubmodules { if baseOid, exists := baseSubmodules[pkg]; !exists || baseOid != headOid { if pkg != "rpms" && pkg != "dependencies" { + _, spkg := filepath.Split(pkg) if exists { - modifiedPackages = append(modifiedPackages, pkg) + modifiedPackages = append(modifiedPackages, spkg) } else { - newPackages = append(newPackages, pkg) + newPackages = append(newPackages, spkg) } } common.LogDebug(pkg, ":", baseOid, "->", headOid) -- 2.51.1 From c73c37e39df99ba8af94355641bde2ba00da57117269ff49690478312611eb83 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Fri, 17 Oct 2025 16:34:13 +0200 Subject: [PATCH 08/11] obs-staging-bot: fix wrong acceptance step 2 remove more code to ignore reference project entirely --- obs-staging-bot/main.go | 81 +++++------------------------------------ 1 file changed, 10 insertions(+), 71 deletions(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index a53e0d2..3912646 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -135,55 +135,15 @@ func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatus panic("Should not happen -- BuiltResultSorter equal repos?") } slices.SortFunc(project.Result, BuildResultSorter) - if refProject == nil { - // just return if buid finished and have some successes, since new package - common.LogInfo("New package. Only need some success...") - SomeSuccess := false - for i := 0; i < len(project.Result); i++ { - repoRes := project.Result[i] - repoResStatus, ok := common.ObsRepoStatusDetails[repoRes.Code] - if !ok { - common.LogDebug("cannot find code:", repoRes.Code) - return BuildStatusSummaryUnknown - } - if !repoResStatus.Finished { - return BuildStatusSummaryBuilding - } - - for _, pkg := range repoRes.Status { - pkgStatus, ok := common.ObsBuildStatusDetails[pkg.Code] - if !ok { - common.LogInfo("Unknown package build status:", pkg.Code, "for", pkg.Package) - common.LogDebug("Details:", pkg.Details) - } - - if pkgStatus.Success { - SomeSuccess = true - } - } - } - - if SomeSuccess { - return BuildStatusSummarySuccess - } - return BuildStatusSummaryFailed - } - - slices.SortFunc(refProject.Result, BuildResultSorter) common.LogDebug("comparing results", len(project.Result), "vs. ref", len(refProject.Result)) for i := 0; i < len(project.Result); i++ { common.LogDebug("searching for", project.Result[i].Repository, "/", project.Result[i].Arch) j := 0 found: - for j = 0; j < len(refProject.Result); j++ { - if project.Result[i].Repository != refProject.Result[j].Repository || - project.Result[i].Arch != refProject.Result[j].Arch { - continue - } - + for j = 0; j < len(project.Result); j++ { common.LogDebug(" found match for @ idx:", j) - res := ProcessRepoBuildStatus(project.Result[i].Status, refProject.Result[j].Status) + res := ProcessRepoBuildStatus(project.Result[i].Status) switch res { case BuildStatusSummarySuccess: break found @@ -205,22 +165,19 @@ func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatus return BuildStatusSummarySuccess } -func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status BuildStatusSummary) { - PackageBuildStatusSorter := func(a, b *common.PackageBuildStatus) int { - return strings.Compare(a.Package, b.Package) - } +func ProcessRepoBuildStatus(results []*common.PackageBuildStatus) (status BuildStatusSummary) { + + PackageBuildStatusSorter := func(a, b *common.PackageBuildStatus) int { + return strings.Compare(a.Package, b.Package) + } - common.LogDebug("******** REF: ") - data, _ := xml.MarshalIndent(ref, "", " ") - common.LogDebug(string(data)) common.LogDebug("******* RESULTS: ") - data, _ = xml.MarshalIndent(results, "", " ") + data, _ := xml.MarshalIndent(results, "", " ") common.LogDebug(string(data)) common.LogDebug("*******") // compare build result slices.SortFunc(results, PackageBuildStatusSorter) - slices.SortFunc(ref, PackageBuildStatusSorter) for i := 0; i < len(results); i++ { res, ok := common.ObsBuildStatusDetails[results[i].Code] @@ -234,26 +191,7 @@ func ProcessRepoBuildStatus(results, ref []*common.PackageBuildStatus) (status B } if !res.Success { - // not failed if reference project also failed for same package here - j := 0 - for ; j < len(ref) && strings.Compare(results[i].Package, ref[j].Package) != 0; j++ { - } - - if j < len(ref) && results[i].Package == ref[j].Package { - refRes, ok := common.ObsBuildStatusDetails[ref[j].Code] - if !ok { - common.LogInfo("unknown ref package result code:", ref[j].Code, "package:", ref[j].Package) - return BuildStatusSummaryUnknown - } - - if !refRes.Finished { - common.LogDebug("Not finished building in reference project?") - } - - if refRes.Success { - return BuildStatusSummaryFailed - } - } + return BuildStatusSummaryFailed } } @@ -316,6 +254,7 @@ func GenerateObsPrjMeta(git common.Git, gitea common.Gitea, pr *models.PullReque common.LogError("error fetching project meta for", buildPrj, ". Err:", err) return nil, err } + common.LogInfo("Meta: ", meta) // generate new project with paths pointinig back to original repos -- 2.51.1 From 1877b4ef787b92753178cbcd221c6178575171ad42eaf50b5e00e92302e21a08 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Tue, 4 Nov 2025 17:10:47 +0100 Subject: [PATCH 09/11] HACK: disable close of pullrequests for now it hit too many ones --- workflow-pr/pr_processor.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/workflow-pr/pr_processor.go b/workflow-pr/pr_processor.go index b100f75..a0c81b4 100644 --- a/workflow-pr/pr_processor.go +++ b/workflow-pr/pr_processor.go @@ -443,11 +443,11 @@ func (pr *PRProcessor) Process(req *models.PullRequest) error { if _, ok := err.(*repository.RepoMergePullRequestConflict); !ok { common.PanicOnError(err) } - } else { - Gitea.AddComment(pr.PR, "Closing here because the associated Project PR has been closed.") - Gitea.UpdatePullRequest(org, repo, idx, &models.EditPullRequestOption{ - State: "closed", - }) +// } else { +// Gitea.AddComment(pr.PR, "Closing here because the associated Project PR has been closed.") +// Gitea.UpdatePullRequest(org, repo, idx, &models.EditPullRequestOption{ +// State: "closed", +// }) } } } -- 2.51.1 From 7d45bff32cd5aefc0438cef989f438bb9e78c2ebe02675d6d43ae1057f7773f2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Thu, 20 Nov 2025 12:58:01 +0100 Subject: [PATCH 10/11] workflow-direct: use relative path when adding a submodule This solves the issue of using the right credentials based on the main repo. Also it allows to rename the organisation. --- workflow-direct/main.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/workflow-direct/main.go b/workflow-direct/main.go index 041ae56..fde24b4 100644 --- a/workflow-direct/main.go +++ b/workflow-direct/main.go @@ -107,7 +107,7 @@ func processConfiguredRepositoryAction(action *common.RepositoryWebhookEvent, co common.LogError(" - ", action.Repository.Name, "repo is not sha256. Ignoring.") return } - common.PanicOnError(git.GitExec(gitPrj, "submodule", "--quiet", "add", "--depth", "1", action.Repository.Clone_Url, action.Repository.Name)) + common.PanicOnError(git.GitExec(gitPrj, "submodule", "--quiet", "add", "--depth", "1", "../" + action.Repository.Name, action.Repository.Name)) defer git.GitExecQuietOrPanic(gitPrj, "submodule", "deinit", "--all", "-f") branch := strings.TrimSpace(git.GitExecWithOutputOrPanic(path.Join(gitPrj, action.Repository.Name), "branch", "--show-current")) @@ -198,7 +198,7 @@ func processConfiguredPushAction(action *common.PushWebhookEvent, config *common } if stat, err := os.Stat(filepath.Join(git.GetPath(), gitPrj, action.Repository.Name)); err != nil { - git.GitExecOrPanic(gitPrj, "submodule", "--quiet", "add", "--depth", "1", action.Repository.Clone_Url, action.Repository.Name) + git.GitExecOrPanic(gitPrj, "submodule", "--quiet", "add", "--depth", "1", "../" + action.Repository.Name, action.Repository.Name) common.LogDebug("Pushed to package that is not part of the project. Re-adding...", err) } else if !stat.IsDir() { common.LogError("Pushed to a package that is not a submodule but exists in the project. Ignoring.") @@ -402,7 +402,7 @@ next_repo: } // add repository to git project - common.PanicOnError(git.GitExec(gitPrj, "submodule", "--quiet", "add", "--depth", "1", r.CloneURL, r.Name)) + common.PanicOnError(git.GitExec(gitPrj, "submodule", "--quiet", "add", "--depth", "1", "../" + r.Name, r.Name)) curBranch := strings.TrimSpace(git.GitExecWithOutputOrPanic(path.Join(gitPrj, r.Name), "branch", "--show-current")) if branch != curBranch { -- 2.51.1 From 00d9a9d68217808d37e8cca7d3df7aff3543138fffc283c2910eb3935b520503 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adrian=20Schr=C3=B6ter?= Date: Thu, 20 Nov 2025 16:38:15 +0100 Subject: [PATCH 11/11] obs-staging-bot: Get rid of reference project results entirely Currently all our release managers insist anyway of only green states --- obs-staging-bot/main.go | 37 ++++--------------------------------- 1 file changed, 4 insertions(+), 33 deletions(-) diff --git a/obs-staging-bot/main.go b/obs-staging-bot/main.go index 3912646..68275d5 100644 --- a/obs-staging-bot/main.go +++ b/obs-staging-bot/main.go @@ -110,33 +110,15 @@ const ( BuildStatusSummaryUnknown = 4 ) -func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatusSummary { - if _, finished := refProject.BuildResultSummary(); !finished { - common.LogDebug("refProject not finished building??") - return BuildStatusSummaryUnknown - } - +func ProcessBuildStatus(project *common.BuildResultList) BuildStatusSummary { if _, finished := project.BuildResultSummary(); !finished { common.LogDebug("Still building...") return BuildStatusSummaryBuilding } - // the repositories should be setup equally between the projects. We - // need to verify that packages that are building in `refProject` are not - // failing in the `project` - BuildResultSorter := func(a, b *common.BuildResult) int { - if c := strings.Compare(a.Repository, b.Repository); c != 0 { - return c - } - if c := strings.Compare(a.Arch, b.Arch); c != 0 { - return c - } + //slices.SortFunc(project.Result, BuildResultSorter) - panic("Should not happen -- BuiltResultSorter equal repos?") - } - slices.SortFunc(project.Result, BuildResultSorter) - - common.LogDebug("comparing results", len(project.Result), "vs. ref", len(refProject.Result)) + common.LogDebug("build results", len(project.Result)) for i := 0; i < len(project.Result); i++ { common.LogDebug("searching for", project.Result[i].Repository, "/", project.Result[i].Arch) j := 0 @@ -153,13 +135,6 @@ func ProcessBuildStatus(project, refProject *common.BuildResultList) BuildStatus return res } } - - if j >= len(refProject.Result) { - common.LogDebug("Cannot find results...") - common.LogDebug(project.Result[i]) - common.LogDebug(refProject.Result) - return BuildStatusSummaryUnknown - } } return BuildStatusSummarySuccess @@ -869,15 +844,11 @@ func ProcessPullRequest(gitea common.Gitea, org, repo string, id int64) (bool, e gitea.AddComment(pr, msg) } - baseResult, err := ObsClient.LastBuildResults(stagingConfig.ObsProject, modifiedPackages...) - if err != nil { - common.LogError("failed fetching ref project status for", stagingConfig.ObsProject, ":", err) - } stagingResult, err := ObsClient.BuildStatus(stagingProject) if err != nil { common.LogError("failed fetching stage project status for", stagingProject, ":", err) } - buildStatus := ProcessBuildStatus(stagingResult, baseResult) + buildStatus := ProcessBuildStatus(stagingResult) done := false switch buildStatus { -- 2.51.1