You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ze...@apache.org on 2023/06/15 16:12:20 UTC

[arrow] branch main updated: MINOR: [Go] [Benchmarking] fix repo specifier (#36104)

This is an automated email from the ASF dual-hosted git repository.

zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 99dd998068 MINOR: [Go] [Benchmarking] fix repo specifier (#36104)
99dd998068 is described below

commit 99dd998068d29ae528704154c499ed246f27dde8
Author: Austin Dickey <au...@gmail.com>
AuthorDate: Thu Jun 15 11:12:14 2023 -0500

    MINOR: [Go] [Benchmarking] fix repo specifier (#36104)
    
    The Arrow Conbench deployment recently imposed stricter validation, that the `repository` key must be a URL.
    
    This PR fixes the Go benchmarking script to comply.
    
    Authored-by: Austin Dickey <au...@gmail.com>
    Signed-off-by: Matt Topol <zo...@gmail.com>
---
 ci/scripts/go_bench_adapt.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci/scripts/go_bench_adapt.py b/ci/scripts/go_bench_adapt.py
index e642082b8a..103712f6d8 100644
--- a/ci/scripts/go_bench_adapt.py
+++ b/ci/scripts/go_bench_adapt.py
@@ -41,7 +41,7 @@ if os.environ.get("CONBENCH_REF") == "main":
     # Assume GitHub Actions CI. The environment variable lookups below are
     # expected to fail when not running in GitHub Actions.
     github_commit_info = {
-        "repository": os.environ["GITHUB_REPOSITORY"],
+        "repository": f'{os.environ["GITHUB_SERVER_URL"]}/{os.environ["GITHUB_REPOSITORY"]}',
         "commit": os.environ["GITHUB_SHA"],
         "pr_number": None,  # implying default branch
     }