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
}