Copilot commented on code in PR #51326:
URL: https://github.com/apache/arrow/pull/51326#discussion_r4004019711
##########
ci/scripts/install_minio.sh:
##########
@@ -62,9 +62,16 @@ if [ "${version}" != "latest" ]; then
exit 1
fi
+# The binaries are no longer available from https://dl.min.io (HTTP 410),
+# so they are fetched from the GitHub releases instead. See GH-47908.
# Use specific versions for minio server and client to avoid CI failures on
new releases.
-minio_version="minio.RELEASE.2025-01-20T14-49-07Z"
-mc_version="mc.RELEASE.2024-09-16T17-43-14Z"
+minio_version="RELEASE.2025-01-20T14-49-07Z"
Review Comment:
The Windows C++ job still downloads this pinned server from
`https://dl.min.io/...` at `.github/workflows/cpp.yml:411`, so it will continue
to fail with HTTP 410 even after this script is updated. Please update that job
to the corresponding GitHub release asset (or reuse this installer) as part of
this change.
##########
ci/scripts/install_minio.sh:
##########
@@ -80,13 +87,13 @@ download()
}
if [[ ! -x ${prefix}/bin/minio ]]; then
-
url="https://dl.min.io/server/minio/release/${platform}-${arch}/archive/${minio_version}";
+
url="https://github.com/minio/minio/releases/download/${minio_version}/minio.${platform}-${arch}.${minio_version}${exe_suffix}";
Review Comment:
This template still maps `s390x` to `linux-s390x`, but the pinned MinIO
GitHub release has no `minio.linux-s390x.RELEASE.2025-01-20T14-49-07Z` asset.
An invocation on s390x will therefore fail with a 404; please provide a valid
source for that architecture or remove it from the supported architecture map
instead of constructing an unavailable URL.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
