[openSUSE/open-build-service] 8df494: Report back build state to the correct scm repository
Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 8df4946b358175d57f2bd3bd4b6dc93c091bd4b9 https://github.com/openSUSE/open-build-service/commit/8df4946b358175d57f2bd3... Author: Lukas Krause <lkrause@suse.de> Date: 2021-05-28 (Fri, 28 May 2021) Changed paths: M src/api/app/models/workflow/step/branch_package_step.rb M src/api/app/services/scm_status_reporter.rb M src/api/app/services/trigger_controller_service/scm_extractor.rb M src/api/app/services/workflows/yaml_downloader.rb M src/api/spec/services/trigger_controller_service/scm_extractor_spec.rb M src/api/spec/services/workflows/yaml_downloader_spec.rb Log Message: ----------- Report back build state to the correct scm repository We were using the head (source) repository instead of the base (target) repository to report back the build state of a workflow. This is wrong since the state is reported using the scm token of the target repo owner/maintainer, which leads to permission problems (since it doesn't associate the reported state to the PR, but directly to the commit on the source repository). We also changed the download url, we should only take the "workflow.yml" file from the target repository. Co-authored-by: Saray Cabrera Padrón <scabrerapadron@suse.de> Commit: fa305dd445d49fbf8c9ba198736e948f9f1d8a9d https://github.com/openSUSE/open-build-service/commit/fa305dd445d49fbf8c9ba1... Author: Lukas Krause <lkrause@suse.de> Date: 2021-05-28 (Fri, 28 May 2021) Changed paths: M src/api/app/services/trigger_controller_service/scm_extractor.rb M src/api/spec/services/trigger_controller_service/scm_extractor_spec.rb Log Message: ----------- Remove 'repo_url' from scm extractor for the github payload We dont use this field anywhere. Co-authored-by: Saray Cabrera Padrón <scabrerapadron@suse.de> Commit: a9412cefc319cf362df2bd464f658ce11c236a22 https://github.com/openSUSE/open-build-service/commit/a9412cefc319cf362df2bd... Author: Victor Pereira <vpereira@suse.de> Date: 2021-05-31 (Mon, 31 May 2021) Changed paths: M src/api/app/models/workflow/step/branch_package_step.rb M src/api/app/services/scm_status_reporter.rb M src/api/app/services/trigger_controller_service/scm_extractor.rb M src/api/app/services/workflows/yaml_downloader.rb M src/api/spec/services/trigger_controller_service/scm_extractor_spec.rb M src/api/spec/services/workflows/yaml_downloader_spec.rb Log Message: ----------- Merge pull request #11192 from krauselukas/fix_scm_reporting Report back build state to the correct scm repository Compare: https://github.com/openSUSE/open-build-service/compare/c8cc3303d41f...a9412c...
participants (1)
-
Victor Pereira